From 5dfc64a62d3425ec1b9d91de4df9b9fe586b2bba Mon Sep 17 00:00:00 2001 From: Emmanuel Viennet Date: Tue, 21 Sep 2021 13:36:56 +0200 Subject: [PATCH] sco_debouche APi views return empty OK responses / removed useless REQUEST --- app/scodoc/sco_debouche.py | 28 +++++++++++++++------------- app/scodoc/sco_edt_cal.py | 2 +- app/scodoc/sco_exceptions.py | 8 +++----- app/scodoc/sco_recapcomplet.py | 2 +- app/scodoc/sco_saisie_notes.py | 2 +- app/scodoc/sco_tag_module.py | 2 +- app/scodoc/sco_undo_notes.py | 2 +- app/scodoc/sco_utils.py | 6 +++--- sco_version.py | 2 +- 9 files changed, 27 insertions(+), 27 deletions(-) diff --git a/app/scodoc/sco_debouche.py b/app/scodoc/sco_debouche.py index 2dc2f10a87..b32b39deec 100644 --- a/app/scodoc/sco_debouche.py +++ b/app/scodoc/sco_debouche.py @@ -47,10 +47,10 @@ from app.scodoc import sco_etud import sco_version -def report_debouche_date(start_year=None, format="html", REQUEST=None): +def report_debouche_date(start_year=None, format="html"): """Rapport (table) pour les débouchés des étudiants sortis à partir de l'année indiquée.""" if not start_year: - return report_debouche_ask_date(REQUEST=REQUEST) + return report_debouche_ask_date() if format == "xls": keep_numeric = True # pas de conversion des notes en strings else: @@ -193,7 +193,7 @@ def table_debouche_etudids(etudids, keep_numeric=True): return tab -def report_debouche_ask_date(REQUEST=None): +def report_debouche_ask_date(): """Formulaire demande date départ""" return ( html_sco_header.sco_header() @@ -248,7 +248,7 @@ def itemsuivi_get(cnx, itemsuivi_id, ignore_errors=False): return None -def itemsuivi_suppress(itemsuivi_id, REQUEST=None): +def itemsuivi_suppress(itemsuivi_id): """Suppression d'un item""" if not sco_permissions_check.can_edit_suivi(): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") @@ -258,9 +258,10 @@ def itemsuivi_suppress(itemsuivi_id, REQUEST=None): _itemsuivi_delete(cnx, itemsuivi_id) logdb(cnx, method="itemsuivi_suppress", etudid=item["etudid"]) log("suppressed itemsuivi %s" % (itemsuivi_id,)) + return ("", 204) -def itemsuivi_create(etudid, item_date=None, situation="", REQUEST=None, format=None): +def itemsuivi_create(etudid, item_date=None, situation="", format=None): """Creation d'un item""" if not sco_permissions_check.can_edit_suivi(): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") @@ -272,11 +273,11 @@ def itemsuivi_create(etudid, item_date=None, situation="", REQUEST=None, format= log("created itemsuivi %s for %s" % (itemsuivi_id, etudid)) item = itemsuivi_get(cnx, itemsuivi_id) if format == "json": - return scu.sendJSON(REQUEST, item) + return scu.sendJSON(item) return item -def itemsuivi_set_date(itemsuivi_id, item_date, REQUEST=None): +def itemsuivi_set_date(itemsuivi_id, item_date): """set item date item_date is a string dd/mm/yyyy """ @@ -287,9 +288,10 @@ def itemsuivi_set_date(itemsuivi_id, item_date, REQUEST=None): item = itemsuivi_get(cnx, itemsuivi_id) item["item_date"] = item_date _itemsuivi_edit(cnx, item) + return ("", 204) -def itemsuivi_set_situation(object, value, REQUEST=None): +def itemsuivi_set_situation(object, value): """set situation""" if not sco_permissions_check.can_edit_suivi(): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") @@ -303,14 +305,14 @@ def itemsuivi_set_situation(object, value, REQUEST=None): return situation or scu.IT_SITUATION_MISSING_STR -def itemsuivi_list_etud(etudid, format=None, REQUEST=None): +def itemsuivi_list_etud(etudid, format=None): """Liste des items pour cet étudiant, avec tags""" cnx = ndb.GetDBConnexion() items = _itemsuivi_list(cnx, {"etudid": etudid}) for it in items: it["tags"] = ", ".join(itemsuivi_tag_list(it["itemsuivi_id"])) if format == "json": - return scu.sendJSON(REQUEST, items) + return scu.sendJSON(items) return items @@ -327,7 +329,7 @@ def itemsuivi_tag_list(itemsuivi_id): return [x["title"] for x in r] -def itemsuivi_tag_search(term, REQUEST=None): +def itemsuivi_tag_search(term): """List all used tag names (for auto-completion)""" # restrict charset to avoid injections if not scu.ALPHANUM_EXP.match(term): @@ -342,10 +344,10 @@ def itemsuivi_tag_search(term, REQUEST=None): ) data = [x["title"] for x in r] - return scu.sendJSON(REQUEST, data) + return scu.sendJSON(data) -def itemsuivi_tag_set(itemsuivi_id="", taglist=[], REQUEST=None): +def itemsuivi_tag_set(itemsuivi_id="", taglist=None): """taglist may either be: a string with tag names separated by commas ("un;deux") or a list of strings (["un", "deux"]) diff --git a/app/scodoc/sco_edt_cal.py b/app/scodoc/sco_edt_cal.py index 4f97bcbbdf..ba744840f6 100644 --- a/app/scodoc/sco_edt_cal.py +++ b/app/scodoc/sco_edt_cal.py @@ -149,7 +149,7 @@ def group_edt_json(group_id, start="", end="", REQUEST=None): # actuellement in } J.append(d) - return scu.sendJSON(REQUEST, J) + return scu.sendJSON(J) """XXX diff --git a/app/scodoc/sco_exceptions.py b/app/scodoc/sco_exceptions.py index 5ae9ace4b0..ec487b0990 100644 --- a/app/scodoc/sco_exceptions.py +++ b/app/scodoc/sco_exceptions.py @@ -55,11 +55,9 @@ class InvalidNoteValue(ScoException): # Exception qui stoque dest_url, utilisee dans Zope standard_error_message class ScoValueError(ScoException): - def __init__(self, msg, dest_url=None, REQUEST=None): + def __init__(self, msg, dest_url=None): ScoException.__init__(self, msg) self.dest_url = dest_url - if REQUEST and dest_url: - REQUEST.set("dest_url", dest_url) class FormatError(ScoValueError): @@ -79,7 +77,7 @@ class ScoConfigurationError(ScoValueError): class ScoLockedFormError(ScoException): - def __init__(self, msg="", REQUEST=None): + def __init__(self, msg=""): msg = ( "Cette formation est verrouillée (car il y a un semestre verrouillé qui s'y réfère). " + str(msg) @@ -90,7 +88,7 @@ class ScoLockedFormError(ScoException): class ScoGenError(ScoException): "exception avec affichage d'une page explicative ad-hoc" - def __init__(self, msg="", REQUEST=None): + def __init__(self, msg=""): ScoException.__init__(self, msg) diff --git a/app/scodoc/sco_recapcomplet.py b/app/scodoc/sco_recapcomplet.py index b03df0c5a8..9780821ef8 100644 --- a/app/scodoc/sco_recapcomplet.py +++ b/app/scodoc/sco_recapcomplet.py @@ -973,4 +973,4 @@ def formsemestres_bulletins(annee_scolaire, REQUEST=None): ) jslist.append(J) - return scu.sendJSON(REQUEST, jslist) + return scu.sendJSON(jslist) diff --git a/app/scodoc/sco_saisie_notes.py b/app/scodoc/sco_saisie_notes.py index 8e6f1bb6f6..213a487a66 100644 --- a/app/scodoc/sco_saisie_notes.py +++ b/app/scodoc/sco_saisie_notes.py @@ -1261,7 +1261,7 @@ def save_note(etudid=None, evaluation_id=None, value=None, comment="", REQUEST=N else: result["history_menu"] = "" # no update needed result["status"] = "ok" - return scu.sendJSON(REQUEST, result) + return scu.sendJSON(result) def get_note_history_menu(evaluation_id, etudid): diff --git a/app/scodoc/sco_tag_module.py b/app/scodoc/sco_tag_module.py index d8a78478fd..d4b1fc0f41 100644 --- a/app/scodoc/sco_tag_module.py +++ b/app/scodoc/sco_tag_module.py @@ -214,7 +214,7 @@ def module_tag_search(term, REQUEST=None): ) data = [x["title"] for x in r] - return scu.sendJSON(REQUEST, data) + return scu.sendJSON(data) def module_tag_list(module_id=""): diff --git a/app/scodoc/sco_undo_notes.py b/app/scodoc/sco_undo_notes.py index 60c15d534b..16d6a34f85 100644 --- a/app/scodoc/sco_undo_notes.py +++ b/app/scodoc/sco_undo_notes.py @@ -263,7 +263,7 @@ def get_note_history(evaluation_id, etudid, REQUEST=None, fmt=""): x["user_name"] = sco_users.user_info(x["uid"])["nomcomplet"] if fmt == "json": - return scu.sendJSON(REQUEST, history) + return scu.sendJSON(history) else: return history diff --git a/app/scodoc/sco_utils.py b/app/scodoc/sco_utils.py index fc851d39cd..93efdb771e 100644 --- a/app/scodoc/sco_utils.py +++ b/app/scodoc/sco_utils.py @@ -560,7 +560,7 @@ class ScoDocJSONEncoder(json.JSONEncoder): return json.JSONEncoder.default(self, o) -def sendJSON(REQUEST, data): +def sendJSON(data): js = json.dumps(data, indent=1, cls=ScoDocJSONEncoder) return send_file(js, filename="sco_data.json", mime=JSON_MIMETYPE, attached=False) @@ -585,7 +585,7 @@ def sendResult(REQUEST, data, name=None, format=None, force_outer_xml_tag=True): REQUEST, data, tagname=name, force_outer_xml_tag=force_outer_xml_tag ) elif format == "json": - return sendJSON(REQUEST, data) + return sendJSON(data) else: raise ValueError("invalid format: %s" % format) @@ -809,7 +809,7 @@ def return_text_if_published(val, REQUEST): La présence de l'argument REQUEST indique la publication. """ if REQUEST and not isinstance(val, str): - return sendJSON(REQUEST, val) + return sendJSON(val) return val diff --git a/sco_version.py b/sco_version.py index 71c0ee8813..e1ab1c57d0 100644 --- a/sco_version.py +++ b/sco_version.py @@ -1,7 +1,7 @@ # -*- mode: python -*- # -*- coding: utf-8 -*- -SCOVERSION = "9.0.33" +SCOVERSION = "9.0.34" SCONAME = "ScoDoc"