View Issue Details

This bug affects 1 person(s).
 2
IDProjectCategoryView StatusLast Update
06635Bug reportsImport/Exportpublic2012-10-09 22:39
Reporternwinter Assigned Toc_schmitz  
PrioritynormalSeverityminor 
Status closedResolutionfixed 
Product Version2.00+ 
Fixed in Version2.00+ 
Summary06635: Export to R headers include hidden fields
Description

The export to R data (.csv) file includes headers at the top; those headers include hidden fields that are not included with the data.

Steps To Reproduce

Export a dataset to R.

Additional Information

The problem is in this block of code in application/helpers/export_helper.php. The -- if ($i<$num_fields && !$field['hide']) -- should apply to the --echo-- of the field, not just the delimiter:

// Add column headers (used by R export)
if($header==TRUE)
{
    $i = 1;
    foreach ($fields as $field) {
        echo $q.strtoupper($field['sql_name']).$q;
        if ($i<$num_fields && !$field['hide']) echo ',';
        $i++;
    }
    echo("\n");
TagsNo tags attached.
Bug heat2
Complete LimeSurvey version number (& build)120931
I will donate to the project if issue is resolvedNo
Browser
Database type & version??
Server OS (if known)Linux
Webserver software & version (if known)Apache/2.2.17 (Fedora)
PHP Version5.3.8

Users monitoring this issue

There are no users monitoring this issue.

Activities

c_schmitz

c_schmitz

2012-10-07 14:12

administrator   ~21110

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

c_schmitz

c_schmitz

2012-10-09 22:39

administrator   ~21144

2.00+ Build 121009 released. Please update.

Related Changesets

LimeSurvey: master 18a09b50

2012-10-07 07:12

c_schmitz


Details Diff
Fixed issue 06635: Export to R headers include hidden fields Affected Issues
06635
mod - application/helpers/export_helper.php Diff File

Issue History

Date Modified Username Field Change
2012-10-03 17:31 nwinter New Issue
2012-10-05 10:10 c_schmitz Assigned To => c_schmitz
2012-10-05 10:10 c_schmitz Status new => assigned
2012-10-07 14:12 c_schmitz Changeset attached => LimeSurvey master 18a09b50
2012-10-07 14:12 c_schmitz Note Added: 21110
2012-10-07 14:12 c_schmitz Resolution open => fixed
2012-10-07 14:13 c_schmitz Status assigned => resolved
2012-10-07 14:13 c_schmitz Fixed in Version => 2.00+
2012-10-09 22:39 c_schmitz Note Added: 21144
2012-10-09 22:39 c_schmitz Status resolved => closed