search

chnm / thatcamp-registrations

Easily create an event registration on your multisite-enabled WordPress instance.
http://thatcamp.org
GNU General Public License v3.0
7 stars 3 forks source link

Fix for #32 #34

Closed boonebgorges closed 12 years ago

boonebgorges commented 12 years ago

It looks like your issue #32 is not caused by my recent changes - it's a coincidence that you found it now.

Your guess that it's caused by people using similar email addresses is correct. The plugin is set up to create users with user_logins that are generated from the part of the email address before the @. There was nothing built into the plugin that checked for duplicates, and when attempt to create a duplicate user, WP spits out an error, and the plugin was not set up to check for these errors.

I added a little logic to do numeric iteration. If amandafrench is taken, try amandafrench1; if that's taken, try amandafrench2; and so on.

I should note that I'm not 100% sure that this is the extent of the problem and solution. However, it does fix the specified issue. Please give it a go and let me know what you find out.

amandafrench commented 12 years ago

Sounds great. I guess now that we're auto-creating so many users, the duplicate user thing is an issue. I'll test it. Thanks, yo.

amandafrench commented 12 years ago

Seems to work just great. Thanks for fixing that -- that should be all for now, since all other issues can wait.

boonebgorges commented 12 years ago

Whoop!

On 2/23/12 1:15 PM, Amanda French wrote:

Seems to work just great. Thanks for fixing that -- that should be all for now, since all other issues can wait.

Reply to this email directly or view it on GitHub: https://github.com/chnm/thatcamp-registrations/pull/34#issuecomment-4141812