Bug 299163

Summary: Effects cannot be configured
Product: [Plasma] kwin Reporter: Martin Flöser <mgraesslin>
Component: compositingAssignee: KWin default assignee <kwin-bugs-null>
Status: RESOLVED FIXED    
Severity: normal Keywords: regression
Priority: NOR    
Version First Reported In: git master   
Target Milestone: 4.9   
Platform: unspecified   
OS: Linux   
Latest Commit: http://commits.kde.org/kde-workspace/8f31f53ce7e942884f1d9ac30783745785102875 Version Fixed/Implemented In:
Sentry Crash Report:

Description Martin Flöser 2012-05-01 12:03:10 UTC 
Any change in the configuration of an effect is not applied due to change in 
236d4c81bc75768dbc8cf5e334cd78f855e31b06

The effects configuration use EffectsHandler::effectConfig which returns a wrong config file when used from kcmshell.

Trivial fix: revert the commit.Better fix: make config modules not use EffectsHandler, as that would also mean they don't have to link the kwineffects lib.
Comment 1 Martin Flöser 2012-05-02 15:45:33 UTC 
Git commit 8f31f53ce7e942884f1d9ac30783745785102875 by Martin Gräßlin.
Committed on 01/05/2012 at 16:15.
Pushed by graesslin into branch 'master'.

Revert 236d4c8 for EffectsHanlder::effectConfig

EffectConfig is used also in the config modules which
means kwinrc needs to be explicitly specified.
REVIEW: 104808

M  +2    -1    kwin/libkwineffects/kwineffects.cpp

http://commits.kde.org/kde-workspace/8f31f53ce7e942884f1d9ac30783745785102875