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

List:       kde-commits
Subject:    branches/KDE/3.5/kdewebdev/kommander
From:       Andras Mantia <amantia () kde ! org>
Date:       2007-10-06 16:42:42
Message-ID: 1191688962.554264.29786.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 722103 by amantia:

Fix ExecButton.setEnabled().
BUG: 113624

 M  +1 -0      ChangeLog  
 M  +1 -1      widget/kommanderwidget.cpp  
 M  +2 -2      widgets/execbutton.cpp  


--- branches/KDE/3.5/kdewebdev/kommander/ChangeLog #722102:722103
@@ -3,6 +3,7 @@
 	o do not remove the whole text after undoing the changes in the editor [#147817]
 	o don't crash if opening a file the second time when the first try failed [#141110]
 	o don't close the dialogs with ESC (just like in real applications)
+	o fix ExecButton.setEnabled() [#113624]
 
 Thu Sep 14 20:01:18 2006 - Michal Rudolf <mrudolf@kdewebdev.org>
 (actually imported from a work branch on Sep 29, 2007 by Andras Mantia <amantia@kde.org>)
--- branches/KDE/3.5/kdewebdev/kommander/widget/kommanderwidget.cpp #722102:722103
@@ -672,7 +672,7 @@
     return QString();
   switch(function) {
     case DCOP::setEnabled:
-      current->setEnabled(args[0] != "false" && args[0] != "0");
+      current->setEnabled( args[0] != "false" && args[0] != "0");
       break;
     case DCOP::setVisible:
       current->setShown(args[0] != "false" && args[0] != "0");
--- branches/KDE/3.5/kdewebdev/kommander/widgets/execbutton.cpp #722102:722103
@@ -96,7 +96,7 @@
 void ExecButton::startProcess()
 {
   QString at = evalAssociatedText().stripWhiteSpace();
-  
+  bool enabledStatus = isEnabled();
   if (m_blockGUI != None)
     setEnabled(false);
   if (m_blockGUI == GUI)
@@ -107,11 +107,11 @@
   m_output = process->run(at);
   if (m_blockGUI == GUI)
   {
-    setEnabled(true);
     KApplication::restoreOverrideCursor();
     if (writeStdout())
       cout << m_output << flush;
   }
+  setEnabled(enabledStatus);
 }
 
 
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic