openstack
/
python-troveclient
Code Issues Proposed changes

Merge "Avoid use xx=[] for parameter in function"

This commit is contained in:
Jenkins 2016-11-01 23:05:44 +00:00 committed by Gerrit Code Review
parent 47705cadec 91e098e0ea
commit ea93edaf83
3 changed files with 10 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
troveclient/common.py
View File

@ -42,7 +42,9 @@ def quote_user_host(user, host):
class Paginated(list):
def __init__(self, items=[], next_marker=None, links=[]):
def __init__(self, items=None, next_marker=None, links=None):
items = items or []
links = links or []
super(Paginated, self).__init__(items)
self.next = next_marker
self.links = links

6
troveclient/compat/common.py
View File

@ -399,10 +399,10 @@ class Paginated(object):
next property you can use to get the next page of data.
"""
def __init__(self, items=[], next_marker=None, links=[]):
self.items = items
def __init__(self, items=None, next_marker=None, links=None):
self.items = items or []
self.next = next_marker
self.links = links
self.links = links or []
def __len__(self):
return len(self.items)

6
troveclient/v1/management.py
View File

@ -272,7 +272,8 @@ class MgmtDatastoreVersions(base.ManagerWithFind):
"version")
def create(self, name, datastore_name, datastore_manager, image,
packages=[], active='true', default='false'):
packages=None, active='true', default='false'):
packages = packages or []
"""Create a new datastore version."""
body = {"version": {
"name": name,
@ -287,7 +288,8 @@ class MgmtDatastoreVersions(base.ManagerWithFind):
return self._create("/mgmt/datastore-versions", body, None, True)
def edit(self, datastore_version_id, datastore_manager=None, image=None,
packages=[], active=None, default=None):
packages=None, active=None, default=None):
packages = packages or []
"""Update a datastore-version."""
body = {}
if datastore_manager is not None: