change default argument list to None (#413)

Author: kenta-takeuchi

linebot/api.py

@@ -1291,7 +1291,7 @@ def get_bot_info(self, timeout=None):
 
         return BotInfo.new_from_json_dict(response.json)
 
-    def create_audience_group(self, audience_group_name, audiences=[],
+    def create_audience_group(self, audience_group_name, audiences=None,
                               is_ifa=False, timeout=None):
         """Create an audience group.
 
@@ -1302,6 +1302,9 @@ def create_audience_group(self, audience_group_name, audiences=[],
         :param bool is_ifa: true | false
         :return: audience group id
         """
+        if audiences is None:
+            audiences = []
+
         if audiences:
             audiences = [Audience.new_from_json_dict(audience) for audience in audiences]
         response = self._post(

linebot/async_api.py

@@ -1338,7 +1338,7 @@ async def get_bot_info(self, timeout=None):
         return BotInfo.new_from_json_dict((await response.json))
 
     async def create_audience_group(
-        self, audience_group_name, audiences=[], is_ifa=False, timeout=None
+        self, audience_group_name, audiences=None, is_ifa=False, timeout=None
     ):
         """Create an audience group.
 
@@ -1349,6 +1349,9 @@ async def create_audience_group(
         :param bool is_ifa: true | false
         :return: audience group id
         """
+        if audiences is None:
+            audiences = []
+
         if audiences:
             audiences = [
                 Audience.new_from_json_dict(audience) for audience in audiences

linebot/models/filter.py

@@ -68,7 +68,7 @@ def __init__(self, **kwargs):
 class GenderFilter(DemographicFilter):
     """GenderFilter."""
 
-    def __init__(self, one_of=[], **kwargs):
+    def __init__(self, one_of=None, **kwargs):
         """__init__ method.
 
         :param one_of: Send messages to users of a given gender. One of:
@@ -78,14 +78,17 @@ def __init__(self, one_of=[], **kwargs):
         """
         super(GenderFilter, self).__init__(**kwargs)
 
+        if one_of is None:
+            one_of = []
+
         self.type = "gender"
         self.one_of = one_of
 
 
 class AppTypeFilter(DemographicFilter):
     """AppTypeFilter."""
 
-    def __init__(self, one_of=[], **kwargs):
+    def __init__(self, one_of=None, **kwargs):
         """__init__ method.
 
         :param one_of: Send messages to users of the specified OS. One of:
@@ -95,21 +98,27 @@ def __init__(self, one_of=[], **kwargs):
         """
         super(AppTypeFilter, self).__init__(**kwargs)
 
+        if one_of is None:
+            one_of = []
+
         self.type = "appType"
         self.one_of = one_of
 
 
 class AreaFilter(DemographicFilter):
     """AreaFilter."""
 
-    def __init__(self, one_of=[], **kwargs):
+    def __init__(self, one_of=None, **kwargs):
         """__init__ method.
 
         :param one_of: Send messages to users in the specified region.
         :type one_of: list[str]
         """
         super(AreaFilter, self).__init__(**kwargs)
 
+        if one_of is None:
+            one_of = []
+
         self.type = "area"
         self.one_of = one_of