Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.The title says it all, see attached patch.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | roleswitch.patch_1.txt | 4.93 KB | killes@www.drop.org |
| #1 | roleswitch.patch_0.txt | 4.24 KB | killes@www.drop.org |
| roleswitch.patch.txt | 4.27 KB | killes@www.drop.org |
Comments
Comment #1
killes@www.drop.org commentedpatch contained a stray var-dump
Comment #2
marcp commentedNice patch. I noticed a small problem when moving a user to a role that s/he already belongs to (not something that would normally be done, but the way it fails is to not move the user out of the role they were expecting to be moved from):
It would be great if you fixed this -- then I'll commit the patch.
Ideally, I'd rather see a way so that we can use the "assign user roles" tab (admin/user/userplus/userperms) to make this new patch unnecessary. It can be done now, but it's a pain, if not impossible, with a lot of users to only see the ones that are currently in any role. If the "assign user roles" tab had a filter control at the top to allow the administrator to filter out users based on role, and if we added "check all" and "clear all" links or buttons to each column, then we'd pretty much be there in a more flexible way. What do you think?
- Marc
Comment #3
killes@www.drop.org commentedfixed patch attached. Also added some error checking.
This patch is different from what you propose. The main use is to quickly get several users _out_ of a role. It is used together with evaluation.module where it is desirable to remove the current students to another role at the end of the term.
Comment #4
killes@www.drop.org commentedand the patch...
Comment #5
marcp commentedThanks Gerhard -- I committed the patch with a little bit of change to the wording. Also the list of source roles is now in a table instead of a bullet separated list.
Comment #6
venkat-rk commentedThanks, Killes, for this great feature.
Comment #7
(not verified) commented