[Kmymoney-devel] Review Request: Patch which highlights the online statement balance if different from current file balance

[Marko Käning]

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

(Updated Nov. 27, 2011, 5:15 p.m.)


Review request for KMymoney.


Changes
-------

changes to MyMoneyFile::hasMatchingOnlineBalance() according to Thomas' review comments
    - make MyMoneyFile::hasMatchingOnlineBalance() const
    - function now also returns false when there is no online balance present
    - amend doxygen info accordingly


Description
-------

Highlighting of online statement balance with red background and
highlighted text if file balance and online balance are different.
(This patch was put together from Thomas' and Cristian's advice on ML)


This addresses bug 287494.
    http://bugs.kde.org/show_bug.cgi?id=287494


Diffs (updated)
-----

  kmymoney/mymoney/mymoneyfile.h d44e2ea 
  kmymoney/mymoney/mymoneyfile.cpp f573f9a 
  kmymoney/widgets/register.h 8cd1a32 
  kmymoney/widgets/register.cpp 39d991c 

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


Testing
-------

Yes


Screenshots
-----------

small snapshot showing the highlighted group marker
  http://git.reviewboard.kde.org/r/103264/s/345/


Thanks,

Marko Käning

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