View Issue Details

IDProjectCategoryView StatusLast Update
12131Bug reportsResponse browsingpublic2020-04-09 11:58
Reporterrom Assigned Toc_schmitz  
PriorityurgentSeveritypartial_block 
Status newResolutionreopened 
Product Version2.62.x 
Fixed in Version3.22.7 
Summary12131: filterout_incomplete_answers does not seem to work anymore
Description

We just updated from 2.06 to 2.62.2 and figured that the config.php setting "filterout_incomplete_answers => filter" does not seem to work anymore, but it's still available in the config_defaults.php file.

Steps To Reproduce

set config.php config array to:
'config'=>array(
'filterout_incomplete_answers' => 'filter',
'debug'=>0,
'debugsql'=>0,
)

This seems to have no effect for the "Export" section (CSV) and the "show answers" Table - both are still showing up as "All answers".

Additional Information

There is a rumour about this setting, that the value should be set to 'filterout_incomplete_answers' => 'complete' instead of "filter" to get it working. I tried both values, no luck so far.

TagsNo tags attached.
Complete LimeSurvey version number (& build) 2.62.2+170203
I will donate to the project if issue is resolvedNo
Browser
Database & DB-VersionMySQL
Server OS (if known)Linux
Webserver software & version (if known)Apache
PHP Version5.56

Activities

user14106

2017-03-03 20:58

  ~43153

searched the code and this seems to be the only place this gets used
https://github.com/LimeSurvey/LimeSurvey/blob/4d9562f7eda960274ac802550c4eb7693112fd57/application/controllers/Statistics_userController.php#L215

try true/false options ?

rom

rom

2017-03-06 08:45

reporter   ~43162

thanks for looking up the code. I tried setting it to true or false, but that did not have any effect either.

'config'=>array(
'filterout_incomplete_answers' => true,
'debug'=>0,
'debugsql'=>0,
)

c_schmitz

c_schmitz

2020-03-05 20:37

administrator   ~56337

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=29649

lime_release_bot

lime_release_bot

2020-03-09 09:54

administrator   ~56362

Fixed in Release 3.22.8+200309

Mazi

Mazi

2020-04-09 11:58

partner   ~57042

@c_schmitz / @cdorin: This commit (https://github.com/LimeSurvey/LimeSurvey/commit/367a4ce4ec72d262c70d94bf0fae2017270f6f4f) breaks existing plugins!

Can we please NOT rename configuration settings? I know that the current naming may not be perfect but there are external tools or plugins reading such settings from the core config and they will not work anymore if such details are changed.
E.g. we have several custom data analysis or PDF plugins which make use of this setting and now that the parameters have changed, they are broken!

Related Changesets

LimeSurvey: master 290e01f0

2020-03-05 20:32:40

c_schmitz

Details Diff
Fixed issue 12131: Config parameter filterout_incomplete_answers does not seem to work anymore Affected Issues
12131
mod - application/config/config-defaults.php Diff File
mod - application/controllers/Statistics_userController.php Diff File

Issue History

Date Modified Username Field Change
2017-02-16 16:20 rom New Issue
2017-03-03 20:58 user14106 Note Added: 43153
2017-03-06 08:45 rom Note Added: 43162
2018-05-23 12:21 LouisGac Assigned To => LouisGac
2018-05-23 12:21 LouisGac Status new => assigned
2018-05-23 12:44 LouisGac Priority none => low
2020-03-05 20:37 c_schmitz Changeset attached => LimeSurvey master 290e01f0
2020-03-05 20:37 c_schmitz Note Added: 56337
2020-03-05 20:37 c_schmitz Assigned To LouisGac => c_schmitz
2020-03-05 20:37 c_schmitz Resolution open => fixed
2020-03-05 20:38 c_schmitz Status assigned => resolved
2020-03-05 20:38 c_schmitz Fixed in Version => 3.22.7
2020-03-09 09:54 lime_release_bot Note Added: 56362
2020-03-09 09:54 lime_release_bot Status resolved => closed
2020-04-09 11:58 Mazi Priority low => urgent
2020-04-09 11:58 Mazi Severity minor => partial_block
2020-04-09 11:58 Mazi Status closed => new
2020-04-09 11:58 Mazi Resolution fixed => reopened
2020-04-09 11:58 Mazi Note Added: 57042