06787: Multiple numerical input w/ sub-question equation validation + array filter - LimeSurvey bugs, patches and feature requests

(21727)
spacejanitor (reporter)
2012-10-27 14:52

http://www.limesurvey.org/en/forum/design-issues/87424-bug-multiple-numerical-input-w-sub-question-equation-validation--array-filter#87433 [^]

(21729)
TMSWhite (developer)
2012-10-27 15:34

Not a bug but a mis-use of the 'that' macro variable. Please read the documentation and try the sample surveys

(21731)
spacejanitor (reporter)
2012-10-27 17:10

I believe the key issue still remains. I have re-tested several different ways with the same result.

EM is trying to assess all subquestions regardless of whether or not they were array-filtered. So if Q2_b was not checked, EM will try to assess it against Q3_b as per Q3's equation, making it impossible to proceed.

(21732)
spacejanitor (reporter)
2012-10-27 17:18

I tried also adding the is_empty function with an OR, but that didn't resolve it either. If the option at Q2 was selected but empty, it would validate, but otherwise, it would not proceed.

(21733)
TMSWhite (developer)
2012-10-27 17:25

Still not a bug. Works exactly as designed. To avoid having filtered variables prevent validation, use .NAOK on all variables.

(21734)
spacejanitor (reporter)
2012-10-27 17:30

I did that:

(Q3_A.NAOK> Q2_A.NAOK) && (Q3_B.NAOK> Q2_B.NAOK) && (Q3_C.NAOK> Q2_C.NAOK)

Still the same result. Sorry to keep re-opening the issue. If it's some basic mistake I'm making I can just continue to reply on the forum thread.

(21735)
TMSWhite (developer)
2012-10-27 18:37

I can look at this on Tuesday if somebody wants to look at it earlier that would be appreciated

(21740)
spacejanitor (reporter)
2012-10-28 14:57

Willing to donate money or my time for speedier resolution. I am not a PHP developer but can help in debugging or diagnosing, and can enlist the help of my PHP developer (has some experience with LS).

(22038)
TMSWhite (developer)
2012-11-09 01:59

I'm not going to have time to look into this for at least another week. Freeing this up for another developer.

(22046)
DenisChenu (developer)
2012-11-09 19:11

Hello,

With included lss file : change
((this/ 3) == floor(this/3))
By
((this.NAOK / 3) == floor(this.NAOK/3))
Like Thomas already say and it'sOK.

For:
(Q3_A.NAOK> Q2_A.NAOK) : you ask Q3_A>Q2_A if exist or not : it works exactly as designed. If you do the same thing in javascript : javascript error.

If you do the same in PHP : error.
Use:
(Q3_A.NAOK> Q2_A.NAOK) and !(is_empty(Q3_A.NAOK) or is_empty(Q2_A.NAOK))

Not a bug.

Issue History
Date Modified	Username	Field	Change
2012-10-27 14:50	spacejanitor	New Issue
2012-10-27 14:50	spacejanitor	File Added: limesurvey_survey_899147.lss
2012-10-27 14:52	spacejanitor	Note Added: 21727
2012-10-27 15:34	TMSWhite	Note Added: 21729
2012-10-27 15:34	TMSWhite	Status	new => closed
2012-10-27 15:34	TMSWhite	Assigned To	=> TMSWhite
2012-10-27 15:34	TMSWhite	Resolution	open => no change required
2012-10-27 17:10	spacejanitor	Note Added: 21731
2012-10-27 17:10	spacejanitor	Status	closed => feedback
2012-10-27 17:10	spacejanitor	Resolution	no change required => reopened
2012-10-27 17:18	spacejanitor	Note Added: 21732
2012-10-27 17:18	spacejanitor	Status	feedback => assigned
2012-10-27 17:25	TMSWhite	Note Added: 21733
2012-10-27 17:25	TMSWhite	Status	assigned => closed
2012-10-27 17:25	TMSWhite	Resolution	reopened => no change required
2012-10-27 17:30	spacejanitor	Note Added: 21734
2012-10-27 17:30	spacejanitor	Status	closed => feedback
2012-10-27 17:30	spacejanitor	Resolution	no change required => reopened
2012-10-27 18:37	TMSWhite	Note Added: 21735
2012-10-28 14:57	spacejanitor	Note Added: 21740
2012-10-28 14:57	spacejanitor	Status	feedback => assigned
2012-11-09 01:59	TMSWhite	Note Added: 22038
2012-11-09 01:59	TMSWhite	Assigned To	TMSWhite =>
2012-11-09 01:59	TMSWhite	Status	assigned => new
2012-11-09 08:32	c_schmitz	Assigned To	=> DenisChenu
2012-11-09 08:32	c_schmitz	Status	new => assigned
2012-11-09 19:11	DenisChenu	Note Added: 22046
2012-11-09 19:11	DenisChenu	Status	assigned => closed
2012-11-09 19:11	DenisChenu	Resolution	reopened => no change required
2012-11-09 19:11	DenisChenu	Fixed in Version	=> 2.00+

Notes
(21727) spacejanitor (reporter) 2012-10-27 14:52	http://www.limesurvey.org/en/forum/design-issues/87424-bug-multiple-numerical-input-w-sub-question-equation-validation--array-filter#87433 [^]

(21729) TMSWhite (developer) 2012-10-27 15:34	Not a bug but a mis-use of the 'that' macro variable. Please read the documentation and try the sample surveys

(21731) spacejanitor (reporter) 2012-10-27 17:10	I believe the key issue still remains. I have re-tested several different ways with the same result. EM is trying to assess all subquestions regardless of whether or not they were array-filtered. So if Q2_b was not checked, EM will try to assess it against Q3_b as per Q3's equation, making it impossible to proceed.

(21732) spacejanitor (reporter) 2012-10-27 17:18	I tried also adding the is_empty function with an OR, but that didn't resolve it either. If the option at Q2 was selected but empty, it would validate, but otherwise, it would not proceed.

(21733) TMSWhite (developer) 2012-10-27 17:25	Still not a bug. Works exactly as designed. To avoid having filtered variables prevent validation, use .NAOK on all variables.

(21734) spacejanitor (reporter) 2012-10-27 17:30	I did that: (Q3_A.NAOK> Q2_A.NAOK) && (Q3_B.NAOK> Q2_B.NAOK) && (Q3_C.NAOK> Q2_C.NAOK) Still the same result. Sorry to keep re-opening the issue. If it's some basic mistake I'm making I can just continue to reply on the forum thread.

(21735) TMSWhite (developer) 2012-10-27 18:37	I can look at this on Tuesday if somebody wants to look at it earlier that would be appreciated

(21740) spacejanitor (reporter) 2012-10-28 14:57	Willing to donate money or my time for speedier resolution. I am not a PHP developer but can help in debugging or diagnosing, and can enlist the help of my PHP developer (has some experience with LS).

(22038) TMSWhite (developer) 2012-11-09 01:59	I'm not going to have time to look into this for at least another week. Freeing this up for another developer.

(22046) DenisChenu (developer) 2012-11-09 19:11	Hello, With included lss file : change ((this/ 3) == floor(this/3)) By ((this.NAOK / 3) == floor(this.NAOK/3)) Like Thomas already say and it'sOK. For: (Q3_A.NAOK> Q2_A.NAOK) : you ask Q3_A>Q2_A if exist or not : it works exactly as designed. If you do the same thing in javascript : javascript error. If you do the same in PHP : error. Use: (Q3_A.NAOK> Q2_A.NAOK) and !(is_empty(Q3_A.NAOK) or is_empty(Q2_A.NAOK)) Not a bug.

Relationships