| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 13881 | Bug reports | Expression Manager | public | 2018-07-18 10:11 | 2023-10-31 17:27 |
| Reporter | DenisChenu | Assigned To | |||
| Priority | none | Severity | minor | ||
| Status | new | Resolution | open | ||
| Product Version | 3.13.x | ||||
| Summary | 13881: EM : Not number question replace . by , (javascript) | ||||
| Description | When entering 1.1 in a short text question : if show the value inside same page : 1,1. In another page 1.1 (in french and other , for separator) | ||||
| Steps To Reproduce | Import included survey and test I test wth text.NAOK, text.shown | ||||
| Additional Information | The biggest issue i think : the value shown is different in js and php. This must be same value shown in php and js. The best solution can be :
2.6lts + 2.73 have same issue To know if value is a number : we have it in
| ||||
| Tags | No tags attached. | ||||
| Bug heat | 4 | ||||
| Complete LimeSurvey version number (& build) | 3.13.2 github | ||||
| I will donate to the project if issue is resolved | No | ||||
| Browser | not relevant | ||||
| Database type & version | not relevant | ||||
| Server OS (if known) | not relevant | ||||
| Webserver software & version (if known) | not relevant | ||||
| PHP Version | not relevant | ||||
 |
Another things : using str_replace(",",".",Text.NAOK) didn't work … when EM have it it's . , then need str_replace("."," ",Text.NAOK) to remove the , . No way to see 1.1 … |
|
|
Issue is here: https://github.com/LimeSurvey/LimeSurvey/commit/49aff574e363cf659e79101b16f59d839427755f Replacement Action is done even if attr.onlynum = 0 This must be done here : https://github.com/LimeSurvey/LimeSurvey/blob/108cf3522890408673977783c9bce8564fdfe189/assets/scripts/expressions/em_javascript.js#L754 |
 |
Feel free to give me an lss file later so I can make a unit test. |
|
|
@ollehar : i can do :) I already have https://bugs.limesurvey.org/view.php?id=15012 and https://bugs.limesurvey.org/view.php?id=14995 to do … but hope i found one day to do it both … |
|
|
Oups :) think it's the another bug … This one update API (even if it's a broken API) unsure we can do it for 3.X |
|
|
Shit … more complex than i think … The issue is here : But since there are no related mantis : why this is added is unknow … |
|
|
OK : EM core to fix … Just … … don't set LEMRadix to , |
