View Issue Details

This bug affects 1 person(s).
 4
IDProjectCategoryView StatusLast Update
14267Bug reportsSurvey takingpublic2018-11-27 19:11
ReporterDenisChenu Assigned ToDenisChenu  
PrioritynoneSeverityminor 
Status closedResolutionduplicate 
Summary14267: Bad evaluation of hidden question with default value and relevance (PHP7.2)
Description

With an hidden question , with subquestion with relevance and default value : evaluation after this count all question, not only relevant one.

Steps To Reproduce

Import included lss : test

  1. Check 1st item only on 1st question : move next
  2. look at list returned

With PHP5.6 :

list : SQ1, SQ2, SQ3
count : 3

with PHP7.2

list : SQ1, SQ2, SQ3,SQ4,SQ5
count : 5

Data saved is OK

Additional Information

I'm unsure it's related to 7.2, but testing in 5.6 with same survey didn't show the issue …

TagsNo tags attached.
Attached Files
Bug heat4
Complete LimeSurvey version number (& build)3.15.5 github
I will donate to the project if issue is resolvedNo
Browsernot relevant
Database type & versionmysql
Server OS (if known)fedora/linux
Webserver software & version (if known)nginx
PHP Version7.2.12

Relationships

duplicate of 14270 closedDenisChenu value, valueNAOK etc … broken with GET params 

Users monitoring this issue

There are no users monitoring this issue.

Activities

jelo

jelo

2018-11-22 15:42

partner   ~49737

With PHP 7.2.12 under CentOS 7
list : SQ1, SQ2, SQ3
count : 3

DenisChenu

DenisChenu

2018-11-22 16:15

developer   ~49738

Argl … tested with PHP 7.2.12 under Fedora …

I review …

jelo

jelo

2018-11-22 16:25

partner   ~49739

Mhm. PHP settings? Extensions? Different packages? Or a LimeSurvey setting?

On demo.limesurvey.org
3 Return
list : SQ1, SQ2, SQ3, SQ4, SQ5
count : 5

DenisChenu

DenisChenu

2018-11-22 16:53

developer   ~49740

Oh, it broke on demo … there are really something : urlParam ?
Your Centos is on apache ? If yes : you can manually set url param to GET : https://manual.limesurvey.org/Optional_settings#URL_settings

Related Changesets

LimeSurvey: master 0c41165a

2018-11-22 15:14:00

DenisChenu

Details Diff
Fixed issue 14270: value, valueNAOK etc … broken with GET params
Dev: remove usage of LimeExpressionManager in prefillFromCommandLine function
Dev: since it can broke a lot EM : log potential error in setVariableAndTokenMappingsForExpressionManager
Affected Issues
14267, 14270
mod - application/helpers/expressions/em_manager_helper.php Diff File
mod - application/helpers/frontend_helper.php Diff File

Issue History

Date Modified Username Field Change
2018-11-21 14:57 DenisChenu New Issue
2018-11-21 14:57 DenisChenu File Added: limesurvey_survey_defaultValueRelevanceHidden.lss
2018-11-22 15:42 jelo Note Added: 49737
2018-11-22 16:15 DenisChenu Note Added: 49738
2018-11-22 16:25 jelo Note Added: 49739
2018-11-22 16:53 DenisChenu Note Added: 49740
2018-11-27 19:09 DenisChenu Assigned To => DenisChenu
2018-11-27 19:09 DenisChenu Status new => assigned
2018-11-27 19:11 DenisChenu Changeset attached => LimeSurvey master 0c41165a
2018-11-27 19:11 DenisChenu Relationship added duplicate of 14270
2018-11-27 19:11 DenisChenu Status assigned => closed
2018-11-27 19:11 DenisChenu Resolution open => duplicate