View Issue Details

This bug affects 1 person(s).
 6
IDProjectCategoryView StatusLast Update
08093Bug reportsImport/Exportpublic2013-08-16 13:54
Reportermfaber Assigned Tomdekker  
PrioritynormalSeverityminor 
Status closedResolutionfixed 
Product Version2.05 RC 
Fixed in Version2.05 RC 
Summary08093: Missing R-export (syntax)...
Description

Hi Menno, please find attached test-.lsa as requested

TagsNo tags attached.
Attached Files
Bug heat6
Complete LimeSurvey version number (& build)130809
I will donate to the project if issue is resolvedNo
BrowserFF
Database type & versionMySQL (recent XAMP)
Server OS (if known)WinXP
Webserver software & version (if known)Apache (recent XAMP)
PHP VersionPHP (recent XAMP)

Users monitoring this issue

There are no users monitoring this issue.

Activities

mdekker

mdekker

2013-08-12 11:02

reporter   ~25954

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

mdekker

mdekker

2013-08-12 11:05

reporter   ~25955

Can you please confirm that this fixes the problem? There is more cleanup to be done but it should be working for now. Thanks for the report and the test survey it really helped in finding the bug. (commented out the headers to show errors)

mfaber

mfaber

2013-08-12 11:53

reporter   ~25964

The good news: It fixes the problem with the test survey and some other smaller test surveys which I just tried.

The bad news is that I have another survey (around 50 questions and 100 responses) where the problem still exists. When I narrowed down the export to the first 2 responses and the first 20 questions, the export still does not work, so it seems a not too uncommon problem. Some other combinations seem to work. I will gladly do more debugging myself but I probably will only find time next week...

Unfortunately I cannot share this data with anyone (neither the questionnaire nor the data). Maybe you have more test data where you could try the export?

Will keep you updated!
M

mdekker

mdekker

2013-08-12 11:59

reporter   ~25965

Using a partial filled out demo survey it still works for me. If you could disable the lines with header in the syntaxwriter (line 27-30) you will get errors on screen.

Hope you can track it down to a question type or answerset (maybe 0 coded) for me.

mfaber

mfaber

2013-08-12 13:04

reporter   ~25969

OK: here's one problem...if variable names have to be renamed...$errors is not defined...

PHP notice

Undefined variable: errors

E:\survey\htdocs\github\LimeSurvey\application\core\plugins\ExportR\RSyntaxWriter.php(179)

177 if ($ftitle != $field['title'])
178 {
179 $errors .= "# Variable name was incorrect and was changed from {$field['title']} to $ftitle .\n";
180 }
181

I have commented out this line and then my survey results export fine :)

mfaber

mfaber

2013-08-12 13:06

reporter   ~25970

The thing with the headers still puzzles me. Do you have an explanation for this behaviour?

mdekker

mdekker

2013-08-12 13:30

reporter   ~25974

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

mdekker

mdekker

2013-08-12 13:30

reporter   ~25975

The headers force a download, but there is nothing to download because of errors. Normally the errors will end up in the downloaded file but I think the Yii error trapping causes trouble here. I pushed another fix, please check.

mfaber

mfaber

2013-08-12 14:21

reporter   ~25980

R export seems to work with the fix! At least no error anymore. Yey! :)

c_schmitz

c_schmitz

2013-08-16 13:54

administrator   ~26010

2.05 beta 6 released

Related Changesets

LimeSurvey: 2.05 812228f7

2013-08-12 09:02:26

mdekker

Details Diff
Fixed 08093: Error on R-export (syntax) while running as a plugin Affected Issues
08093
mod - application/core/plugins/ExportR/RSyntaxWriter.php Diff File
mod - application/helpers/export_helper.php Diff File

LimeSurvey: 2.05 9ec4cdd6

2013-08-12 11:30:15

mdekker

Details Diff
Fixed 08093: Error on R-export (syntax) while running as a plugin Affected Issues
08093
mod - application/core/plugins/ExportR/RSyntaxWriter.php Diff File

Issue History

Date Modified Username Field Change
2013-08-09 15:34 mfaber New Issue
2013-08-09 15:34 mfaber Status new => assigned
2013-08-09 15:34 mfaber Assigned To => mdekker
2013-08-09 15:34 mfaber File Added: survey_archive_834486.lsa
2013-08-12 11:02 mdekker Changeset attached => LimeSurvey 2.05 812228f7
2013-08-12 11:02 mdekker Note Added: 25954
2013-08-12 11:02 mdekker Resolution open => fixed
2013-08-12 11:05 mdekker Note Added: 25955
2013-08-12 11:06 mdekker Status assigned => resolved
2013-08-12 11:06 mdekker Fixed in Version => 2.05 RC
2013-08-12 11:53 mfaber Note Added: 25964
2013-08-12 11:54 mdekker Status resolved => assigned
2013-08-12 11:59 mdekker Note Added: 25965
2013-08-12 13:04 mfaber Note Added: 25969
2013-08-12 13:06 mfaber Note Added: 25970
2013-08-12 13:30 mdekker Changeset attached => LimeSurvey 2.05 9ec4cdd6
2013-08-12 13:30 mdekker Note Added: 25974
2013-08-12 13:30 mdekker Note Added: 25975
2013-08-12 13:52 mdekker Status assigned => resolved
2013-08-12 14:21 mfaber Note Added: 25980
2013-08-16 13:54 c_schmitz Note Added: 26010
2013-08-16 13:54 c_schmitz Status resolved => closed