Fix relation between MultipleChoiceOption and FeedbackOption

When a MultipleChoiceOption is created via the API call PUT /mult-choice, a new FeedbackOption is now automatically inserted into the database that links to the same problem as the MultipleChoiceOption.

The response of PUT /mult-choice will now contain an mc_id that shows the ID of the new MultipleChoiceQuestion and a feedback_id with the ID of the new FeedbackOption.

The relation between MultipleChoiceOption and Problem is now removed. A new relation from FeedbackOption to MultipleChoiceOption is created. However, it is still possible to use problem.mc_options with a hybrid attribute that essentially runs a query in the background.

TODO:

• Ensure that if a FeedbackOption is deleted, the related MultipleChoiceOption is also deleted.

Closes #35 (closed) and #36 (closed).

added 1 commit

• 8507d35e - Added some documentation

Compare with previous version

unmarked as a Work In Progress

marked as a Work In Progress

added 1 commit

• 73cba00b - Added relationship between mc_option and feedbackoption, problem_id is no longer used

Compare with previous version

added 1 commit

• 0be3d496 - Changed mult_choice API

Compare with previous version

added 1 commit

• b9a66ed2 - Changed api and added some doc

Compare with previous version

added 1 commit

• 74899114 - Added hybrid property to problem, removed direct relation between mc_option...

Compare with previous version

added 1 commit

• 3ad2ab41 - Changed migrations, changed API since put now no longer requires a...

Compare with previous version

unmarked as a Work In Progress

changed the description

If a mc option is deleted, the associated feedback isn't. Is this what we want?

Fixed in 3e15bec9, the API call DELETE /feedback/problem_id/feedback_id now deletes the related MultipleChoiceOption if present.

I also found out that deleting a FeedbackOption dit not return a response, which is changed now.

Now we delete a checkbox if the feedback is deleted but what about the other way around? Do we want that if a checkbox is deleted that the feedback is still visible?

I think the feedback should also be deleted.

Is it possible to use cascade delete on these (@Hugo mentioned it recently)?

This can simply be done by adding cascase='delete' in the database relationship.

marked as a Work In Progress

changed the description

added 1 commit

• 3e15bec9 - MultipleChoiceOption is now deleted if the related FeedbackOption is deleted

Compare with previous version

unmarked as a Work In Progress

marked the checklist item Ensure that if a FeedbackOption is deleted, the related MultipleChoiceOption is also deleted. as completed

changed the description

assigned to @jtimotei

I think deleting a feedback with an attached MC option shouldn't be possible in a finalized exam.