View Issue Details

Related ChangesetsJump to NotesJump to History
This bug affects 1 person(s).
 4
IDProjectCategoryView StatusDate SubmittedLast Update
09859Bug reportsSurvey takingpublic2015-09-01 12:362015-09-30 11:01
ReporterDenisChenu Assigned ToDenisChenu  
PrioritynormalSeverityminor 
Status closedResolutionfixed 
Product Version2.06+ 
Fixed in Version2.06+ 
Summary09859: Broken JS with thousands_separator
Description

We have a javascript error with thousands_separator, this can break survey logic system.

Steps To Reproduce

Import included lss and look at console.

Additional Information

In preview : url for numerical_input.js is false (or in sub directory for survey)

App()->clientScript->registerScriptFile('scripts/numerical_input.js');
MUST BE
App()->clientScript->registerScriptFile(Yii::app()->getConfig('generalscripts').'numerical_input.js');

And jquery-price-format is not included.
But more : it's not in third_party directory (or i don't found it).

I can fix , but i need to know the jquery plugin used (It's an easy plugin to do ...)

TagsNo tags attached.
Attached Files
limesurvey_survey_thousands_separator.lss (12,066 bytes)
Bug heat4
Complete LimeSurvey version number (& build)150901
I will donate to the project if issue is resolvedNo
Browsernot relevant
Database type & versionnot relevant
Server OS (if known)not relevant
Webserver software & version (if known)not relevant
PHP Versionnot relevant

Users monitoring this issue

There are no users monitoring this issue.

Activities

DenisChenu

DenisChenu

2015-09-01 14:27

developer   ~33050

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

DenisChenu

2015-09-01 14:57

developer   ~33052

Fixed in 2.06 .
Review composer for 3.0 (see https://github.com/flaviosilveira/Jquery-Price-Format/issues/34 and https://github.com/flaviosilveira/Jquery-Price-Format/issues/49)
DenisChenu

DenisChenu

2015-09-01 15:33

developer   ~33053

Fix committed to develop branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=15855
c_schmitz

c_schmitz

2015-09-30 11:01

administrator   ~33230

Version 2.06+ Build 150930 released

Related Changesets

LimeSurvey: master 063a3bcd

2015-09-01 14:25

DenisChenu


Details Diff		 Fixed issue 09859: Broken javascript with thousands_separator
Dev: add third_party
Dev: Fix script link call		 Affected Issues
09859
mod - application/config/third_party.php Diff File
mod - application/helpers/qanda_helper.php Diff File

LimeSurvey: develop e14cda41

2015-09-01 15:32

DenisChenu


Details Diff		 Fixed issue 09859: Broken Javascript with thousands_separator Affected Issues
09859
mod - application/config/third_party.php Diff File
mod - application/models/questions/MultipleNumberQuestion.php Diff File
mod - application/models/questions/NumericalQuestion.php Diff File

Issue History

Date Modified Username Field Change
2015-09-01 12:36 DenisChenu New Issue
2015-09-01 12:36 DenisChenu Status new => assigned
2015-09-01 12:36 DenisChenu Assigned To => DenisChenu
2015-09-01 12:36 DenisChenu File Added: limesurvey_survey_thousands_separator.lss
2015-09-01 14:27 DenisChenu Changeset attached => LimeSurvey master 063a3bcd
2015-09-01 14:27 DenisChenu Note Added: 33050
2015-09-01 14:27 DenisChenu Resolution open => fixed
2015-09-01 14:57 DenisChenu Note Added: 33052
2015-09-01 15:33 DenisChenu Changeset attached => LimeSurvey develop e14cda41
2015-09-01 15:33 DenisChenu Note Added: 33053
2015-09-01 15:34 DenisChenu Status assigned => resolved
2015-09-01 15:34 DenisChenu Fixed in Version => 2.06+
2015-09-30 11:01 c_schmitz Note Added: 33230
2015-09-30 11:01 c_schmitz Status resolved => closed