From e4f7bc7216c155e04a42eb36f93dc7bd9b4e8513 Mon Sep 17 00:00:00 2001 From: danielgrippi Date: Wed, 14 Sep 2011 22:18:08 -0700 Subject: [PATCH] local_people should always return an array (re: injection issues) --- lib/postzord/dispatcher.rb | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/postzord/dispatcher.rb b/lib/postzord/dispatcher.rb index bb941a9e7..e981269b7 100644 --- a/lib/postzord/dispatcher.rb +++ b/lib/postzord/dispatcher.rb @@ -68,9 +68,9 @@ class Postzord::Dispatcher end # @param local_people [Array] - # @return [ActiveRecord::Association] + # @return [ActiveRecord::Association, Array] def fetch_local_users(people) - return if people.blank? + return [] if people.blank? user_ids = people.map{|x| x.owner_id } User.where(:id => user_ids) end @@ -142,13 +142,13 @@ class Postzord::Dispatcher # @param services [Array] def notify_users(users) - return unless users && @object.respond_to?(:persisted?) + return unless users.present? && @object.respond_to?(:persisted?) Resque.enqueue(Job::NotifyLocalUsers, users.map{|u| u.id}, @object.class.to_s, @object.id, @object.author.id) end # @param services [Array] def socket_to_users(users) - return unless users && @object.respond_to?(:socket_to_user) + return unless users.present? && @object.respond_to?(:socket_to_user) users.each do |user| @object.socket_to_user(user) end