Ticket #7847: 7847.patch

src/bp-templates/bp-nouveau/buddypress/common/js-templates/messages/index.php

@@ -272 +272 @@
 
         </div>
 
-        <?php bp_nouveau_messages_hook( 'before', 'content' ); ?>
+        <# if ( data.beforeContent ) { #>
+                <div class="before-message-content">{{{data.beforeContent}}}</div>
+        <# } #>
 
-        <div class="message-content"><# print( data.content ) #></div>
+        <div class="message-content">{{{data.content}}}</div>
 
-        <?php bp_nouveau_messages_hook( 'after', 'content' ); ?>
+        <# if ( data.afterContent ) { #>
+                <div class="after-message-content">{{{data.afterContent}}}</div>
+        <# } #>
 
 </script>
 

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

@@ -182 +182 @@
                 $reply['is_starred'] = array_search( 'unstar', explode( '/', $star_link ) );
         }
 
+        $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();
 
@@ -422 +431 @@
                         $thread->messages[ $i ]['star_nonce'] = wp_create_nonce( 'bp-messages-star-' . bp_get_the_thread_message_id() );
                 }
 
+                $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;
 

src/bp-templates/bp-nouveau/includes/messages/functions.php

@@ -444 +444 @@
                 )
         );
 }
+
+/**
+ * Fires Messages Legacy hooks to catch the content and add them
+ * as extra keys to the JSON Messages UI reply.
+ *
+ * @since 3.0.1
+ *
+ * @param array $hooks The list of hooks to fire.
+ * @return array       An associative containing the caught content.
+ */
+function bp_nouveau_messages_catch_hook_content( $hooks = array() ) {
+        $content = array();
+
+        ob_start();
+        foreach ( $hooks as $js_key => $hook ) {
+                if ( ! has_action( $hook ) ) {
+                        continue;
+                }
+
+                // Fire the hook.
+                do_action( $hook );
+
+                // Catch the content.
+                $content[ $js_key ] = ob_get_contents();
+
+                // Clean the buffer.
+                ob_clean();
+        }
+        ob_end_clean();
+
+        return $content;
+}