[buddypress-trac] [BuddyPress Trac] #7300: xprofile date field year regression on existing fields

buddypress-trac noreply at wordpress.org
Tue Oct 25 15:11:45 UTC 2016


#7300: xprofile date field year regression on existing fields
------------------------------+-----------------------
 Reporter:  DJPaul            |       Owner:  djpaul
     Type:  defect (bug)      |      Status:  assigned
 Priority:  high              |   Milestone:  2.7.1
Component:  Extended Profile  |     Version:
 Severity:  major             |  Resolution:
 Keywords:  commit            |
------------------------------+-----------------------
Changes (by boonebgorges):

 * keywords:   => commit
 * owner:   => djpaul
 * status:  new => assigned


Comment:

 Thanks for tracking this down, @DJPaul. Fix looks good to me.

 > This contradicts our PHPDoc block for bp_xprofile_get_meta which says it
 returns a boolean false if the request value is not set... which is how
 WP's https://codex.wordpress.org/Function_Reference/get_user_meta works,
 so we probably need to go through all our custom meta functions, fix the
 docblocks, and check other instances we might be handling type strict.
 This second part itself should be its own ticket, but I want confirmation
 that I've looked at this correctly first. :)

 Ugh. I feel like I specifically looked at this when migrating the meta
 functions to WP's. I wonder if xprofile is a special case, because it
 translates object_id. In any case, definitely worth a separate ticket.

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


More information about the buddypress-trac mailing list