Filter data not synced

Hey @roadsunknown ,

That’s an interesting issue. Thanks for bringing it up.

Yes we do sync the unfiltered $_POST data during a profile update. That ended up giving us the best compatibility with other plugins, as we don’t always know where the profile_update is being triggered from, and whether or not the POST’ed values have been saved to the database yet.

But I can see how that doesn’t work for your scenario.

What I’ve done is in the push_user_meta() function in WPF_User I’ve added

foreach ( $user_meta as $field => $value ) {
	$user_meta[ $field ] = apply_filters( "pre_user_{$field}", $value );
}

so that any data being sent to the CRM will be passed through the pre_user_ filter. That’s working now with your pre_user_first_name example.

I think that will be reliable… Unless you can think of a better idea?

Hi @verygoodplugins. I understand why you use unfiltered to optimize compatibility. Using this method however would mean any other custom filters applied by any other plugin would not be compatible.

It’s a compromise but I can work with it for my use case since WordPress core does provide pre_ filters for the fields I need before sync: first_name, last_name and user_email. (I do apply filters to other fields but in my use case I do not sync these.)

https://developer.www.ads-software.com/reference/hooks/pre_user_email/

Alternatively, checking if matches database and/or sync after save to database for new users via https://developer.www.ads-software.com/reference/hooks/user_register/ or updated users: https://developer.www.ads-software.com/reference/hooks/profile_update/ (which there is reference too but is removed?).

Thanks for the help.

Hey @roadsunknown ,

Well… it’s an interesting problem. I’m not sure our solution is the best one.

We do actually hook into profile_update: https://github.com/verygoodplugins/wp-fusion-lite/blob/master/includes/class-user.php#L249. But it doesn’t tell us which fields were updated.

We could at that point make a call to get_user_meta() (https://github.com/verygoodplugins/wp-fusion-lite/blob/master/includes/class-user.php#L441) to get all the data that’s in the database for the user. That would ensure that everything has passed through any relevant filters already.

But:

a.) We don’t know that every plugin has saved everything at that point
b.) The idea was to only sync the data to the CRM that has *just* been updated. For performance and simplicity reasons. If you have 100 fields enabled for sync we wouldn’t want to send all of them just because someone updated their billing address.

So then I thought about array_intersect-ing the $_POST data with the data from the database to get the filtered values from the DB *only* for the fields that were POST’ed.

But the field keys also sometimes change between the POST data and the database. For example on user-edit.php in the admin, the input is “url”, but it’s saved to the DB as “user_url”.

So *then* I thought, instead of using the user_update action, we could use the insert_user_meta filter to get only the values that were being changed. But it felt weird to me to attach functional code to a filter… an action is more appropriate.

Any thoughts?

Also do note we have our own filter that lets you modify just data going to the CRM, in case it helps: https://wpfusion.com/documentation/filters/wpf_user_update/

Jack

Hey @verygoodplugins / Jack,

Thanks for the continued thoughts on this. Perhaps the simplest solution for this use case is to hook into the wpf_user_update and wpf_user_register filters as you suggest. This does require duplicating the code but its only a few simple lines. I’ve tested and deployed this and all appears to be working as expected. If you do decide to make any more complex architectural changes do let me know but for now we’re at least getting cleaner data into the CRM ??

// WP Fusion Sync - Raw Form Data is Synced so Format (again)
function wpf_format_user($user_meta, $user_id) {
	if( isset( $user_meta['first_name'] ) ) {
		$user_meta['first_name'] = (ctype_upper(str_replace(' ', '', $user_meta['first_name'])) ? ucwords(strtolower($user_meta['first_name'])) : ucwords($user_meta['first_name'])); 
	}
	if( isset( $user_meta['last_name'] ) ) {
		$user_meta['last_name'] = (ctype_upper(str_replace(' ', '', $user_meta['last_name'])) ? ucwords(strtolower($user_meta['last_name'])) : ucwords($user_meta['last_name']));
	}
	if( isset( $user_meta['email'] ) ) {
		$user_meta['email'] = strtolower(sanitize_email($user_meta['email']));
	}
	return $user_meta;
}
add_filter('wpf_user_update', 'wpf_format_user', 10, 2);
add_filter('wpf_user_register', 'wpf_format_user', 10, 2);