Commits

Tetsuya Morimoto committed ddf37b0

fixed to save traccron settings about 'endswith' method

  • Participants
  • Parent commits 1e46203

Comments (0)

Files changed (1)

File src/traccron/core.py

                 arg_name_list.append('%s.%s.%s' % (task_id, schedule_id,
                                      self.cronconf.SCHEDULE_ARGUMENT_KEY))
 
+        enabled_keys = ('.%s' % self.cronconf.TASK_ENABLED_KEY,
+                        '.%s' % self.cronconf.SCHEDULE_ENABLED_KEY)
         for arg_name in arg_name_list:
             arg_value = req.args.get(arg_name, '').strip()
             self.env.log.debug('request args: %s=[%s]' % (arg_name, arg_value))
             if (arg_value == ''):
                 # dont't remove the key because of default value may be True
-                if arg_name.endswith('.' + self.cronconf.TASK_ENABLED_KEY,
-                                     '.' + self.cronconf.SCHEDULE_ENABLED_KEY):
+                if arg_name.endswith(enabled_keys):
                     self.cronconf.set_value(arg_name, 'False')
                 else:
                     # otherwise we can remove the key
             self.env.log.debug('request args: %s=[%s]' % (arg_name, arg_value))
             if (arg_value == ''):
                 # dont't remove the key because of default value may be True
-                if (arg_name.endswith('.' + self.cronconf.TASK_ENABLED_KEY)):
+                if arg_name.endswith('.%s' % self.cronconf.TASK_ENABLED_KEY):
                     self.cronconf.set_value(arg_name, 'False')
                 else:
                     # otherwise we can remove the key