View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
07881 | Bug reports | RemoteControl | public | 2013-05-27 06:10 | 2013-10-23 14:40 |
Reporter | allinonesoftware | Assigned To | trougakoss | ||
Priority | normal | Severity | minor | ||
Status | closed | Resolution | fixed | ||
Product Version | 2.00+ | ||||
Fixed in Version | 2.05 RC | ||||
Summary | 07881: Only survey owner has access to the survey | ||||
Description | In the LimeSurvey backend it is possible to grant another user access to a survey. Therefore when logging into the remote control with user B who was granted access (but isn't owner) to survey A, will only return survey B | ||||
Steps To Reproduce |
| ||||
Tags | No tags attached. | ||||
Bug heat | 10 | ||||
Complete LimeSurvey version number (& build) | 130514 | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | |||||
Database type & version | MySQL 5.1.41 | ||||
Server OS (if known) | Ubuntu 10.04 | ||||
Webserver software & version (if known) | Apache 2.2.14 | ||||
PHP Version | 5.3.2 | ||||
Hi muhzak, do you need any further help with solving this? When can we expect a fix? |
|
trougakoss, I am assigning this one to you since it is related to the API working differently than the Limesurvey admin backend. Let me know if you have any further questions or need any more details. |
|
I ll give you a response as soon as possible. |
|
The remotecontrol in list_surveys, lists only the surveys that belong to the user. That means that there is no check being made for surveys that the user in question has some privileges on. This of course is different from the behaviour of the Limesurvey admin backend and if we think that the behaviour should be exactly the same, i will make the appropriate changes. |
|
I assume "The remotecontrol in list_surveys, lists only the surveys that belong to the user" means that the user's UID is listed at the table listing the survey rights and IDs? Since a user might be assigned to a survey but not have assigned any rights, we should better query for certain survey rights as well, depending on the action. c_schmitz, what do you think? |
|
Hi, Think behaviours must be exacltly the same. list_surcey : show survey. no ? |
|
I agree to Mazi. If needed you will have to refactor the function behind the GUI survey list so it can be used by RemoteControl, too. |
|
So in other words the list_survey function will be defined like this By default the privileges are set to null so only the ownership will be checked (current behaviour). So in other words the current behaviour wont be changed, but we will accommodate for the extra needs. |
|
Almost. ;) |
|
So.... i did not understand correctly... :P |
|
Yes, you are correct. Thank you for your diligence, Spiros! |
|
Pull request #135 |
|
I assume you were not able to use the GUI function? Anyway, looks good - please merge. |
|
trougakoss, you will need to patch the 2.05 branch too. Can you do this please and then set this issue to resolved? Thank you! |
|
Carsten as you might saw i did not use the gui function because it did not provide me with the extra functionality that the RC has. |
|
2.05RC3 released. |
|
Date Modified | Username | Field | Change |
---|---|---|---|
2013-05-27 06:10 | allinonesoftware | New Issue | |
2013-05-27 14:33 | c_schmitz | Assigned To | => muhzak |
2013-05-27 14:33 | c_schmitz | Status | new => assigned |
2013-06-27 09:14 | Mazi | Note Added: 25675 | |
2013-06-27 09:15 | Mazi | Issue Monitored: trougakoss | |
2013-09-18 11:22 | Mazi | Assigned To | muhzak => trougakoss |
2013-09-18 11:23 | Mazi | Note Added: 26268 | |
2013-09-18 18:48 | trougakoss | Note Added: 26283 | |
2013-09-26 16:53 | trougakoss | Note Added: 26388 | |
2013-09-26 17:11 | Mazi | Note Added: 26389 | |
2013-09-26 17:11 | Mazi | Issue Monitored: c_schmitz | |
2013-09-26 17:50 | DenisChenu | Note Added: 26390 | |
2013-09-26 21:30 | c_schmitz | Note Added: 26391 | |
2013-09-27 21:40 | c_schmitz | Issue End Monitor: trougakoss | |
2013-09-29 16:49 | trougakoss | Note Added: 26435 | |
2013-09-29 17:04 | c_schmitz | Note Added: 26436 | |
2013-09-29 17:04 | c_schmitz | Note Edited: 26436 | |
2013-09-29 17:54 | trougakoss | Note Added: 26437 | |
2013-09-29 17:56 | c_schmitz | Note Added: 26438 | |
2013-10-01 12:40 | trougakoss | Note Added: 26455 | |
2013-10-07 21:46 | c_schmitz | Note Added: 26606 | |
2013-10-07 21:46 | c_schmitz | Note Edited: 26606 | |
2013-10-07 21:46 | c_schmitz | Note Edited: 26606 | |
2013-10-09 09:45 | c_schmitz | Changeset attached | => LimeSurvey master 3c147910 |
2013-10-09 09:46 | c_schmitz | Note Added: 26641 | |
2013-10-09 09:46 | c_schmitz | Status | assigned => feedback |
2013-10-09 17:22 | trougakoss | Note Added: 26673 | |
2013-10-11 12:42 | trougakoss | Status | feedback => resolved |
2013-10-11 12:42 | trougakoss | Fixed in Version | => 2.05 RC |
2013-10-11 12:42 | trougakoss | Resolution | open => fixed |
2013-10-23 14:40 | c_schmitz | Note Added: 26957 | |
2013-10-23 14:40 | c_schmitz | Status | resolved => closed |
2021-08-02 16:09 | guest | Bug heat | 8 => 10 |