Changeset 2020 for trunk/bp-messages.php

Timestamp:

09/30/2009 06:53:29 PM (16 years ago)

Author:

apeatling

Message:

Typecasting message arrays and checking to make sure duplicates do not slip through.

File:

• trunk/bp-messages.php (modified) (2 diffs)

trunk/bp-messages.php

@@ -440 +440 @@
 
         /* Strip the sender from the recipient list if they exist */
-        if ( $key = array_search( $sender_id, $recipient_ids ) )
-            unset( $recipient_ids[$key] );  
-
+        if ( $key = array_search( $sender_id, (array)$recipient_ids ) )
+            unset( $recipient_ids[$key] );
+
+        /* Remove duplicates */
+        $recipient_ids = array_unique( (array)$recipient_ids );
+        
+        if ( empty( $recipient_ids ) )
+            return false;
+        
         $message->recipients = $recipient_ids; 
     }
@@ -450 +456 @@
 
         // Send screen notifications to the recipients
-        foreach ( $message->recipients as $recipient ) {
+        foreach ( (array)$message->recipients as $recipient ) {
             bp_core_add_notification( $message->id, $recipient, 'messages', 'new_message' );    
         }