[buddypress-trac] [BuddyPress Trac] #6707: Member - Settings - Email - radio buttons

buddypress-trac noreply at wordpress.org
Mon Nov 2 19:02:40 UTC 2015


#6707: Member - Settings - Email - radio buttons
----------------------------------+------------------
 Reporter:  slaFFik               |       Owner:
     Type:  defect (bug)          |      Status:  new
 Priority:  normal                |   Milestone:  2.5
Component:  Component - Settings  |     Version:  1.0
 Severity:  normal                |  Resolution:
 Keywords:  has-patch             |
----------------------------------+------------------
Changes (by r-a-y):

 * keywords:   => has-patch
 * version:   => 1.0
 * milestone:  Awaiting Review => 2.5


Comment:

 Yeah, this isn't great.  This code has been in place since v1.0!

 On the other hand though, the majority of plugins do direct checks for
 `'yes'` or `'no'` values only; this value is never outputted.

 `6707.01.patch` just sets the value of the notification setting before
 saving it into user meta.

 Filtering `bp_get_user_meta( $user_id, 'notification_XXX' )` might be
 overkill here, so I haven't done this in `01.patch`.  If we feel it is
 necessary, then we could do it.

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


More information about the buddypress-trac mailing list