Fixed error calling #notify for callbacks

When tracked_option was expanded into the argument list, if non-empty it pushed extraneous arguments
simukappu · Nov 8, 2017 · 8f6f0c3 · 8f6f0c3
1 parent e21d7ee
commit 8f6f0c3
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/activity_notification/roles/acts_as_notifiable.rb b/lib/activity_notification/roles/acts_as_notifiable.rb
@@ -259,8 +259,8 @@ def add_tracked_callbacks(target_type, tracked_option = {})
           add_tracked_callback(tracked_callbacks, :create, ->{ notify target_type, tracked_option })
           add_tracked_callback(tracked_callbacks, :update, ->{ notify target_type, tracked_option })
         else
-          add_tracked_callback(tracked_callbacks, :create, ->{ notify target_type, *tracked_option, key: notification_key_for_tracked_creation })
-          add_tracked_callback(tracked_callbacks, :update, ->{ notify target_type, *tracked_option, key: notification_key_for_tracked_update })
+          add_tracked_callback(tracked_callbacks, :create, ->{ notify target_type, tracked_option.merge(key: notification_key_for_tracked_creation) })
+          add_tracked_callback(tracked_callbacks, :update, ->{ notify target_type, tracked_option.merge(key: notification_key_for_tracked_update) })
         end
         { tracked: tracked_callbacks }
       end