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 |