Forum Replies Created
-
AuthorPosts
-
May 23, 2017 at 8:56 pm in reply to: username not setting unique. Allowing people to steal accounts #8311Cole GeissingerMember
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/
Cole GeissingerMemberTesting 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 functiongetUserID()
. I would set that toUserMeta\User
so that way you access it by adding below the namespace declarationuse \UserMeta\User;
and then change line 45 toUser\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.
Cole GeissingerMemberI’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…
- This reply was modified 8 years, 8 months ago by Cole Geissinger.
-
AuthorPosts