Do not use mutable lists in function definitions

diff --git a/tasklib/task.py b/tasklib/task.py
index ed89653eab7e593a396bcaa6720cc2266e859599..35fc2b710c7d99fde6cd2397a02fbe442aa2a0ec 100644 (file)
--- a/tasklib/task.py
+++ b/tasklib/task.py
@@ -744,7 +744,7 @@ class Task(TaskResource):
 
         return args
 
 
         return args
 

-    def refresh(self, only_fields=[], after_save=False):
+    def refresh(self, only_fields=None, after_save=False):

         # Raise error when trying to refresh a task that has not been saved
         if not self.saved:
             raise Task.NotSaved("Task needs to be saved to be refreshed")
         # Raise error when trying to refresh a task that has not been saved
         if not self.saved:
             raise Task.NotSaved("Task needs to be saved to be refreshed")


@@ -798,8 +798,8 @@ class TaskFilter(SerializingObject):
     A set of parameters to filter the task list with.
     """
 
     A set of parameters to filter the task list with.
     """
 

-    def __init__(self, warrior, filter_params=[]):
-        self.filter_params = filter_params
+    def __init__(self, warrior, filter_params=None):
+        self.filter_params = filter_params or []

         super(TaskFilter, self).__init__(warrior)
 
     def add_filter(self, filter_str):
         super(TaskFilter, self).__init__(warrior)
 
     def add_filter(self, filter_str):