From d5149f75db4c0aa6ee3f916e219d4db6ac8cf801 Mon Sep 17 00:00:00 2001 From: Emmanuel Viennet Date: Thu, 26 Oct 2023 17:28:36 +0200 Subject: [PATCH] Fix API (mostly revert previous commit) --- app/api/assiduites.py | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/app/api/assiduites.py b/app/api/assiduites.py index 6c2b63dfc5..f5195f529a 100644 --- a/app/api/assiduites.py +++ b/app/api/assiduites.py @@ -652,7 +652,8 @@ def _create_one( return (200, {"assiduite_id": nouv_assiduite.id}) except ScoValueError as excp: - return json_error(404, message=excp.args[0]) + # ici on utilise pas json_error car on doit renvoyer status, message + return 404, excp.args[0] @bp.route("/assiduite/delete", methods=["POST"]) @@ -691,10 +692,12 @@ def assiduite_delete(): return output -def _delete_singular(assiduite_id: int, database): +def _delete_singular(assiduite_id: int, database) -> tuple[int, str]: + """@iziram PLEASE COMMENT THIS F*CKING CODE""" assiduite_unique: Assiduite = Assiduite.query.filter_by(id=assiduite_id).first() if assiduite_unique is None: - return json_error(404, "Assiduite non existante") + # on ne peut pas utiliser json_error ici car on est déclaré (int, str) + return 404, "Assiduite non existante" if g.scodoc_dept is None and assiduite_unique.etudiant.dept_id is not None: # route sans département set_sco_dept(assiduite_unique.etudiant.departement.acronym) @@ -707,7 +710,7 @@ def _delete_singular(assiduite_id: int, database): ) database.session.delete(assiduite_unique) scass.simple_invalidate_cache(ass_dict) - return (200, "OK") + return 200, "OK" @bp.route("/assiduite//edit", methods=["POST"])