Merge branch...

Merge branch '241-substitution-planning-cancelled-checkbox-does-not-have-false-as-value-when-unchecked-actively' into 'master'

Resolve "substitution planning "cancelled" checkbox does not have false as value when unchecked actively"

Closes #241

See merge request !357

aleksis/apps/chronos/frontend/components/AmendLesson.vue

@@ -159,10 +159,13 @@ export default {
         amends: this.selectedEvent.meta.id,
         datetimeStart: this.selectedEvent.startDateTime.toUTC().toISO(),
         datetimeEnd: this.selectedEvent.endDateTime.toUTC().toISO(),
+        // Normalize cancelled, v-checkbox returns null & does not
+        // honor false-value.
+        cancelled: item.cancelled ? true : false,
       };
     },
     transformPatchData(item) {
-      let { id, __typename, cancelled, ...patchItem } = item;
+      let { __typename, cancelled, ...patchItem } = item;
       return {
         ...patchItem,
         // Normalize cancelled, v-checkbox returns null & does not

aleksis/apps/chronos/schema/__init__.py

@@ -107,7 +107,7 @@ class AmendLessonBatchPatchMutation(DatetimeTimezoneMixin, DjangoBatchPatchMutat
     class Meta:
         model = LessonEvent
         permissions = ("chronos.edit_substitution_rule",)
-        only_fields = ("subject", "teachers", "groups", "rooms", "cancelled", "comment")
+        only_fields = ("id", "subject", "teachers", "groups", "rooms", "cancelled", "comment")
 
     @classmethod
     def before_save(cls, root, info, input, updated_objects):  # noqa: A002