Merge pull request ManageIQ#17875 from skateman/send-notification-id

Send the notification ID when propagating a notification through WS
andyvesel · Aug 17, 2018 · fa93766 · fa93766
2 parents 85cdfe6 + 42a4233
commit fa93766
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/app/models/notification.rb b/app/models/notification.rb
@@ -55,8 +55,7 @@ def self.notification_text(name, message_params)
   def emit_message
     return unless ::Settings.server.asynchronous_notifications
     notification_recipients.pluck(:id, :user_id).each do |id, user|
-      to_h[:id] = id
-      ActionCable.server.broadcast("notifications_#{user}", to_h)
+      ActionCable.server.broadcast("notifications_#{user}", to_h.merge(:id => id.to_s))
     end
   end