View Issue Details

Related ChangesetsJump to NotesJump to History
This bug affects 1 person(s).
 4
IDProjectCategoryView StatusDate SubmittedLast Update
06102Bug reportsSurvey takingpublic2012-05-16 16:422012-05-17 10:27
ReporterTMSWhite Assigned ToTMSWhite  
PrioritynormalSeverityminor 
Status closedResolutionfixed 
Product Version1.92+ 
Target Version1.92+Fixed in Version1.92+ 
Summary06102: "Print your answers" shows EM syntax errors
Description

In Sample Census survey, if have more then 1 co-habitant, those above 1 are shown with syntax errors on "print your answers", but not during survey taking.

Additional Information

Root cause is LimeExpressionManager.php (7178) - does recursive call to $this->ProcessString() from within GetVarAttribute() when trying to do dynamic substitution of answer values. This messes up local state. This should be handled by using higher recursion level instead.

TagsNo tags attached.
Bug heat4
Complete LimeSurvey version number (& build)120509
I will donate to the project if issue is resolvedNo
Browser
Database type & versionMysql 5.3
Server OS (if known)Windows XP
Webserver software & version (if known)XAMPP
PHP Version5.3

Relationships

Relationship GraphDependency Graph
related to 06075 closedTMSWhite Content of previous answer is not replaced, instead placeholder of selected answer is shown: eg. {TOKEN:ATTRIBUTE_2} 

Users monitoring this issue

There are no users monitoring this issue.

Activities

TMSWhite

TMSWhite

2012-05-16 19:23

reporter   ~18790

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

TMSWhite

2012-05-16 19:28

reporter   ~18792

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

c_schmitz

2012-05-17 10:27

administrator   ~18810

New 1.92+ version released.

Related Changesets

LimeSurvey: master c01f90b5

2012-05-16 12:22

TMSWhite


Details Diff		 Fixed issue 06102: "Print your answers" shows EM syntax errors
Dev had to fix issue 06075 in a different way to solve this issue: Needed to do 3-deep recursive replacement instead of 2-deep recursion.		 Affected Issues
06075, 06102
mod - classes/expressions/LimeExpressionManager.php Diff File
mod - replacements.php Diff File

LimeSurvey: Yii 7092bfda

2012-05-16 12:27

TMSWhite


Details Diff		 Fixed issue 06102: "Print your answers" shows EM syntax errors
Dev had to fix issue 06075 in a different way to solve this issue: Needed to do 3-deep recursive replacement instead of 2-deep recursion.		 Affected Issues
06075, 06102
mod - application/helpers/expressions/em_manager_helper.php Diff File
mod - application/helpers/replacements_helper.php Diff File

Issue History

Date Modified Username Field Change
2012-05-16 16:42 TMSWhite New Issue
2012-05-16 16:42 TMSWhite Status new => assigned
2012-05-16 16:42 TMSWhite Assigned To => TMSWhite
2012-05-16 19:21 TMSWhite Relationship added related to 06075
2012-05-16 19:23 TMSWhite Changeset attached => LimeSurvey master c01f90b5
2012-05-16 19:23 TMSWhite Note Added: 18790
2012-05-16 19:23 TMSWhite Resolution open => fixed
2012-05-16 19:28 TMSWhite Changeset attached => LimeSurvey Yii 7092bfda
2012-05-16 19:28 TMSWhite Note Added: 18792
2012-05-16 19:29 TMSWhite Status assigned => resolved
2012-05-16 19:29 TMSWhite Fixed in Version => 1.92+
2012-05-17 10:27 c_schmitz Note Added: 18810
2012-05-17 10:27 c_schmitz Status resolved => closed