Dependency Graph View Issue Relation Graph Vertical
related to child of duplicate of

View Issue Details

IDProjectCategoryView StatusLast Update
14200Bug reports[All Projects] Expression Managerpublic2019-08-20 16:31
ReporterTonisOrmisson Assigned Tocdorin  
PrioritynoneSeveritymajor 
Status assignedResolutionopen 
Product Version3.16.x 
Target VersionFixed in Version 
Summary14200: EM missing data for multiple choice questions
Description

Expression manager is missing question info for multiple-response questions

eg {Q1.qid}, {Q1.sgqa} are missing. Possibly some other data also

Steps To Reproduce

See attached survey example.

TagsNo tags attached.
Complete LimeSurvey version number (& build)32aef4b01ae08f2364d82a8389d1e6dc025f4db5
I will donate to the project if issue is resolvedNo
Browser
Database & DB-Versionn/a
Server OS (if known)n/a
Webserver software & version (if known)n/a
PHP Versionn/a

Activities

TonisOrmisson

TonisOrmisson

2018-11-01 14:29

developer  

limesurvey_survey_791116.lss (18,367 bytes)
TonisOrmisson

TonisOrmisson

2018-11-01 14:29

developer  

TonisOrmisson

TonisOrmisson

2018-11-01 14:31

developer   ~49502

as far as I understand the $LEM->qcode2sgqa array does not contain multiple choice question codes
here:
https://github.com/LimeSurvey/LimeSurvey/blob/master/application/helpers/expressions/em_manager_helper.php#L8978

DenisChenu

DenisChenu

2018-11-02 11:24

developer   ~49512

Last edited: 2018-11-02 11:25

View 2 revisions

SGQA is : database name (or must be near). It's a related to an input inside HTML. Parent question didn't have input.

For expression manager : ONLY value save in DB exist.

For a single question Q1, for a multiple question : Q1_SQ01,Q1_SQ02, for a multiplexmultiple question (array number, array text) : Q1_SY1_SX1 etc …

For Q1 : maybe we can add something with "static" set to true with only .question, .relevance etc unsure about sgqa : since sgqa didn't exist for Q1.

TonisOrmisson

TonisOrmisson

2018-11-02 12:12

developer   ~49513

Last edited: 2018-11-02 12:14

View 2 revisions

ok nevermind the SGQA

the whole {Q1.*} is missing for multis. {Q1.gid}, {Q1.qcode}, {Q1.type} etc etc

DenisChenu

DenisChenu

2018-11-02 14:47

developer   ~49517

Yes, but explanation still valid :).

What to shown with Q1.NAOK ? that.Q1.NAOK ?

It really a complete new feature … i already think of adding a plugin event here https://github.com/LimeSurvey/LimeSurvey/blob/2d58085ced72e2bb471aa8b4493b3730aa9e4749/application/helpers/expressions/em_manager_helper.php#L4378

To allow plugin to update, create new knowVars … something like $event->set('knownVars',$this->knownVars); $this->knownVars = $event->get('knownVars'); (in fact : i think adding Q1.help, and some Q1.number, Q1.medium (see https://gitlab.com/SondagesPro/getStatInSurvey#documentation). Then more to update current knownVars, but adding can be done too.

What did you think of such feature ? This can broke survey with badly devel plugin, but it can does a lot.

DenisChenu

DenisChenu

2019-08-05 16:28

developer   ~53111

@TonisOrmisson : maybe we can add it in https://github.com/LimeSurvey/LimeSurvey/commit/011890d128084176074421cbe82cdd342706f729#diff-1a3bea26d4d8bfcd4a0422ab1b3b9864

Not complex

jelo

jelo

2019-08-13 15:12

partner   ~53171

Is that issue solved in LS4? What is the position of core devs?

DenisChenu

DenisChenu

2019-08-20 16:31

developer   ~53199

My personal opinion : it can be interesting BUT

  1. Take place in memory
  2. Take place in SESSION (currently)
  3. Can be done in plugin (eventually core plugin (new one or update the current)

Issue History

Date Modified Username Field Change
2018-11-01 14:29 TonisOrmisson New Issue
2018-11-01 14:29 TonisOrmisson File Added: limesurvey_survey_791116.lss
2018-11-01 14:29 TonisOrmisson File Added: Screenshot from 2018-11-01 15-24-07.png
2018-11-01 14:31 TonisOrmisson Note Added: 49502
2018-11-02 11:24 DenisChenu Note Added: 49512
2018-11-02 11:25 DenisChenu Note Edited: 49512 View Revisions
2018-11-02 12:12 TonisOrmisson Note Added: 49513
2018-11-02 12:14 TonisOrmisson Note Edited: 49513 View Revisions
2018-11-02 14:47 DenisChenu Note Added: 49517
2018-11-03 15:58 DenisChenu Relationship added related to 14206
2019-08-05 16:28 DenisChenu Note Added: 53111
2019-08-13 15:12 jelo Note Added: 53171
2019-08-13 15:15 cdorin Assigned To => cdorin
2019-08-13 15:15 cdorin Status new => assigned
2019-08-14 09:01 Mazi Relationship added related to 08858
2019-08-20 16:31 DenisChenu Note Added: 53199