View Issue Details

Jump to NotesJump to History
This bug affects 1 person(s).
 2
IDProjectCategoryView StatusDate SubmittedLast Update
14932Bug reportsExpression Managerpublic2019-05-23 11:282020-03-19 09:10
ReporterDenisChenu Assigned To 
PrioritynoneSeverityminor 
Status newResolutionopen 
Product Version3.17.x 
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.
Attached Files
Capture d’écran du 2019-05-23 11-28-07.png (16,163 bytes)   
Capture d’écran du 2019-05-23 11-28-07.png (16,163 bytes)   
Capture d’écran du 2019-05-23 11-28-14.png (4,590 bytes)   
Capture d’écran du 2019-05-23 11-28-14.png (4,590 bytes)   
limesurvey_survey_LemAnyNa.lss (25,550 bytes)
Bug heat2
Complete LimeSurvey version number (& build)3.17.3 github
I will donate to the project if issue is resolvedNo
Browsernot relevant
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 14818 new Feature requests Default suffix to NAOK not code 

Users monitoring this issue

There are no users monitoring this issue.

Activities

DenisChenu

DenisChenu

2019-05-23 11:29

developer   ~52066

Last edited: 2019-05-23 11:29

@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 ?
DenisChenu

DenisChenu

2020-03-19 09:10

developer   ~56661

Q1 == 1 or Q2 == 2 with condition on Q2 send false in PHP

See sample lss

limesurvey_survey_457919.lss (40,958 bytes)

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
2019-05-23 11:30 DenisChenu Description Updated
2019-05-23 11:30 DenisChenu Relationship added related to 14818
2020-03-19 09:10 DenisChenu File Added: limesurvey_survey_457919.lss
2020-03-19 09:10 DenisChenu Note Added: 56661