View Issue Details

Related ChangesetsJump to NotesJump to History
This bug affects 1 person(s).
 4
IDProjectCategoryView StatusDate SubmittedLast Update
11396Bug reportsSurvey takingpublic2016-06-16 10:012016-06-20 15:31
ReporterDenisChenu Assigned ToDenisChenu  
PrioritynoneSeverityminor 
Status closedResolutionfixed 
Product Version2.50.x 
Fixed in Version2.50.x 
Summary11396: Expression manager foir max_answers ranking are not comptletly used
Description

If you use Expression in max_answers for ranking question type : it was not used in JS , only in PHP

Steps To Reproduce

Importe incluided survey and test

Additional Information

With 2.06 : it work without any issue with different page (see screenshot), still some issue with same page.

TagsNo tags attached.
Attached Files
limesurvey_survey_maxInRanking.lss (19,891 bytes)
Capture du 2016-06-16 09-59-34.png (33,788 bytes)   
Capture du 2016-06-16 09-59-34.png (33,788 bytes)   
Capture du 2016-06-16 09-59-48.png (37,329 bytes)   
Capture du 2016-06-16 09-59-48.png (37,329 bytes)   
Bug heat4
Complete LimeSurvey version number (& build)160616
I will donate to the project if issue is resolvedNo
BrowserFirefox
Database type & versionnot relevant
Server OS (if known)not relevant
Webserver software & version (if known)not relevant
PHP Versionnot relevant

Relationships

Relationship GraphDependency Graph
related to 09828 closedDenisChenu Ranking question : update allowed can broke Survey DB 

Users monitoring this issue

There are no users monitoring this issue.

Activities

DenisChenu

DenisChenu

2016-06-16 10:14

developer   ~39620

See https://github.com/LimeSurvey/LimeSurvey/pull/507/commits/0d0cbb3cbd035c1ac969c3543c745bb3edbf5868 for the partial fix (this survey)
c_schmitz

c_schmitz

2016-06-17 09:01

administrator   ~39654

Denis, can you fix it completely?
DenisChenu

DenisChenu

2016-06-17 09:24

developer   ~39658

You look at pull request ?

Because fis are different with the pull request or not ....
https://github.com/LimeSurvey/LimeSurvey/pull/507

I really like the idea to allow user to limit column number in ranking question : for example allow 150 possible answers but save only 4. But i don't like to remove other dev work witout discussuon.
DenisChenu

DenisChenu

2016-06-17 16:47

developer   ~39679

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

c_schmitz

2016-06-20 15:31

administrator   ~39732

Version 2.50 Build 160620 released

Related Changesets

LimeSurvey: master 0d0cbb3c

2016-06-16 10:04

DenisChenu


Details Diff		 Fixed issue 11396: Expression manager in max_answers for ranking don't show error
Dev: different page step
Dev: ranking.js must be fixed for same page
Dev: EM tips are empty if in same page .... must fix it before ranking.js		 Affected Issues
11396
mod - application/helpers/qanda_helper.php Diff File

Issue History

Date Modified Username Field Change
2016-06-16 10:01 DenisChenu New Issue
2016-06-16 10:01 DenisChenu File Added: limesurvey_survey_maxInRanking.lss
2016-06-16 10:02 DenisChenu File Added: Capture du 2016-06-16 09-59-34.png
2016-06-16 10:02 DenisChenu File Added: Capture du 2016-06-16 09-59-48.png
2016-06-16 10:14 DenisChenu Note Added: 39620
2016-06-17 09:01 c_schmitz Note Added: 39654
2016-06-17 09:02 c_schmitz Assigned To => DenisChenu
2016-06-17 09:02 c_schmitz Status new => feedback
2016-06-17 09:24 DenisChenu Note Added: 39658
2016-06-17 09:24 DenisChenu Status feedback => assigned
2016-06-17 09:24 DenisChenu Relationship added related to 09828
2016-06-17 16:47 DenisChenu Changeset attached => LimeSurvey master 0d0cbb3c
2016-06-17 16:47 DenisChenu Note Added: 39679
2016-06-17 16:47 DenisChenu Resolution open => fixed
2016-06-17 16:58 DenisChenu Status assigned => resolved
2016-06-17 16:58 DenisChenu Fixed in Version => 2.5
2016-06-20 15:31 c_schmitz Note Added: 39732
2016-06-20 15:31 c_schmitz Status resolved => closed