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

Ship it!


Ship It!

- Cristian Oneț


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
> 
>

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

Reply via email to