#165 CSV Exports need quoted values

closed-fixed
nobody
None
5
2011-06-12
2009-11-13
No

When exporting to a CSV file format the values should be quoted to avoid commas in the value being interpretted as field separators.

Discussion

  • Nikolay

    Nikolay - 2009-12-27

    Hi,

    It should be fixed in MMEX ver. 0.9.5.1

     
  • Comment has been marked as spam. 
    Undo

    You can see all pending comments posted by this user  here

    Anonymous - 2010-02-27

    I just checked in the latest trunk version (5.9.0), and it does not seem to have been fixed. However, the change is really easy, here is a patch:

    --- mmex/src/util.cpp
    +++ mmex/src/util.cpp
    @@ -578,9 +578,9 @@

    st2.Finalize();
    } else {
    - text << dateString << delimit << payee << delimit << sign << amount << delimit
    - << categ << delimit << subcateg << delimit << transNum << delimit
    - << notes << delimit << origtype << endl;
    + text << wxT("\"") << dateString << wxT("\"") << delimit << wxT("\"") << payee << wxT("\"") << delimit << wxT("\"")
    + << sign << amount << wxT("\"") << delimit << wxT("\"") << categ << wxT("\"") << delimit << wxT("\"") << subcateg << wxT("\"")
    + << delimit << transNum << delimit << wxT("\"") << notes << wxT("\"") << delimit << wxT("\"") << origtype << wxT("\"") << endl;
    }

    numRecords++;

     
  • Sascha

    Sascha - 2010-02-28

    Sorry, I meant version 9.6.0, not 5.9.0

     
  • Pat O

    Pat O - 2011-06-12

    This appears to be resolved.

     
  • Pat O

    Pat O - 2011-06-12
    • labels: 1264369 -->
    • status: open --> closed-fixed
     
  • Stefano

    Stefano - 2011-06-17

    Discovered this was still a problem and is now fixed.

     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:

JavaScript is required for this form.





No, thanks