> On Jan. 1, 2014, 2:40 p.m., Thomas Baumgart wrote: > > kmymoney/dialogs/investtransactioneditor.cpp, line 1170 > > <https://git.reviewboard.kde.org/r/113427/diff/1/?file=205822#file205822line1170> > > > > Isn't that simply > > > > if(memo) { > > d->m_activity->m_memoChanged = (memo->toPlainText() != > > d->m_activity->m_memoText); > > }
done > On Jan. 1, 2014, 2:40 p.m., Thomas Baumgart wrote: > > kmymoney/dialogs/transactioneditor.cpp, line 278 > > <https://git.reviewboard.kde.org/r/113427/diff/1/?file=205824#file205824line278> > > > > You should also check for memo being a NULL ptr here just like in > > InvestTransactionEditor::slotUpdateInvestMemoState(void) Of course. Done - Allan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/113427/#review46543 ----------------------------------------------------------- On Oct. 24, 2013, 11:06 p.m., Allan Anderson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/113427/ > ----------------------------------------------------------- > > (Updated Oct. 24, 2013, 11:06 p.m.) > > > 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: https://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