View Issue Details

Jump to NotesJump to History
This bug affects 1 person(s).
 8
IDProjectCategoryView StatusDate SubmittedLast Update
10940Bug reportsSurvey takingpublic2016-04-13 17:512016-06-09 13:53
Reportermarkbao Assigned To 
PrioritylowSeverityminor 
Status closedResolutionfixed 
Product Version2.06+ 
Fixed in Version2.50.x 
Summary10940: Numerical input with "Thousands separator" enabled rejects input that have a thousands separator
Description

Hello, I have a question (attached) that is Numerical and has this configuration:

Thousands separator: Yes
Minimum value: 0
Maximum value: 100000

However, whenever I input a number 1000 or above (which triggers the automatic thousands separator), it produces a number like "1,000", and it is rejected by the validation. Changing it back to something like 999 makes it go though.

Error message: One or more questions have not been answered in a valid manner. You cannot proceed until these answers are valid.

This seems to be fixed in 2.5 but 2.5 has another bug with inputting numbers (see http://www.giphy.com/gifs/xT1XGDYH3Vfsed7o5i) and we are not able to use 2.5 at this time.

Steps To Reproduce
  1. Use attached question file.
  2. Preview survey.
  3. Type a number larger than 1000 (don't type thousands separator) and try to submit.

Expected behavior: These numbers should go through correctly, since the thousands separator should be valid (besides, it's being inserted by LimeSurvey).

Actual behavior: It fails validation.

TagsNo tags attached.
Attached Files
limesurvey_survey_818111.lss (12,946 bytes)
Bug heat8
Complete LimeSurvey version number (& build)Build 151215
I will donate to the project if issue is resolvedNo
Browser
Database type & versionMySQL latest
Server OS (if known)Ubuntu latest
Webserver software & version (if known)Apache latest
PHP VersionPHP 5 latest

Users monitoring this issue

There are no users monitoring this issue.

Activities

markbao

markbao

2016-04-13 17:52

reporter   ~37352

Correction in steps to reproduce: "Use attached question file" should be "Use attached survey file".
trivender

trivender

2016-04-13 18:53

updater   ~37353

Version 2.50+ Build 160414
Tried to reproduce the problem:
1: Import attached LSS.
2: Preview survey.
3: Pressing number 1 .. input change --> 100.
4: Again pressing number 1 ...input change --> 100,100 (Error:Only an integer value may be entered in this field.)
ollehar

ollehar

2016-04-19 12:34

administrator   ~37538

This is not a bug, since you had set "Integers only" to "Yes" in Advanced settings.
c_schmitz

c_schmitz

2016-04-21 11:40

administrator   ~37640

Version 2.50+ Build 160421 released
markbao

markbao

2016-04-21 17:44

reporter   ~37667

Last edited: 2016-04-21 17:45

Hmm - so the "thousands separator" seems like it should be working with numbers, since that's the only place you would use a thousands separator, correct? So it doesn't seem to make sense that if you want a thousands separator, the integer validation has to be turned off.

Expected general behavior should be to add the thousands separator in the survey UI, but strip it out in validation and in the response data.

In addition, expected behavior in LS 2.5 should be that adding an extra "0" to 100 should lead to it being 1,000, not 100,100 and throwing an error as it currently is.

(Also, I didn't see this in the 2.5 build 160421 changelog so I am posting this follow-up.)
markbao

markbao

2016-05-14 20:09

reporter   ~38572

Thoughts on this?
ollehar

ollehar

2016-05-17 10:13

administrator   ~38587

IMO, it should not be possible to enable both integers only AND thousand separator.
c_schmitz

c_schmitz

2016-06-09 13:53

administrator   ~39355

The thousands separator feature was removed as there is no clean way to make it work.

Issue History

Date Modified Username Field Change
2016-04-13 17:51 markbao New Issue
2016-04-13 17:51 markbao File Added: limesurvey_survey_818111.lss
2016-04-13 17:52 markbao Note Added: 37352
2016-04-13 18:53 trivender Note Added: 37353
2016-04-13 19:17 c_schmitz Assigned To => ollehar
2016-04-13 19:17 c_schmitz Status new => assigned
2016-04-13 19:18 c_schmitz Priority none => high
2016-04-19 12:34 ollehar Note Added: 37538
2016-04-19 12:34 ollehar Status assigned => resolved
2016-04-19 12:34 ollehar Fixed in Version => 2.5
2016-04-19 12:34 ollehar Resolution open => no change required
2016-04-21 11:40 c_schmitz Note Added: 37640
2016-04-21 11:40 c_schmitz Status resolved => closed
2016-04-21 17:44 markbao Note Added: 37667
2016-04-21 17:44 markbao Status closed => feedback
2016-04-21 17:44 markbao Resolution no change required => reopened
2016-04-21 17:45 markbao Note Edited: 37667
2016-04-21 17:45 markbao Note Edited: 37667
2016-05-14 20:09 markbao Note Added: 38572
2016-05-14 20:09 markbao Status feedback => assigned
2016-05-17 10:13 ollehar Note Added: 38587
2016-06-07 13:19 LouisGac Priority high => low
2016-06-09 13:53 c_schmitz Note Added: 39355
2016-06-09 13:53 c_schmitz Status assigned => closed
2016-06-09 13:53 c_schmitz Assigned To ollehar =>
2016-06-09 13:53 c_schmitz Resolution reopened => fixed