diff options
| author | Petr Viktorin <pviktori@redhat.com> | 2013-01-21 08:39:09 -0500 |
|---|---|---|
| committer | Martin Kosek <mkosek@redhat.com> | 2013-03-01 16:59:45 +0100 |
| commit | 4779865ea30dba0b5f0594cdc4633b0cc93bf36a (patch) | |
| tree | 17708b9facca822d26570de65cd691e12bf3eeff /ipaserver/install/adtrustinstance.py | |
| parent | f5c404c65d81d9a28f171fabe7c5749d6c37f102 (diff) | |
| download | freeipa-4779865ea30dba0b5f0594cdc4633b0cc93bf36a.tar.gz freeipa-4779865ea30dba0b5f0594cdc4633b0cc93bf36a.tar.xz freeipa-4779865ea30dba0b5f0594cdc4633b0cc93bf36a.zip | |
Replace getList by a get_entries method
The find_entries method is cumbersome to use: it requires keyword arguments
for simple uses, and callers are tempted to ignore the 'truncated' flag
it returns.
Introduce a simpler method, get_entries, that returns the found
list directly, and raises an errors if the list is truncated.
Replace the getList method by get_entries.
Part of the work for: https://fedorahosted.org/freeipa/ticket/2660
Diffstat (limited to 'ipaserver/install/adtrustinstance.py')
| -rw-r--r-- | ipaserver/install/adtrustinstance.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/ipaserver/install/adtrustinstance.py b/ipaserver/install/adtrustinstance.py index 52534c19c..fe1ef53a3 100644 --- a/ipaserver/install/adtrustinstance.py +++ b/ipaserver/install/adtrustinstance.py @@ -261,9 +261,9 @@ class ADTRUSTInstance(service.Service): """ try: - res = self.admin_conn.getList(DN(api.env.container_ranges, self.suffix), - ldap.SCOPE_ONELEVEL, - "(objectclass=ipaDomainIDRange)") + res = self.admin_conn.get_entries( + DN(api.env.container_ranges, self.suffix), + ldap.SCOPE_ONELEVEL, "(objectclass=ipaDomainIDRange)") if len(res) != 1: root_logger.critical("Found more than one ID range for the " \ "local domain.") |