bp_after_message_content hook not working with BP Nouveau

[pareshradadiya]

bp_after_message_content hook does not work when BP Nouveau is active.

I use bp_after_message_content to display some custom stuff after each messages.

So when BP Nouveau is active, my callback function hooked into bp_after_message_content never get call.

I have done something like this:

add_action( 'bp_after_message_content', 'test_message_meta_display' ) );

function test_message_meta_display() {

	$message_id = (int) bp_get_the_thread_message_id();
	$meta = bp_messages_get_meta( $message_id, 'message_meta' );

	ob_start();

	// ..some more custom code

	$content = ob_get_contents();

	ob_end_clean();

	echo str_replace("\n","",$content);
}

[DJPaul]

Another missing action - a similar problem to #7837

[DJPaul]

Which is going to be tricky because we'll have to make sure however those templates are loaded in Javascript, that the message content filter is run in the AJAX callback(?).

[imath]

I thought this ​https://buddypress.trac.wordpress.org/browser/trunk/src/bp-templates/bp-nouveau/buddypress/common/js-templates/messages/index.php#L279 was handling this part. I’ll look at the function to understand what’s going wrong. I think the hook is fired into the JS template. So it’s an approach a bit different, I’ll add an example with messages meta.

[imath]

After a deeper look, i confirm bp_after_message_content is fired. But it's unusable the way @pareshradadiya tries to use it, because data is available in JavaScript and not in PHP, see screenshot below:

What needs to be done imho:

1. improve the Ajax functions that are fetching the messages data :
   1. by adding a filter to allow plugins to get meta data
   2. or by populating a metas property with all the meta for the message using bp_messages_get_meta( bp_get_the_thread_message_id() )
2. Add some documentation about how to use it to display the meta value into the UI.

I doubt 1.a. can be done for the 3.0.1 maintenance release, 1.b. is easier but might cause some performance troubles.

[pareshradadiya]

I confirm bp_after_message_content callback function get called. I was returning output instead of echoing it on a page in my old test. However, I cannot fetch message meta inside from callback like this.

$message_id = (int) bp_get_the_thread_message_id();
$meta = bp_messages_get_meta( $message_id, 'message_meta' );

[imath]

I chose a 3rd approach. The idea is to change as fiew as possible for plugin authors. I think the new Messages UI is a perfect candidate for the ​BP Rest API, so untill we can count on it and why not deprecate some actions in favor of this API, i think we should use the existing actions and catch their content using the buffer. As this is done within a messages loop, data is accessible via PHP and it doesn't change anything for plugin authors. Here's a screenshot showing injected content using bp_before_message_content & bp_after_message_content :

@pareshradadiya can you test ​7847.patch and tell me if it's fixing the issue for you ?

[DJPaul]

Milestone renamed

[imath]

Improves first patch by making sure bp_is_messages_conversation() is true in AJAX

[imath]

Introduces a new function bp_strip_script_and_style_tags() to sanitize hooks buffer

[pareshradadiya]

@imath I confirm 7847.3.patch fixing issue for me.

[imath]

Thanks a lot for your feedback 👍 I'll commit it asap.

[imath]

In 12121:

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

See #7847 (Trunk)

[imath]

In 12122:

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)