diff options
| author | Steve Martinelli <stevemar@ca.ibm.com> | 2015-04-19 02:41:04 -0400 |
|---|---|---|
| committer | Steve Martinelli <stevemar@ca.ibm.com> | 2015-04-19 02:41:04 -0400 |
| commit | 00eeb3593ca6cdd4a3c714ce49435ed74d5e0630 (patch) | |
| tree | bad68ec8bd3cb78d73784ce2f53fc25a9b0c1764 /openstackclient/identity/v3/group.py | |
| parent | fd8b284164da76a1a40726a2af6490b24a53e5d8 (diff) | |
| download | python-openstackclient-00eeb3593ca6cdd4a3c714ce49435ed74d5e0630.tar.gz | |
remove unnecessary conditionals
In several places we had else branches where a reasonable default
would do the job. This makes the code a mean cleaer and easier to
read.
Change-Id: I231e09aab85fd32b8300bc33c48d0899b728b96e
Diffstat (limited to 'openstackclient/identity/v3/group.py')
| -rw-r--r-- | openstackclient/identity/v3/group.py | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/openstackclient/identity/v3/group.py b/openstackclient/identity/v3/group.py index a2afecb9..91acf3e5 100644 --- a/openstackclient/identity/v3/group.py +++ b/openstackclient/identity/v3/group.py @@ -137,11 +137,11 @@ class CreateGroup(show.ShowOne): def take_action(self, parsed_args): self.log.debug('take_action(%s)', parsed_args) identity_client = self.app.client_manager.identity + + domain = None if parsed_args.domain: domain = common.find_domain(identity_client, parsed_args.domain).id - else: - domain = None try: group = identity_client.groups.create( @@ -228,11 +228,10 @@ class ListGroup(lister.Lister): self.log.debug('take_action(%s)', parsed_args) identity_client = self.app.client_manager.identity + domain = None if parsed_args.domain: domain = common.find_domain(identity_client, parsed_args.domain).id - else: - domain = None if parsed_args.user: user = utils.find_resource( |