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

Hrvoje Senjan hrvoje.senjan at gmail.com
Tue Apr 7 18:39:42 UTC 2015


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

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/20150407/5a3e357c/attachment.html>


More information about the Plasma-devel mailing list