View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
08093 | Bug reports | Import/Export | public | 2013-08-09 15:34 | 2013-08-16 13:54 |
Reporter | mfaber | Assigned To | mdekker | ||
Priority | normal | Severity | minor | ||
Status | closed | Resolution | fixed | ||
Product Version | 2.05 RC | ||||
Fixed in Version | 2.05 RC | ||||
Summary | 08093: Missing R-export (syntax)... | ||||
Description | Hi Menno, please find attached test-.lsa as requested | ||||
Tags | No tags attached. | ||||
Attached Files | |||||
Bug heat | 6 | ||||
Complete LimeSurvey version number (& build) | 130809 | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | FF | ||||
Database type & version | MySQL (recent XAMP) | ||||
Server OS (if known) | WinXP | ||||
Webserver software & version (if known) | Apache (recent XAMP) | ||||
PHP Version | PHP (recent XAMP) | ||||
Fix committed to 2.05 branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=12837 |
|
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) |
|
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! |
|
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. |
|
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']) I have commented out this line and then my survey results export fine :) |
|
The thing with the headers still puzzles me. Do you have an explanation for this behaviour? |
|
Fix committed to 2.05 branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=12844 |
|
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. |
|
R export seems to work with the fix! At least no error anymore. Yey! :) |
|
2.05 beta 6 released |
|
LimeSurvey: 2.05 812228f7 2013-08-12 11:02 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 13:30 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 |
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 |