[Kmymoney-devel] Review Request: Add payee source selection to import OFX file open dialog

[Thomas Baumgart]

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

Review request for kmymoney.


Summary
-------

This patch allows to select the source of the payee information for file imports. It supports the same settings as the OFX direct connect version. The selection is (currently) non-persistent and defaults to the payee id field.


Diffs
-----

  trunk/extragear/office/kmymoney/kmymoney/kmymoney.h 1248770 
  trunk/extragear/office/kmymoney/kmymoney/kmymoney.cpp 1248770 
  trunk/extragear/office/kmymoney/kmymoney/plugins/importinterface.h 1248770 
  trunk/extragear/office/kmymoney/kmymoney/plugins/interfaces/kmmimportinterface.h 1248770 
  trunk/extragear/office/kmymoney/kmymoney/plugins/interfaces/kmmimportinterface.cpp 1248770 
  trunk/extragear/office/kmymoney/kmymoney/plugins/ofximport/CMakeLists.txt 1248770 
  trunk/extragear/office/kmymoney/kmymoney/plugins/ofximport/ofximporterplugin.cpp 1248770 

Diff: http://svn.reviewboard.kde.org/r/6777/diff


Testing
-------

Due to the lack of OFX files that require the setting to be changed, I only added some debug statements (which have been removed in the patch) to verify that the controlling variable d->m_preferName is setup correctly after the dialog is closed. Further processing should be identical to the OFX direct connect version.

Someone who needs this option should do some more testing and report about the results.


Screenshots
-----------

OFX file open dialog
  http://svn.reviewboard.kde.org/r/6777/s/618/


Thanks,

Thomas

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kmymoney-devel/attachments/20110823/44c4e9d3/attachment-0001.html>