diff options
author | Pavel Zuna <pzuna@redhat.com> | 2009-06-24 15:18:34 +0200 |
---|---|---|
committer | Rob Crittenden <rcritten@redhat.com> | 2009-07-02 13:33:03 -0400 |
commit | 57123f2a9993be9e9ea83c1203f0670e67986de9 (patch) | |
tree | 295b2cc697f4d4eee150a9f647673a4df772141e /ipalib/plugins/group.py | |
parent | 341a47f5a3786f96602bcc0c0ecd12aa86f5b9c1 (diff) | |
download | freeipa-57123f2a9993be9e9ea83c1203f0670e67986de9.tar.gz freeipa-57123f2a9993be9e9ea83c1203f0670e67986de9.tar.xz freeipa-57123f2a9993be9e9ea83c1203f0670e67986de9.zip |
Fix minor bugs, typos, etc. discovered by unit tests in plugins.
Diffstat (limited to 'ipalib/plugins/group.py')
-rw-r--r-- | ipalib/plugins/group.py | 17 |
1 files changed, 10 insertions, 7 deletions
diff --git a/ipalib/plugins/group.py b/ipalib/plugins/group.py index 1a5e0ef6f..db2462d15 100644 --- a/ipalib/plugins/group.py +++ b/ipalib/plugins/group.py @@ -105,14 +105,16 @@ class group_del(basegroup_del): :param kw: Unused """ ldap = self.api.Backend.ldap2 - dn = get_dn_by_attr(ldap, 'cn', cn, self.filter_class, self.container) + (dn, entry_attrs) = ldap.find_entry_by_attr( + 'cn', cn, self.filter_class, [''], self.container + ) # Don't allow the default user group to be removed try: config = ldap.get_ipa_config()[1] def_group_cn = config.get('ipadefaultprimarygroup') - def_group_dn = get_dn_by_attr( - ldap, 'cn', def_group_cn, self.filter_class, self.container + (def_group_dn, entry_attrs) = ldap.find_entry_by_attr( + 'cn', def_group_cn, self.filter_class, [''], self.container ) if dn == def_group_dn: raise errors.DefaultGroup() @@ -154,8 +156,9 @@ class group_mod(basegroup_mod): ldap = self.api.Backend.ldap2 if kw['posix'] or 'gidnumber' in kw: - dn = get_dn_by_attr(ldap, 'cn', cn, self.filter_class, self.container) - (dn, entry_attrs) = ldap.get_entry(dn, ['objectclass']) + (dn, entry_attrs) = ldap.find_entry_by_attr( + 'cn', cn, self.filter_class, ['objectclass'], self.container + ) if 'posixgroup' in entry_attrs['objectclass']: if kw['posix'] in entry_attrs['objectclass']: raise errors.AlreadyPosixGroup() @@ -176,8 +179,8 @@ class group_find(basegroup_find): container = _container_dn filter_class = _default_class - def execute(self, cn, **kw): - return super(group_find, self).execute(cn, **kw) + def execute(self, term, **kw): + return super(group_find, self).execute(term, **kw) api.register(group_find) |