Bug 119631

Summary: Once I launch KSysV, I can not close it down unless I issue a kill command.
Product: [Unmaintained] ksysv Reporter: John Andino <jgplt3095>
Component: generalAssignee: Niels Sascha Reedijk <niels.reedijk>
Status: RESOLVED FIXED    
Severity: normal CC: america69, madcoder
Priority: NOR    
Version First Reported In: 1.3.9   
Target Milestone: ---   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description John Andino 2006-01-06 15:08:11 UTC 
Version:           1.3.9 (using KDE 3.5.0, Mandrake Linux Cooker i586 - Cooker)
Compiler:          gcc version 3.3.1 (Mandrake Linux 9.2 3.3.1-2mdk)
OS:                Linux (i686) release 2.6.12-oci6.mdk-i586-up-1GB

When I launch KSysV I can not close it down so I would have to go to the terminal to issue a kill command, so far this is the only issue I am having.
Comment 1 crispo matteo 2006-02-24 23:34:41 UTC 
I have this problem with kde 3.5.1
Comment 2 Al Settle 2006-03-01 04:40:38 UTC 
ksysv version 1.3.9 (kde 3.5.1, Debian/GNU Linux etch/sid)
OS: Linux (i686)release 2.6.14-2-k7
Target: i486-linux-gnu

same here.
Comment 3 Dirk Mueller 2006-05-23 17:50:23 UTC 
*** Bug 125802 has been marked as a duplicate of this bug. ***
Comment 4 Dirk Mueller 2006-05-23 17:51:33 UTC 
SVN commit 544098 by mueller:

make it possible to close ksysv
BUG: 119631


 M  +0 -1      ServiceDlg.cpp  
 M  +4 -5      TopWidget.cpp  
 M  +1 -1      TopWidget.h  
 M  +1 -0      main.cpp  


--- branches/KDE/3.5/kdeadmin/ksysv/ServiceDlg.cpp #544097:544098
@@ -32,7 +32,6 @@
   : KDialogBase (parent, name, false, action, Apply|Close, Apply, true)
 {
   QWidget* page = new QWidget (this);
-  setMainWidget (page);
 
   QBoxLayout* top = new QVBoxLayout (page, 0, spacingHint());
 
--- branches/KDE/3.5/kdeadmin/ksysv/TopWidget.cpp #544097:544098
@@ -128,8 +128,6 @@
 							  this)),
     mVisible (new QCheckBox*[ksv::runlevelNumber])
 {
-  kapp->setMainWidget (this);
-
   setCaption(false);
 
   initStatusBar(); // order dependency
@@ -290,8 +288,10 @@
   createGUI(xmlFile());
 }
 
-bool KSVTopLevel::queryExit() {
-  uint res = KMessageBox::Yes;
+//
+bool KSVTopLevel::queryExit() 
+{
+  uint res = KMessageBox::Continue;
 
   if (mChanged) {
     res = KMessageBox::warningContinueCancel(kapp->mainWidget(),
@@ -301,7 +301,6 @@
 
   }
 
-  qApp->processEvents();
   return res == KMessageBox::Continue;
 }
 
--- branches/KDE/3.5/kdeadmin/ksysv/TopWidget.h #544097:544098
@@ -47,7 +47,7 @@
 public slots:
   void setCaption (bool changed);
 
-private:
+protected:
   virtual bool queryExit();
   virtual void closeEvent( QCloseEvent* e );
   void initTools();
--- branches/KDE/3.5/kdeadmin/ksysv/main.cpp #544097:544098
@@ -93,6 +93,7 @@
         }
 
       KSVTopLevel* top = new KSVTopLevel();
+      app.setMainWidget(top);
       top->show();
     }
   // end session-management