diff options
| author | Rob Crittenden <rcritten@redhat.com> | 2010-08-02 18:10:01 -0400 |
|---|---|---|
| committer | Rob Crittenden <rcritten@redhat.com> | 2010-08-06 15:34:09 -0400 |
| commit | 8ad88b411921ba90bdd149eb278639e1cbbe1a1a (patch) | |
| tree | 1c5c56a0f1a0a490a3499201a5427250697528b6 /ipalib/plugins/netgroup.py | |
| parent | d0740fb08b7b9d910a38c56a812aaad2c2df7f50 (diff) | |
| download | freeipa-8ad88b411921ba90bdd149eb278639e1cbbe1a1a.tar.gz freeipa-8ad88b411921ba90bdd149eb278639e1cbbe1a1a.tar.xz freeipa-8ad88b411921ba90bdd149eb278639e1cbbe1a1a.zip | |
Properly show the members when an add/remove operation fails.
The remove member function in baseldap was not returning failures at all.
The add member function was only showing them in the group object.
Most of the magic is handled in baseldap. Each plugin just needs to define
object_name and object_name_plural. object_name must be all lower-case
because fake-attributes are created so membership can be broken out
per-object type. I left the plural name lower case as well.
ticket 85
Diffstat (limited to 'ipalib/plugins/netgroup.py')
| -rw-r--r-- | ipalib/plugins/netgroup.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ipalib/plugins/netgroup.py b/ipalib/plugins/netgroup.py index 755c5f4da..d2ffc404e 100644 --- a/ipalib/plugins/netgroup.py +++ b/ipalib/plugins/netgroup.py @@ -186,7 +186,7 @@ class netgroup_add_member(LDAPAddMember): """ Add members to netgroup. """ - has_output_params = output_params + has_output_params = LDAPAddMember.has_output_params + output_params member_attributes = ['memberuser', 'memberhost'] def post_callback(self, ldap, completed, failed, dn, entry_attrs, *keys, **options): completed_external = 0 @@ -222,7 +222,7 @@ class netgroup_remove_member(LDAPRemoveMember): """ Remove members from netgroup. """ - has_output_params = output_params + has_output_params = LDAPRemoveMember.has_output_params + output_params member_attributes = ['memberuser', 'memberhost'] def post_callback(self, ldap, completed, failed, dn, entry_attrs, *keys, **options): # Run through the host failures and gracefully remove any defined as |