In the UI we don't want to display Edit links unless someone can actually

edit things. We use the 'editors' group for this. This group itself grants
no permission other than displaying certain things in the UI.

In order to be in the editors group a user must be a member of a group that
is the source group in a delegation. The memberof plugin will do all the
hard work to be sure that a user's memberof contains cn=editors if they
are in a delegated group.

432874

1 files changed, 16 insertions, 1 deletions

diff --git a/ipa-admintools/ipa-deldelegation b/ipa-admintools/ipa-deldelegation
index ac0ae798e..bc05b2837 100644
--- a/ipa-admintools/ipa-deldelegation
+++ b/ipa-admintools/ipa-deldelegation
@@ -51,12 +51,15 @@ def main():
         aci_str_list = [aci_str_list]
 
     acistr = None
+    aci_list = []
     for aci_str in aci_str_list:
         try:
             aci = ipa.aci.ACI(aci_str)
             if aci.name == args[1]:
                 acistr = aci_str
-                break
+                source_group = aci.source_group
+            else:
+                aci_list.append(aci)
         except SyntaxError:
             # ignore aci_str's that ACI can't parse
             pass
@@ -72,6 +75,18 @@ def main():
     aci_entry.setValue('aci', new_aci_str_list)
 
     client.update_entry(aci_entry)
+
+    last = True
+    # If this is the last delegation for a group, remove it from editors
+    for a in aci_list:
+        if source_group == a.source_group:
+            last = False
+            break
+
+    if last:
+        group = client.get_entry_by_cn("editors")
+        client.remove_member_from_group(source_group, group.dn)
+
     print "Delegation removed."
 
     return 0