February 29, 2016 at 10:07 pm #7768
I am using UMP on a multisite install. I see the option under Registration in settings that “Allow user registration if user already exists in any of the other sites in the network”. However, I get kicked back an error saying “Sorry, that email address is already used!” when that user email is registered in another site in the network.
Isn’t this what that option is supposed to fix? Can’t help but feel this is broken.
Any thoughts what I could be doing wrong?March 4, 2016 at 8:12 pm #7776KhaledMember
Adding existing user to blog is working with username. For email also, please try our dev version.
Thanks.April 8, 2016 at 10:29 pm #7792
I’ll test the development version, but on the latest version it’s not working. Any idea when the latest changes will be rolled out? I’m not a fan of throwing the dev version on a website that sees millions of hits in a week…
April 11, 2016 at 8:53 pm #7794
- This reply was modified 5 years, 9 months ago by Cole Geissinger.
Testing the dev version I get a fatal error when trying to register with an existing user
Fatal error: Call to undefined function UserMeta\UserMeta\getUserID() in /srv/www/htdocs/wp-content/plugins/user-meta/models/pro/umMultisiteUserInsert.php on line 45
Notice how PHP is trying to access the namespace?
UserMeta\UserMeta\getUserID(). Looks like you are targeting the namespacing wrong. I was able to fix this by changing line 45 to
$userID = \UserMeta\getUserID($userData);
If you don’t add that backslash to the beginning, you aren’t targeting the global namespace,
UserMeta, so PHP is doubling the namespace like we see. IMO, you should consider breaking up the various sections into deeper levels. Tatke this helper function
getUserID(). I would set that to
UserMeta\Userso that way you access it by adding below the namespace declaration
use \UserMeta\User;and then change line 45 to
User\getUserID(). It’s a bit more architecturally sound and informative of what you are accessing in the name space.
Over all, I see this issue is working (thus far) in the dev version. Still unsure of pushing a development version up on a high traffic website, but the fact this feature is an option in the latest stable but yet it doesn’t work is a huge blocker. I am paying for this plugin, so hoping to see a release soon.April 11, 2016 at 9:11 pm #7796
Never mind, this issue is not resolved. Dev version is broken. See here http://user-meta.com/forums/topic/several-breaking-issues-with-dev-version/May 21, 2016 at 2:50 pm #7817KhaledMember
We have applied fixes into dev version. Recently we applied namespace over almost whole plugin. Some lines of code missed backslash unintentionally.
One of the realise candidate will be rolled out soon.
- You must be logged in to reply to this topic.