View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
12463 | Bug reports | Survey editing | public | 2017-06-09 11:44 | 2017-06-12 17:16 |
Reporter | sickpig | Assigned To | DenisChenu | ||
Priority | none | Severity | partial_block | ||
Status | closed | Resolution | fixed | ||
Product Version | 2.65.x | ||||
Fixed in Version | 2.65.x | ||||
Summary | 12463: "Quick add.." replace function does not work anymore | ||||
Description | For all questions where you could use the the above function (single list, array, etc etc) every time you get an alert error saying that an error happened in the ajax request... this an example of the URL LS fails to fetch via ajax | ||||
Steps To Reproduce | Install the latest version of lime survey | ||||
Additional Information | I have git bisected the limesurvey repo and the commit that introduced the issue is this one: commit 68815bf6acf995363fb260ff395bc620882d741d
reverting this change fix the problem. But of course something that we don't want to do due the security implications | ||||
Tags | No tags attached. | ||||
Bug heat | 4 | ||||
Complete LimeSurvey version number (& build) | Version 2.65.3 | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | chrome | ||||
Database type & version | postgresql 9.5.7 | ||||
Server OS (if known) | Ubuntu Linux 16.04 | ||||
Webserver software & version (if known) | Apache 2.4.7 | ||||
PHP Version | 7.0.18 | ||||
Maybe some isset VS empty. I thing we don't have to send empty param ( sid//gid for example). |
|
Ouo, you tell me the issue : {{quid_placeholder}} :/ |
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=22947 |
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=22948 |
|
Version 2.65.4 (build 170612) released. |
|
LimeSurvey: master 74eb67f4 2017-06-09 18:59:36 Details Diff |
Fixed issue 12463: "Quick add.." replace function does not work anymore Dev: qid is not used here, but fix it |
Affected Issues 12463 |
|
mod - scripts/admin/subquestions.js | Diff File | ||
LimeSurvey: master a45e4aa6 2017-06-09 19:38:52 Details Diff |
Dev: really fix 12463: "Quick add.." Dev: maybe accept new[0-9] for qid/gid/sid ? Dev: this force to create a new one ? |
Affected Issues 12463 |
|
mod - application/controllers/admin/questions.php | Diff File |
Date Modified | Username | Field | Change |
---|---|---|---|
2017-06-09 11:44 | sickpig | New Issue | |
2017-06-09 12:04 | DenisChenu | Assigned To | => DenisChenu |
2017-06-09 12:04 | DenisChenu | Status | new => assigned |
2017-06-09 12:06 | DenisChenu | Note Added: 43862 | |
2017-06-09 18:35 | DenisChenu | Note Added: 43864 | |
2017-06-09 19:03 | DenisChenu | Changeset attached | => LimeSurvey master 74eb67f4 |
2017-06-09 19:03 | DenisChenu | Note Added: 43865 | |
2017-06-09 19:03 | DenisChenu | Resolution | open => fixed |
2017-06-09 19:40 | DenisChenu | Changeset attached | => LimeSurvey master a45e4aa6 |
2017-06-09 19:40 | DenisChenu | Note Added: 43866 | |
2017-06-09 19:41 | DenisChenu | Status | assigned => resolved |
2017-06-09 19:41 | DenisChenu | Fixed in Version | => 2.65.x |
2017-06-12 17:16 | c_schmitz | Note Added: 43878 | |
2017-06-12 17:16 | c_schmitz | Status | resolved => closed |
2019-11-01 17:25 | c_schmitz | Category | Survey design => Survey editing |