1
0
forked from ScoDoc/ScoDoc

Fix API (mostly revert previous commit)

This commit is contained in:
Emmanuel Viennet 2023-10-26 17:28:36 +02:00 committed by Iziram
parent 1d64680c55
commit b28a257129

View File

@ -652,7 +652,8 @@ def _create_one(
return (200, {"assiduite_id": nouv_assiduite.id}) return (200, {"assiduite_id": nouv_assiduite.id})
except ScoValueError as excp: 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"]) @bp.route("/assiduite/delete", methods=["POST"])
@ -691,10 +692,12 @@ def assiduite_delete():
return output 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() assiduite_unique: Assiduite = Assiduite.query.filter_by(id=assiduite_id).first()
if assiduite_unique is None: 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: if g.scodoc_dept is None and assiduite_unique.etudiant.dept_id is not None:
# route sans département # route sans département
set_sco_dept(assiduite_unique.etudiant.departement.acronym) set_sco_dept(assiduite_unique.etudiant.departement.acronym)
@ -707,7 +710,7 @@ def _delete_singular(assiduite_id: int, database):
) )
database.session.delete(assiduite_unique) database.session.delete(assiduite_unique)
scass.simple_invalidate_cache(ass_dict) scass.simple_invalidate_cache(ass_dict)
return (200, "OK") return 200, "OK"
@bp.route("/assiduite/<int:assiduite_id>/edit", methods=["POST"]) @bp.route("/assiduite/<int:assiduite_id>/edit", methods=["POST"])