Skip to content
Snippets Groups Projects

Fix import of supervisions for non-existing breaks

Merged Jonathan Weth requested to merge fix/break-slots into master
1 file
+ 7
2
Compare changes
  • Side-by-side
  • Inline
@@ -386,13 +386,18 @@ def import_supervision_areas(
weekday = int(raw_supervision_2[2]) - 1
before_period_number = int(raw_supervision_2[3])
teacher = teachers_ref[teacher_id]
slot = breaks_ref[weekday][before_period_number]
logger.info(
f"Import supervision on weekday {weekday} before the "
f"{before_period_number}. period (teacher {teacher})"
)
slot = breaks_ref.get(weekday, {}).get(before_period_number)
if not slot:
logger.warning("Skip supervision as slot doesn't exist.")
continue
# Get or create
possible_supervisions = lesrooster_models.Supervision.objects.filter(
break_slot=slot,
@@ -449,7 +454,7 @@ def import_supervision_areas(
with create_revision():
set_comment(_("Deleted by Untis import"))
supervision.delete()
logger.info(f" Supervision {supervision} deleted")
logger.info(f" Supervision {supervision.id} deleted")
ref[import_ref] = {"area": new_area, "supervisions": supervisions_ref}
Loading