Skip to content

Commit 3691c3e

Browse files
committed
FIX order of access for sqlalchemy repo object
1 parent 62cc145 commit 3691c3e

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

api/anubis/views/admin/repo.py

+4-2
Original file line numberDiff line numberDiff line change
@@ -48,18 +48,20 @@ def admin_repos_delete(repo_id: str):
4848
assert_course_context(assignment)
4949
assert_course_context(student)
5050

51-
5251
# If the repo is shared, then student can not delete
5352
req_assert(not repo.shared, message="Repo is shared. Please reach out to Anubis support to delete/reset this repo.")
5453

54+
# save this now, or get a database error after the delete operation
55+
assignment_id = repo.assignment_id
56+
5557
# Delete the repo
5658
delete_assignment_repo(student, assignment)
5759

5860
# Delete cache entry
5961
cache.delete_memoized(get_repos, student.id)
6062

6163
# Clear cache entry
62-
cache.delete_memoized(get_assignment_data, student.id, repo.assignment_id)
64+
cache.delete_memoized(get_assignment_data, student.id, assignment_id)
6365

6466
# Pass them back
6567
return success_response({"status": "Github Repo & Submissions deleted"})

0 commit comments

Comments
 (0)