[buddypress-trac] [BuddyPress Trac] #5500: Date xprofile field enhancement

buddypress-trac noreply at wordpress.org
Tue Aug 30 15:11:33 UTC 2016


#5500: Date xprofile field enhancement
-------------------------------------+------------------
 Reporter:  sooskriszta              |       Owner:
     Type:  enhancement              |      Status:  new
 Priority:  normal                   |   Milestone:  2.7
Component:  Extended Profile         |     Version:
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch needs-testing  |
-------------------------------------+------------------

Comment (by boonebgorges):

 @DJPaul Thanks for the careful review! All your points are good ones.

 You're correct about `validate_settings()`. It doesn't really sanitize in
 the SQL-injection sense (this happens in the `meta` functions) but it does
 ensure that the passed values are valid for the given fields. I've left a
 comment to that effect.

 I'm going to commit the first pass. @mercime If you have a few moments, an
 accessibility review of the admin interface would be great - especially
 the "from/to" dialogs.

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


More information about the buddypress-trac mailing list