View Issue Details

IDProjectCategoryView StatusLast Update
16741Bug reportsStatisticspublic2020-10-15 16:16
Reportercdorin Assigned To 
PrioritynoneSeverityminor 
Status confirmedResolutionopen 
Summary16741: Filtering on responses for questions of type Array Dual Scale was not working in Statistics
Description

In Statistics, Filtering on responses for questions of type Array Dual Scale was not working at all: Results were not respecting those filters, and the selections in those selectors were lost on the results page.

Steps To Reproduce

To reproduce: Just create a one-question survey with a dual array dual scale question and their sub-questions. Or use an example from the man page https://manual.limesurvey.org/Question_type_-_Array_dual_scale . Then fill-in one survey, and go to admin results statistics. In Response Filters select one of the dual-scale answers that was NOT selected in that test survey.

Expected result: 0 out of 1 shown and selection in Response Filter still selected.
Actual result: 1 out of 1 shown and Response Filter selection lost.
After applying PR: Result becomes as expected.

Additional Information

Analysis and fixes:
Those filters' multi-selects were ignored in the filtering because their select names were wrong: E.g.:
Actual wrong: select name="956915X67X519SQ001#{0}[]" and select name="956915X67X519SQ001#{1}[]"
Expected correct: select name="956915X67X519SQ001#0[]" and select name="956915X67X519SQ001#1[]"

Additionally, once fixed, an html comment above was generating a warning because they supposed strings and got arrays of strings. This PR fixes both of them.

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

Activities

Issue History

Date Modified Username Field Change
2020-10-15 16:16 cdorin New Issue
2020-10-15 16:16 cdorin Note Added: 60216
2020-10-15 16:16 cdorin Status new => confirmed