[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    kdeextragear-1/amarok/src
From:       Mark Kretschmann <markey () web ! de>
Date:       2005-04-28 8:56:42
Message-ID: 20050428085642.43B4A662 () office ! kde ! org
[Download RAW message or body]

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


[prev in list] [next in list] [prev in thread] [next in thread]