[buddypress-trac] [BuddyPress Trac] #7023: Administrator not able to delete user notifications

buddypress-trac noreply at wordpress.org
Mon May 16 16:55:15 UTC 2016


#7023: Administrator not able to delete user notifications
---------------------------------------+--------------------
 Reporter:  VibeThemes                 |       Owner:
     Type:  defect (bug)               |      Status:  new
 Priority:  normal                     |   Milestone:  2.6
Component:  Component - Notifications  |     Version:  2.5.0
 Severity:  normal                     |  Resolution:
 Keywords:  has-patch has-screenshots  |
---------------------------------------+--------------------

Comment (by dcavins):

 @r-a-y: Nice! Works well for me and is a definite improvement.

 Comments: There's an errant backslash opening line 626 of `bp-
 notifications-template.php`.

 For my own education, can you say why you chose this pattern:
 {{{#!php
 function bp_get_notifications_unread_permalink( $user_id = 0 ) {
         if ( 0 === $user_id ) {
                 $user_id = bp_loggedin_user_id();
                 $domain  = bp_loggedin_user_domain();
         } else {
                 $domain = bp_core_get_user_domain( (int) $user_id );
         }
 }}}

 instead of
 {{{#!php
         if ( $user_id ) {
                 $domain = bp_core_get_user_domain( (int) $user_id );
         } else {
                 $user_id = bp_loggedin_user_id();
                 $domain  = bp_loggedin_user_domain();
         }
 }}}

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


More information about the buddypress-trac mailing list