View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update | 
|---|---|---|---|---|---|
| 10883 | Bug reports | Other | public | 2016-04-07 11:20 | 2016-04-07 23:55 | 
| Reporter | maxoumat | Assigned To | ollehar | ||
| Priority | high | Severity | crash | ||
| Status | closed | Resolution | fixed | ||
| Product Version | 2.50.x | ||||
| Fixed in Version | 2.50.x | ||||
| Summary | 10883: Error 500 survey permission | ||||
| Description | when you want to change the permissions of a survey, we have an 500 page error : /admin/surveypermission/sa/view/surveyid/33364  | ||||
| Tags | No tags attached. | ||||
| Bug heat | 8 | ||||
| Complete LimeSurvey version number (& build) | 160405 | ||||
| I will donate to the project if issue is resolved | No | ||||
| Browser | |||||
| Database type & version | mysql 5.5 | ||||
| Server OS (if known) | debian | ||||
| Webserver software & version (if known) | Apache 2 | ||||
| PHP Version | 5.5 | ||||
| 
	 Could you attach the relevant survey, please?  | 
|
| 
	 Or enable debug=2 in config.php, reload and send us the error message on screen.  | 
|
| 
	 Fatal error: Can't use method return value in write context in /var/www/application/controllers/admin/surveypermission.php on line 596  | 
|
| 
	 That line is in the delete function. So the error appears when you delete rights?  | 
|
| 
	 Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=18374  | 
|
| 
	 sorry, one question: in string check for Gid param, but set Uid param is it right?  | 
|
| 
	 Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=18375  | 
|
| 
	 Could you pull and check again?  | 
|
| 
	 $postusergroupid is fixed in last commit.  | 
|
| 
	 error is the same... $postuserid = !(App()->getRequest()->getParam('uid')) ? App()->getRequest()->getParam('uid') : false; it's ok now  | 
|
| 
	 Is it OK or the same error?  | 
|
| 
	 better i think : $postuserid = (App()->getRequest()->getParam('uid')) ? App()->getRequest()->getParam('uid') : false;  | 
|
| 
	 no error add user : ok  | 
|
| 
	 You can check the commit: 
 Great, I'll mark it as resolved!  | 
|
| 
	 Version 2.50 Build 160407 released  | 
|
| 
	
	 LimeSurvey: master 98e736d2 2016-04-07 13:39 Details Diff  | 
Fixed issue 10883: Error 500 survey permission | 
Affected Issues 10883  | 
|
| mod - application/controllers/admin/surveypermission.php | Diff File | ||
| 
	
	 LimeSurvey: master f0efa6a6 2016-04-07 14:01 Details Diff  | 
Fixed issue 10883: Error 500 survey permission | 
Affected Issues 10883  | 
|
| mod - application/controllers/admin/surveypermission.php | Diff File | ||
| Date Modified | Username | Field | Change | 
|---|---|---|---|
| 2016-04-07 11:20 | maxoumat | New Issue | |
| 2016-04-07 12:57 | ollehar | Note Added: 37056 | |
| 2016-04-07 12:57 | ollehar | Note Added: 37057 | |
| 2016-04-07 12:58 | ollehar | Assigned To | => ollehar | 
| 2016-04-07 12:58 | ollehar | Status | new => acknowledged | 
| 2016-04-07 12:58 | ollehar | Status | acknowledged => feedback | 
| 2016-04-07 13:23 | asch | Note Added: 37059 | |
| 2016-04-07 13:37 | ollehar | Note Added: 37060 | |
| 2016-04-07 13:40 | ollehar | Changeset attached | => LimeSurvey master 98e736d2 | 
| 2016-04-07 13:40 | ollehar | Note Added: 37061 | |
| 2016-04-07 13:40 | ollehar | Resolution | open => fixed | 
| 2016-04-07 14:01 | asch | Note Added: 37062 | |
| 2016-04-07 14:01 | ollehar | Changeset attached | => LimeSurvey master f0efa6a6 | 
| 2016-04-07 14:01 | ollehar | Note Added: 37063 | |
| 2016-04-07 14:08 | ollehar | Note Added: 37064 | |
| 2016-04-07 14:09 | ollehar | Note Added: 37065 | |
| 2016-04-07 14:17 | maxoumat | Note Added: 37066 | |
| 2016-04-07 14:17 | maxoumat | Status | feedback => assigned | 
| 2016-04-07 14:20 | c_schmitz | Priority | none => high | 
| 2016-04-07 14:20 | c_schmitz | Description Updated | |
| 2016-04-07 14:26 | ollehar | Note Added: 37071 | |
| 2016-04-07 14:30 | maxoumat | Note Added: 37074 | |
| 2016-04-07 14:30 | maxoumat | Note Added: 37075 | |
| 2016-04-07 14:38 | ollehar | Note Added: 37077 | |
| 2016-04-07 14:38 | ollehar | Status | assigned => resolved | 
| 2016-04-07 14:38 | ollehar | Fixed in Version | => 2.5 | 
| 2016-04-07 23:55 | c_schmitz | Note Added: 37106 | |
| 2016-04-07 23:55 | c_schmitz | Status | resolved => closed |