Merge PR #534 into 12.0

Signed-off-by huguesdk

beesdoo_shift_swap/models/planning.py

@@ -43,11 +43,13 @@ def _prepare_task_day(self):
 
         swap_subscription_done = []
         for shift in shifts:
-            for rec in changes:
-                shift, swap_subscription_done, done = rec.update_shift_data(
-                    shift, swap_subscription_done
-                )
-                if done:
-                    changes.remove(rec)
+            # process the remaining changes and create a new list containing
+            # only the changes that have not be processed yet.
+            changes = [
+                rec
+                for rec in changes
+                # 3rd returned element is a boolean telling whether it was done
+                if not rec.update_shift_data(shift, swap_subscription_done)[2]
+            ]
 
         return shifts