View Issue Details

This bug affects 1 person(s).
 8
IDProjectCategoryView StatusLast Update
09909Bug reportsImport/Exportpublic2016-08-29 12:38
Reporterhvgard Assigned ToDenisChenu  
PrioritynormalSeverityminor 
Status closedResolutionfixed 
Product Version2.05+ 
Target Version2.50.xFixed in Version2.50.x 
Summary09909: Empty colums in export file while data visible in GUI
Description

When we view responses there is data in the [MWhyStop_time] column (answer = Y) but on export (to CSV, XLS, PDF) that column is empty in the output.

Steps To Reproduce

View responses
Export data (any combination of settings)
Inspect output file

Additional Information

I'm a chemist not a programmer. Hesitant to use things like comfort-upgrade while running surveys for customers :-)

TagsNo tags attached.
Attached Files
survey_archive_976576.lsa (157,736 bytes)
survey_archive_TIME.lsa (157,736 bytes)
Bug heat8
Complete LimeSurvey version number (& build)Version 2.05+ Build 141229
I will donate to the project if issue is resolvedNo
BrowserSeveral, mainly Firefox
Database type & versionmySQL
Server OS (if known)Linux
Webserver software & version (if known)I guess apache
PHP VersionPHP Version 5.3.10-1ubuntu3.11

Relationships

duplicate of 08396 closedDenisChenu Bug reports Sub Question code TIME with Save timings survey : unable to export survey 
related to 09272 closedDenisChenu Feature requests Impossible to use the string "other" as answer code 
related to 09534 closedDenisChenu Feature requests Remove uneended javascript disalowing other in sub question code 
related to 09533 closedDenisChenu Bug reports TSV export/import multi with other add a other sub question 

Users monitoring this issue

There are no users monitoring this issue.

Activities

hvgard

hvgard

2015-09-23 19:32

reporter   ~33144

Someone I work with (a looonnnngggg-time programmer said): The only unique thing I can come up with that might explain why it happened is: it's the first answer after another multiple-choice question with an "other" choice. Maybe the "other" choice messed up something in the export. If I was the programmer I'd look there.

DenisChenu

DenisChenu

2015-09-24 11:24

developer   ~33148

Last edited: 2015-09-24 11:25

Same issue with your survey and master GIT version

PS: vv export seems fine

DenisChenu

DenisChenu

2015-09-24 11:46

developer   ~33149

It's due to time : you have 2 times the same column ....

hvgard

hvgard

2015-09-24 15:06

reporter   ~33153

Will admit right away that I don't know / never used the VV export. Will try later today.

Which 2 columns are the same? I don't see it. They all some unique.

DenisChenu

DenisChenu

2015-09-24 15:13

developer   ~33154

No : the "time" column used by Survey timing have same name than the coluln used for the sub question time.

hvgard

hvgard

2015-09-24 15:16

reporter   ~33155

Last edited: 2015-09-24 15:17

OK, so to AVOID that issue we could (for the time being, next survey) not use [time] as a sub-question name. Good to know and happy fixing :-)

DenisChenu

DenisChenu

2015-09-24 15:29

developer   ~33156

Last edited: 2015-09-24 15:29

Hard to fix this one .....
PS: i think you can update the column name in surveys_timing_XXXX table

hvgard

hvgard

2015-09-24 15:37

reporter   ~33157

For this one we only had 200 responses and typed in the 28 "Y" answers in the XLS. So we are fine. But suppose one has 10.000 responses ..... thanks.

c_schmitz

c_schmitz

2016-08-17 17:36

administrator   ~40330

Prohibit usage of keywords 'time' and 'other' as subquestion codes.

hvgard

hvgard

2016-08-17 22:18

reporter   ~40332

Thanks Carsten. Good to see it will be solved.

DenisChenu

DenisChenu

2016-08-22 11:45

developer   ~40373

@Carsten : other is already managed in Question rules : prohibit only if Question is set to "other"/Y.

Then : we can use "array filter" with other on another question. It's OK with this.

markusfluer

markusfluer

2016-08-24 15:17

administrator   ~40456

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

DenisChenu

DenisChenu

2016-08-24 16:25

developer   ~40464

2 issues with the fix:

Not done for import and Remote
Other can be used for Array- question type : and a lot of user want it in the survey (and use it with good reason)
Exemple of usage : http://demonstration.sondages.pro/522714?newtest=Y#

DenisChenu

DenisChenu

2016-08-25 16:41

developer   ~40505

https://bugs.limesurvey.org/view.php?id=8396

DenisChenu

DenisChenu

2016-08-25 16:45

developer   ~40506

I can not confirm this issue : export function does the job in last version.

See the screenshot

DenisChenu

DenisChenu

2016-08-26 09:45

developer   ~40517

Uploaded

  • survey_archive_TIME.lsa
  • results-survey832841-1.csv

having time or TIME is not the issue. I revert the fix

DenisChenu

DenisChenu

2016-08-26 18:04

developer   ~40523

OK,
Got it ...

time !== TIME

We can allow TIME, but disallow time .....

Actually : best seems to update manually the DB (rename the colum time to Time and the Subquestion code to Time).

DenisChenu

DenisChenu

2016-08-26 18:35

developer   ~40524

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

DenisChenu

DenisChenu

2016-08-26 18:36

developer   ~40525

@C_schmitz : https://github.com/LimeSurvey/LimeSurvey/pull/528 fix it really. BUt seems awfull for maintenance ....

c_schmitz

c_schmitz

2016-08-29 12:38

administrator   ~40532

Version 2.51.0 Build 160829 released

Related Changesets

LimeSurvey: master 4432d56f

2016-08-24 15:17

markusfluer


Details Diff
Fixed issue 09909: Empty colums in export file while data visible in GUI Affected Issues
09909
mod - application/views/admin/survey/Question/_subQuestionsAndAnwsersJsVariables.php Diff File
mod - scripts/admin/subquestions.js Diff File

LimeSurvey: master 6268ab17

2016-08-26 18:35

DenisChenu


Details Diff
Fixed issue 09909 : disallow time for SubQuestion code (allow Time/TIME ...) for export issue
Dev: Done in models rules directly
Dev: Must validate it before submit (and show a beautifull error), using ajax ?
Dev: if we use https://github.com/LimeSurvey/LimeSurvey/pull/528 : we can allow time
Affected Issues
09909
mod - application/models/Question.php Diff File

Issue History

Date Modified Username Field Change
2015-09-23 01:08 hvgard New Issue
2015-09-23 01:08 hvgard File Added: survey_archive_976576.lsa
2015-09-23 19:32 hvgard Note Added: 33144
2015-09-24 11:24 DenisChenu Note Added: 33148
2015-09-24 11:24 DenisChenu Status new => confirmed
2015-09-24 11:25 DenisChenu Note Edited: 33148
2015-09-24 11:46 DenisChenu Note Added: 33149
2015-09-24 15:06 hvgard Note Added: 33153
2015-09-24 15:13 DenisChenu Note Added: 33154
2015-09-24 15:16 hvgard Note Added: 33155
2015-09-24 15:17 hvgard Note Edited: 33155
2015-09-24 15:29 DenisChenu Note Added: 33156
2015-09-24 15:29 DenisChenu Note Edited: 33156
2015-09-24 15:37 hvgard Note Added: 33157
2016-08-17 17:36 c_schmitz Note Added: 40330
2016-08-17 17:36 c_schmitz Assigned To => c_schmitz
2016-08-17 17:36 c_schmitz Status confirmed => assigned
2016-08-17 17:36 c_schmitz Target Version => 2.5
2016-08-17 22:18 hvgard Note Added: 40332
2016-08-22 11:45 DenisChenu Note Added: 40373
2016-08-24 14:23 c_schmitz Assigned To c_schmitz => markusfluer
2016-08-24 15:17 markusfluer Changeset attached => LimeSurvey master 4432d56f
2016-08-24 15:17 markusfluer Note Added: 40456
2016-08-24 15:17 markusfluer Resolution open => fixed
2016-08-24 15:17 markusfluer Status assigned => resolved
2016-08-24 15:17 markusfluer Fixed in Version => 2.5
2016-08-24 16:25 DenisChenu Note Added: 40464
2016-08-24 16:25 DenisChenu Status resolved => feedback
2016-08-24 16:25 DenisChenu Resolution fixed => reopened
2016-08-24 16:57 DenisChenu Relationship added related to 09272
2016-08-24 16:58 DenisChenu Relationship added related to 09534
2016-08-24 16:58 DenisChenu Relationship added related to 09533
2016-08-25 11:42 DenisChenu Assigned To markusfluer => DenisChenu
2016-08-25 11:42 DenisChenu Status feedback => assigned
2016-08-25 16:41 DenisChenu Note Added: 40505
2016-08-25 16:43 DenisChenu Relationship added duplicate of 08396
2016-08-25 16:45 DenisChenu Note Added: 40506
2016-08-25 16:45 DenisChenu File Added: Capture du 2016-08-25 16-44-04.png
2016-08-26 09:36 DenisChenu File Added: survey_archive_TIME.lsa
2016-08-26 09:44 DenisChenu File Added: results-survey832841-1.csv
2016-08-26 09:45 DenisChenu Note Added: 40517
2016-08-26 10:31 DenisChenu Fixed in Version 2.50.x =>
2016-08-26 10:31 DenisChenu Severity partial_block => minor
2016-08-26 10:31 DenisChenu Reproducibility have not tried => sometimes
2016-08-26 18:04 DenisChenu Note Added: 40523
2016-08-26 18:35 DenisChenu Changeset attached => LimeSurvey master 6268ab17
2016-08-26 18:35 DenisChenu Note Added: 40524
2016-08-26 18:36 DenisChenu Note Added: 40525
2016-08-26 18:36 DenisChenu Status assigned => resolved
2016-08-26 18:36 DenisChenu Fixed in Version => 2.50.x
2016-08-26 18:36 DenisChenu Resolution reopened => fixed
2016-08-29 12:38 c_schmitz Note Added: 40532
2016-08-29 12:38 c_schmitz Status resolved => closed