diff --git a/app/__init__.py b/app/__init__.py index ac256d8a..a63f9a23 100644 --- a/app/__init__.py +++ b/app/__init__.py @@ -25,7 +25,7 @@ from flask_moment import Moment from flask_caching import Cache import sqlalchemy -from app.scodoc.sco_exceptions import ScoValueError, APIInvalidParams +from app.scodoc.sco_exceptions import ScoGenError, ScoValueError, APIInvalidParams from config import DevConfig import sco_version @@ -163,6 +163,7 @@ def create_app(config_class=DevConfig): cache.init_app(app) sco_cache.CACHE = cache + app.register_error_handler(ScoGenError, handle_sco_value_error) app.register_error_handler(ScoValueError, handle_sco_value_error) app.register_error_handler(500, internal_server_error) app.register_error_handler(503, postgresql_server_error) diff --git a/app/scodoc/sco_edit_module.py b/app/scodoc/sco_edit_module.py index 26799f25..8adc3557 100644 --- a/app/scodoc/sco_edit_module.py +++ b/app/scodoc/sco_edit_module.py @@ -259,12 +259,13 @@ def do_module_delete(oid): # S'il y a des moduleimpls, on ne peut pas detruire le module ! mods = sco_moduleimpl.do_moduleimpl_list(module_id=oid) if mods: - err_page = scu.confirm_dialog( - message="""
Il faut d'abord supprimer le semestre. Mais il est peut être préférable de + laisser ce programme intact et d'en créer une nouvelle version pour la modifier. +
+ reprendre + """ raise ScoGenError(err_page) # delete cnx = ndb.GetDBConnexion() diff --git a/app/scodoc/sco_etape_apogee_view.py b/app/scodoc/sco_etape_apogee_view.py index a4440bf7..62f22108 100644 --- a/app/scodoc/sco_etape_apogee_view.py +++ b/app/scodoc/sco_etape_apogee_view.py @@ -770,7 +770,7 @@ def view_apo_csv(etape_apo="", semset_id="", format="html", REQUEST=None): return "\n".join(H) -# called from Web +# called from Web (GET) def apo_csv_export_results( semset_id, block_export_res_etape=False, diff --git a/app/scodoc/sco_etud.py b/app/scodoc/sco_etud.py index 04f312cf..a75bad88 100644 --- a/app/scodoc/sco_etud.py +++ b/app/scodoc/sco_etud.py @@ -338,31 +338,33 @@ def _check_duplicate_code( ) if etudid: OK = "retour à la fiche étudiant" - dest_url = "ficheEtud" + dest_endpoint = "scolar.ficheEtud" parameters = {"etudid": etudid} else: if "tf_submitted" in args: del args["tf_submitted"] OK = "Continuer" - dest_url = "etudident_create_form" + dest_endpoint = "scolar.etudident_create_form" parameters = args else: OK = "Annuler" - dest_url = "" + dest_endpoint = "notes.index_html" parameters = {} if not disable_notify: - err_page = scu.confirm_dialog( - message="""", - OK=OK, - dest_url=dest_url, - parameters=parameters, - ) + err_page = f"""
Le {code_name} {args[code_name]} est déjà utilisé: un seul étudiant peut avoir + ce code. Vérifier votre valeur ou supprimer l'autre étudiant avec cette valeur. +
++ {OK} +
+ """ else: - err_page = """