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] 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>