[Kmymoney-devel] Review Request 114767: Add ability to modify loan institution attribute.

Jeremy Whiting jpwhiting at kde.org
Fri Jan 3 17:06:24 UTC 2014


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

(Updated Jan. 3, 2014, 10:06 a.m.)


Review request for KMymoney.


Changes
-------

Updated as per review. One more thing is missing however, if I edit a loan that has been closed, the finish button is throwing an exception because the loan's schedule is invalid or something. I guess I need to change what happens in the finish a bit also.


Bugs: 257619
    http://bugs.kde.org/show_bug.cgi?id=257619


Repository: kmymoney


Description (updated)
-------

Add ability to modify loan institution attribute.

Add a new choice to editselectionwizardpage to modify only loan attributes.
Add a new loanattributes page to modify just institution for now, will add other attributes next.

BUG:257619


Diffs (updated)
-----

  kmymoney/wizards/newloanwizard/CMakeLists.txt 939d569c2bba56ccfd3c9d6b28f8b90d414b9332 
  kmymoney/wizards/newloanwizard/editselectionwizardpage.cpp e8cf0353a6aa357f6fc3d818aa3819aa13cf25c5 
  kmymoney/wizards/newloanwizard/editselectionwizardpagedecl.ui 2e482c47f9a3642a880f0dcb047ca577c6e0446c 
  kmymoney/wizards/newloanwizard/keditloanwizard.cpp 5590a359ffc16c92fc3dc24b6484aac1180b1507 
  kmymoney/wizards/newloanwizard/knewloanwizard.h 6b3f049912605f4f9cb0e78ee83b14e2767586b8 
  kmymoney/wizards/newloanwizard/knewloanwizarddecl.ui 831d85ce388b51a904c76cd998f54ca6a099b347 
  kmymoney/wizards/newloanwizard/loanattributeswizardpage.h PRE-CREATION 
  kmymoney/wizards/newloanwizard/loanattributeswizardpage.cpp PRE-CREATION 
  kmymoney/wizards/newloanwizard/loanattributeswizardpagedecl.ui PRE-CREATION 

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


Testing
-------

It works here to set and to show the current institution.


Thanks,

Jeremy Whiting

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


More information about the KMyMoney-devel mailing list