View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
10860 | Bug reports | Other | public | 2016-04-04 08:58 | 2018-09-28 14:12 |
Reporter | fvanderstarre | Assigned To | DenisChenu | ||
Priority | none | Severity | minor | ||
Status | closed | Resolution | fixed | ||
Product Version | 2.06+ | ||||
Fixed in Version | 2.50.x | ||||
Summary | 10860: Regular expression fail (2.06+) | ||||
Description | I have a short text question where I want people to enter "+" or "-" followed by a number (three digits max), or zero. The regular expression I built to validate works OK when I test it on https://regex101.com/, but not in LimeSurvey it seems. There, any number entered after "+" except zero(es), is rejected. | ||||
Steps To Reproduce | Create short text question with input box width 4 and max chars 4. | ||||
Tags | No tags attached. | ||||
Attached Files | |||||
Bug heat | 4 | ||||
Complete LimeSurvey version number (& build) | 2.06+ build 160129 | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | Firefox/Chrome | ||||
Database type & version | MS SQL server 2008R2 10.50.6220 (X64) | ||||
Server OS (if known) | DB server: Windows NT 6.1 <X64> build 7601 SP1 | ||||
Webserver software & version (if known) | Linux RHEL 7, Apache 2.4.6 | ||||
PHP Version | 5.6.19 | ||||
related to | 11620 | closed | Validation equation bug | |
related to | 11731 | closed | ollehar | JavaScript error when the field has a validation regexp, and the input starts with a plus sign |
related to | 11894 | closed | Regex validation not working as expected when leading zeros are present in short text questions | |
child of | 08324 | closed | DenisChenu | Broken numeric interpretation of answer codes. |
This works fine in 2.50. |
|
@c_schmitz : you are sure ? Tested see screenshot. |
|
Did someone know why we force NON numeric question type to numeric here We already test if it's numeric : |
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=18498 |
|
Fix committed to 2.06lts branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=18499 |
|
This fix whole "string" to "number" for non numeric question type. Think other system are not broken with this fix . |
|
Version 2.50+ Build 160415 released |
|
Fix committed to 2.06lts branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=28078 |
|
LimeSurvey: master 16a95712 2016-04-12 13:10 Details Diff |
Fixed issue 10860 : Regular expression fail with specific value Dev: before 01 return a string and 1 a number Dev : now (if question is not numeric) : +1, 30 number, 01 .0 return string Dev: still issue with big number (but with big number : even when forcing to number : sum is broken). |
Affected Issues 10860 |
|
mod - application/helpers/expressions/em_core_helper.php | Diff File | ||
mod - scripts/expressions/em_javascript.js | Diff File | ||
LimeSurvey: master 8c6ed4e4 2016-04-13 08:56 Details Diff |
Merge pull request #474 from Shnoulle/master_fix10860 Fixed issue 10860 : Regular expression fail with specific value - Time to comment is closed. Tested, update a little the system, but seems better (hope) |
Affected Issues 10860 |
|
mod - application/helpers/expressions/em_core_helper.php | Diff File | ||
mod - scripts/expressions/em_javascript.js | Diff File | ||
LimeSurvey: 2.06lts 163878fb 2016-04-13 10:04 Details Diff |
Fixed issue 10860 : Regular expression fail with specific value Dev: same fix than master, and some space removed .... Dev: before 01 return a string and 1 a number Dev : now (if question is not numeric) : +1, 30 number, 01 .0 return string Dev: still issue with big number (but with big number : even when forcing to number : sum is broken). |
Affected Issues 10860 |
|
mod - application/helpers/expressions/em_core_helper.php | Diff File | ||
mod - scripts/expressions/em_javascript.js | Diff File | ||
LimeSurvey: 2.06lts 3279a102 2016-04-13 12:04 Details Diff |
Fixed issue 10860 : Regular expression fail with specific value Dev: same fix than master, and some space removed .... Dev: before 01 return a string and 1 a number Dev : now (if question is not numeric) : +1, 30 number, 01 .0 return string Dev: still issue with big number (but with big number : even when forcing to number : sum is broken). |
Affected Issues 10860 |
|
mod - application/helpers/expressions/em_core_helper.php | Diff File | ||
mod - scripts/expressions/em_javascript.js | Diff File |
Date Modified | Username | Field | Change |
---|---|---|---|
2016-04-04 08:58 | fvanderstarre | New Issue | |
2016-04-04 08:58 | fvanderstarre | File Added: limesurvey_survey_491491.lss | |
2016-04-11 12:23 | c_schmitz | Note Added: 37208 | |
2016-04-11 13:37 | DenisChenu | Assigned To | => DenisChenu |
2016-04-11 13:37 | DenisChenu | Status | new => assigned |
2016-04-11 13:54 | DenisChenu | Note Added: 37219 | |
2016-04-11 13:54 | DenisChenu | File Added: Capture du 2016-04-11 13-53-46.png | |
2016-04-11 14:12 | DenisChenu | Note Added: 37220 | |
2016-04-11 14:26 | DenisChenu | Relationship added | child of 08324 |
2016-04-12 13:31 | DenisChenu | Note Added: 37261 | |
2016-04-13 08:56 | DenisChenu | Changeset attached | => LimeSurvey master 16a95712 |
2016-04-13 08:56 | DenisChenu | Changeset attached | => LimeSurvey master 8c6ed4e4 |
2016-04-13 08:56 | DenisChenu | Note Added: 37278 | |
2016-04-13 08:56 | DenisChenu | Resolution | open => fixed |
2016-04-13 10:04 | DenisChenu | Changeset attached | => LimeSurvey 2.06lts 163878fb |
2016-04-13 10:04 | DenisChenu | Note Added: 37281 | |
2016-04-13 10:12 | DenisChenu | Note Added: 37282 | |
2016-04-13 10:12 | DenisChenu | Status | assigned => resolved |
2016-04-13 10:12 | DenisChenu | Fixed in Version | => 2.5 |
2016-04-13 12:24 | Mazi | Relationship added | related to 10934 |
2016-04-13 12:34 | DenisChenu | Relationship deleted | related to 10934 |
2016-04-15 10:24 | c_schmitz | Note Added: 37421 | |
2016-04-15 10:24 | c_schmitz | Status | resolved => closed |
2016-09-03 18:45 | DenisChenu | Relationship added | related to 11620 |
2016-10-02 16:25 | DenisChenu | Relationship added | related to 11731 |
2016-11-13 17:21 | DenisChenu | Relationship added | related to 11894 |
2018-09-28 14:12 | DenisChenu | Changeset attached | => LimeSurvey 2.06lts 3279a102 |
2018-09-28 14:12 | DenisChenu | Note Added: 49183 |