After the last dev-update I noticed that my new users get the "admin"-badge assigned (not the role, just the badge) when they regiserter. When I make a change to the wrongly assigned "admin"-badge, the users badge is corrected to "member"...

Any idea what I may look for in order to resolve this?

CommentFileSizeAuthor
#1 user_badges-1092920.patch676 bytesvkareh

Comments

vkareh’s picture

vkareh commented
StatusFileSize
new user_badges-1092920.patch676 bytes

Having a similar error, realized that the function that returns the role badges is defining a boolean value as a string ('FALSE', which is processed as TRUE). Changing that fixed it for me.

Attached is a patch.

vkareh’s picture

vkareh commented
Status: Active » Needs review

changing status

hlopes’s picture

hlopes commented

Same here.

vkareh’s picture

vkareh commented

Did the patch fix it for you or are you still having the problem?

hlopes’s picture

hlopes commented

Don't know yet, but as soon as i find out, i'll report it here.

hlopes’s picture

hlopes commented

Yep, everything's ok now.

Just a side note, the issue seems to occur whenever a user changes roles.

This patch can be changed to "reviewed by the community" or "patch to be ported", IMHO.

Thanks

vkareh’s picture

vkareh commented
Status: Needs review » Reviewed & tested by the community

Thanks.
Changing status.

tinatz’s picture

tinatz commented

Can I ask ? How can I use the patch ?

Do I only have to paste it on the .module ?
Sorry I just don't know.
Please Reply. :)

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented
Priority: Critical » Major

http://drupal.org/node/620014

The patch may be major, but it is not keeping your site down, so it is not critical.

tinatz’s picture

tinatz commented

whats with this error in cygwin ?
"bash : patch: command not found.

im on the module directory

vkareh’s picture

vkareh commented

That means that you don't have the program to patch files installed. Run your cygwin setup file and make sure you select "patch" from the included software list.

Disclaimer: I have not used cygwin in many years, so it might have changed in how it does this.

ckng’s picture

ckng
Primary language English
commented

I would actually upgrade this to a security, access bypass priority.
Anyway patch #1 solved it.

shabana.navas’s picture

shabana.navas commented
Status: Reviewed & tested by the community » Closed (fixed)