API function "change_complete_action" should save to the INI files.

This bug made Plush forget the latest EOQ action.
Set default value of special queue_complete_pers to False (leads to less mistakes).

sabnzbd/__init__.py

@@ -655,6 +655,10 @@ def change_queue_complete_action(action, new=True):
     else:
         action = None
 
+    if new:
+        cfg.queue_complete.set(action)
+        config.save_config()
+
     #keep the name of the action for matching the current select in queue.tmpl
     QUEUECOMPLETE = action
 

sabnzbd/cfg.py

@@ -108,7 +108,7 @@ bookmark_rate = OptionNumber('newzbin', 'bookmark_rate', 60, minval=15, maxval=2
 top_only = OptionBool('misc', 'top_only', True)
 autodisconnect = OptionBool('misc', 'auto_disconnect', True)
 queue_complete = OptionStr('misc', 'queue_complete')
-queue_complete_pers = OptionBool('misc', 'queue_complete_pers', True)
+queue_complete_pers = OptionBool('misc', 'queue_complete_pers', False)
 
 replace_spaces = OptionBool('misc', 'replace_spaces', False)
 replace_dots = OptionBool('misc', 'replace_dots', False)

sabnzbd/interface.py

@@ -674,8 +674,6 @@ class QueuePage(object):
         if msg: return msg
         action = kwargs.get('action')
         sabnzbd.change_queue_complete_action(action)
-        cfg.queue_complete.set(action)
-        config.save_config()
         raise queueRaiser(self.__root, kwargs)
 
     @cherrypy.expose