View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
11516 | Bug reports | Survey taking | public | 2016-08-02 11:57 | 2016-08-10 14:45 |
Reporter | vkuzmin | Assigned To | |||
Priority | urgent | Severity | minor | ||
Status | closed | Resolution | fixed | ||
Product Version | 2.50.x | ||||
Fixed in Version | 2.50.x | ||||
Summary | 11516: Sliders (Multiple numerical input). It's possible continue without answering to mandatory question. | ||||
Description | There's special hidden field to check user activity, for example: When respondent change slider value first time special hidden field value is set to 0. Limesurvey check that field and if its value is "1" respondent will get an error and should set any value next time. But after submitting when survey is shown again with error message ALL that field now have default "0" value instead of correct values from last try. So respondent can click Submit button twice and continue without answering. | ||||
Steps To Reproduce | 1) Make "Multiple numerical input" question. | ||||
Tags | No tags attached. | ||||
Attached Files | |||||
Bug heat | 10 | ||||
Complete LimeSurvey version number (& build) | 160728 | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | Firefox, Chrome | ||||
Database type & version | mysqlnd 5.0.10 - 20111026 | ||||
Server OS (if known) | Windows 10 | ||||
Webserver software & version (if known) | Apache/2.4.6 (Win32) | ||||
PHP Version | PHP/5.4.17 | ||||
Step 3 should be: |
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=19961 |
|
Thank you a lot! Works: Cause new bugs: |
|
Uploaded some screenshoots with your changes and two versions of bootstrap-slider.js |
|
new 9.1.1 verson of bootstrap-slider.js cause bugs. Need more changes or roll back to the bootstrap-slider.js v. 7.0.1. |
|
so after clarification: it seems that there the commit : is mixing the update and the hack. As said in the comment, please separate the operations in two commits so we can keep a clear trace of the hack. If the hack can be avoid: better. |
|
After closer examination and just to clarify, the old 7.1.1 bootstrap slider states that it cannot work with non-numeric values. Which as it seems in not correct. I cannot really reproduce the issue, so you would be so kind to show me the output of your debug console, though I could see where the error occurs in your system. |
|
Fresh installation of Limesurvey from Github without any changes. Last commit: https://github.com/LimeSurvey/LimeSurvey/commit/457cdbac944edd72dde297caf245c7cca0963101 1) Old chrome on win 7 (Chrome/38.0.2125.111) https://aboutmybrowser.com/MvMuXTt2 2) Yandex browser on win 7 (Chrome/51.0.2704.106 YaBrowser/16.7.0.3342) https://aboutmybrowser.com/wjio0Lp5 4) Firefox 48 on win 10 https://aboutmybrowser.com/dIQbxS82 5) Firefox ESR 45 on win 10 https://aboutmybrowser.com/I2ldzrCm Later I'll try to run LimeSurvey on php 7 instead of 5.4. And another bug with sliders (i'l make new issue later becouse current one is more important): |
|
https://github.com/seiyria/bootstrap-slider/releases bootstrap-slider.css: This issue can be closed now and i'l make new one about predefined slider position. |
|
I have updated the css to use the latest github version |
|
Version 2.50+ Build 160810 released |
|
LimeSurvey: master 9bd89f30 2016-08-04 16:53
Details Diff |
Fixed issue 11516: Sliders (Multiple numerical input). It's possible continue without answering to mandatory question |
Affected Issues 11516 |
|
mod - application/helpers/qanda_helper.php | Diff File | ||
mod - application/views/survey/questions/multiplenumeric/rows/sliders/answer_row.php | Diff File | ||
mod - third_party/bootstrap-slider/bootstrap-slider.js | Diff File | ||
mod - third_party/bootstrap-slider/bootstrap-slider.min.js | Diff File |
Date Modified | Username | Field | Change |
---|---|---|---|
2016-08-02 11:57 | vkuzmin | New Issue | |
2016-08-02 12:15 | vkuzmin | Issue Monitored: vkuzmin | |
2016-08-02 12:21 | vkuzmin | Note Added: 40164 | |
2016-08-02 12:22 | vkuzmin | File Added: limesurvey_survey_368443.lss | |
2016-08-04 16:54 |
|
Status | new => resolved |
2016-08-04 16:54 |
|
Fixed in Version | => 2.5 |
2016-08-04 16:54 |
|
Resolution | open => fixed |
2016-08-04 16:54 |
|
Assigned To | => markusfluer |
2016-08-04 16:59 |
|
Changeset attached | => LimeSurvey master 9bd89f30 |
2016-08-04 16:59 |
|
Note Added: 40171 | |
2016-08-05 13:54 | vkuzmin | Note Added: 40191 | |
2016-08-05 14:02 | vkuzmin | File Added: Fix-with-new-js.png | |
2016-08-05 14:02 | vkuzmin | File Added: Fix-with-old-js1.png | |
2016-08-05 14:02 | vkuzmin | File Added: Fix-with-old-js2.png | |
2016-08-05 14:06 | vkuzmin | Note Added: 40192 | |
2016-08-05 14:16 | vkuzmin | Note Added: 40193 | |
2016-08-05 14:16 | vkuzmin | Status | resolved => feedback |
2016-08-05 14:16 | vkuzmin | Resolution | fixed => reopened |
2016-08-05 15:01 |
|
Note Added: 40195 | |
2016-08-05 15:02 |
|
Priority | none => urgent |
2016-08-05 15:10 |
|
Note Edited: 40195 | |
2016-08-08 10:26 |
|
Note Added: 40198 | |
2016-08-08 16:11 | vkuzmin | Note Added: 40205 | |
2016-08-08 16:11 | vkuzmin | Status | feedback => assigned |
2016-08-08 17:11 | vkuzmin | Note Edited: 40205 | |
2016-08-08 17:22 | vkuzmin | Note Edited: 40205 | |
2016-08-08 17:22 | vkuzmin | Note Edited: 40205 | |
2016-08-09 11:45 | vkuzmin | Note Added: 40213 | |
2016-08-09 11:46 | vkuzmin | Note Edited: 40213 | |
2016-08-09 11:53 | vkuzmin | Note Edited: 40213 | |
2016-08-09 11:53 | vkuzmin | Note Edited: 40213 | |
2016-08-09 11:53 | vkuzmin | Note Edited: 40213 | |
2016-08-09 11:54 | vkuzmin | Note Edited: 40213 | |
2016-08-09 11:54 | vkuzmin | Note Edited: 40213 | |
2016-08-09 11:55 | vkuzmin | Note Edited: 40213 | |
2016-08-09 12:51 | vkuzmin | Note Edited: 40213 | |
2016-08-09 12:51 | vkuzmin | Note Edited: 40213 | |
2016-08-09 13:11 |
|
Note Added: 40214 | |
2016-08-09 13:11 |
|
Status | assigned => resolved |
2016-08-09 13:11 |
|
Resolution | reopened => fixed |
2016-08-10 14:45 | c_schmitz | Note Added: 40226 | |
2016-08-10 14:45 | c_schmitz | Status | resolved => closed |
2021-08-05 06:03 | guest | Bug heat | 8 => 10 |