.

generic/views.py

@@ -195,6 +195,7 @@ def generic_exam_json_edit(request):
                     add_count += 1
                     exam.exam_questions.add(i)
 
+            if add_count > 0:
                 exam.save()
             data = {"status": "success", "questions added": add_count}
             return JsonResponse(data, status=200)

rapids/forms.py

@@ -80,13 +80,13 @@ class RapidForm(ModelForm):
         self.user = kwargs.pop(
             "user"
         )  # To get request.user. Do not use kwargs.pop('user', None) due to potential security hole
-        if kwargs.get("instance"):
+        #if kwargs.get("instance"):
-            # We get the 'initial' keyword argument or initialize it
+        #    # We get the 'initial' keyword argument or initialize it
-            # as a dict if it didn't exist.
+        #    # as a dict if it didn't exist.
-            initial = kwargs.setdefault("initial", {})
+        #    initial = kwargs.setdefault("initial", {})
-            # The widget for a ModelMultipleChoiceField expects
+        #    # The widget for a ModelMultipleChoiceField expects
-            # a list of primary key for the selected data.
+        #    # a list of primary key for the selected data.
-            initial["exams"] = [t.pk for t in kwargs["instance"].exams.all()]
+        #    initial["exams"] = [t.pk for t in kwargs["instance"].exams.all()]
 
         ModelForm.__init__(self, *args, **kwargs)
         super(RapidForm, self).__init__(*args, **kwargs)
@@ -132,16 +132,16 @@ class RapidForm(ModelForm):
         instance.save()
 
         self.save_m2m()
-        old_exams = instance.exams.all()
+        #old_exams = instance.exams.all()
 
-        new_exams = self.cleaned_data["exams"]
+        #new_exams = self.cleaned_data["exams"]
 
-        for exam in old_exams:
+        #for exam in old_exams:
-            if exam in new_exams:
+        #    if exam in new_exams:
-                exam.exam_questions.add(instance)
+        #        exam.exam_questions.add(instance)
-            else:
+        #    else:
-                exam.exam_questions.remove(instance)
+        #        exam.exam_questions.remove(instance)
-            exam.save()
+        #    exam.save()
 
         #        # Prepare a 'save_m2m' method for the form,
         #        old_save_m2m = self.save_m2m