[REF] Redefining built-in 'id'

beesdoo_shift_attendance/models/attendance_sheet.py

@@ -377,8 +377,8 @@ class AttendanceSheet(models.Model):
             )
 
         # Expected shifts status update
-        for id in self.expected_shift_ids.ids:
-            shift = self.env["beesdoo.shift.sheet.expected"].browse(id)
+        for id_ in self.expected_shift_ids.ids:
+            shift = self.env["beesdoo.shift.sheet.expected"].browse(id_)
             if (
                 shift.worker_id == worker and not shift.replaced_id
             ) or shift.replaced_id == worker:

beesdoo_shift_attendance/tests/test_beesdoo_shift.py

@@ -252,8 +252,8 @@ class TestBeesdooShift(TransactionCase):
             sheet_1.on_barcode_scanned(barcode)
 
         # Check expected shifts update
-        for id in sheet_1.expected_shift_ids.ids:
-            shift = sheet_1.expected_shift_ids.browse(id)
+        for id_ in sheet_1.expected_shift_ids.ids:
+            shift = sheet_1.expected_shift_ids.browse(id_)
             self.assertEqual(shift.state, "done")
 
         """
@@ -275,8 +275,8 @@ class TestBeesdooShift(TransactionCase):
         self.assertEqual(len(sheet_1.added_shift_ids), 2)
 
         # Check added shifts fields
-        for id in sheet_1.added_shift_ids.ids:
-            shift = sheet_1.added_shift_ids.browse(id)
+        for id_ in sheet_1.added_shift_ids.ids:
+            shift = sheet_1.added_shift_ids.browse(id_)
             self.assertEqual(sheet_1, shift.attendance_sheet_id)
             self.assertEqual(shift.state, "done")
             self.assertEqual(