[buddypress-trac] [BuddyPress Trac] #6331: Star Private Messages

buddypress-trac noreply at wordpress.org
Thu Apr 30 17:35:42 UTC 2015


#6331: Star Private Messages
------------------------------------+-----------------------
 Reporter:  r-a-y                   |       Owner:  r-a-y
     Type:  enhancement             |      Status:  assigned
 Priority:  normal                  |   Milestone:  2.3
Component:  Component - Messaging   |     Version:
 Severity:  normal                  |  Resolution:
 Keywords:  dev-feedback has-patch  |
------------------------------------+-----------------------

Comment (by imath):

 About the filter to disable, in your comment announcing the patch you
 shared
 `add_filter( 'bp_messages_is_starred_enabled', '__return_false' );`

 If i do that having the patch applied, the star functionality is still
 loading, because your function is :
 {{{
 392  function bp_messages_is_starred_enabled() {
 393       return (bool) apply_filters( 'bp_messages_enable_starred', true
 );
 394 }
 }}}

 That's why i suggested to change the filter to
 'bp_messages_is_starred_enabled' or change the function name to the filter
 as you like :)

 > I'm not opposed to moving the CSS class and enqueue styles stuff into
 the core BP messages component. I'll move it in just for you, imath :)

 Thank you, but it was related to the file organization, so if everyone is
 fine with having all in one file, i'm fine with it :)

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/6331#comment:16>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac


More information about the buddypress-trac mailing list