diff --git a/app/api/departements.py b/app/api/departements.py index 787193e23..5965a03a1 100644 --- a/app/api/departements.py +++ b/app/api/departements.py @@ -96,7 +96,7 @@ def departement_get(dept_id: int): /departement/id/1; """ - dept = Departement.query.get_or_404(dept_id) + dept = Departement.get_or_404(dept_id) return dept.to_dict() @@ -212,7 +212,7 @@ def departement_etudiants_by_id(dept_id: int): """ Retourne la liste des étudiants d'un département d'id donné. """ - dept = Departement.query.get_or_404(dept_id) + dept = Departement.get_or_404(dept_id) return [etud.to_dict_short() for etud in dept.etudiants] @@ -246,7 +246,7 @@ def departement_formsemestres_ids_by_id(dept_id: int): /departement/id/1/formsemestres_ids; """ - dept = Departement.query.get_or_404(dept_id) + dept = Departement.get_or_404(dept_id) return [formsemestre.id for formsemestre in dept.formsemestres] @@ -273,7 +273,7 @@ def departement_formsemestres_courants(acronym: str = "", dept_id: int | None = dept = ( Departement.query.filter_by(acronym=acronym).first_or_404() if acronym - else Departement.query.get_or_404(dept_id) + else Departement.get_or_404(dept_id) ) date_courante = request.args.get("date_courante") date_courante = datetime.fromisoformat(date_courante) if date_courante else None diff --git a/app/api/evaluations.py b/app/api/evaluations.py index 498d886cc..36d48abcb 100644 --- a/app/api/evaluations.py +++ b/app/api/evaluations.py @@ -215,7 +215,7 @@ def evaluation_create(moduleimpl_id: int): /moduleimpl/1/evaluation/create;{""description"":""Exemple éval.""} """ - moduleimpl: ModuleImpl = ModuleImpl.query.get_or_404(moduleimpl_id) + moduleimpl: ModuleImpl = ModuleImpl.get_or_404(moduleimpl_id) if not moduleimpl.can_edit_evaluation(current_user): return scu.json_error(403, "opération non autorisée") data = request.get_json(force=True) # may raise 400 Bad Request diff --git a/app/api/formations.py b/app/api/formations.py index d2de7ce2e..e618716b0 100644 --- a/app/api/formations.py +++ b/app/api/formations.py @@ -199,7 +199,7 @@ def ue_set_parcours(ue_id: int): parcours = [] else: parcours = [ - ApcParcours.query.get_or_404(int(parcour_id)) for parcour_id in parcours_ids + ApcParcours.get_or_404(int(parcour_id)) for parcour_id in parcours_ids ] log(f"ue_set_parcours: ue_id={ue.id} parcours_ids={parcours_ids}") ok, error_message = ue.set_parcours(parcours) @@ -226,7 +226,7 @@ def ue_assoc_niveau(ue_id: int, niveau_id: int): if g.scodoc_dept: query = query.join(Formation).filter_by(dept_id=g.scodoc_dept_id) ue: UniteEns = query.first_or_404() - niveau: ApcNiveau = ApcNiveau.query.get_or_404(niveau_id) + niveau: ApcNiveau = ApcNiveau.get_or_404(niveau_id) ok, error_message = ue.set_niveau_competence(niveau) if not ok: if g.scodoc_dept: # "usage web" diff --git a/app/api/formsemestres.py b/app/api/formsemestres.py index afd2e9bad..4d91bc4f5 100644 --- a/app/api/formsemestres.py +++ b/app/api/formsemestres.py @@ -615,13 +615,13 @@ def formsemestre_etat_evaluations(formsemestre_id: int): result = [] for modimpl_id in nt.modimpls_results: modimpl_results: ModuleImplResults = nt.modimpls_results[modimpl_id] - modimpl: ModuleImpl = ModuleImpl.query.get_or_404(modimpl_id) + modimpl: ModuleImpl = ModuleImpl.get_or_404(modimpl_id) modimpl_dict = modimpl.to_dict(convert_objects=True, with_module=False) list_eval = [] for evaluation_id in modimpl_results.evaluations_etat: eval_etat = modimpl_results.evaluations_etat[evaluation_id] - evaluation = Evaluation.query.get_or_404(evaluation_id) + evaluation = Evaluation.get_or_404(evaluation_id) eval_dict = evaluation.to_dict_api() eval_dict["etat"] = eval_etat.to_dict() diff --git a/app/api/jury.py b/app/api/jury.py index 528faa435..8005910b6 100644 --- a/app/api/jury.py +++ b/app/api/jury.py @@ -277,7 +277,7 @@ def validation_rcue_record(etudid: int): except ValueError: return json_error(API_CLIENT_ERROR, "invalid date string") if parcours_id is not None: - parcours: ApcParcours = ApcParcours.query.get_or_404(parcours_id) + parcours: ApcParcours = ApcParcours.get_or_404(parcours_id) if parcours.referentiel_id != ue1.niveau_competence.competence.referentiel_id: return json_error(API_CLIENT_ERROR, "niveau et parcours incompatibles") diff --git a/app/api/partitions.py b/app/api/partitions.py index d9ac1cef1..9008285fa 100644 --- a/app/api/partitions.py +++ b/app/api/partitions.py @@ -159,7 +159,7 @@ def group_etudiants_query(group_id: int): @as_json def group_set_etudiant(group_id: int, etudid: int): """Affecte l'étudiant au groupe indiqué.""" - etud = Identite.query.get_or_404(etudid) + etud = Identite.get_or_404(etudid) query = GroupDescr.query.filter_by(id=group_id) if g.scodoc_dept: query = ( @@ -192,7 +192,7 @@ def group_set_etudiant(group_id: int, etudid: int): @as_json def group_remove_etud(group_id: int, etudid: int): """Retire l'étudiant de ce groupe. S'il n'y est pas, ne fait rien.""" - etud = Identite.query.get_or_404(etudid) + etud = Identite.get_or_404(etudid) query = GroupDescr.query.filter_by(id=group_id) if g.scodoc_dept: query = ( @@ -224,7 +224,7 @@ def partition_remove_etud(partition_id: int, etudid: int): (NB: en principe, un étudiant ne doit être que dans 0 ou 1 groupe d'une partition) """ - etud = Identite.query.get_or_404(etudid) + etud = Identite.get_or_404(etudid) query = Partition.query.filter_by(id=partition_id) if g.scodoc_dept: query = query.join(FormSemestre).filter_by(dept_id=g.scodoc_dept_id) @@ -533,7 +533,7 @@ def formsemestre_set_partitions_order(formsemestre_id: int): message="paramètre liste des partitions invalide", ) for p_id, numero in zip(partition_ids, range(len(partition_ids))): - partition = Partition.query.get_or_404(p_id) + partition = Partition.get_or_404(p_id) partition.numero = numero db.session.add(partition) db.session.commit() @@ -579,7 +579,7 @@ def partition_order_groups(partition_id: int): message="paramètre liste de groupe invalide", ) for group_id, numero in zip(group_ids, range(len(group_ids))): - group = GroupDescr.query.get_or_404(group_id) + group = GroupDescr.get_or_404(group_id) group.numero = numero db.session.add(group) db.session.commit() diff --git a/app/api/users.py b/app/api/users.py index f446c7279..d5cb2f7ac 100644 --- a/app/api/users.py +++ b/app/api/users.py @@ -188,7 +188,7 @@ def user_edit(uid: int): ``` """ args = request.get_json(force=True) # may raise 400 Bad Request - user: User = User.query.get_or_404(uid) + user: User = User.get_or_404(uid) # L'utilisateur doit avoir le droit dans le département de départ et celui d'arrivée orig_dept = user.dept dest_dept = args.get("dept", False) @@ -241,7 +241,7 @@ def user_password(uid: int): /user/3/password;{""password"" : ""rePlaCemeNT456averylongandcomplicated""} """ data = request.get_json(force=True) # may raise 400 Bad Request - user: User = User.query.get_or_404(uid) + user: User = User.get_or_404(uid) password = data.get("password") if not password: return json_error(404, "user_password: missing password") @@ -272,7 +272,7 @@ def user_password(uid: int): @as_json def user_role_add(uid: int, role_name: str, dept: str = None): """Ajoute un rôle à l'utilisateur dans le département donné.""" - user: User = User.query.get_or_404(uid) + user: User = User.get_or_404(uid) role: Role = Role.query.filter_by(name=role_name).first_or_404() if dept is not None: # check _ = Departement.query.filter_by(acronym=dept).first_or_404() @@ -301,7 +301,7 @@ def user_role_add(uid: int, role_name: str, dept: str = None): @as_json def user_role_remove(uid: int, role_name: str, dept: str = None): """Retire le rôle (dans le département donné) à cet utilisateur.""" - user: User = User.query.get_or_404(uid) + user: User = User.get_or_404(uid) role: Role = Role.query.filter_by(name=role_name).first_or_404() if dept is not None: # check _ = Departement.query.filter_by(acronym=dept).first_or_404() diff --git a/app/auth/models.py b/app/auth/models.py index 465169c57..ba1c6701a 100644 --- a/app/auth/models.py +++ b/app/auth/models.py @@ -626,7 +626,7 @@ class AnonymousUser(AnonymousUserMixin): login.anonymous_user = AnonymousUser -class Role(db.Model): +class Role(ScoDocModel): """Roles for ScoDoc""" id = db.Column(db.Integer, primary_key=True) @@ -730,7 +730,7 @@ class Role(db.Model): return Role.query.filter_by(name=name).first() -class UserRole(db.Model): +class UserRole(ScoDocModel): """Associate user to role, in a dept. If dept is None, the role applies to all departments (eg super admin). """ diff --git a/app/but/bulletin_but_court.py b/app/but/bulletin_but_court.py index d9772d630..0254d102b 100644 --- a/app/but/bulletin_but_court.py +++ b/app/but/bulletin_but_court.py @@ -57,7 +57,7 @@ from app.views import ScoData @permission_required(Permission.ScoView) def bulletin_but(formsemestre_id: int, etudid: int = None, fmt="html"): """Page HTML affichant le bulletin BUT simplifié""" - etud: Identite = Identite.query.get_or_404(etudid) + etud: Identite = Identite.get_or_404(etudid) formsemestre: FormSemestre = ( FormSemestre.query.filter_by(id=formsemestre_id) .join(FormSemestreInscription) diff --git a/app/but/cursus_but.py b/app/but/cursus_but.py index 6afa8f82e..b321cc46b 100644 --- a/app/but/cursus_but.py +++ b/app/but/cursus_but.py @@ -508,7 +508,7 @@ def but_validations_ues_parcours( # Les UEs associées au tronc commun (à aucun parcours) # UniteEns.query.filter(~UniteEns.id.in_(UEParcours.query.with_entities(UEParcours.ue_id))) - parcour = ApcParcours.query.get(parcour_id) + parcour: ApcParcours = db.session.get(ApcParcours, parcour_id) if not parcour: raise ScoValueError(f"but_validations_ues_parcours: {parcour_id} inexistant") # Les validations d'UE de ce parcours ou du tronc commun pour cet étudiant: diff --git a/app/entreprises/models.py b/app/entreprises/models.py index b41d6b5ce..91f31f3fc 100644 --- a/app/entreprises/models.py +++ b/app/entreprises/models.py @@ -1,7 +1,7 @@ -from app import db +from app import db, models -class Entreprise(db.Model): +class Entreprise(models.ScoDocModel): __tablename__ = "are_entreprises" id = db.Column(db.Integer, primary_key=True) siret = db.Column(db.Text, index=True, unique=True) @@ -45,7 +45,7 @@ class Entreprise(db.Model): } -class EntrepriseSite(db.Model): +class EntrepriseSite(models.ScoDocModel): __tablename__ = "are_sites" id = db.Column(db.Integer, primary_key=True) entreprise_id = db.Column( @@ -65,7 +65,7 @@ class EntrepriseSite(db.Model): ) def to_dict(self): - entreprise = Entreprise.query.get_or_404(self.entreprise_id) + entreprise = Entreprise.get_or_404(self.entreprise_id) return { "siret_entreprise": entreprise.siret, "id_site": self.id, @@ -77,7 +77,7 @@ class EntrepriseSite(db.Model): } -class EntrepriseCorrespondant(db.Model): +class EntrepriseCorrespondant(models.ScoDocModel): __tablename__ = "are_correspondants" id = db.Column(db.Integer, primary_key=True) site_id = db.Column(db.Integer, db.ForeignKey("are_sites.id", ondelete="cascade")) @@ -92,7 +92,7 @@ class EntrepriseCorrespondant(db.Model): notes = db.Column(db.Text) def to_dict(self): - site = EntrepriseSite.query.get_or_404(self.site_id) + site = EntrepriseSite.get_or_404(self.site_id) return { "id": self.id, "civilite": self.civilite, @@ -108,7 +108,7 @@ class EntrepriseCorrespondant(db.Model): } -class EntrepriseContact(db.Model): +class EntrepriseContact(models.ScoDocModel): __tablename__ = "are_contacts" id = db.Column(db.Integer, primary_key=True) date = db.Column(db.DateTime(timezone=True)) @@ -119,7 +119,7 @@ class EntrepriseContact(db.Model): notes = db.Column(db.Text) -class EntrepriseOffre(db.Model): +class EntrepriseOffre(models.ScoDocModel): __tablename__ = "are_offres" id = db.Column(db.Integer, primary_key=True) entreprise_id = db.Column( @@ -147,7 +147,7 @@ class EntrepriseOffre(db.Model): } -class EntrepriseHistorique(db.Model): +class EntrepriseHistorique(models.ScoDocModel): __tablename__ = "are_historique" id = db.Column(db.Integer, primary_key=True) date = db.Column(db.DateTime(timezone=True), server_default=db.func.now()) @@ -158,7 +158,7 @@ class EntrepriseHistorique(db.Model): text = db.Column(db.Text) -class EntrepriseStageApprentissage(db.Model): +class EntrepriseStageApprentissage(models.ScoDocModel): __tablename__ = "are_stages_apprentissages" id = db.Column(db.Integer, primary_key=True) entreprise_id = db.Column( @@ -176,7 +176,7 @@ class EntrepriseStageApprentissage(db.Model): notes = db.Column(db.Text) -class EntrepriseTaxeApprentissage(db.Model): +class EntrepriseTaxeApprentissage(models.ScoDocModel): __tablename__ = "are_taxe_apprentissage" id = db.Column(db.Integer, primary_key=True) entreprise_id = db.Column( @@ -187,7 +187,7 @@ class EntrepriseTaxeApprentissage(db.Model): notes = db.Column(db.Text) -class EntrepriseEnvoiOffre(db.Model): +class EntrepriseEnvoiOffre(models.ScoDocModel): __tablename__ = "are_envoi_offre" id = db.Column(db.Integer, primary_key=True) sender_id = db.Column(db.Integer, db.ForeignKey("user.id", ondelete="cascade")) @@ -196,7 +196,7 @@ class EntrepriseEnvoiOffre(db.Model): date_envoi = db.Column(db.DateTime(timezone=True), server_default=db.func.now()) -class EntrepriseEnvoiOffreEtudiant(db.Model): +class EntrepriseEnvoiOffreEtudiant(models.ScoDocModel): __tablename__ = "are_envoi_offre_etudiant" id = db.Column(db.Integer, primary_key=True) sender_id = db.Column(db.Integer, db.ForeignKey("user.id", ondelete="cascade")) @@ -207,14 +207,14 @@ class EntrepriseEnvoiOffreEtudiant(db.Model): date_envoi = db.Column(db.DateTime(timezone=True), server_default=db.func.now()) -class EntrepriseOffreDepartement(db.Model): +class EntrepriseOffreDepartement(models.ScoDocModel): __tablename__ = "are_offre_departement" id = db.Column(db.Integer, primary_key=True) offre_id = db.Column(db.Integer, db.ForeignKey("are_offres.id", ondelete="cascade")) dept_id = db.Column(db.Integer, db.ForeignKey("departement.id", ondelete="cascade")) -class EntreprisePreferences(db.Model): +class EntreprisePreferences(models.ScoDocModel): __tablename__ = "are_preferences" id = db.Column(db.Integer, primary_key=True) name = db.Column(db.Text) diff --git a/app/entreprises/routes.py b/app/entreprises/routes.py index 537e86b96..e5696adf4 100644 --- a/app/entreprises/routes.py +++ b/app/entreprises/routes.py @@ -1489,7 +1489,7 @@ def add_stage_apprentissage(entreprise_id): ) if form.validate_on_submit(): etudid = form.etudid.data - etudiant = Identite.query.get_or_404(etudid) + etudiant = Identite.get_or_404(etudid) formation = etudiant.inscription_courante_date( form.date_debut.data, form.date_fin.data ) @@ -1552,7 +1552,7 @@ def edit_stage_apprentissage(entreprise_id, stage_apprentissage_id): ) if form.validate_on_submit(): etudid = form.etudid.data - etudiant = Identite.query.get_or_404(etudid) + etudiant = Identite.get_or_404(etudid) formation = etudiant.inscription_courante_date( form.date_debut.data, form.date_fin.data ) diff --git a/app/formations/edit_formation.py b/app/formations/edit_formation.py index 3b9a26368..593199a09 100644 --- a/app/formations/edit_formation.py +++ b/app/formations/edit_formation.py @@ -50,7 +50,7 @@ from app.scodoc import codes_cursus def formation_delete(formation_id=None, dialog_confirmed=False): """Delete a formation""" - formation: Formation = Formation.query.get_or_404(formation_id) + formation: Formation = Formation.get_or_404(formation_id) H = [ f"""

Suppression de la formation {formation.titre} ({formation.acronyme})

""", @@ -159,7 +159,7 @@ def formation_edit(formation_id=None, create=False): is_locked = False else: # edit an existing formation - formation: Formation = Formation.query.get_or_404(formation_id) + formation: Formation = Formation.get_or_404(formation_id) form_dict = formation.to_dict() form_dict["commentaire"] = form_dict["commentaire"] or "" initvalues = form_dict @@ -347,7 +347,7 @@ def do_formation_edit(args) -> bool: if "formation_code" in args and not args["formation_code"]: del args["formation_code"] - formation: Formation = Formation.query.get_or_404(args["formation_id"]) + formation: Formation = Formation.get_or_404(args["formation_id"]) # On autorise la modif de la formation meme si elle est verrouillee # car cela ne change que du cosmetique, (sauf eventuellement le code formation ?) # mais si verrouillée on ne peut changer le type de parcours @@ -386,7 +386,7 @@ def do_formation_edit(args) -> bool: def module_move(module_id, after=0, redirect=True): """Move before/after previous one (decrement/increment numero)""" redirect = bool(redirect) - module = Module.query.get_or_404(module_id) + module = Module.get_or_404(module_id) after = int(after) # 0: deplace avant, 1 deplace apres if after not in (0, 1): raise ValueError(f'invalid value for "after" ({after})') @@ -430,7 +430,7 @@ def module_move(module_id, after=0, redirect=True): def ue_move(ue_id, after=0, redirect=1): """Move UE before/after previous one (decrement/increment numero)""" - ue = UniteEns.query.get_or_404(ue_id) + ue = UniteEns.get_or_404(ue_id) redirect = int(redirect) after = int(after) # 0: deplace avant, 1 deplace apres if after not in (0, 1): diff --git a/app/formations/edit_matiere.py b/app/formations/edit_matiere.py index d758ee195..517e35707 100644 --- a/app/formations/edit_matiere.py +++ b/app/formations/edit_matiere.py @@ -45,7 +45,7 @@ from app.scodoc.sco_exceptions import ( def matiere_create(ue_id=None): """Formulaire création d'une matiere""" - ue: UniteEns = UniteEns.query.get_or_404(ue_id) + ue: UniteEns = UniteEns.get_or_404(ue_id) default_numero = max([mat.numero for mat in ue.matieres] or [9]) + 1 H = [ f"""

Création d'une matière dans l'UE {ue.titre or ''} ({ue.acronyme})

diff --git a/app/formations/edit_module.py b/app/formations/edit_module.py index 34673579d..388552774 100644 --- a/app/formations/edit_module.py +++ b/app/formations/edit_module.py @@ -52,7 +52,7 @@ from app.scodoc import codes_cursus def module_delete(module_id=None): """Formulaire suppression d'un module""" - module = Module.query.get_or_404(module_id) + module = Module.get_or_404(module_id) if not module.can_be_deleted(): raise ScoNonEmptyFormationObject( @@ -149,13 +149,13 @@ def module_edit( formation = ue.formation orig_semestre_idx = ue.semestre_idx if semestre_id is None else semestre_id else: - formation = Formation.query.get_or_404(formation_id) + formation = Formation.get_or_404(formation_id) module = None unlocked = True else: if not module_id: raise ValueError("missing module_id !") - module = models.Module.query.get_or_404(module_id) + module = models.Module.get_or_404(module_id) ue = module.ue module_dict = module.to_dict() formation = module.formation @@ -714,7 +714,7 @@ def module_edit( old_ue_id = module.ue.id new_ue_id = tf[2]["ue_id"] if (old_ue_id != new_ue_id) and in_use: - new_ue = UniteEns.query.get_or_404(new_ue_id) + new_ue = UniteEns.get_or_404(new_ue_id) if new_ue.semestre_idx != module.ue.semestre_idx: # pas changer de semestre un module utilisé ! raise ScoValueError( @@ -808,7 +808,7 @@ def formation_add_malus_modules( ): """Création d'un module de "malus" dans chaque UE d'une formation""" - formation = Formation.query.get_or_404(formation_id) + formation = Formation.get_or_404(formation_id) nb = 0 ues = formation.ues diff --git a/app/formations/edit_ue.py b/app/formations/edit_ue.py index e02edf607..4c77666af 100644 --- a/app/formations/edit_ue.py +++ b/app/formations/edit_ue.py @@ -219,7 +219,7 @@ def ue_edit(ue_id=None, create=False, formation_id=None, default_semestre_idx=No """Formulaire modification ou création d'une UE""" create = int(create) if not create: - ue: UniteEns = UniteEns.query.get_or_404(ue_id) + ue: UniteEns = UniteEns.get_or_404(ue_id) ue_dict = ue.to_dict() formation_id = ue.formation_id title = f"Modification de l'UE {ue.acronyme} {ue.titre}" @@ -573,7 +573,7 @@ def next_ue_numero(formation_id, semestre_id=None) -> int: def ue_delete(ue_id=None, delete_validations=False, dialog_confirmed=False): """Delete an UE""" - ue = UniteEns.query.get_or_404(ue_id) + ue = UniteEns.get_or_404(ue_id) if ue.modules.all(): raise ScoValueError( f"""Suppression de l'UE {ue.titre} impossible car @@ -1370,7 +1370,7 @@ def ue_sharing_code(ue_code: str = "", ue_id: int = None, hide_ue_id: int = None hide_ue_id spécifie un id à retirer de la liste. """ if ue_id is not None: - ue = UniteEns.query.get_or_404(ue_id) + ue = UniteEns.get_or_404(ue_id) if not ue_code: ue_code = ue.ue_code formation_code = ue.formation.formation_code diff --git a/app/formations/formation_io.py b/app/formations/formation_io.py index 38fc53d13..5f3b1e1b8 100644 --- a/app/formations/formation_io.py +++ b/app/formations/formation_io.py @@ -653,7 +653,7 @@ def formation_list_table(detail: bool) -> GenTable: def formation_create_new_version(formation_id, redirect=True): "duplicate formation, with new version number" - formation = Formation.query.get_or_404(formation_id) + formation = Formation.get_or_404(formation_id) resp = formation_export( formation_id, export_ids=True, export_external_ues=True, fmt="xml" ) diff --git a/app/formations/formation_versions.py b/app/formations/formation_versions.py index afc41b293..41596aaf0 100644 --- a/app/formations/formation_versions.py +++ b/app/formations/formation_versions.py @@ -60,7 +60,7 @@ def formsemestre_associate_new_version( formsemestre_id: optionnel, formsemestre de départ, qui sera associé à la nouvelle version """ formsemestre_id = int(formsemestre_id) if formsemestre_id else None - formation: Formation = Formation.query.get_or_404(formation_id) + formation: Formation = Formation.get_or_404(formation_id) other_formsemestre_ids = {int(x) for x in (other_formsemestre_ids or [])} if request.method == "GET": # dresse la liste des semestres non verrouillés de la même formation @@ -191,7 +191,7 @@ def do_formsemestres_associate_new_version( log(f"do_formsemestres_associate_new_version {formation_id} {formsemestre_ids}") # Check: tous les semestres de la formation - formsemestres = [FormSemestre.query.get_or_404(i) for i in formsemestre_ids] + formsemestres = [FormSemestre.get_or_404(i) for i in formsemestre_ids] if not all( [formsemestre.formation_id == formation_id for formsemestre in formsemestres] ): diff --git a/app/models/__init__.py b/app/models/__init__.py index c70d2fbdf..0ca9a70cd 100644 --- a/app/models/__init__.py +++ b/app/models/__init__.py @@ -5,6 +5,8 @@ from flask import abort, g import sqlalchemy +from sqlalchemy import select +from sqlalchemy.exc import NoResultFound import app from app import db @@ -164,6 +166,16 @@ class ScoDocModel(db.Model): return query.first() return query.first_or_404() + # Compatibilité avec SQLAlchemy 2.0 + @classmethod + def get_or_404(cls, oid: int | str): + """Get instance or abort 404""" + stmt = select(cls).where(cls.id == oid) + try: + return db.session.execute(stmt).scalar_one() + except NoResultFound: + abort(404) + from app.models.absences import Absence, AbsenceNotification, BilletAbsence from app.models.departements import Departement diff --git a/app/models/absences.py b/app/models/absences.py index 27aab114b..c3265a771 100644 --- a/app/models/absences.py +++ b/app/models/absences.py @@ -3,10 +3,10 @@ """Gestion des absences """ -from app import db +from app import db, models -class Absence(db.Model): +class Absence(models.ScoDocModel): """LEGACY Ce modèle n'est PLUS UTILISE depuis ScoDoc 9.6 et remplacé par assiduité. une absence (sur une demi-journée) @@ -45,7 +45,7 @@ class Absence(db.Model): return data -class AbsenceNotification(db.Model): +class AbsenceNotification(models.ScoDocModel): """Notification d'absence émise""" __tablename__ = "absences_notifications" @@ -67,7 +67,7 @@ class AbsenceNotification(db.Model): ) -class BilletAbsence(db.Model): +class BilletAbsence(models.ScoDocModel): """Billet d'absence (signalement par l'étudiant)""" __tablename__ = "billet_absence" diff --git a/app/models/assiduites.py b/app/models/assiduites.py index 5c36a1710..44cf1fc1e 100644 --- a/app/models/assiduites.py +++ b/app/models/assiduites.py @@ -752,7 +752,7 @@ def get_assiduites_justif(assiduite_id: int, long: bool) -> list[int | dict]: list[int | dict]: La liste des justificatifs (par défaut uniquement les identifiants, sinon les dict si long est vrai) """ - assi: Assiduite = Assiduite.query.get_or_404(assiduite_id) + assi: Assiduite = Assiduite.get_or_404(assiduite_id) return get_justifs_from_date(assi.etudid, assi.date_debut, assi.date_fin, long) diff --git a/app/models/but_refcomp.py b/app/models/but_refcomp.py index f072bfb6c..d89e19dab 100644 --- a/app/models/but_refcomp.py +++ b/app/models/but_refcomp.py @@ -15,7 +15,7 @@ from flask_sqlalchemy.query import Query from sqlalchemy.orm import class_mapper import sqlalchemy -from app import db, log +from app import db, log, models from app.scodoc.sco_utils import ModuleType from app.scodoc.sco_exceptions import ScoNoReferentielCompetences, ScoValueError @@ -56,7 +56,7 @@ class XMLModel: return f'<{self.__class__.__name__} {self.id} "{self.titre if hasattr(self, "titre") else ""}">' -class ApcReferentielCompetences(db.Model, XMLModel): +class ApcReferentielCompetences(models.ScoDocModel, XMLModel): "Référentiel de compétence d'une spécialité" id = db.Column(db.Integer, primary_key=True) dept_id = db.Column( @@ -339,7 +339,7 @@ class ApcReferentielCompetences(db.Model, XMLModel): return doc.get(self.specialite, {}) -class ApcCompetence(db.Model, XMLModel): +class ApcCompetence(models.ScoDocModel, XMLModel): "Compétence" id = db.Column(db.Integer, primary_key=True) referentiel_id = db.Column( @@ -410,7 +410,7 @@ class ApcCompetence(db.Model, XMLModel): } -class ApcSituationPro(db.Model, XMLModel): +class ApcSituationPro(models.ScoDocModel, XMLModel): "Situation professionnelle" id = db.Column(db.Integer, primary_key=True) competence_id = db.Column( @@ -425,7 +425,7 @@ class ApcSituationPro(db.Model, XMLModel): return {"libelle": self.libelle} -class ApcComposanteEssentielle(db.Model, XMLModel): +class ApcComposanteEssentielle(models.ScoDocModel, XMLModel): "Composante essentielle" id = db.Column(db.Integer, primary_key=True) competence_id = db.Column( @@ -439,7 +439,7 @@ class ApcComposanteEssentielle(db.Model, XMLModel): return {"libelle": self.libelle} -class ApcNiveau(db.Model, XMLModel): +class ApcNiveau(models.ScoDocModel, XMLModel): """Niveau de compétence Chaque niveau peut être associé à deux UE, des semestres impair et pair de la même année. @@ -608,7 +608,7 @@ app_critiques_modules = db.Table( ) -class ApcAppCritique(db.Model, XMLModel): +class ApcAppCritique(models.ScoDocModel, XMLModel): "Apprentissage Critique BUT" id = db.Column(db.Integer, primary_key=True) niveau_id = db.Column( @@ -694,7 +694,7 @@ parcours_formsemestre = db.Table( """Association parcours <-> formsemestre (many-to-many)""" -class ApcParcours(db.Model, XMLModel): +class ApcParcours(models.ScoDocModel, XMLModel): "Un parcours BUT" id = db.Column(db.Integer, primary_key=True) referentiel_id = db.Column( @@ -749,7 +749,7 @@ class ApcParcours(db.Model, XMLModel): ) -class ApcAnneeParcours(db.Model, XMLModel): +class ApcAnneeParcours(models.ScoDocModel, XMLModel): id = db.Column(db.Integer, primary_key=True) parcours_id = db.Column( db.Integer, db.ForeignKey("apc_parcours.id", ondelete="CASCADE"), nullable=False @@ -774,7 +774,7 @@ class ApcAnneeParcours(db.Model, XMLModel): } -class ApcParcoursNiveauCompetence(db.Model): +class ApcParcoursNiveauCompetence(models.ScoDocModel): """Association entre année de parcours et compétence. Le "niveau" de la compétence est donné ici (convention Orébut) diff --git a/app/models/config.py b/app/models/config.py index 25d9b9ccb..f137ccccb 100644 --- a/app/models/config.py +++ b/app/models/config.py @@ -8,7 +8,7 @@ import re import urllib.parse from flask import flash -from app import current_app, db, log +from app import current_app, db, log, models from app.comp import bonus_spo from app.scodoc.sco_exceptions import ScoValueError from app.scodoc import sco_utils as scu @@ -68,7 +68,7 @@ def code_scodoc_to_apo_default(code): return CODES_SCODOC_TO_APO.get(code, "DEF") -class ScoDocSiteConfig(db.Model): +class ScoDocSiteConfig(models.ScoDocModel): """Config. d'un site Nouveau en ScoDoc 9: va regrouper les paramètres qui dans les versions antérieures étaient dans scodoc_config.py diff --git a/app/models/departements.py b/app/models/departements.py index 64127f6e3..18f0181d1 100644 --- a/app/models/departements.py +++ b/app/models/departements.py @@ -4,7 +4,7 @@ """ import re -from app import db +from app import db, models from app.models import SHORT_STR_LEN from app.models.preferences import ScoPreference from app.scodoc.sco_exceptions import ScoValueError @@ -12,7 +12,7 @@ from app.scodoc.sco_exceptions import ScoValueError VALID_DEPT_EXP = re.compile(r"^[\w@\\\-\.]+$") -class Departement(db.Model): +class Departement(models.ScoDocModel): """Un département ScoDoc""" id = db.Column(db.Integer, primary_key=True) @@ -61,7 +61,7 @@ class Departement(db.Model): dept_id = None if dept_id is None: return cls.query.filter_by(acronym=dept_ident).first_or_404() - return cls.query.get_or_404(dept_id) + return cls.get_or_404(dept_id) def to_dict(self, with_dept_name=True, with_dept_preferences=False): data = { diff --git a/app/models/evaluations.py b/app/models/evaluations.py index 6ec196697..06db8075d 100644 --- a/app/models/evaluations.py +++ b/app/models/evaluations.py @@ -526,7 +526,7 @@ class Evaluation(models.ScoDocModel): ) -class EvaluationUEPoids(db.Model): +class EvaluationUEPoids(models.ScoDocModel): """Poids des évaluations (BUT) association many to many """ diff --git a/app/models/events.py b/app/models/events.py index 2773a7ec0..e0285d9c9 100644 --- a/app/models/events.py +++ b/app/models/events.py @@ -62,7 +62,7 @@ class Scolog(ScoDocModel): } -class ScolarNews(db.Model): +class ScolarNews(ScoDocModel): """Nouvelles pour page d'accueil""" NEWS_ABS = "ABS" # saisie absence diff --git a/app/models/formsemestre.py b/app/models/formsemestre.py index 0de2eb854..863b5d8ce 100644 --- a/app/models/formsemestre.py +++ b/app/models/formsemestre.py @@ -1575,7 +1575,7 @@ class FormSemestreUECoef(models.ScoDocModel): coefficient = db.Column(db.Float, nullable=False) -class FormSemestreUEComputationExpr(db.Model): +class FormSemestreUEComputationExpr(models.ScoDocModel): """Formules utilisateurs pour calcul moyenne UE (désactivées en 9.2+).""" __tablename__ = "notes_formsemestre_ue_computation_expr" diff --git a/app/models/groups.py b/app/models/groups.py index 4a07a8753..55cd8ecf9 100644 --- a/app/models/groups.py +++ b/app/models/groups.py @@ -372,11 +372,3 @@ group_membership = db.Table( db.Column("group_id", db.Integer, db.ForeignKey("group_descr.id")), db.UniqueConstraint("etudid", "group_id"), ) -# class GroupMembership(db.Model): -# """Association groupe / étudiant""" - -# __tablename__ = "group_membership" -# __table_args__ = (db.UniqueConstraint("etudid", "group_id"),) -# id = db.Column(db.Integer, primary_key=True) -# etudid = db.Column(db.Integer, db.ForeignKey("identite.id", ondelete="CASCADE")) -# group_id = db.Column(db.Integer, db.ForeignKey("group_descr.id")) diff --git a/app/models/modules.py b/app/models/modules.py index 339b59d5a..2df6a8b2c 100644 --- a/app/models/modules.py +++ b/app/models/modules.py @@ -624,7 +624,7 @@ class Module(models.ScoDocModel): return "", http.HTTPStatus.NO_CONTENT -class ModuleUECoef(db.Model): +class ModuleUECoef(models.ScoDocModel): """Coefficients des modules vers les UE (APC, BUT) En mode APC, ces coefs remplacent le coefficient "PPN" du module. """ diff --git a/app/models/notes.py b/app/models/notes.py index 63e3b23f5..5f5c93ddb 100644 --- a/app/models/notes.py +++ b/app/models/notes.py @@ -83,7 +83,7 @@ class NotesNotes(models.ScoDocModel): } {db.session.get(Evaluation, self.evaluation_id) if self.evaluation_id else "X" }>""" -class NotesNotesLog(db.Model): +class NotesNotesLog(models.ScoDocModel): """Historique des modifs sur notes (anciennes entrees de notes_notes)""" __tablename__ = "notes_notes_log" diff --git a/app/models/scolar_event.py b/app/models/scolar_event.py index 4294efb12..dbc593b35 100644 --- a/app/models/scolar_event.py +++ b/app/models/scolar_event.py @@ -1,10 +1,11 @@ """évènements scolaires dans la vie d'un étudiant(inscription, ...) """ -from app import db + +from app import db, models from app.models import SHORT_STR_LEN -class ScolarEvent(db.Model): +class ScolarEvent(models.ScoDocModel): """Evenement dans le parcours scolaire d'un étudiant""" __tablename__ = "scolar_events" diff --git a/app/models/ues.py b/app/models/ues.py index dce90b4e6..4430c0a1f 100644 --- a/app/models/ues.py +++ b/app/models/ues.py @@ -573,7 +573,7 @@ class UniteEns(models.ScoDocModel): return self.set_parcours(self.parcours + [parcour]) -class UEParcours(db.Model): +class UEParcours(models.ScoDocModel): """Association ue <-> parcours, indiquant les ECTS""" __tablename__ = "ue_parcours" @@ -593,7 +593,7 @@ class UEParcours(db.Model): return f"" -class DispenseUE(db.Model): +class DispenseUE(models.ScoDocModel): """Dispense d'UE Utilisé en APC (BUT) pour indiquer - les étudiants redoublants avec une UE capitalisée qu'ils ne refont pas. diff --git a/app/models/validations.py b/app/models/validations.py index 150bb94a2..fee681ee2 100644 --- a/app/models/validations.py +++ b/app/models/validations.py @@ -4,8 +4,7 @@ """ from flask_sqlalchemy.query import Query -from app import db -from app import log +from app import db, log, models from app.models import SHORT_STR_LEN from app.models import CODE_STR_LEN from app.models.events import Scolog @@ -16,7 +15,7 @@ from app.scodoc import sco_utils as scu from app.scodoc.codes_cursus import CODES_UE_VALIDES -class ScolarFormSemestreValidation(db.Model): +class ScolarFormSemestreValidation(models.ScoDocModel): """Décisions de jury (sur semestre ou UEs)""" __tablename__ = "scolar_formsemestre_validation" @@ -158,7 +157,7 @@ class ScolarFormSemestreValidation(db.Model): ) -class ScolarAutorisationInscription(db.Model): +class ScolarAutorisationInscription(models.ScoDocModel): """Autorisation d'inscription dans un semestre""" __tablename__ = "scolar_autorisation_inscription" diff --git a/app/scodoc/sco_apogee_csv.py b/app/scodoc/sco_apogee_csv.py index f2dba69c9..f054a296c 100644 --- a/app/scodoc/sco_apogee_csv.py +++ b/app/scodoc/sco_apogee_csv.py @@ -282,7 +282,7 @@ class ApoEtud(dict): ): res = self.autre_res else: - formsemestre = FormSemestre.query.get_or_404(sem["formsemestre_id"]) + formsemestre = FormSemestre.get_or_404(sem["formsemestre_id"]) res: NotesTableCompat = res_sem.load_formsemestre_results(formsemestre) if etudid not in res.identdict: @@ -761,7 +761,7 @@ class ApoData: if not self.sems_etape: raise ScoValueError("aucun semestre trouvé !") self.formsemestres_etape = [ - FormSemestre.query.get_or_404(s["formsemestre_id"]) for s in self.sems_etape + FormSemestre.get_or_404(s["formsemestre_id"]) for s in self.sems_etape ] apcs = { formsemestre.formation.is_apc() for formsemestre in self.formsemestres_etape @@ -903,9 +903,7 @@ class ApoData: """ codes_by_sem = {} for sem in self.sems_etape: - formsemestre: FormSemestre = FormSemestre.query.get_or_404( - sem["formsemestre_id"] - ) + formsemestre: FormSemestre = FormSemestre.get_or_404(sem["formsemestre_id"]) # L'ensemble des codes apo associés aux éléments: codes_semestre = formsemestre.get_codes_apogee() codes_modules = set().union( diff --git a/app/scodoc/sco_archives_formsemestre.py b/app/scodoc/sco_archives_formsemestre.py index 31548bcd6..892e9438a 100644 --- a/app/scodoc/sco_archives_formsemestre.py +++ b/app/scodoc/sco_archives_formsemestre.py @@ -416,7 +416,7 @@ def formsemestre_get_archived_file(formsemestre_id, archive_name, filename): def formsemestre_delete_archive(formsemestre_id, archive_name, dialog_confirmed=False): """Delete an archive""" - formsemestre: FormSemestre = FormSemestre.query.get_or_404(formsemestre_id) + formsemestre: FormSemestre = FormSemestre.get_or_404(formsemestre_id) if not formsemestre.can_edit_pv(): raise ScoPermissionDenied( dest_url=url_for( diff --git a/app/scodoc/sco_bulletins.py b/app/scodoc/sco_bulletins.py index e2b4f3c81..e59249926 100644 --- a/app/scodoc/sco_bulletins.py +++ b/app/scodoc/sco_bulletins.py @@ -159,7 +159,7 @@ def formsemestre_bulletinetud_dict(formsemestre_id, etudid, version="long"): # Formation et parcours if I["sem"]["formation_id"]: - formation_dict = Formation.query.get_or_404(I["sem"]["formation_id"]).to_dict() + formation_dict = Formation.get_or_404(I["sem"]["formation_id"]).to_dict() else: # what's the fuck ? formation_dict = { "acronyme": "?", diff --git a/app/scodoc/sco_bulletins_pdf.py b/app/scodoc/sco_bulletins_pdf.py index 678c7d96d..8b11a0701 100644 --- a/app/scodoc/sco_bulletins_pdf.py +++ b/app/scodoc/sco_bulletins_pdf.py @@ -225,7 +225,7 @@ def get_formsemestre_bulletins_pdf( from app.but import bulletin_but_court from app.scodoc import sco_bulletins - formsemestre: FormSemestre = FormSemestre.query.get_or_404(formsemestre_id) + formsemestre: FormSemestre = FormSemestre.get_or_404(formsemestre_id) versions = ( scu.BULLETINS_VERSIONS_BUT if formsemestre.formation.is_apc() diff --git a/app/scodoc/sco_cursus_dut.py b/app/scodoc/sco_cursus_dut.py index e4f7746b3..d4605454e 100644 --- a/app/scodoc/sco_cursus_dut.py +++ b/app/scodoc/sco_cursus_dut.py @@ -313,7 +313,7 @@ class SituationEtudCursusClassic(SituationEtudCursus): sont validés. En sortie, sem_idx_set contient ceux qui n'ont pas été validés.""" for sem in self.get_semestres(): if sem["formation_code"] == self.formation.formation_code: - formsemestre = FormSemestre.query.get_or_404(sem["formsemestre_id"]) + formsemestre = FormSemestre.get_or_404(sem["formsemestre_id"]) nt: NotesTableCompat = res_sem.load_formsemestre_results(formsemestre) decision = nt.get_etud_decision_sem(self.etudid) if decision and code_semestre_validant(decision["code"]): @@ -408,7 +408,7 @@ class SituationEtudCursusClassic(SituationEtudCursus): if not sem: code = "" # non inscrit à ce semestre else: - formsemestre = FormSemestre.query.get_or_404(sem["formsemestre_id"]) + formsemestre = FormSemestre.get_or_404(sem["formsemestre_id"]) nt: NotesTableCompat = res_sem.load_formsemestre_results(formsemestre) decision = nt.get_etud_decision_sem(self.etudid) if decision: @@ -949,7 +949,7 @@ def do_formsemestre_validate_ue( ): """Ajoute ou change validation UE""" if semestre_id is None: - ue = UniteEns.query.get_or_404(ue_id) + ue = UniteEns.get_or_404(ue_id) semestre_id = ue.semestre_idx args = { "formsemestre_id": formsemestre_id, diff --git a/app/scodoc/sco_etape_apogee_view.py b/app/scodoc/sco_etape_apogee_view.py index 1dda5aaab..0e7913fd2 100644 --- a/app/scodoc/sco_etape_apogee_view.py +++ b/app/scodoc/sco_etape_apogee_view.py @@ -347,7 +347,7 @@ def apo_semset_maq_status( if missing: formation_ids = {sem["formation_id"] for sem in semset.sems} formations = [ - Formation.query.get_or_404(formation_id) for formation_id in formation_ids + Formation.get_or_404(formation_id) for formation_id in formation_ids ] H.append( f"""
diff --git a/app/scodoc/sco_export_results.py b/app/scodoc/sco_export_results.py index b372c72c0..aafeabe72 100644 --- a/app/scodoc/sco_export_results.py +++ b/app/scodoc/sco_export_results.py @@ -64,7 +64,7 @@ def _build_results_table(start_date=None, end_date=None, types_parcours=()): semlist = [dpv["formsemestre"] for dpv in dpv_by_sem.values() if dpv] semlist_parcours = [] for sem in semlist: - sem["formation"] = Formation.query.get_or_404(sem["formation_id"]).to_dict() + sem["formation"] = Formation.get_or_404(sem["formation_id"]).to_dict() sem["parcours"] = codes_cursus.get_cursus_from_code( sem["formation"]["type_parcours"] ) diff --git a/app/scodoc/sco_formsemestre.py b/app/scodoc/sco_formsemestre.py index 31dffb736..8ca5a42fa 100644 --- a/app/scodoc/sco_formsemestre.py +++ b/app/scodoc/sco_formsemestre.py @@ -148,7 +148,7 @@ def _formsemestre_enrich(sem): # imports ici pour eviter refs circulaires from app.scodoc import sco_formsemestre_edit - formation: Formation = Formation.query.get_or_404(sem["formation_id"]) + formation: Formation = Formation.get_or_404(sem["formation_id"]) parcours = codes_cursus.get_cursus_from_code(formation.type_parcours) # 'S1', 'S2', ... ou '' pour les monosemestres if sem["semestre_id"] != NO_SEMESTRE_ID: diff --git a/app/scodoc/sco_formsemestre_edit.py b/app/scodoc/sco_formsemestre_edit.py index 02227ab6b..74f1d7481 100644 --- a/app/scodoc/sco_formsemestre_edit.py +++ b/app/scodoc/sco_formsemestre_edit.py @@ -181,7 +181,7 @@ def do_formsemestre_createwithmodules(edit=False, formsemestre: FormSemestre = N formation = formsemestre.formation else: formation_id = int(vals["formation_id"]) - formation = Formation.query.get_or_404(formation_id) + formation = Formation.get_or_404(formation_id) is_apc = formation.is_apc() if not edit: @@ -1076,10 +1076,10 @@ def _formsemestre_check_module_list(module_ids, semestre_idx): """ # vérification de la cohérence / modules / semestre mod_sems_idx = { - Module.query.get_or_404(module_id).ue.semestre_idx for module_id in module_ids + Module.get_or_404(module_id).ue.semestre_idx for module_id in module_ids } if mod_sems_idx and mod_sems_idx != {semestre_idx}: - modules = [Module.query.get_or_404(module_id) for module_id in module_ids] + modules = [Module.get_or_404(module_id) for module_id in module_ids] log( f"""_formsemestre_check_module_list: {chr(10).join( str(module) + " " + str(module.ue) for module in modules )} @@ -1097,7 +1097,7 @@ def _formsemestre_check_module_list(module_ids, semestre_idx): def _formsemestre_check_ue_bonus_unicity(module_ids): """Vérifie qu'il n'y a qu'une seule UE bonus associée aux modules choisis""" - ues = [Module.query.get_or_404(module_id).ue for module_id in module_ids] + ues = [Module.get_or_404(module_id).ue for module_id in module_ids] ues_bonus = {ue.id for ue in ues if ue.type == codes_cursus.UE_SPORT} if len(ues_bonus) > 1: raise ScoValueError( @@ -1294,9 +1294,7 @@ def do_formsemestre_clone( New dates, responsable_id """ log(f"do_formsemestre_clone: {orig_formsemestre_id}") - formsemestre_orig: FormSemestre = FormSemestre.query.get_or_404( - orig_formsemestre_id - ) + formsemestre_orig: FormSemestre = FormSemestre.get_or_404(orig_formsemestre_id) # 1- create sem args = formsemestre_orig.to_dict() del args["formsemestre_id"] diff --git a/app/scodoc/sco_formsemestre_exterieurs.py b/app/scodoc/sco_formsemestre_exterieurs.py index da71ee64b..1e4f7a976 100644 --- a/app/scodoc/sco_formsemestre_exterieurs.py +++ b/app/scodoc/sco_formsemestre_exterieurs.py @@ -61,7 +61,7 @@ def formsemestre_ext_create(etud: Identite | None, sem_params: dict) -> FormSeme sem_params: dict nécessaire à la création du formsemestre """ # Check args - _ = Formation.query.get_or_404(sem_params["formation_id"]) + _ = Formation.get_or_404(sem_params["formation_id"]) # Create formsemestre sem_params["modalite"] = "EXT" @@ -230,7 +230,7 @@ def formsemestre_ext_edit_ue_validations(formsemestre_id, etudid): La moyenne générale indicative du semestre est calculée et affichée, mais pas enregistrée. """ - formsemestre: FormSemestre = FormSemestre.query.get_or_404(formsemestre_id) + formsemestre: FormSemestre = FormSemestre.get_or_404(formsemestre_id) etud = Identite.get_etud(etudid) ues = formsemestre.formation.ues.filter(UniteEns.type != UE_SPORT).order_by( UniteEns.semestre_idx, UniteEns.numero diff --git a/app/scodoc/sco_formsemestre_inscriptions.py b/app/scodoc/sco_formsemestre_inscriptions.py index 33ffb4bdb..9e9324255 100644 --- a/app/scodoc/sco_formsemestre_inscriptions.py +++ b/app/scodoc/sco_formsemestre_inscriptions.py @@ -248,9 +248,7 @@ def do_formsemestre_inscription_with_modules( group_ids = [group_ids] # Check that all groups exist before creating the inscription groups = [ - GroupDescr.query.get_or_404(group_id) - for group_id in group_ids - if group_id != "" + GroupDescr.get_or_404(group_id) for group_id in group_ids if group_id != "" ] formsemestre = FormSemestre.get_formsemestre(formsemestre_id, dept_id=dept_id) # Inscription au semestre @@ -377,7 +375,7 @@ def formsemestre_inscription_with_modules( ) if multiple_ok: multiple_ok = int(multiple_ok) - formsemestre: FormSemestre = FormSemestre.query.get_or_404(formsemestre_id) + formsemestre: FormSemestre = FormSemestre.get_or_404(formsemestre_id) etud = Identite.get_etud(etudid) if etud.dept_id != formsemestre.dept_id: raise ScoValueError("l'étudiant n'est pas dans ce département") diff --git a/app/scodoc/sco_formsemestre_status.py b/app/scodoc/sco_formsemestre_status.py index 1d971fe8e..e4a404d18 100755 --- a/app/scodoc/sco_formsemestre_status.py +++ b/app/scodoc/sco_formsemestre_status.py @@ -1079,7 +1079,7 @@ def formsemestre_status(formsemestre_id=None, check_parcours=True): raise ScoInvalidIdType( "formsemestre_bulletinetud: formsemestre_id must be an integer !" ) - formsemestre: FormSemestre = FormSemestre.query.get_or_404(formsemestre_id) + formsemestre: FormSemestre = FormSemestre.get_or_404(formsemestre_id) # S'assure que les groupes de parcours sont à jour: if int(check_parcours): formsemestre.setup_parcours_groups() diff --git a/app/scodoc/sco_formsemestre_validation.py b/app/scodoc/sco_formsemestre_validation.py index 675abed9e..093fc168c 100644 --- a/app/scodoc/sco_formsemestre_validation.py +++ b/app/scodoc/sco_formsemestre_validation.py @@ -1383,7 +1383,7 @@ def do_formsemestre_validate_previous_ue( cette UE (utile seulement pour les semestres extérieurs). """ nt: NotesTableCompat = res_sem.load_formsemestre_results(formsemestre) - ue: UniteEns = UniteEns.query.get_or_404(ue_id) + ue: UniteEns = UniteEns.get_or_404(ue_id) cnx = ndb.GetDBConnexion() if ue_coefficient is not None: diff --git a/app/scodoc/sco_groups.py b/app/scodoc/sco_groups.py index ae9d7e1ca..3c83e9d0c 100644 --- a/app/scodoc/sco_groups.py +++ b/app/scodoc/sco_groups.py @@ -663,7 +663,7 @@ def change_etud_group_in_partition(etudid: int, group: GroupDescr) -> bool: (et le désinscrit d'autres groupes de cette partition) Return True si changement, False s'il était déjà dans ce groupe. """ - etud: Identite = Identite.query.get_or_404(etudid) + etud: Identite = Identite.get_or_404(etudid) if not group.partition.set_etud_group(etud, group): return # pas de changement @@ -742,7 +742,7 @@ groupsToDelete={groupsToDelete} except ValueError: log(f"setGroups: ignoring invalid group_id={group_id}") continue - group: GroupDescr = GroupDescr.query.get_or_404(group_id) + group: GroupDescr = GroupDescr.get_or_404(group_id) # Anciens membres du groupe: old_members_set = {etud.id for etud in group.etuds} # Place dans ce groupe les etudiants indiqués: @@ -807,7 +807,7 @@ def create_group(partition_id, group_name="", default=False) -> GroupDescr: If default, create default partition (with no name) Obsolete: utiliser Partition.create_group """ - partition = Partition.query.get_or_404(partition_id) + partition = Partition.get_or_404(partition_id) if not partition.formsemestre.can_change_groups(): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") # @@ -840,7 +840,7 @@ def delete_group(group_id, partition_id=None): est bien dans cette partition. S'il s'agit d'un groupe de parcours, affecte l'inscription des étudiants aux parcours. """ - group = GroupDescr.query.get_or_404(group_id) + group = GroupDescr.get_or_404(group_id) if partition_id: if partition_id != group.partition_id: raise ValueError("inconsistent partition/group") @@ -1096,7 +1096,7 @@ def partition_set_attr(partition_id, attr, value): if attr not in {"bul_show_rank", "show_in_lists"}: raise ValueError(f"invalid partition attribute: {attr}") - partition = Partition.query.get_or_404(partition_id) + partition = Partition.get_or_404(partition_id) if not partition.formsemestre.can_change_groups(): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") diff --git a/app/scodoc/sco_groups_edit.py b/app/scodoc/sco_groups_edit.py index 9a6290b8c..0db4d638f 100644 --- a/app/scodoc/sco_groups_edit.py +++ b/app/scodoc/sco_groups_edit.py @@ -42,7 +42,7 @@ def affect_groups(partition_id): Permet aussi la creation et la suppression de groupes. """ # réécrit pour 9.0.47 avec un template - partition = Partition.query.get_or_404(partition_id) + partition = Partition.get_or_404(partition_id) formsemestre = partition.formsemestre if not formsemestre.can_change_groups(): raise AccessDenied("vous n'avez pas la permission de modifier les groupes") @@ -63,7 +63,7 @@ def affect_groups(partition_id): def group_rename(group_id): """Form to rename a group""" - group: GroupDescr = GroupDescr.query.get_or_404(group_id) + group: GroupDescr = GroupDescr.get_or_404(group_id) formsemestre_id = group.partition.formsemestre_id formsemestre = FormSemestre.get_formsemestre(formsemestre_id) if not formsemestre.can_change_groups(): diff --git a/app/scodoc/sco_groups_view.py b/app/scodoc/sco_groups_view.py index f38c4cf90..adc145b58 100644 --- a/app/scodoc/sco_groups_view.py +++ b/app/scodoc/sco_groups_view.py @@ -370,7 +370,7 @@ class DisplayedGroupsInfos: if not group_ids: # appel sans groupe (eg page accueil) if not formsemestre_id: - raise ValueError("missing parameter formsemestre_id or group_ids") + raise ScoValueError("missing parameter formsemestre_id or group_ids") if empty_list_select_all: if select_all_when_unspecified: group_ids = [ diff --git a/app/scodoc/sco_moduleimpl_inscriptions.py b/app/scodoc/sco_moduleimpl_inscriptions.py index 8ea83d7b0..505409861 100644 --- a/app/scodoc/sco_moduleimpl_inscriptions.py +++ b/app/scodoc/sco_moduleimpl_inscriptions.py @@ -399,7 +399,7 @@ def moduleimpl_inscriptions_stats(formsemestre_id): H.append( '

Étudiants avec UEs capitalisées (ADM):