diff options
| author | Martin Basti <mbasti@redhat.com> | 2015-12-17 16:07:10 +0100 |
|---|---|---|
| committer | Martin Basti <mbasti@redhat.com> | 2016-01-14 13:15:26 +0100 |
| commit | 2320be18a34628fa6d05ffc42e695da5dd6dab6e (patch) | |
| tree | 0362878e96249442f90a8ab09ead6bdc3c3b7354 | |
| parent | 267bad10a81f101db8b645abf01b2cdd62c91775 (diff) | |
| download | freeipa-2320be18a34628fa6d05ffc42e695da5dd6dab6e.tar.gz freeipa-2320be18a34628fa6d05ffc42e695da5dd6dab6e.tar.xz freeipa-2320be18a34628fa6d05ffc42e695da5dd6dab6e.zip | |
Enable pylint unbalanced-tuple-unpacking check
Enables check and fixes one issue.
get_entries can be replaced by get_entry, filter is not needed because
check of 'originfilter' attribute is done later.
Reviewed-By: Martin Babinsky <mbabinsk@redhat.com>
| -rwxr-xr-x | install/tools/ipa-managed-entries | 3 | ||||
| -rwxr-xr-x | make-lint | 1 |
2 files changed, 1 insertions, 3 deletions
diff --git a/install/tools/ipa-managed-entries b/install/tools/ipa-managed-entries index c786e883a..8a89da8a8 100755 --- a/install/tools/ipa-managed-entries +++ b/install/tools/ipa-managed-entries @@ -132,8 +132,7 @@ def main(): disabled = True try: - [entry] = conn.get_entries(def_dn, conn.SCOPE_BASE, - filter, ['originfilter']) + entry = conn.get_entry(def_dn) disable_attr = '(objectclass=disable)' try: org_filter = entry.single_value.get('originfilter') @@ -279,7 +279,6 @@ def main(): linter.set_option('disable', 'unused-argument') linter.set_option('disable', 'unused-variable') linter.set_option('disable', 'useless-else-on-loop') - linter.set_option('disable', 'unbalanced-tuple-unpacking') linter.set_option('enable', 'C') linter.set_option('disable', 'bad-classmethod-argument') |
