fix database deletion bug

WING-NUS / SSID

Student Submission Integrity Diagnosis

18 stars 23 forks source link

fix database deletion bug #357

Closed huyuxin0429 closed 11 months ago

huyuxin0429 commented 12 months ago

Description

fix database deletion bug #347 with api key

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

sibinhho99 commented 12 months ago

Does this fix the delete cascade issue in general (i.e. if I delete the user the corresponding API key is deleted as well, if I delete an assignment its submission similarities would be deleted as well), or it just changes the order of deletion for this specific test?

huyuxin0429 commented 11 months ago

Does this fix the delete cascade issue in general (i.e. if I delete the user the corresponding API key is deleted as well, if I delete an assignment its submission similarities would be deleted as well), or it just changes the order of deletion for this specific test?

This fixes the root dependency between api keys and assignment. When an assignment is deleted, all api keys assigned to that account are also deleted. The tests are updated to reflect this relation.

sibinhho99 commented 11 months ago

Does this fix the delete cascade issue in general (i.e. if I delete the user the corresponding API key is deleted as well, if I delete an assignment its submission similarities would be deleted as well), or it just changes the order of deletion for this specific test?

This fixes the root dependency between api keys and assignment. When an assignment is deleted, all api keys assigned to that account are also deleted. The tests are updated to reflect this relation.

You mean between api keys and users, and when an user is deleted? Looks like it's still not working for me locally