CVS commit by markey: Delete KNewStuff's config entries before showing the download dialog. This solves the problem that the check-marks get out of sync after uninstalling scripts, until we find a real solution. I think it's acceptable this way, as the ScriptManager detects when a script is already installing, and prevents from installing it twice. M +5 -0 scriptmanager.cpp 1.82 --- kdeextragear-1/amarok/src/scriptmanager.cpp #1.81:1.82 @@ -339,4 +339,9 @@ void ScriptManager::slotRetrieveScript() { + // Delete KNewStuff's configuration entries. These entries reflect what has + // already been installed. As we cannot yet keep them in sync after uninstalling + // scripts, we deactivate the check marks entirely. + amaroK::config()->deleteGroup( "KNewStuffStatus" ); + // we need this because KNewStuffGeneric's install function isn't clever enough AmarokScriptNewStuff *kns = new AmarokScriptNewStuff( "amarok/script", this );