From: Tomas Babej <tomasbabej@gmail.com>
Date: Thu, 6 Aug 2015 21:20:51 +0000 (+0200)
Subject: Do not use mutable lists in function definitions
X-Git-Url: https://git.madduck.net/etc/taskwarrior.git/commitdiff_plain/272ac3f6ece5a39abfa20d1da3dceb48be4685f6?ds=sidebyside;hp=15aaa065bcea43ea45069ce4114d031b70119344

Do not use mutable lists in function definitions
---

diff --git a/tasklib/task.py b/tasklib/task.py
index ed89653..35fc2b7 100644
--- a/tasklib/task.py
+++ b/tasklib/task.py
@@ -744,7 +744,7 @@ class Task(TaskResource):
 
         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")
@@ -798,8 +798,8 @@ class TaskFilter(SerializingObject):
     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):