gpo: Clarify the contents of deleted_gpo_list in process_group_policy

Signed-off-by: David Mulder <dmulder@suse.com>
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>

diff --git a/python/samba/gp_scripts_ext.py b/python/samba/gp_scripts_ext.py
index 9bd828d0687d796ca3e5f01a4971b86a451028e9..f36470800c6d0070a06fb1e4f67762c5f76ed932 100644 (file)
--- a/python/samba/gp_scripts_ext.py
+++ b/python/samba/gp_scripts_ext.py
@@ -35,10 +35,10 @@ class gp_scripts_ext(gp_pol_ext):
         return 'Unix Settings/Scripts'
 
     def process_group_policy(self, deleted_gpo_list, changed_gpo_list, cdir=None):
-        for gpo in deleted_gpo_list:
-            self.gp_db.set_guid(gpo[0])
-            if str(self) in gpo[1]:
-                for attribute, script in gpo[1][str(self)].items():
+        for guid, settings in deleted_gpo_list:
+            self.gp_db.set_guid(guid)
+            if str(self) in settings:
+                for attribute, script in settings[str(self)].items():
                     os.unlink(script)
                     self.gp_db.delete(str(self), attribute)
             self.gp_db.commit()

diff --git a/python/samba/gp_sec_ext.py b/python/samba/gp_sec_ext.py
index 5e230f73c3ca9953e1bd4a7adb1cfef382d86b22..5e49dec75e1c8f4b676c38ed6c905083a1d286f9 100644 (file)
--- a/python/samba/gp_sec_ext.py
+++ b/python/samba/gp_sec_ext.py
@@ -34,11 +34,11 @@ class gp_krb_ext(gp_inf_ext):
         if self.lp.get('server role') != 'active directory domain controller':
             return
         inf_file = 'MACHINE/Microsoft/Windows NT/SecEdit/GptTmpl.inf'
-        for gpo in deleted_gpo_list:
-            self.gp_db.set_guid(gpo[0])
-            for section in gpo[1].keys():
+        for guid, settings in deleted_gpo_list:
+            self.gp_db.set_guid(guid)
+            for section in settings.keys():
                 if section == str(self):
-                    for att, value in gpo[1][section].items():
+                    for att, value in settings[section].items():
                         update_samba, _ = self.mapper().get(att)
                         update_samba(att, value)
                         self.gp_db.delete(section, att)
@@ -127,11 +127,11 @@ class gp_access_ext(gp_inf_ext):
         if self.lp.get('server role') != 'active directory domain controller':
             return
         inf_file = 'MACHINE/Microsoft/Windows NT/SecEdit/GptTmpl.inf'
-        for gpo in deleted_gpo_list:
-            self.gp_db.set_guid(gpo[0])
-            for section in gpo[1].keys():
+        for guid, settings in deleted_gpo_list:
+            self.gp_db.set_guid(guid)
+            for section in settings.keys():
                 if section == str(self):
-                    for att, value in gpo[1][section].items():
+                    for att, value in settings[section].items():
                         update_samba, _ = self.mapper().get(att)
                         update_samba(att, value)
                         self.gp_db.delete(section, att)

diff --git a/python/samba/gp_sudoers_ext.py b/python/samba/gp_sudoers_ext.py
index 6eb033a67c09dd21b651325f2be9dab69570d2f2..5c607564a3fd037d4edc1c6f45ca1c5afc7759a9 100644 (file)
--- a/python/samba/gp_sudoers_ext.py
+++ b/python/samba/gp_sudoers_ext.py
@@ -40,10 +40,10 @@ class gp_sudoers_ext(gp_pol_ext):
 
     def process_group_policy(self, deleted_gpo_list, changed_gpo_list,
             sdir='/etc/sudoers.d'):
-        for gpo in deleted_gpo_list:
-            self.gp_db.set_guid(gpo[0])
-            if str(self) in gpo[1]:
-                for attribute, sudoers in gpo[1][str(self)].items():
+        for guid, settings in deleted_gpo_list:
+            self.gp_db.set_guid(guid)
+            if str(self) in settings:
+                for attribute, sudoers in settings[str(self)].items():
                     os.unlink(sudoers)
                     self.gp_db.delete(str(self), attribute)
             self.gp_db.commit()