diff options
author | Jan Barta <55042barta@sstebrno.eu> | 2016-06-02 09:58:52 +0200 |
---|---|---|
committer | Martin Basti <mbasti@redhat.com> | 2016-09-22 16:52:57 +0200 |
commit | 36484e8672f5ee1fdc2bd57622e330ab8dbb7671 (patch) | |
tree | 1f06a3e4aebbc0e6e7fe28e15c9e677e396932a3 /ipaserver/plugins/dogtag.py | |
parent | 929086e0992cc32a654b4dfa435f536ecb0c665b (diff) | |
download | freeipa-36484e8672f5ee1fdc2bd57622e330ab8dbb7671.tar.gz freeipa-36484e8672f5ee1fdc2bd57622e330ab8dbb7671.tar.xz freeipa-36484e8672f5ee1fdc2bd57622e330ab8dbb7671.zip |
pylint: fix simplifiable-if-statement warnings
fix inefficient if statements, enable pylint check
Reviewed-By: Tomas Krizek <tkrizek@redhat.com>
Reviewed-By: Florence Blanc-Renaud <frenaud@redhat.com>
Diffstat (limited to 'ipaserver/plugins/dogtag.py')
-rw-r--r-- | ipaserver/plugins/dogtag.py | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/ipaserver/plugins/dogtag.py b/ipaserver/plugins/dogtag.py index 644b41e90..ffe6ead26 100644 --- a/ipaserver/plugins/dogtag.py +++ b/ipaserver/plugins/dogtag.py @@ -1750,10 +1750,7 @@ class ra(rabase.rabase, RestClient): # Return command result cmd_result = {} - if parse_result.get('revoked') == 'yes': - cmd_result['revoked'] = True - else: - cmd_result['revoked'] = False + cmd_result['revoked'] = parse_result.get('revoked') == 'yes' return cmd_result @@ -1814,10 +1811,7 @@ class ra(rabase.rabase, RestClient): if 'error_string' in parse_result: cmd_result['error_string'] = parse_result['error_string'] - if parse_result.get('unrevoked') == 'yes': - cmd_result['unrevoked'] = True - else: - cmd_result['unrevoked'] = False + cmd_result['unrevoked'] = parse_result.get('unrevoked') == 'yes' return cmd_result |