View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|18281||Bug reports||User / Groups / Roles||public||2022-07-27 11:04||2022-09-09 16:24|
|Summary||18281: Users in group are not deleted|
When you click the menu entry “user groups“ in the top menu configuration you get an error (page is not loading) Screenshot attached
|Steps To Reproduce|
Steps to reproduce
(Replace this text with detailed step-by-step instructions on how to reproduce the issue)
(Write here what you expected to happen)
(Write here what happened instead)
|Tags||No tags attached.|
|Complete LimeSurvey version number (& build)||5|
|I will donate to the project if issue is resolved||No|
|Database type & version||MySQL|
|Server OS (if known)|
|Webserver software & version (if known)|
58aefca2-a89d-4515-b277-d4cfdd7b17a9.png (278,676 bytes)
Can you provide more steps for rerproducing?
So the problem happens when you create a usergrp as a user and then delete the user, when you then go to the usergroups page the error is visible. The main problem is that the users dont get deleted at all from the tables "user_groups" and "user_in_groups".
Part of the problems had to do with the fact that the code assumed that if you were the owner you were in the group. In LTS it is like this, the owner is always in the group. But in master no.
My opinion : we must have a clean decision about $userGroup->owner_id == Yii::app()->user->id on the right part
I think we must have UserGroup->hasPermission with something like this
When decision is done about owner : we can easily add it here.
So, you wouldn't care about who owns the group, right?
Currently no, be cause we don't care in 3.X and in current 5.X.
Then : need decision about usage of owner_id before adding it.
See the last sentence
Not sure if we are talking about the same thing... I see it is being used a lot...
Are we talking about the same stuff?
No worries, I am not saying I will, I will not... Why would I?
Seems to didn't work with specific permission then …
Time to update to real Permission system …
Look forward for decisions and directions ...
|2022-07-27 11:04||p_teichmann||New Issue|
|2022-07-27 11:04||p_teichmann||File Added: 58aefca2-a89d-4515-b277-d4cfdd7b17a9.png|
|2022-07-27 11:05||p_teichmann||Assigned To||=> gabrieljenik|
|2022-07-27 11:05||p_teichmann||Status||new => assigned|
|2022-07-27 22:32||gabrieljenik||Note Added: 71204|
|2022-07-27 22:32||gabrieljenik||Bug heat||0 => 2|
|2022-07-27 22:32||gabrieljenik||Status||assigned => feedback|
|2022-08-04 15:26||gabrieljenik||Status||feedback => assigned|
|2022-08-04 15:26||gabrieljenik||Note Added: 71322|
|2022-08-04 15:28||gabrieljenik||Note Edited: 71322|
|2022-08-09 19:28||gabrieljenik||Assigned To||gabrieljenik => DenisChenu|
|2022-08-09 19:28||gabrieljenik||Status||assigned => ready for code review|
|2022-08-09 19:28||gabrieljenik||Note Added: 71391|
|2022-08-23 15:49||DenisChenu||Relationship added||child of 18289|
|2022-08-23 16:00||DenisChenu||Note Added: 71519|
|2022-08-23 16:00||DenisChenu||Bug heat||2 => 4|
|2022-08-23 16:00||DenisChenu||Note Edited: 71519|
|2022-08-23 16:30||DenisChenu||Assigned To||DenisChenu => gabrieljenik|
|2022-08-23 16:30||DenisChenu||Status||ready for code review => in code review|
|2022-08-23 17:17||gabrieljenik||Note Added: 71520|
|2022-08-24 08:16||DenisChenu||Note Added: 71522|
|2022-08-24 14:30||gabrieljenik||Note Added: 71525|
|2022-08-27 19:49||DenisChenu||Note Added: 71544|
|2022-08-29 15:17||gabrieljenik||Assigned To||gabrieljenik => c_schmitz|
|2022-08-29 15:17||gabrieljenik||Status||in code review => feedback|
|2022-08-29 15:17||gabrieljenik||Note Added: 71555|
|2022-09-09 16:24||DenisChenu||Relationship added||related to 18294|