From 5fdaf73fdf43a1711519ae8bf202713e39bdd36a Mon Sep 17 00:00:00 2001
From: Calum Lind <calumlind+deluge@gmail.com>
Date: Mon, 7 Jul 2014 18:50:27 +0100
Subject: [PATCH] [GTKUI] Fix quitting bypassing password lock

---
 deluge/ui/gtkui/mainwindow.py | 31 +++++++++++++++++++++----------
 deluge/ui/gtkui/systemtray.py |  6 ------
 2 files changed, 21 insertions(+), 16 deletions(-)

diff --git a/deluge/ui/gtkui/mainwindow.py b/deluge/ui/gtkui/mainwindow.py
index e5337bc1a..923f40f62 100644
--- a/deluge/ui/gtkui/mainwindow.py
+++ b/deluge/ui/gtkui/mainwindow.py
@@ -236,18 +236,29 @@ class MainWindow(component.Component):
         :param shutdown: whether or not to shutdown the daemon as well
         :type shutdown: boolean
         """
-        def stop_gtk_reactor(result=None):
-            try:
-                reactor.stop()
-            except ReactorNotRunning:
-                log.debug("Attempted to stop the reactor but it is not running...")
+        def quit_gtkui():
+            def stop_gtk_reactor(result=None):
+                try:
+                    reactor.stop()
+                except ReactorNotRunning:
+                    log.debug("Attempted to stop the reactor but it is not running...")
 
-        if shutdown:
-            client.daemon.shutdown().addCallback(stop_gtk_reactor)
-        elif not client.is_classicmode() and client.connected():
-            client.disconnect().addCallback(stop_gtk_reactor)
+            if shutdown:
+                client.daemon.shutdown().addCallback(stop_gtk_reactor)
+            elif not client.is_classicmode() and client.connected():
+                client.disconnect().addCallback(stop_gtk_reactor)
+            else:
+                stop_gtk_reactor()
+
+        if self.config["tray_password"] and not self.visible():
+            dialog = PasswordDialog("Enter your pasword to Quit Deluge...")
+            def on_dialog_response(response_id):
+                if response_id == gtk.RESPONSE_OK:
+                    if self.config["tray_password"] == sha(dialog.get_password()).hexdigest():
+                        quit_gtkui()
+            dialog.run().addCallback(on_dialog_response)
         else:
-            stop_gtk_reactor()
+            quit_gtkui()
 
     def load_window_state(self):
         x = self.config["window_x_pos"]
diff --git a/deluge/ui/gtkui/systemtray.py b/deluge/ui/gtkui/systemtray.py
index 916d2248c..25865dbe1 100644
--- a/deluge/ui/gtkui/systemtray.py
+++ b/deluge/ui/gtkui/systemtray.py
@@ -375,16 +375,10 @@ class SystemTray(component.Component):
 
     def on_menuitem_quit_activate(self, menuitem):
         log.debug("on_menuitem_quit_activate")
-        if self.config["lock_tray"] and not self.window.visible():
-            self.window.present()
-
         self.window.quit()
 
     def on_menuitem_quitdaemon_activate(self, menuitem):
         log.debug("on_menuitem_quitdaemon_activate")
-        if self.config["lock_tray"] and not self.window.visible():
-            self.window.present()
-
         self.window.quit(shutdown=True)
 
     def on_tray_setbwdown(self, widget, data=None):