Changeset 8052 for trunk/bp-templates/bp-legacy/buddypress-functions.php

Timestamp:

03/06/2014 01:15:18 AM (11 years ago)

Author:

r-a-y

Message:

bp-legacy: When bulk-deleting message threads, use messages_delete_thread().

Previously, a foreach loop and a call to BP_Messages_Thread::delete()
was used to bulk-delete message threads from the inbox with AJAX.

This commit:

• Gets rid of the foreach loop and replaces the direct BP_Messages_Thread::delete() call with BP's messages_delete_thread() function.
• Properly sanitizes the posted 'thread_ids' with wp_parse_id_list().

File:

• trunk/bp-templates/bp-legacy/buddypress-functions.php (modified) (1 diff)

trunk/bp-templates/bp-legacy/buddypress-functions.php

@@ -1330 +1330 @@
 
     } else {
-        $thread_ids = explode( ',', $_POST['thread_ids'] );
-
-        for ( $i = 0, $count = count( $thread_ids ); $i < $count; ++$i ) {
-            BP_Messages_Thread::delete( (int) $thread_ids[$i] );
-        }
+        $thread_ids = wp_parse_id_list( $_POST['thread_ids'] );
+        messages_delete_thread( $thread_ids );
 
         _e( 'Messages deleted.', 'buddypress' );