Changeset 12142 for branches/3.0/src/bp-templates/bp-nouveau/includes/messages/ajax.php

Timestamp:

05/28/2018 05:37:38 PM (8 years ago)

Author:

imath

Message:

Messages: catch Legacy hooks to customize the Active conversation box.

In order to preserve in the BP Nouveau Messages UI the level of PHP customization that developers are used to get in Legacy using do_action hooks, The content generated for the following hooks will be caught during Ajax requests and added as new properties of their JSON replies:

• bp_before_message_meta
• bp_after_message_meta
• bp_messages_inbox_list_item
• bp_messages_thread_options

About the bp_messages_inbox_list_item hook: as the Messages UI of BP Nouveau, unlike Legacy, is not using a table tag to display the list of messages received/starred/sent. All extra actions about messages into these loop views are done within the Active conversation box (eg: star/unstar). To inject the content of this specific hook, we are adding a table element under the message content preview as the expected output is a td tag.

Fixes #7851 (Branch 3.0)

File:

• branches/3.0/src/bp-templates/bp-nouveau/includes/messages/ajax.php (modified) (5 diffs)

branches/3.0/src/bp-templates/bp-nouveau/includes/messages/ajax.php

@@ -190 +190 @@
 
     $extra_content = bp_nouveau_messages_catch_hook_content( array(
+        'beforeMeta'    => 'bp_before_message_meta',
+        'afterMeta'     => 'bp_after_message_meta',
         'beforeContent' => 'bp_before_message_content',
         'afterContent'  => 'bp_after_message_content',
@@ -263 +265 @@
 
     while ( bp_message_threads() ) : bp_message_thread();
+        $last_message_id = (int) $messages_template->thread->last_message_id;
+
         $threads->threads[ $i ] = array(
             'id'            => bp_get_message_thread_id(),
-            'message_id'    => (int) $messages_template->thread->last_message_id,
+            'message_id'    => (int) $last_message_id,
             'subject'       => html_entity_decode( bp_get_message_thread_subject() ),
             'excerpt'       => html_entity_decode( bp_get_message_thread_excerpt() ),
@@ -314 +318 @@
 
             // Defaults to last
-            $sm_id = (int) $messages_template->thread->last_message_id;
+            $sm_id = $last_message_id;
 
             if ( $threads->threads[ $i ]['is_starred'] ) {
@@ -322 +326 @@
             $threads->threads[ $i ]['star_nonce'] = wp_create_nonce( 'bp-messages-star-' . $sm_id );
             $threads->threads[ $i ]['starred_id'] = $sm_id;
+        }
+
+        $thread_extra_content = bp_nouveau_messages_catch_hook_content( array(
+            'inboxListItem' => 'bp_messages_inbox_list_item',
+            'threadOptions' => 'bp_messages_thread_options',
+        ) );
+
+        if ( array_filter( $thread_extra_content ) ) {
+            $threads->threads[ $i ] = array_merge( $threads->threads[ $i ], $thread_extra_content );
         }
 
@@ -472 +485 @@
 
         $extra_content = bp_nouveau_messages_catch_hook_content( array(
+            'beforeMeta'    => 'bp_before_message_meta',
+            'afterMeta'     => 'bp_after_message_meta',
             'beforeContent' => 'bp_before_message_content',
             'afterContent'  => 'bp_after_message_content',