Review Request 123288: Fix some more KConfig usage (wrt bug 340691)

[David Faure]

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




kcms/componentchooser/componentchooserbrowser.cpp (line 86)
<https://git.reviewboard.kde.org/r/123288/#comment64716>

    This change makes it impossible for me to choose another browser than firefox on OpenSuSE, where /etc/xdg/kdeglobals says BrowserApplication=firefox.
    
    (comment added in https://bugs.kde.org/show_bug.cgi?id=340691#c9)
    
    SimpleConfig means that upon saving, kconfig doesn't know it has to write
    BrowserApplication[$d]
    It does when I remove SimpleConfig, which fixes the bug for me.
    
    This seems pretty logical to me, so this fix is clearly wrong. If something has to be fixed around some "copyTo" that the bug mentions, then it has to be fixed differently.


- David Faure


On April 7, 2015, 6:50 p.m., Hrvoje Senjan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123288/
> -----------------------------------------------------------
> 
> (Updated April 7, 2015, 6:50 p.m.)
> 
> 
> Review request for Plasma, David Edmundson, Eike Hein, and Marco Martin.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> -------
> 
> We only need to sync the changes to kde4 kdeglobals once users changes the defaults, hence only users KF5 kdeglobals need to be evaluated.
> This workarounds (though the change is IMO correct anyway) the kconfig bug, as we can otherwise pass both local and systemwide kdeglobals into copyTo method.
> 
> 
> Diffs
> -----
> 
>   kcms/componentchooser/componentchooserbrowser.cpp f2b81ef 
>   kcms/componentchooser/componentchooserterminal.cpp be52334 
>   kcms/input/mouse.cpp 69759bd 
> 
> Diff: https://git.reviewboard.kde.org/r/123288/diff/
> 
> 
> Testing
> -------
> 
> W/o the patch i had both
> BrowserApplication[$e]=chromium-browser.desktop (local adjustement)
> BrowserApplication=firefox.desktop (distro branding)
> 
> in ~/.kde4/share/config/kdeglobals
> now i have only locally adjusted value
> 
> 
> Thanks,
> 
> Hrvoje Senjan
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160513/38e96452/attachment.html>