[FIX] hr_employee_lastnames - Updating the employee with the same nam…

…e should not modify firstname lastname and lastname2 fields

hr_employee_lastnames/models/hr_employee.py

@@ -66,10 +66,12 @@ def _prepare_vals_on_write_firstname_lastname(self, vals):
                 lastname2 = self.lastname2
             vals["name"] = self._get_name_lastnames(lastname, firstname, lastname2)
         elif vals.get("name"):
-            name_splitted = self.split_name(vals["name"])
-            vals["lastname"] = name_splitted["lastname"]
-            vals["firstname"] = name_splitted["firstname"]
-            vals["lastname2"] = name_splitted["lastname2"]
+            for rec in self:
+                if rec.name != vals.get("name"):
+                    name_splitted = self.split_name(vals["name"])
+                    vals["lastname"] = name_splitted["lastname"]
+                    vals["firstname"] = name_splitted["firstname"]
+                    vals["lastname2"] = name_splitted["lastname2"]
         return res
 
     def _update_partner_firstname(self):

hr_employee_lastnames/tests/test_hr_employee_lastnames.py

@@ -161,3 +161,16 @@ def test_change_lastname_with_set_last_first_comma(self):
         self.employee1_id.refresh()
 
         self.assertEqual(self.employee1_id.name, "Lopez Gonzalez, Manuel")
+
+    def test_change_name_with_the_same_name(self):
+        self.employee1_id.write(
+            {
+                "firstname": "Maxime Françis Patrick",
+                "lastname": "Dubois",
+                "lastname2": "",
+            }
+        )
+        self.employee1_id.write({"name": "Maxime Françis Patrick Dubois"})
+        self.assertEqual(self.employee1_id.firstname, "Maxime Françis Patrick")
+        self.assertEqual(self.employee1_id.lastname, "Dubois")
+        self.assertEqual(self.employee1_id.lastname2, "")