[FIX] hr_employee_calendar_planning: fill calendar if empty

We should only fill the calendar with sync_user if and only if you don't have one Sync user is used also when we modify the user of an employee
OCA · Jun 27, 2023 · b488b72 · b488b72
1 parent cb29ccf
commit b488b72
Showing 1 changed file with 12 additions and 11 deletions.
diff --git a/hr_employee_calendar_planning/models/hr_employee.py b/hr_employee_calendar_planning/models/hr_employee.py
@@ -125,17 +125,18 @@ def copy(self, default=None):
     def _sync_user(self, user):
         res = super()._sync_user(user=user)
         # set calendar_ids from Create employee button from user
-        res.update(
-            {
-                "calendar_ids": [
-                    (
-                        0,
-                        0,
-                        {"calendar_id": user.company_id.resource_calendar_id.id,},
-                    ),
-                ]
-            }
-        )
+        if not self.calendar_ids:
+            res.update(
+                {
+                    "calendar_ids": [
+                        (
+                            0,
+                            0,
+                            {"calendar_id": user.company_id.resource_calendar_id.id,},
+                        ),
+                    ]
+                }
+            )
         return res
 
     @api.model_create_multi