[buddypress-trac] [BuddyPress Trac] #7847: bp_after_message_content hook not working with BP Nouveau
buddypress-trac
noreply at wordpress.org
Tue May 22 10:21:04 UTC 2018
#7847: bp_after_message_content hook not working with BP Nouveau
-----------------------------------------+-----------------------
Reporter: pareshradadiya | Owner: imath
Type: defect (bug) | Status: assigned
Priority: normal | Milestone: 3.0.1
Component: Messages | Version:
Severity: normal | Resolution:
Keywords: has-patch reporter-feedback |
-----------------------------------------+-----------------------
Changes (by imath):
* keywords: => has-patch reporter-feedback
Comment:
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
[https://github.com/buddypress/BP-REST/blob/master/includes/bp-
messages/classes/class-bp-rest-messages-endpoint.php BP Rest API], so
untill we can count on it and why not deprecate some actions in favor of
this API, i think we shoul 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}}} :
[[Image(https://cldup.com/wXf19swDlL.png)]]
@pareshradadiya can you test [https://buddypress.trac.wordpress.org/raw-
attachment/ticket/7847/7847.patch 7847.patch] and tell me if it's fixing
the issue for you ?
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/7847#comment:7>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac
More information about the buddypress-trac
mailing list