[Kmymoney-devel] Review Request 109803: Add CSV export capability.

[Allan Anderson]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109803/
-----------------------------------------------------------

(Updated April 24, 2013, 4:16 p.m.)


Review request for KMymoney.


Changes
-------

Attached new complete patch with Thomas's recommendations.  Also, removed two unneeded Date SLOTs from csveportdlg.cpp, and removed superfluous line from investprocessing.cpp.


Description
-------

Add CSV export capability.  Modify existing KMyMoney File menu in order to make menu item positions more logical.


This addresses bug 317614.
    http://bugs.kde.org/show_bug.cgi?id=317614


Diffs (updated)
-----

  kmymoney/kmymoneyui.rc f353641 
  kmymoney/plugins/CMakeLists.txt 81ca458 
  kmymoney/plugins/csvexport/CMakeLists.txt PRE-CREATION 
  kmymoney/plugins/csvexport/csvexportdlg.h PRE-CREATION 
  kmymoney/plugins/csvexport/csvexportdlg.cpp PRE-CREATION 
  kmymoney/plugins/csvexport/csvexportdlg.ui PRE-CREATION 
  kmymoney/plugins/csvexport/csvexporterplugin.h PRE-CREATION 
  kmymoney/plugins/csvexport/csvexporterplugin.cpp PRE-CREATION 
  kmymoney/plugins/csvexport/csvwriter.h PRE-CREATION 
  kmymoney/plugins/csvexport/csvwriter.cpp PRE-CREATION 
  kmymoney/plugins/csvexport/kmm_csvexport.desktop PRE-CREATION 
  kmymoney/plugins/csvexport/kmm_csvexport.rc PRE-CREATION 
  kmymoney/plugins/csvimport/investprocessing.cpp 07d4d82 
  kmymoney/plugins/csvimport/kmm_csvimport.rc d678169 

Diff: http://git.reviewboard.kde.org/r/109803/diff/


Testing
-------

Exported numerous checking and investment CSV files, and then imported into KMyMoney via CSV import (discovering a few minor issues in the existing KMyMoney accounts in the process.)
Ran astyle and Krazy2.


Thanks,

Allan Anderson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kmymoney-devel/attachments/20130424/fc7172e1/attachment.html>