#165 CSV Exports need quoted values

closed-fixed
nobody
None
5
2011-06-12
2009-11-13
John Henderson
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

     

  • 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

    • labels: 1264369 -->
    • status: open --> closed-fixed
     
  • Stefano
    Stefano
    2011-06-17

    Discovered this was still a problem and is now fixed.

     


Anonymous


Cancel   Add attachments