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

Review request for KMymoney.


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


Repository: kmymoney


Description
-------

There was a previous fix for this problem - Git commit 
9485826cfb50816d2df4dac9709b4beb255b8b75 by Cristian One?. Unfortunately, this 
was inadvertently disabled when BUG:311481 REVIEW:107714 was committed.  This 
is now fixed.
This review adds the same capability for investment transactions.
In addition, there have been requests that when clearing the memo field, it 
should when required be empty rather that containing a blank character.  It is 
necessary to enter a character in the memo field in order to signal that an 
edit has occurred, but now that character may then be deleted to leave the 
field empty, if that is what is required.
With investment transactions, it is always necessary to enter the security name 
as well. This requirement could probably be removed, but it is probably 
sensible to be editing just a single security.
As with all editing of multiple items, all fields appear as blank initially. 
This has been retained here with multiple memo editing, but if desired, this 
could be changed and the field could be left showing previous content, which 
could be more intuitive for a user wanting to have an empty field.


Diffs
-----

  kmymoney/dialogs/investactivities.h aa4800f 
  kmymoney/dialogs/investactivities.cpp e4760e5 
  kmymoney/dialogs/investtransactioneditor.h 20e3819 
  kmymoney/dialogs/investtransactioneditor.cpp 805bd8d 
  kmymoney/dialogs/transactioneditor.h f07dafb 
  kmymoney/dialogs/transactioneditor.cpp 71d94ec 

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


Testing
-------

Numerous groups of investment and checking transactions entered and edited 
correctly.


Thanks,

Allan Anderson

_______________________________________________
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Reply via email to