[Kmymoney-devel] Review Request 119635: Replaced KMyMoneyGlobalSettings::setSubstringSearch(QWidget* w) to make kmm_config independent of widgets

Cristian Oneț onet.cristian at gmail.com
Fri Aug 8 10:24:29 UTC 2014



> On Aug. 8, 2014, 10:23 a.m., Cristian Oneț wrote:
> > Ship It!

This has been anoying me to for a while now and your patch looks nice.


- Cristian


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


On Aug. 6, 2014, 1:41 p.m., Christian David wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119635/
> -----------------------------------------------------------
> 
> (Updated Aug. 6, 2014, 1:41 p.m.)
> 
> 
> Review request for KMymoney.
> 
> 
> Repository: kmymoney
> 
> 
> Description
> -------
> 
> Replaced
> void KMyMoneyGlobalSettings::setSubstringSearch(QWidget* w)
> by void KMyMoneyMVCCombo::setSubstringSearchForChildren(QWidget*const widget, bool enabled)
> 
> To use this commit you must replace
> KMyMoneyGlobalSettings::setSubstringSearch(widgetPtr);
> by
> KMyMoneyMVCCombo::setSubstringSearchForChildren(widgetPtr, !KMyMoneySettings::stringMatchFromStart());
> 
> So KMyMoneyMVCCombo is not needed in KMyMoneyGlobalSettings anymore and
> the target kmm_config can be used without linking the target "widgets".
> 
> My goal is to clean up the build of all widgets. Later there should be
> 1) a shared library which contains all KMyMoney widgets which are interessting for plugins,
> 2) a static library which contains widgets which are not interessting for plugins (if there are such widgets, at the moment nearly everything is put here)
> 3)?a shared library for QtDesigner (which is not working at the moment)
> 
> 
> Diffs
> -----
> 
>   kmymoney/kmymoney.cpp c4bbf0cbf46250f69f7e2e9c13471c4e6c7cb261 
>   kmymoney/kmymoney.kcfg 5d5e09cf9b1d94cec07fe97cda388819fd8a97ed 
>   kmymoney/kmymoneyglobalsettings.cpp a6742fa9e329579fd81548b20bfda766ebb27863 
>   kmymoney/widgets/kmymoneymvccombo.h a9d8e37ee0a600240c7ca4544964a448770d0a24 
>   kmymoney/widgets/kmymoneymvccombo.cpp cf7c21b25716aa99d9e72ee19883f371da816c0c 
>   kmymoney/wizards/endingbalancedlg/kendingbalancedlg.cpp adb2adae4c8e086df0a1ce5e62a36cb26859dc76 
>   kmymoney/wizards/newloanwizard/knewloanwizard.cpp 5b8134e1345a863a13469dfd32eb56ed99b9bdc2 
> 
> Diff: https://git.reviewboard.kde.org/r/119635/diff/
> 
> 
> Testing
> -------
> 
> 1. kwriteconfig --file kmymoneyrc --group "General Options" --key "stringMatchFromStart" --type bool true
> 2. restarted KMyMoney
> 3. tried completion for payees in register
>  -- worked as expected --
> 4. removed option again
> 5. restarted KMyMoney
> 6. tried completion for payees in register
>  -- worked as expected --
> 
> 
> Thanks,
> 
> Christian David
> 
>

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


More information about the KMyMoney-devel mailing list