Review Request 129081: Simplify redefinedlg

Łukasz Wojniłowicz lukasz.wojnilowicz at gmail.com
Sun Oct 2 09:55:52 UTC 2016


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

Review request for KMymoney.


Repository: kmymoney


Description
-------

1) renamed redefinedlg.cpp to transactiondlg.cpp,
2) merged RedefineDlgDecl and RedefineDlg,
3) initialize transactiondlg only when it's needed,
4) fit size of transactiondlg to content,
5) connected slotAccepted and slotRejected in ui file,
6) removed redundant variables,
7) displayLine doesn't need to know what it displays,
8) add column headers to transactiondlg,
9) it doesn't matter if transaction type is unrecognised or invalid,


Diffs
-----

  kmymoney/plugins/csvimport/CMakeLists.txt 6c57307 
  kmymoney/plugins/csvimport/csvwizard.cpp 3867cd8 
  kmymoney/plugins/csvimport/investmentwizardpage.h 574c4b6 
  kmymoney/plugins/csvimport/investmentwizardpage.cpp 5917e65 
  kmymoney/plugins/csvimport/redefinedlg.h 9594a90 
  kmymoney/plugins/csvimport/redefinedlg.cpp b7ca3f4 
  kmymoney/plugins/csvimport/redefinedlgdecl.ui 12c6191 
  kmymoney/plugins/csvimport/transactiondlg.h PRE-CREATION 
  kmymoney/plugins/csvimport/transactiondlg.cpp PRE-CREATION 
  kmymoney/plugins/csvimport/transactiondlg.ui PRE-CREATION 

Diff: https://git.reviewboard.kde.org/r/129081/diff/


Testing
-------


Thanks,

Łukasz Wojniłowicz

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


More information about the KMyMoney-devel mailing list