View Issue Details

Related ChangesetsJump to NotesJump to History
This issue affects 1 person(s).
 6
IDProjectCategoryView StatusDate SubmittedLast Update
09876Bug reportsSurvey takingpublic2015-09-09 10:162018-09-28 14:12
ReporterDenisChenu Assigned Toollehar  
PrioritynormalSeverityminor 
Status closedResolutionfixed 
Product Version2.06+ 
Target Version2.50.xFixed in Version2.06+ 
Summary09876: Different bogus with thousand seperator
Description

1 : With radix to "," and integre : unable to submit a number up to 1000
2 : Unable to have thousand for a question and no thousand for another one
3 : Unable to remove (empty) an already set input (always put 0)
4 : actual value can be badly updated if up to 1000
5 : unable to use maximum_chars

Steps To Reproduce

Import survey and so some test.

Additional Information

Seems with Android 5.05 default browser : unable to put any number after put a alpha caratcer. Can not test in live : have only Android 4 (CyanogenMod).

I can put 5 bug if needed : but whole need same file fix

With 4 : each time you try to submit : value is updated .... (see pic)

TagsNo tags attached.
Attached Files
limesurvey_bogus_thousand.lss (17,497 bytes)
Capture du 2015-09-09 10-15-14.png (17,598 bytes)   
Capture du 2015-09-09 10-15-14.png (17,598 bytes)   
Capture du 2015-09-09 10-15-31.png (18,881 bytes)   
Capture du 2015-09-09 10-15-31.png (18,881 bytes)   
Bug heat6
Complete LimeSurvey version number (& build)150909
I will donate to the project if issue is resolvedNo
Browserfirefox + Chromium
Database type & versionnot relevant
Server OS (if known)not relevant
Webserver software & version (if known)not relevant
PHP Versionnot relevant

Relationships

Relationship GraphDependency Graph
parent of 10830 closedDenisChenu In certain conditions, slider submission record 0.XY instead of X.Y 
related to 10189 closedollehar Enabling "Thousands separator" in "numerical input" questions results in an error and data loss 
related to 10211 closedollehar Thousands separator - numeric input and multiple numeric input in the same group 
related to 10394 closedollehar Thousand Separator appears to be broken - causes other issues (numerical input) 

Users monitoring this issue

There are no users monitoring this issue.

Activities

DenisChenu

DenisChenu

2016-02-19 12:30

developer   ~35445

@olle : you start to work on thousand_separator : then i give it to you.

2 : Unable to have thousand for a question and no thousand for another one (NUM4)
3 : Unable to remove (empty) an already set input (always put 0) (NUM3)
is still here with last GIT

Maybe some other error two but maybe fix 2 and 3 can fix it : http://limesurvey.sondages.pro/index.php/811525

Here : i can nbot submit and no error is shown.
ollehar

ollehar

2016-02-19 12:31

administrator   ~35446

Same problem in 2.5, I guess? Because we will only do security fixes on 2.06.
DenisChenu

DenisChenu

2016-02-19 12:37

developer   ~35447

@olle : i assign 3 pull for you :
https://github.com/LimeSurvey/LimeSurvey/pull/412
https://github.com/LimeSurvey/LimeSurvey/pull/423
https://github.com/LimeSurvey/LimeSurvey/pull/421

The 3 have different method, and really hard to choose the best for future ...; Pull must be updated according to update in qanda_helper

One don't touch EM : only do whole in js
One touch EM : i really dislike to touch EM
One is more updating the function used.
ollehar

ollehar

2016-02-22 12:45

administrator   ~35529

I agree with not touching the EM unnecessarily.

It would be nice if the pull requests exactly matched an issue in the bug tracker. That would make it easier to test.
c_schmitz

c_schmitz

2016-02-29 11:45

administrator   ~35769

Version 2.50+ Build 160228 released
DenisChenu

DenisChenu

2016-03-25 12:15

developer   ~36805

Fix committed to 2.06lts branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=18181
DenisChenu

DenisChenu

2016-03-25 12:16

developer   ~36807

Reopen to set fixed in 2.06 . Sorry for the mail olle ;)
DenisChenu

DenisChenu

2018-09-28 14:12

developer   ~49178

Fix committed to 2.06lts branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=28068

Related Changesets

LimeSurvey: 2.06lts da7da69d

2016-03-25 12:15

DenisChenu


Details Diff		 Fixed issue 10830 : thousand separator break slider in some condition
Fixed issue 09876: Different bogus with thousand seperator
Dev: use the JS only fix to be more quick
Dev: adapt the js to apply only to .thousandsseparator input.numeric		 Affected Issues
09876, 10830
mod - scripts/numerical_input.js Diff File

LimeSurvey: 2.06lts 36c59125

2016-03-25 13:15

DenisChenu


Details Diff		 Fixed issue 10830 : thousand separator break slider in some condition
Fixed issue 09876: Different bogus with thousand seperator
Dev: use the JS only fix to be more quick
Dev: adapt the js to apply only to .thousandsseparator input.numeric		 Affected Issues
09876, 10830
mod - scripts/numerical_input.js Diff File

Issue History

Date Modified Username Field Change
2015-09-09 10:16 DenisChenu New Issue
2015-09-09 10:16 DenisChenu File Added: limesurvey_bogus_thousand.lss
2015-09-09 10:18 DenisChenu File Added: Capture du 2015-09-09 10-15-14.png
2015-09-09 10:18 DenisChenu File Added: Capture du 2015-09-09 10-15-31.png
2015-09-09 10:20 DenisChenu Browser firefox + => firefox + Chromium
2016-01-06 17:35 DenisChenu Relationship added related to 10189
2016-01-08 17:48 DenisChenu Relationship added related to 10211
2016-02-19 12:27 DenisChenu Assigned To => ollehar
2016-02-19 12:27 DenisChenu Status new => assigned
2016-02-19 12:30 DenisChenu Note Added: 35445
2016-02-19 12:31 ollehar Note Added: 35446
2016-02-19 12:31 ollehar Target Version => 2.5
2016-02-19 12:37 DenisChenu Note Added: 35447
2016-02-19 12:38 DenisChenu Relationship added related to 10394
2016-02-22 12:45 ollehar Note Added: 35529
2016-02-24 11:19 DenisChenu Status assigned => resolved
2016-02-24 11:19 DenisChenu Fixed in Version => 2.5
2016-02-24 11:19 DenisChenu Resolution open => fixed
2016-02-29 11:45 c_schmitz Note Added: 35769
2016-02-29 11:45 c_schmitz Status resolved => closed
2016-03-25 11:56 DenisChenu Relationship added parent of 10830
2016-03-25 12:15 DenisChenu Changeset attached => LimeSurvey 2.06lts da7da69d
2016-03-25 12:15 DenisChenu Note Added: 36805
2016-03-25 12:16 DenisChenu Note Added: 36807
2016-03-25 12:16 DenisChenu Status closed => feedback
2016-03-25 12:16 DenisChenu Resolution fixed => reopened
2016-03-25 12:17 DenisChenu Fixed in Version 2.5 => 2.06+
2016-03-25 12:17 DenisChenu Summary Diffrenrent bogus with thousand seperator => Different bogus with thousand seperator
2016-03-25 12:17 DenisChenu Status feedback => resolved
2016-03-25 12:17 DenisChenu Resolution reopened => fixed
2016-03-25 12:18 DenisChenu Status resolved => closed
2018-09-28 14:12 DenisChenu Changeset attached => LimeSurvey 2.06lts 36c59125
2018-09-28 14:12 DenisChenu Note Added: 49178