From 3ea52b7bf1d96294439c37485374f916a4b21fe9 Mon Sep 17 00:00:00 2001
From: Holger Brunn <mail@hunki-enterprises.com>
Date: Mon, 8 Jan 2024 15:47:14 +0100
Subject: [PATCH 1/2] [FIX] #90 don't reset default user's groups

---
 verdigado_attendance/__manifest__.py          |  2 +-
 .../migrations/15.0.1.0.2/post-migration.py   | 28 +++++++++++++++++++
 .../security/verdigado_attendance.xml         |  5 +++-
 3 files changed, 33 insertions(+), 2 deletions(-)
 create mode 100644 verdigado_attendance/migrations/15.0.1.0.2/post-migration.py

diff --git a/verdigado_attendance/__manifest__.py b/verdigado_attendance/__manifest__.py
index 88ba645..474e764 100644
--- 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
new file mode 100644
index 0000000..ebeb939
--- /dev/null
+++ b/verdigado_attendance/migrations/15.0.1.0.2/post-migration.py
@@ -0,0 +1,28 @@
+from openupgradelib import openupgrade
+
+
+@openupgrade.migrate()
+def migrate(env, version=None):
+    group_ids = []
+    for xmlid in (
+        "hr_holidays.group_hr_holidays_user",
+        "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.group_hr_user",
+        "hr_attendance.group_hr_attendance_kiosk",
+        "hr_attendance.group_hr_attendance",
+        "hr_attendance.group_hr_attendance_user",
+        "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
index 90d58d3..2e3535a 100644
--- a/verdigado_attendance/security/verdigado_attendance.xml
+++ b/verdigado_attendance/security/verdigado_attendance.xml
@@ -167,6 +167,9 @@
 
     <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_attendance.group_hr_attendance_manager')), (3, ref('hr_holidays.group_hr_holidays_manager')), (4, ref('verdigado_attendance.group_verdigado_employee'))]"
+        />
     </record>
 </odoo>

From 1ca3c352cc55fe27866be50e2dc1a94369b17aca Mon Sep 17 00:00:00 2001
From: Holger Brunn <mail@hunki-enterprises.com>
Date: Mon, 15 Jan 2024 11:00:24 +0100
Subject: [PATCH 2/2] fixup! [FIX] #90 don't reset default user's groups

---
 .../migrations/15.0.1.0.2/post-migration.py           |  4 ----
 .../security/verdigado_attendance.xml                 | 11 ++++++++++-
 2 files changed, 10 insertions(+), 5 deletions(-)

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
index ebeb939..038e5bc 100644
--- a/verdigado_attendance/migrations/15.0.1.0.2/post-migration.py
+++ b/verdigado_attendance/migrations/15.0.1.0.2/post-migration.py
@@ -5,17 +5,13 @@
 def migrate(env, version=None):
     group_ids = []
     for xmlid in (
-        "hr_holidays.group_hr_holidays_user",
         "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.group_hr_user",
-        "hr_attendance.group_hr_attendance_kiosk",
         "hr_attendance.group_hr_attendance",
-        "hr_attendance.group_hr_attendance_user",
         "hr_holidays.group_hr_holidays_responsible",
         "verdigado_attendance.group_verdigado_employee",
     ):
diff --git a/verdigado_attendance/security/verdigado_attendance.xml b/verdigado_attendance/security/verdigado_attendance.xml
index 2e3535a..a59d917 100644
--- a/verdigado_attendance/security/verdigado_attendance.xml
+++ b/verdigado_attendance/security/verdigado_attendance.xml
@@ -169,7 +169,16 @@
         <!-- don't make new users hr managers by default /-->
         <field
             name="groups_id"
-            eval="[(3, ref('hr.group_hr_manager')), (3, ref('hr_attendance.group_hr_attendance_manager')), (3, ref('hr_holidays.group_hr_holidays_manager')), (4, ref('verdigado_attendance.group_verdigado_employee'))]"
+            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>