Merge lp:~mukunde/unifield-server/US-12110 into lp:unifield-server

[jftempo]

[lp:~mukunde/unifield-server/US-12110]

6171. By Gaël Mukunde on 2023-12-07

US-12110 [IMP] Code fix to avoid in-pipe sync data without deleting or blocking usage of the group

=== modified file 'bin/addons/msf_profile/data/patches.xml'
--- bin/addons/msf_profile/data/patches.xml	2023-11-16 13:03:16 +0000
+++ bin/addons/msf_profile/data/patches.xml	2023-12-07 16:09:13 +0000
@@ -1016,6 +1016,10 @@
             <field name="method">us_9321_2_remove_location_colors</field>
         </record>
 
+        <record id="us_12110_remove_sup_fin_read" model="patch.scripts">
+            <field name="method">us_12110_remove_sup_fin_read</field>
+        </record>
+
 
     </data>
 </openerp>

=== modified file 'bin/addons/msf_profile/msf_profile.py'
--- bin/addons/msf_profile/msf_profile.py	2023-11-16 13:03:16 +0000
+++ bin/addons/msf_profile/msf_profile.py	2023-12-07 16:09:13 +0000
@@ -58,6 +58,15 @@
     }
 
 
+    def us_12110_remove_sup_fin_read(self, cr, uid, *a, **b):
+        '''
+        Remove all Sup_Fin_Read groups from all users
+        '''
+        instance = self.pool.get('res.users').browse(cr, uid, uid).company_id.instance_id
+        if instance and instance.level == 'section':
+            self.pool.get('sync.trigger.something').create(cr, uid, {'name': 'US-12110-Sup_Fin_Read'})
+        return True
+
     # python 3
     def us_9321_2_remove_location_colors(self, cr, uid, *a, **b):
         '''
@@ -6654,6 +6663,11 @@
             """)
             _logger.warn('Deletion of %d Dest CC Links created out of HQ' % (cr.rowcount,))
 
+        if vals.get('name') == 'US-12110-Sup_Fin_Read':
+            cr.execute("""
+            DELETE FROM res_groups_users_rel
+            WHERE gid=(SELECT id FROM res_groups WHERE name='Sup_Fin_Read')""")
+
         return super(sync_tigger_something, self).create(cr, uid, vals, context)
 
 sync_tigger_something()