xprofile BP_XProfile_Field populate do not fill data

[eggproject]

$tmp = new BP_XProfile_Field(FIELDID,USERID);

do not fill auto with data...

function populate .... remove $user_id = 0;!

[boonebgorges]

Can you please be more specific about what your problem is? I do not understand what you're trying to say here. (BP_XProfile_Field does not have a populate() method.)

[johnjamesjacoby]

+1 to Boone's assessment. Closing as invalid for now. Reopen if/when you can attach more details.

[r-a-y]

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.

[eggproject]

about i use...

$cfield = new BP_XProfile_Field($kitKeresekFieldId,$cuserId);
$cvalue = maybe_unserialize($dfield->data->value);

original code to the constructor is not charged in $ data ....

function construct( $id = null, $user_id = null, $get_data = true ) {

if ( !empty( $id ) )

$this->populate( $id, $user_id, $get_data );

}

call populate... my code fill this object ...

[boonebgorges]

My bad, I was misreading the code. Let's check the logs to see why we're wiping out $user_id here. Since, as r-a-y notes, we're not using the $user_id param anywhere in BP, this should be pretty harmless to fix (though we may want to check the plugin repo to see if anyone is taking advantage of this mysterious bug in a plugin).

[boonebgorges]

Looks like the bug was inadvertantly introduced in r3369, in an attempt to clear some WP_DEBUG warnings. I'm going to go ahead and revert it.

[boonebgorges]

In 7117:

Don't wipe out the user_id param passed to BP_XProfile_Field::populate()

This fixes a bug introduced in r3369 in an attempt to remove a WP_DEBUG
notice, but inadvertantly prevented plugins from directly calling the
populate() method using the user_id param.

Fixes #4912