[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