.

rapids/forms.py

@@ -134,21 +134,6 @@ class RapidForm(ModelForm):
         for exam in self.cleaned_data["exams"]:
             exam.exam_questions.add(instance)
 
-        #        # Prepare a 'save_m2m' method for the form,
-        #        old_save_m2m = self.save_m2m
-        #
-        #        def save_m2m():
-        #            old_save_m2m()
-        #            # This is where we actually link the pizza with toppings
-        #            instance.exams.clear()
-        #            for exam in self.cleaned_data["exams"]:
-        #                instance.exams.add(exam)
-        #
-        #        self.save_m2m = save_m2m
-        #
-        #        # Do we need to save all changes now?
-        #        # if commit:
-        #        instance.save()
         self.save_m2m()
 
         return instance

rapids/views.py

@@ -370,11 +370,15 @@ class RapidUpdate(
             answer_formset.save()
 
             # restore exam orders
+            wanted_exams = form["exams"].data
             for exam in self.object.exams.all():
-                if exam in exam_orders and self.object in exam_orders[exam]:
-                    print(exam_orders[exam])
-                    exam.exam_questions.set(exam_orders[exam])
-                    exam.save()
+                # Only readd if exam is still added
+                # There is probably a better way to do this
+                if str(exam.pk) in wanted_exams:
+                    if exam in exam_orders and self.object in exam_orders[exam]:
+                        print(exam_orders[exam])
+                        exam.exam_questions.set(exam_orders[exam])
+                        exam.save()
 
             return response
         else: