cmueller/ansible
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Merge pull request #12847 from godfreyduke/devel

Browse source 
Remove mutable default arguments.
This commit is contained in:
Toshio Kuratomi 2015-10-20 23:25:49 -07:00
commit 6b419db9fa
2 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
lib/ansible/module_utils/cloudstack.py
View file

@ -150,7 +150,9 @@ class AnsibleCloudStack(object):
return False return False
def _get_by_key(self, key=None, my_dict={}): def _get_by_key(self, key=None, my_dict=None):
if my_dict is None:
my_dict = {}
if key: if key:
if key in my_dict: if key in my_dict:
return my_dict[key] return my_dict[key]

4
lib/ansible/module_utils/urls.py
View file

@ -400,7 +400,9 @@ class RequestWithMethod(urllib2.Request):
Originally contained in library/net_infrastructure/dnsmadeeasy Originally contained in library/net_infrastructure/dnsmadeeasy
''' '''
def __init__(self, url, method, data=None, headers={}): def __init__(self, url, method, data=None, headers=None):
if headers is None:
headers = {}
self._method = method self._method = method
urllib2.Request.__init__(self, url, data, headers) urllib2.Request.__init__(self, url, data, headers)