[buddypress-trac] [BuddyPress] #4912: xprofile BP_XProfile_Field populate do not fill data

buddypress-trac noreply at wordpress.org
Sat Apr 6 22:15:15 UTC 2013


#4912: xprofile BP_XProfile_Field populate do not fill data
--------------------------+-----------------------------
 Reporter:  eggproject    |       Owner:
     Type:  defect (bug)  |      Status:  reopened
 Priority:  normal        |   Milestone:  Future Release
Component:  XProfile      |     Version:  1.2.8
 Severity:  normal        |  Resolution:
 Keywords:                |
--------------------------+-----------------------------
Changes (by r-a-y):

 * keywords:  reporter-feedback =>
 * status:  closed => reopened
 * version:  1.6.4 => 1.2.8
 * resolution:  invalid =>
 * milestone:   => Future Release


Comment:

 > BP_XProfile_Field does not have a populate() method.

 It does.  See:
 https://buddypress.trac.wordpress.org/browser/tags/1.7-rc1/bp-xprofile/bp-
 xprofile-classes.php#L453

 There's also an inline note denoting why we are even wiping out the
 `$user_id` variable:
 https://buddypress.trac.wordpress.org/browser/tags/1.7-rc1/bp-xprofile/bp-
 xprofile-classes.php#L456

 The good news is we never use the second or third parameters of
 `BP_XProfile_Field::populate()` internally.  So it looks like eggproject
 is doing something specific.

 We should perhaps look at rectifying this.

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/4912#comment:3>
BuddyPress <http://buddypress.org/>
BuddyPress


More information about the buddypress-trac mailing list