Merge pull request verdigado#94 from hbrunn/15.0-90-fix-default-user-…

…groups [FIX] verdigado#90 don't reset default user's groups
hbrunn · Jan 15, 2024 · 172b932 · 172b932
2 parents 66c19f9 + 1ca3c35
commit 172b932
Show file tree

Hide file tree

Showing 3 changed files with 38 additions and 2 deletions.
diff --git a/verdigado_attendance/__manifest__.py b/verdigado_attendance/__manifest__.py
@@ -3,7 +3,7 @@
 
 {
     "name": "Verdigado HR Attendance",
-    "version": "15.0.1.0.1",
+    "version": "15.0.1.0.2",
     "category": "Human Resources",
     "website": "https://github.com/verdigado/odoo-customize",
     "author": "verdigado eG",

diff --git a/verdigado_attendance/migrations/15.0.1.0.2/post-migration.py b/verdigado_attendance/migrations/15.0.1.0.2/post-migration.py
@@ -0,0 +1,24 @@
+from openupgradelib import openupgrade
+
+
+@openupgrade.migrate()
+def migrate(env, version=None):
+    group_ids = []
+    for xmlid in (
+        "base.group_allow_export",
+        "base.group_private_addresses",
+        "base.group_user",
+        "base.group_partner_manager",
+        "base.group_no_one",
+        "mail.group_mail_template_editor",
+        "hr_attendance.group_hr_attendance",
+        "hr_holidays.group_hr_holidays_responsible",
+        "verdigado_attendance.group_verdigado_employee",
+    ):
+        group_ids += env.ref(xmlid).ids
+
+    env.ref("base.default_user").write(
+        {
+            "groups_id": [(6, 0, group_ids)],
+        }
+    )
diff --git a/verdigado_attendance/security/verdigado_attendance.xml b/verdigado_attendance/security/verdigado_attendance.xml
@@ -167,6 +167,18 @@
 
     <record id="base.default_user" model="res.users">
         <!-- don't make new users hr managers by default /-->
-        <field name="groups_id" eval="[(5, ref('hr.group_hr_manager'))]" />
+        <field
+            name="groups_id"
+            eval="[
+                (3, ref('hr.group_hr_manager')),
+                (3, ref('hr.group_hr_user')),
+                (3, ref('hr_attendance.group_hr_attendance_kiosk')),
+                (3, ref('hr_attendance.group_hr_attendance_manager')),
+                (3, ref('hr_attendance.group_hr_attendance_user')),
+                (3, ref('hr_holidays.group_hr_holidays_manager')),
+                (3, ref('hr_holidays.group_hr_holidays_user')),
+                (4, ref('verdigado_attendance.group_verdigado_employee')),
+            ]"
+        />
     </record>
 </odoo>