Usernames frequently not unique when creating users about civicrm-wp-profile-sync HOT 7 CLOSED

@linssen Ah, I see - fair point. Yours is a deliberate override of the username rather than a last resort approach - though the end result is the same. I'll add a new filter where you suggest so as to avoid confusion for those taking a defensive approach.

Another thing that would make it even more useful would be to select more columns, thus the $rows param would have more to play with

Calls for another filter, methinks. Give me a couple of mins...

from civicrm-wp-profile-sync.

@linssen To be accurate, there's an extra attempt to create a user after step (3) where username = first + middle + last and if this too fails then the civicrm_wp_profile_sync_unique_username filter is applied. Can you explain how moving the filter would help avoid duplicate usernames?

from civicrm-wp-profile-sync.

Largely it would just mean a uniform approach to usernames for all contacts. For example, in our case we already have a field for a globally unique member id: we'd want to use that. With the plugin being called later on, we would have 90% of our users with the username firstname.lastname and 10% using their unique member id.

I suggest using it is an override hook at the beginning of the username generation.

Another thing that would make it even more useful would be to select more columns, thus the $rows param would have more to play with a * would be perfect in our case, since it would include all the data we'd want to build a totally unique username.

from civicrm-wp-profile-sync.

@linssen I've added some filters in e366a9b that should help you out. You can now filter the query, contact data rows and username.

from civicrm-wp-profile-sync.

Looks like exactly what I was hoping for, thanks! Worth adding to the readme?

from civicrm-wp-profile-sync.

@linssen I really don't want to do endless free support for non-technical folks on this - hence why the bulk constant is off by default. I'll review the readme, but I doubt I'll highlight this.

from civicrm-wp-profile-sync.

Fair enough. Really appreciate this, thanks @christianwach.

from civicrm-wp-profile-sync.