Merge pull request metabrainz#2192 from rdswift/quit_command_mods

PICARD-2615: Picard does not check for unsaved files when using the QUIT command

picard/tagger.py

@@ -547,6 +547,10 @@ def handle_command_pause(self, argstring):
             log.error("No command pause time specified.")
 
     def handle_command_quit(self, argstring):
+        if not argstring.upper() == 'FORCE' and self.window.show_quit_confirmation():
+            log.info("QUIT command cancelled by the user.")
+            RemoteCommands.set_quit(False)  # Allow queueing more commands.
+            return
         self.exit()
         self.quit()
 

picard/util/remotecommands.py

@@ -74,7 +74,9 @@ def __init__(self, method_name, help_text=None, help_args=None):
     ),
     "QUIT": RemoteCommand(
         "handle_command_quit",
-        help_text="Exit the running instance of Picard.",
+        help_text="Exit the running instance of Picard. "
+        "Use the argument 'FORCE' to bypass Picard's unsaved files check.",
+        help_args="[FORCE]",
     ),
     "REMOVE": RemoteCommand(
         "handle_command_remove",