From e2e184363962b10ddbcd6f04d28f83d92ba67b7c Mon Sep 17 00:00:00 2001 From: Brion Vibber Date: Fri, 11 Dec 2009 13:53:09 -0800 Subject: [PATCH] slight cleanup for a bit in Notice.php where a var was reused for different types, confusing tracking down a bug --- classes/Notice.php | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/classes/Notice.php b/classes/Notice.php index 4aec4ed55c..01ed4e7f41 100644 --- a/classes/Notice.php +++ b/classes/Notice.php @@ -966,6 +966,9 @@ class Notice extends Memcached_DataObject return true; } + /** + * @return array of integer profile IDs + */ function saveReplies() { // Alternative reply format @@ -1044,8 +1047,8 @@ class Notice extends Memcached_DataObject $recipientIds = array_keys($replied); - foreach ($recipientIds as $recipient) { - $user = User::staticGet('id', $recipient); + foreach ($recipientIds as $recipientId) { + $user = User::staticGet('id', $recipientId); if ($user) { mail_notify_attn($user, $this); }