diff options
| author | Jenkins <jenkins@review.openstack.org> | 2016-02-21 01:31:34 +0000 |
|---|---|---|
| committer | Gerrit Code Review <review@openstack.org> | 2016-02-21 01:31:34 +0000 |
| commit | f4ca06cad6348b2a00d72d65c7a1c2f56e8df1d4 (patch) | |
| tree | 052d212647d06a670cd77c9925f99bc3d7d0f331 /openstackclient | |
| parent | 097d35e1303415c0b07fc26c67a77d7fee036692 (diff) | |
| parent | dc5a8faddd2f7f7afb29751b7d00eb6b5474857f (diff) | |
| download | python-openstackclient-f4ca06cad6348b2a00d72d65c7a1c2f56e8df1d4.tar.gz | |
Merge "Fix Mutable default argument"
Diffstat (limited to 'openstackclient')
| -rw-r--r-- | openstackclient/common/utils.py | 14 |
1 files changed, 12 insertions, 2 deletions
diff --git a/openstackclient/common/utils.py b/openstackclient/common/utils.py index 4142f830..840da402 100644 --- a/openstackclient/common/utils.py +++ b/openstackclient/common/utils.py @@ -163,7 +163,7 @@ def get_field(item, field): raise exceptions.CommandError(msg) -def get_item_properties(item, fields, mixed_case_fields=[], formatters={}): +def get_item_properties(item, fields, mixed_case_fields=None, formatters=None): """Return a tuple containing the item properties. :param item: a single item resource (e.g. Server, Project, etc) @@ -172,6 +172,11 @@ def get_item_properties(item, fields, mixed_case_fields=[], formatters={}): :param formatters: dictionary mapping field names to callables to format the values """ + if mixed_case_fields is None: + mixed_case_fields = [] + if formatters is None: + formatters = {} + row = [] for field in fields: @@ -187,7 +192,7 @@ def get_item_properties(item, fields, mixed_case_fields=[], formatters={}): return tuple(row) -def get_dict_properties(item, fields, mixed_case_fields=[], formatters={}): +def get_dict_properties(item, fields, mixed_case_fields=None, formatters=None): """Return a tuple containing the item properties. :param item: a single dict resource @@ -196,6 +201,11 @@ def get_dict_properties(item, fields, mixed_case_fields=[], formatters={}): :param formatters: dictionary mapping field names to callables to format the values """ + if mixed_case_fields is None: + mixed_case_fields = [] + if formatters is None: + formatters = {} + row = [] for field in fields: |