View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 12456 | Bug reports | Conditions | public | 2017-06-05 18:03 | 2018-02-15 12:03 |
| Reporter | sickpig | Assigned To | c_schmitz | ||
| Priority | none | Severity | partial_block | ||
| Status | closed | Resolution | reopened | ||
| Product Version | 2.64.x | ||||
| Fixed in Version | 3.0.x | ||||
| Summary | 12456: Conditional routing doesn't work on scale and array questions | ||||
| Description | Values of scale and array question is treated like string rather than array while evaluating routing conditions. This means that for a 1 to 10 scale, 4 < 10 evaluate to false. | ||||
| Steps To Reproduce | Q1 scale 1 to 20 Q2 has to be filled only if Q1 < 10 for every value of Q1 belonging to [2,9] the routing will not work as intented | ||||
| Additional Information | Use the attached lss file to reproduce the problem | ||||
| Tags | No tags attached. | ||||
| Attached Files | |||||
| Bug heat | 6 | ||||
| Complete LimeSurvey version number (& build) | 2.65.1+170522 | ||||
| I will donate to the project if issue is resolved | No | ||||
| Browser | chrome, firefox | ||||
| Database type & version | myysql 5.7, postresql 9.6 | ||||
| Server OS (if known) | Ubuntu Linux 16.04 | ||||
| Webserver software & version (if known) | Apache 2.4.7 | ||||
| PHP Version | 5.5.9 and 7 | ||||
|
Exactly same issue than https://bugs.limesurvey.org/view.php?id=12452 |
|
|
Sorry to say that but this is not the case. I have updated the instance to the latest release build 170606 and the problem is still there. |
|
|
you need to use the same fix applied to fix 12452 also for the scale question diff --git a/scripts/expressions/em_javascript.js b/scripts/expressions/em_javascript.js
|
|
|
parseFloat will also allow invalid numbers to become numbers. Just number() is better, I think. |
|
|
@c_schmitz: OK to test and fix it ? |
|
|
@c_schmitz number() is ok for me |
|
|
@c_schmitz bug is still there as of today (Version 2.67.2+170719). the patch above fix the problem (using number). Do you want me to submit a PR to the GH repo? |
|
|
Sure, please do. |
|
|
ok great, will do it today |
|
|
done. |
|
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2017-06-05 18:03 | sickpig | New Issue | |
| 2017-06-05 18:03 | sickpig | File Added: limesurvey_survey_123919.lss | |
| 2017-06-06 15:28 | DenisChenu | Relationship added | related to 12452 |
| 2017-06-06 16:35 | DenisChenu | Relationship replaced | duplicate of 12452 |
| 2017-06-06 16:36 | DenisChenu | Assigned To | => DenisChenu |
| 2017-06-06 16:36 | DenisChenu | Status | new => closed |
| 2017-06-06 16:36 | DenisChenu | Resolution | open => duplicate |
| 2017-06-06 16:36 | DenisChenu | Note Added: 43826 | |
| 2017-06-07 10:32 | sickpig | Status | closed => feedback |
| 2017-06-07 10:32 | sickpig | Resolution | duplicate => reopened |
| 2017-06-07 10:32 | sickpig | Note Added: 43843 | |
| 2017-06-07 11:11 | sickpig | Note Added: 43845 | |
| 2017-06-07 11:11 | sickpig | Status | feedback => assigned |
| 2017-06-07 11:17 | c_schmitz | Note Added: 43849 | |
| 2017-06-07 11:47 | DenisChenu | Assigned To | DenisChenu => c_schmitz |
| 2017-06-07 11:47 | DenisChenu | Note Added: 43851 | |
| 2017-06-07 11:57 | sickpig | Note Added: 43853 | |
| 2017-07-19 12:07 | sickpig | Note Added: 44109 | |
| 2017-07-19 14:00 | c_schmitz | Note Added: 44111 | |
| 2017-07-19 14:23 | sickpig | Note Added: 44112 | |
| 2017-07-19 17:43 | sickpig | Note Added: 44118 | |
| 2018-02-15 12:03 | c_schmitz | Status | assigned => closed |
| 2018-02-15 12:03 | c_schmitz | Fixed in Version | => 3.0.x |