Merge pull request #264 from sbrunner/autofix/wrapped2_to3_fix

Fix "Avoid mutable default arguments" issue

tilecloud_chain/__init__.py

@@ -134,7 +134,9 @@ def get_tile_matrix_identifier(grid, resolution=None, zoom=None):
 
 class TileGeneration:
 
-    def __init__(self, config_file, options=None, layer_name=None, base_config={}):
+    def __init__(self, config_file, options=None, layer_name=None, base_config=None):
+        if base_config is None:
+            base_config = {}
         self.close_actions = []
         self.geom = None
         self.error = 0
@@ -432,8 +434,10 @@ def get_all_dimensions(self, layer=None):
         all_dimensions = product(*all_dimensions)
         return [dict(d) for d in all_dimensions]
 
-    def get_store(self, cache, layer, dimensions={}, read_only=False):
+    def get_store(self, cache, layer, dimensions=None, read_only=False):
         # build layout
+        if dimensions is None:  # pragma: no cover
+            dimensions = {}
         grid = layer['grid_ref'] if 'grid_ref' in layer else None
         layout = WMTSTileLayout(
             layer=layer['name'],

tilecloud_chain/copy_.py

@@ -23,8 +23,10 @@ def copy(self, options, gene, layer, source, dest, task_name):
         else:  # pragma: no cover
             self._copy(options, gene, layer, source, dest, task_name)
 
-    def _copy(self, options, gene, layer, source, dest, task_name, dimensions={}):
+    def _copy(self, options, gene, layer, source, dest, task_name, dimensions=None):
         # disable metatiles
+        if dimensions is None:  # pragma: no cover
+            dimensions = {}
         del gene.layers[layer]['meta']
         count_tiles_dropped = Count()
 

tilecloud_chain/generate.py

@@ -46,7 +46,9 @@ def gene(self, options, gene, layer):
         else:  # pragma: no cover
             self._gene(options, gene, layer)
 
-    def _gene(self, options, gene, layer, dimensions={}):
+    def _gene(self, options, gene, layer, dimensions=None):
+        if dimensions is None:  # pragma: no cover
+            dimensions = {}
         count_metatiles = None
         count_metatiles_dropped = Count()
         count_tiles = None

tilecloud_chain/server.py

@@ -384,7 +384,9 @@ def serve(self, path, params, **kwargs):
         else:
             return self.error(204, **kwargs)
 
-    def forward(self, url, headers={}, no_cache=False, **kwargs):
+    def forward(self, url, headers=None, no_cache=False, **kwargs):
+        if headers is None:
+            headers = {}
         if no_cache:
             headers['Cache-Control'] = 'no-cache'
             headers['Pragma'] = 'no-cache'
@@ -424,7 +426,9 @@ def error(self, code, message='', start_response=None):
         start_response(self.HTTP_MESSAGES[code], [])
         return [message]
 
-    def responce(self, data, headers={}, start_response=None):
+    def responce(self, data, headers=None, start_response=None):
+        if headers is None:  # pragma: no cover
+            headers = {}
         headers['Content-Length'] = str(len(data))
         start_response('200 OK', headers.items())
         return [data]
@@ -466,7 +470,9 @@ def __call__(self):
     def error(self, code, message=''):
         raise self.HTTP_EXCEPTIONS[code](message)
 
-    def responce(self, data, headers={}):
+    def responce(self, data, headers=None):
+        if headers is None:  # pragma: no cover
+            headers = {}
         self.request.response.headers = headers
         if type(data) == memoryview:
             self.request.response.body_file = data