diff options
| author | Tomas Babej <tbabej@redhat.com> | 2014-06-13 12:06:07 +0200 |
|---|---|---|
| committer | Petr Viktorin <pviktori@redhat.com> | 2014-06-25 20:14:53 +0200 |
| commit | af4518b72882f88a01de0e5c23d423898ba894b4 (patch) | |
| tree | bab97e723bc1a6e7ffb1e571f63f33063eec663f /ipalib/plugins/netgroup.py | |
| parent | e7969f5af56be1b9163a8f9ee4686becb3fdcb59 (diff) | |
| download | freeipa-af4518b72882f88a01de0e5c23d423898ba894b4.tar.gz freeipa-af4518b72882f88a01de0e5c23d423898ba894b4.tar.xz freeipa-af4518b72882f88a01de0e5c23d423898ba894b4.zip | |
sudorule: Refactor add and remove external_post_callback
Reviewed-By: Petr Viktorin <pviktori@redhat.com>
Diffstat (limited to 'ipalib/plugins/netgroup.py')
| -rw-r--r-- | ipalib/plugins/netgroup.py | 30 |
1 files changed, 19 insertions, 11 deletions
diff --git a/ipalib/plugins/netgroup.py b/ipalib/plugins/netgroup.py index 2d9fdebd0..a7cad1dcb 100644 --- a/ipalib/plugins/netgroup.py +++ b/ipalib/plugins/netgroup.py @@ -75,6 +75,7 @@ output_params = ( ), ) + @register() class netgroup(LDAPObject): """ @@ -206,7 +207,6 @@ class netgroup(LDAPObject): ) - @register() class netgroup_add(LDAPCreate): __doc__ = _('Add a new netgroup.') @@ -243,7 +243,6 @@ class netgroup_add(LDAPCreate): return dn - @register() class netgroup_del(LDAPDelete): __doc__ = _('Delete a netgroup.') @@ -273,7 +272,6 @@ class netgroup_mod(LDAPUpdate): return dn - @register() class netgroup_find(LDAPSearch): __doc__ = _('Search for a netgroup.') @@ -311,7 +309,6 @@ class netgroup_find(LDAPSearch): return (filter, base_dn, scope) - @register() class netgroup_show(LDAPRetrieve): __doc__ = _('Display information about a netgroup.') @@ -319,21 +316,26 @@ class netgroup_show(LDAPRetrieve): has_output_params = LDAPRetrieve.has_output_params + output_params - @register() class netgroup_add_member(LDAPAddMember): __doc__ = _('Add members to a netgroup.') member_attributes = ['memberuser', 'memberhost', 'member'] has_output_params = LDAPAddMember.has_output_params + output_params + def pre_callback(self, ldap, dn, found, not_found, *keys, **options): assert isinstance(dn, DN) return add_external_pre_callback('host', ldap, dn, keys, options) - def post_callback(self, ldap, completed, failed, dn, entry_attrs, *keys, **options): + def post_callback(self, ldap, completed, failed, dn, entry_attrs, + *keys, **options): assert isinstance(dn, DN) - return add_external_post_callback('memberhost', 'host', 'externalhost', ldap, completed, failed, dn, entry_attrs, keys, options) - + return add_external_post_callback(ldap, dn, entry_attrs, + failed=failed, + completed=completed, + memberattr='memberhost', + membertype='host', + externalattr='externalhost') @register() @@ -342,7 +344,13 @@ class netgroup_remove_member(LDAPRemoveMember): member_attributes = ['memberuser', 'memberhost', 'member'] has_output_params = LDAPRemoveMember.has_output_params + output_params - def post_callback(self, ldap, completed, failed, dn, entry_attrs, *keys, **options): - assert isinstance(dn, DN) - return remove_external_post_callback('memberhost', 'host', 'externalhost', ldap, completed, failed, dn, entry_attrs, keys, options) + def post_callback(self, ldap, completed, failed, dn, entry_attrs, + *keys, **options): + assert isinstance(dn, DN) + return remove_external_post_callback(ldap, dn, entry_attrs, + failed=failed, + completed=completed, + memberattr='memberhost', + membertype='host', + externalattr='externalhost') |