View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 18313 | Bug reports | User / Groups / Roles | public | 2022-08-16 12:04 | 2026-10-31 02:10 |
| Reporter | el_grom | Assigned To | c_schmitz | ||
| Priority | none | Severity | minor | ||
| Status | resolved | Resolution | fixed | ||
| Product Version | 5.3.x | ||||
| Summary | 18313: Permissions via drop-down are not shown | ||||
| Description | Hi, On the user management page, if you click on the action button "edit permissions" of any user, you'll see his permission. | ||||
| Steps To Reproduce | Steps to reproduceGo to the user management page (logged as superadmin or anything that has sufficient permissions) Expected resultWe expect that the checkboxes are checked according to the effective permissions the user has. Actual resultAll the checkboxes are unchecked. Like if the user has no permission at all | ||||
| Tags | No tags attached. | ||||
| Bug heat | 6 | ||||
| Complete LimeSurvey version number (& build) | 5.3.30+220808 | ||||
| I will donate to the project if issue is resolved | No | ||||
| Browser | Chrome/Edge | ||||
| Database type & version | Azure SQL DB | ||||
| Server OS (if known) | Windows 2019 | ||||
| Webserver software & version (if known) | Apache | ||||
| PHP Version | 7.4.29 | ||||
| Git branch / Pull Request URL | |||||
|
Thanks. Bug received. |
|
I believe these expectations are not that well. If multiple users have different permissions, how shall we set checkboxes? |
|
|
Hello, Good point. If the target of this drop-down is to be used for multiple item, I sugget to make it grey (/unusable) until 2 or more items are selected. |
|
|
Additional note, due to issue 18316 which makes us use the drop-down for one item, this reinforce the feeling about the 1-item + drop-down list gives the same result as the action button |
|
Sorry, got lost. Which name?
Agree. No matter how many users you select, mass permission setting it is always showing the checkboxes blanked.
Could be 1 or many. Usually many, but there is no restriction
From your experience, what would the banner should say? Thanks! |
|
|
(Sorry, I don't know how to quote)
|
|
@c_schmitz what do yo think about this? |
|
|
A very usual design pattern for this situation would be to show the normal permission selection screen, but where the user permissions are different among the selected users, the related box(es) should be greyed out. |
|
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=41184 |
|
|
LimeSurvey: master b339213a 2026-10-31 02:09 Details Diff |
Fixed issue 18313: Edit permissions for several selected users showed all permissions unchecked and saving removed every permission that was not ticked Dev The mass edit dialog loads the permissions of the selected users: Ticked if all have it, a dash if only some have it (clicking cycles ticked, unticked, dash) Dev Saving only changes the permissions changed in the dialog, new userManagement/batchPermissionStates endpoint, applyPermissionFromArray() has a new $onlyGiven parameter |
Affected Issues 18313 |
|
| mod - application/controllers/UserManagementController.php | Diff File | ||
| mod - application/views/userManagement/massiveAction/_updatepermissions.php | Diff File | ||
| mod - assets/packages/usermanagement/js/usermanagement.js | Diff File | ||
| add - tests/unit/controllers/UserManagementBatchPermissionsTest.php | Diff File | ||
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2022-08-16 12:04 | el_grom | New Issue | |
| 2022-08-16 16:17 | gabrieljenik | Assigned To | => gabrieljenik |
| 2022-08-16 16:17 | gabrieljenik | Status | new => acknowledged |
| 2022-08-16 16:17 | gabrieljenik | Note Added: 71443 | |
| 2022-08-16 16:17 | gabrieljenik | Bug heat | 0 => 2 |
| 2022-08-17 15:33 | gabrieljenik | Status | acknowledged => feedback |
| 2022-08-17 15:33 | gabrieljenik | Note Added: 71466 | |
| 2022-08-18 08:39 | el_grom | Note Added: 71473 | |
| 2022-08-18 08:39 | el_grom | Bug heat | 2 => 4 |
| 2022-08-18 08:39 | el_grom | Status | feedback => assigned |
| 2022-08-18 08:44 | el_grom | Note Added: 71474 | |
| 2022-08-18 19:44 | gabrieljenik | Note Added: 71486 | |
| 2022-08-18 19:44 | gabrieljenik | Status | assigned => feedback |
| 2022-08-19 09:14 | el_grom | Note Added: 71489 | |
| 2022-08-19 09:14 | el_grom | Status | feedback => assigned |
| 2022-08-19 14:56 | gabrieljenik | Note Added: 71498 | |
| 2022-08-19 14:56 | gabrieljenik | Status | assigned => feedback |
| 2022-08-19 15:00 | c_schmitz | Note Added: 71500 | |
| 2022-08-19 15:00 | c_schmitz | Bug heat | 4 => 6 |
| 2022-08-19 15:00 | c_schmitz | Note Edited: 71500 | |
| 2026-10-31 02:10 | c_schmitz | Changeset attached | => LimeSurvey master b339213a |
| 2026-10-31 02:10 | c_schmitz | Note Added: 87033 | |
| 2026-10-31 02:10 | c_schmitz | Assigned To | gabrieljenik => c_schmitz |
| 2026-10-31 02:10 | c_schmitz | Status | feedback => resolved |
| 2026-10-31 02:10 | c_schmitz | Resolution | open => fixed |