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

Timestamp:

05/25/2018 03:52:08 AM (7 years ago)

Author:

imath

Message:

BP Nouveau: Improve Messages UI extensibility

In BP Nouveau, the Messages UI is a Backbone based one. This means current legacy hooks inserted into the JS templates are a bit more different to use for plugin developers as it requires them to use JavaScript to get the data models. If the current AJAX requests still need a way to be extended to fetch extra data such as specific Messages metas, this commit is a first step to help plugin developers to insert content into the Messages UI without changing their habits about using the PHP Messages template global variables to get data about the Messages loop. This is done by introducing the back compatibility function bp_nouveau_messages_catch_hook_content(). This function is used during the AJAX requests that are fetching messages for a specific thread. As a start, 2 legacy hooks have been moved from the JS Template level to this higher level :

• bp_before_message_content
• bp_after_message_content

If actions are attached to these hooks their outputs will be caught using the buffer and will be "transported" as new properties, respectively beforeContent and afterContent, of the JSON reply that is used by the Messages UI. If these 2 properties are set and populated, their content will be output inside the corresponding JS templates.

Props pareshradadiya

Fixes #7847 (Branch 3.0)

File:

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

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

@@ -135 +135 @@
     }
 
-    // Get the message bye pretending we're in the message loop.
+    // Get the message by pretending we're in the message loop.
     global $thread_template;
+
+    $bp           = buddypress();
+    $reset_action = $bp->current_action;
+
+    // Override bp_current_action().
+    $bp->current_action = 'view';
 
     bp_thread_has_messages( array( 'thread_id' => (int) $_POST['thread_id'] ) );
@@ -183 +189 @@
     }
 
+    $extra_content = bp_nouveau_messages_catch_hook_content( array(
+        'beforeContent' => 'bp_before_message_content',
+        'afterContent'  => 'bp_after_message_content',
+    ) );
+
+    if ( array_filter( $extra_content ) ) {
+        $reply = array_merge( $reply, $extra_content );
+    }
+
     // Clean up the loop.
     bp_thread_messages();
+
+    // Remove the bp_current_action() override.
+    $bp->current_action = $reset_action;
 
     wp_send_json_success( array(
@@ -357 +375 @@
     }
 
-    $thread_id = (int) $_POST['id'];
+    $thread_id    = (int) $_POST['id'];
+    $bp           = buddypress();
+    $reset_action = $bp->current_action;
+
+    // Override bp_current_action().
+    $bp->current_action = 'view';
 
     // Simulate the loop.
     if ( ! bp_thread_has_messages( array( 'thread_id' => $thread_id ) ) ) {
+        // Remove the bp_current_action() override.
+        $bp->current_action = $reset_action;
+
         wp_send_json_error( $response );
     }
@@ -423 +449 @@
         }
 
+        $extra_content = bp_nouveau_messages_catch_hook_content( array(
+            'beforeContent' => 'bp_before_message_content',
+            'afterContent'  => 'bp_after_message_content',
+        ) );
+
+        if ( array_filter( $extra_content ) ) {
+            $thread->messages[ $i ] = array_merge( $thread->messages[ $i ], $extra_content );
+        }
+
         $i += 1;
     endwhile;
 
     $thread->messages = array_filter( $thread->messages );
+
+    // Remove the bp_current_action() override.
+    $bp->current_action = $reset_action;
 
     wp_send_json_success( $thread );