View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
17659 | Bug reports | User / Groups / Roles | public | 2021-10-12 09:37 | 2021-10-14 12:10 |
Reporter | Jmantysalo | Assigned To | galads | ||
Priority | none | Severity | minor | ||
Status | confirmed | Resolution | open | ||
Product Version | 5.x | ||||
Summary | 17659: Deleting a user that is owner of a survey group | ||||
Description | According to https://forums.limesurvey.org/forum/can-i-do-this-with-limesurvey/125344-mass-deletion-of-users the systems does not work properly when you delete a user that is owner of a survey group. | ||||
Tags | No tags attached. | ||||
Bug heat | 6 | ||||
Complete LimeSurvey version number (& build) | 5.x | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | |||||
Database type & version | N/A | ||||
Server OS (if known) | |||||
Webserver software & version (if known) | |||||
PHP Version | N/A | ||||
Need example survey and instructions on how to reproduce. |
|
Example survey? The bug is not about one survey but |
|
Example survey? The bug is not about one survey but survey group. To reproduce: Add a user. Add a survey group, set owner to be the user you just created. Delete the user. See attached picture, or the result in the DB: MariaDB [lime]> select gsid,title,owner_id from lime_surveys_groups; MariaDB [lime]> select uid from lime_users; |
|
@Jmantysalo : with debug set to 2 : you don't have error on SurveysGroups list ? |
|
Setting debug to 2 does not make any change. I can set new owner for a group, hence this is not so severe bug. Btw, application/controllers/UserManagementController.php already contains a line //todo REFACTORING user permissions should be deleted also ... (in table permissions) which could be done with this, I guess. |
|
great news !
Permission and owner are not the same thing in my opinion :)
|
|
Usually DB:s contain foreign keys... Those would force developers to think about things like this. A user can be referenced by 1) survey, 2) survey group, 3) permission and 4) label set. At least those, maybe more. |
|
Label set ? |
|
My bad. No owner for a label set. |
|
:) then : must check
Must delete when delete user
|
|
Date Modified | Username | Field | Change |
---|---|---|---|
2021-10-12 09:37 | Jmantysalo | New Issue | |
2021-10-12 10:54 | ollehar | Note Added: 66830 | |
2021-10-12 10:54 | ollehar | Bug heat | 0 => 2 |
2021-10-12 12:39 | Jmantysalo | Note Added: 66831 | |
2021-10-12 12:39 | Jmantysalo | Bug heat | 2 => 4 |
2021-10-12 12:42 | Jmantysalo | Note Added: 66832 | |
2021-10-12 12:42 | Jmantysalo | File Added: survey-group-no-owner.png | |
2021-10-13 09:07 | DenisChenu | Note Added: 66840 | |
2021-10-13 09:07 | DenisChenu | Bug heat | 4 => 6 |
2021-10-14 07:16 | Jmantysalo | Note Added: 66846 | |
2021-10-14 09:05 | DenisChenu | Note Added: 66847 | |
2021-10-14 11:05 | Jmantysalo | Note Added: 66848 | |
2021-10-14 11:12 | galads | Assigned To | => galads |
2021-10-14 11:12 | galads | Status | new => confirmed |
2021-10-14 11:43 | DenisChenu | Note Added: 66850 | |
2021-10-14 11:51 | Jmantysalo | Note Added: 66851 | |
2021-10-14 12:10 | DenisChenu | Note Added: 66852 |