Hi, I'm using BUEditor and have assigned some roles to a default editor. no overlapping roles for my users. I noticed that BUEditor doesn't show on node editing. Further investigations results in a wrong weight management for editors/roles.
The weight is sorted with some "unsigned" comparison, and for "authenticated" and "anonymous" roles weight isn't stored in BUEditor variables.
Regards
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | bueditor.admin_.inc_.patch | 772 bytes | ufku |
| #1 | bueditor-default_roles_weight_fix.patch | 1.25 KB | thepanz |
Comments
Comment #1
thepanz commentedFixed weight sorting function and storing DRUPAL_AUTHENTICATED_RID and DRUPAL_ANONYMOUS_RID into BUEditor variables.
Comment #2
ufku commentedbelow patch has been committed. thanks.
Comment #3
aleksey.tk commentedThe patch you commit doesn't fix anything. Actually the patch from thePanz (the first one) fixing this bug. In the commit comment you said that you applied patch from thePanz, but looking at the diff i can say that this is not right, you applied another patch you attached to last comment. Please check this out!
Comment #4
ufku commentedYou should submit the role-editor assignment form at least once. Then it should work as expected. Actually, without patching you could also achieve the desired behaviour using negative values.
Comment #5
aleksey.tk commentedI was talking about this commit:
This doesn't fix the bug. Please, check this out!
P.S. Sorry for my english, it is not my native language
Comment #6
aleksey.tk commentedHmm, looks like your patch fixing this issue. Sorry for trying to mislead you... but your commit comment was really confusing.
Comment #7
ufku commentedNice to hear it's working. With a few simplifications it's still thePanz's patch inside ;)