[Kmymoney-devel] Review Request 108150: Display the number of not marked transactions for every account on KMM's homepage

[Marko Käning]

> On Jan. 19, 2013, 6:39 a.m., Thomas Baumgart wrote:
> > I have a added a few naming issues. No big deal. Otherwise, it looks good to me.

I'll take care of these naming issues before I commit this patch.

Anyway, on my TODO is still the splitting up of the corresponding test into separate test functions, as suggested by Lukazs.


> On Jan. 19, 2013, 6:39 a.m., Thomas Baumgart wrote:
> > kmymoney/mymoney/mymoneyfile.cpp, line 2933
> > <http://git.reviewboard.kde.org/r/108150/diff/8/?file=107793#file107793line2933>
> >
> >     I am not sure if any optimizer reduces that to
> >     
> >      return transactionList(filter).count();
> >     
> >     which is all we need.

This is what I get if I try it without a QList inserted:

/Users/marko/WC/GIT/kmymoney.homeview-additional_columns/kmymoney/mymoney/mymoneyfile.cpp: In member function ‘int MyMoneyFile::countTransactionsWithSpecificReconcileState(const QString&, MyMoneyTransactionFilter::stateOptionE) const’:
/Users/marko/WC/GIT/kmymoney.homeview-additional_columns/kmymoney/mymoney/mymoneyfile.cpp:2933: error: ‘((const MyMoneyFile*)this)->MyMoneyFile::transactionList’ does not have class type
make[2]: *** [kmymoney/mymoney/CMakeFiles/kmm_mymoney.dir/mymoneyfile.cpp.o] Error 1
make[1]: *** [kmymoney/mymoney/CMakeFiles/kmm_mymoney.dir/all] Error 2


- Marko


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


On Jan. 18, 2013, 10:11 p.m., Marko Käning wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108150/
> -----------------------------------------------------------
> 
> (Updated Jan. 18, 2013, 10:11 p.m.)
> 
> 
> Review request for KMymoney and Łukasz Maszczyński.
> 
> 
> Description
> -------
> 
> This patch introduces 3 new columns in the accounts tables on KMM's homepage (similar to [1]) which will list the number of 
> 
>  a) not marked ('!M'),  
> 
>  b) "only" cleared ('C'), 
> 
>  c) as well as all not reconciled ('!R')
> 
> transactions for each specific account.
> 
> 
> Every one of these columns can be individually switched on.
> 
> 
> This information will give the user the chance to immediately spot accounts which might need to be looked after.
> 
> 
> ---
> [1] "homepage: extend accounts table with additional columns to reflect online-status" https://git.reviewboard.kde.org/r/107364/
> 
> 
> Diffs
> -----
> 
>   kmymoney/dialogs/settings/ksettingshomedecl.ui 4cb36cced4801f5d6ecf35ef84f375a629cd8792 
>   kmymoney/kmymoney.kcfg a60ccad153385c961fa15c5b0964588c14e7ad89 
>   kmymoney/mymoney/mymoneyfile.h 0f114457877e6cebc44ce991741984643e409136 
>   kmymoney/mymoney/mymoneyfile.cpp eb99876a924d13287f3d80786d2da6b05dfd3417 
>   kmymoney/mymoney/mymoneyfiletest.h 5e397981c40eb006449fb73fd58a006b8fc16d82 
>   kmymoney/mymoney/mymoneyfiletest.cpp c831ad3586756b91469567dd934cbd0165df570d 
>   kmymoney/views/khomeview.cpp 7da6980b866ff2523ddfb07e0826d3394d1b7157 
> 
> Diff: http://git.reviewboard.kde.org/r/108150/diff/
> 
> 
> Testing
> -------
> 
> Builds, tests and runs fine.
> 
> TODO:
>  - implement sub-tests as separate test-functions
> 
> 
> Thanks,
> 
> Marko Käning
> 
>

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