[Kmymoney-devel] Review Request 110212: Add Merge Payee Option

[Felipe Tonello]

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

(Updated April 7, 2014, 11:06 a.m.)


Review request for KMymoney.


Changes
-------

* Fixed kmymoney.rc kpartgui version.


Summary (updated)
-----------------

Add Merge Payee Option


Repository: kmymoney


Description
-------

payee: Does not merge when payees are empty

When there is no transactions to be merged we don't need to merge. In fact if
we don't stop there the rest of the code will delete the payees, which is a
behavior for delete payees functionality.

Signed-off-by: Felipe F. Tonello <eu at felipetonello.com>

payee: Show transactions from multiple payees

This patch allow users to have see all transactions that they
will merge.

Signed-off-by: Felipe F. Tonello <eu at felipetonello.com>

payee: Added Payees Merge KButton

Adds a button next to the delete payee and enables it only when multiple
payees are selected.

Signed-off-by: Felipe F. Tonello <eu at felipetonello.com>

payee: Added payee_merge KAction

Added KAction to the context menu and connected to the slotPayeeMerge().

Signed-off-by: Felipe F. Tonello <eu at felipetonello.com>

payee: Added support to Merge Payees

This patch implements the opeartion to merge payees. This is done by reusing
the payee delete slot, modifying it to support merge operation and add support
to it in KPayeeReassignDlg dialog.

The main difference between delete a payee and merging a payee is that when
merging the user can merge to a new payee. So the modification was done to
support this use-case.

Signed-off-by: Felipe F. Tonello <eu at felipetonello.com>

payee: change slotPayeeNew(...) to return boolean value

This is usefull if the user of that function wants to know if the user created
a new payee or not.

Signed-off-by: Felipe F. Tonello <eu at felipetonello.com>

payee: Make PayeeReassignDlg support other operations then delete

This is useful so it can be implemented a merge opration and maybe others in
future as well.

Signed-off-by: Felipe F. Tonello <eu at felipetonello.com>

payee: Better explanation when deleting payee

Signed-off-by: Felipe F. Tonello <eu at felipetonello.com>


Diffs (updated)
-----

  kmymoney/kmymoney.h 56e0bd6a3de42bdaa3fbb2f8586495cbb4d555ca 
  kmymoney/dialogs/kpayeereassigndlgdecl.ui 2cf563d4ea755e8d89ff2587fa677f14453cbaa5 
  kmymoney/dialogs/kpayeereassigndlg.cpp 8bdb25c0b3478c42715fac9f052903ed45b3614c 
  kmymoney/dialogs/kpayeereassigndlg.h a348d2b9fdc68a7c56a616a862a3373ef35aa0ba 
  kmymoney/kmymoney.cpp 8443d3077e59c7f147554026a3c8b915015295f5 
  kmymoney/kmymoneyui.rc 65afc920cef7091e3138c9acf75f73ec96929445 
  kmymoney/views/kpayeesview.h c747e691f6509c7bf60201dfb3cc77e9f939a7c5 
  kmymoney/views/kpayeesview.cpp 7252241886e05f00555d16dc1388b9d6ecef36a2 
  kmymoney/views/kpayeesviewdecl.ui 6efb2ae5f94383851ee2ab15da36187b7079815a 

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


Testing
-------

Normal UI test going trough all the possible options of users input.


Thanks,

Felipe Tonello

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