View Issue Details

IDProjectCategoryView StatusLast Update
14932Bug reports[All Projects] Expression Managerpublic2019-05-23 11:30
ReporterDenisChenu Assigned To 
PrioritynoneSeverityminor 
Status newResolutionopen 
Product Version3.17.x 
Target VersionFixed in Version 
Summary14932: NAOK is not needed in PHP , only in JS
Description

In manual :

For example : count(Q1_SQ1,Q1_SQ2,Q1_SQ3,Q1_SQ4) give always an empty string if one subquestion of Q1 is filtered. To count the number of checked subquestion in such question can be count(Q1_SQ1.NAOK,Q1_SQ2.NAOK,Q1_SQ3.NAOK,Q1_SQ4.NAOK). If the sub question is hidden :Expression manager return an empty string.
https://manual.limesurvey.org/Expression_Manager#Usage_of_NAOK

But : it's false :

  1. In PHP : count is OK
  2. In JS : count is bad ONLY without suffix , with .code : it's OK
Steps To Reproduce

Import included survey, test.

You can check with js deactivated in 1st question …

Additional Information

There are 2 issues here:

  1. JS function LEManyNa don't remove suffix when testing variable : must check if it's an update between 2.06 and 3.X. Then return true for code
  2. There are no of NA system in PHP

Possible fix : just remove the LEManyNa checking …

TagsNo tags attached.
Complete LimeSurvey version number (& build)3.17.3 github
I will donate to the project if issue is resolvedNo
Browsernot relevant
Database & DB-Versionnot relevant
Server OS (if known)not relevant
Webserver software & version (if known)not relevant
PHP Versionnot relevant

Relationships

related to 14818 assignedDenisChenu Feature requests Default suffix to NAOK not code 

Activities

DenisChenu

DenisChenu

2019-05-23 11:28

developer  

limesurvey_survey_LemAnyNa.lss (25,550 bytes)
DenisChenu

DenisChenu

2019-05-23 11:29

developer   ~52066

Last edited: 2019-05-23 11:29

View 2 revisions

@c_schmitz : I already have the fix for .code in JS. But what is the good fix ?
Did we need to send '' for count(…) in PHP too ?

Issue History

Date Modified Username Field Change
2019-05-23 11:28 DenisChenu New Issue
2019-05-23 11:28 DenisChenu File Added: Capture d’écran du 2019-05-23 11-28-07.png
2019-05-23 11:28 DenisChenu File Added: Capture d’écran du 2019-05-23 11-28-14.png
2019-05-23 11:28 DenisChenu File Added: limesurvey_survey_LemAnyNa.lss
2019-05-23 11:29 DenisChenu Note Added: 52066
2019-05-23 11:29 DenisChenu Note Edited: 52066 View Revisions
2019-05-23 11:30 DenisChenu Description Updated View Revisions
2019-05-23 11:30 DenisChenu Relationship added related to 14818