[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