• Hello,

    I’ve discovered a bug in your CSV export code and would like to submit a patch. Where can I do this?

    Thanks!

Viewing 4 replies - 1 through 4 (of 4 total)
  • Plugin Author Takayuki Miyoshi

    (@takayukister)

    Thanks. What kind of bug did you discover?

    Thread Starter buildetech

    (@buildetech)

    The CSV export code in flamingo/admin/admin.php for the “Inbound Messages” items adds an empty column between the last column of form data and the date values. In other words, the output looks like:

    “label1″,”label2″,”label3″,”date”
    “data1″,”data2″,”data3″,””,”timestamp”
    “data1″,”data2″,”data3″,””,”timestamp”

    It looks to me like the simple fix is to change line 605 so that, rather than appending the date to the array, it overwrites the last slot in the $row array with the time stamp string.

    I’m happy to submit a patch through Trac, GitHub, etc, but it didn’t look like there was any activity on the plugin’s GitHub mirror and I didn’t see your plugin in the dropdown when creating a new ticket on Trac.

    Plugin Author Takayuki Miyoshi

    (@takayukister)

    Thank you very much. I confirmed there is a bug. It will be fixed in the next release.

    Thread Starter buildetech

    (@buildetech)

    Fantastic–thank you very much!

Viewing 4 replies - 1 through 4 (of 4 total)

The topic ‘CSV Export Issue Patch’ is closed to new replies.