From 51076574ee78cf7aa34cc84d55ba938956003d1d Mon Sep 17 00:00:00 2001 From: Arthur ZHU Date: Mon, 11 Jul 2022 16:06:38 +0200 Subject: [PATCH] refactoring routes --- app/entreprises/routes.py | 502 +++++++++++------- app/templates/entreprises/_correspondant.html | 4 +- app/templates/entreprises/_offre.html | 14 +- app/templates/entreprises/contacts.html | 6 +- app/templates/entreprises/correspondants.html | 2 +- app/templates/entreprises/entreprises.html | 8 +- .../entreprises/entreprises_validation.html | 4 +- .../entreprises/fiche_entreprise.html | 34 +- .../fiche_entreprise_validation.html | 4 +- ...ts.html => form_ajout_correspondants.html} | 0 ...eprise.html => form_ajout_entreprise.html} | 0 ...ml => form_ajout_stage_apprentissage.html} | 0 ...ation_form.html => form_confirmation.html} | 0 ..._offre_form.html => form_envoi_offre.html} | 0 ...n.html => form_validate_confirmation.html} | 0 app/templates/entreprises/import_donnees.html | 6 +- .../entreprises/logs_entreprise.html | 10 +- .../entreprises/offres_expirees.html | 2 +- app/templates/entreprises/offres_recues.html | 4 +- 19 files changed, 351 insertions(+), 249 deletions(-) rename app/templates/entreprises/{ajout_correspondants.html => form_ajout_correspondants.html} (100%) rename app/templates/entreprises/{ajout_entreprise.html => form_ajout_entreprise.html} (100%) rename app/templates/entreprises/{ajout_stage_apprentissage.html => form_ajout_stage_apprentissage.html} (100%) rename app/templates/entreprises/{confirmation_form.html => form_confirmation.html} (100%) rename app/templates/entreprises/{envoi_offre_form.html => form_envoi_offre.html} (100%) rename app/templates/entreprises/{validate_confirmation.html => form_validate_confirmation.html} (100%) diff --git a/app/entreprises/routes.py b/app/entreprises/routes.py index 5f0cfd3ece..ec863bdf06 100644 --- a/app/entreprises/routes.py +++ b/app/entreprises/routes.py @@ -157,18 +157,18 @@ def correspondants(): ) -@bp.route("/fiche_entreprise/", methods=["GET"]) +@bp.route("/fiche_entreprise/", methods=["GET"]) @permission_required(Permission.RelationsEntreprisesView) -def fiche_entreprise(id): +def fiche_entreprise(entreprise_id): """ Permet d'afficher la fiche entreprise d'une entreprise avec une liste des dernières opérations et l'historique des étudiants ayant réaliser un stage ou une alternance dans cette entreprise. La fiche entreprise comporte les informations de l'entreprise, les correspondants de l'entreprise et les offres de l'entreprise. """ - entreprise = Entreprise.query.filter_by(id=id, visible=True).first_or_404( - description=f"fiche entreprise {id} inconnue" - ) + entreprise = Entreprise.query.filter_by( + id=entreprise_id, visible=True + ).first_or_404(description=f"fiche entreprise {entreprise_id} inconnue") offres_with_files = [] depts = are.get_depts() for offre in entreprise.offres: @@ -185,19 +185,19 @@ def fiche_entreprise(id): sites = entreprise.sites[:] logs = ( EntrepriseHistorique.query.order_by(EntrepriseHistorique.date.desc()) - .filter(EntrepriseHistorique.entreprise_id == id) + .filter(EntrepriseHistorique.entreprise_id == entreprise.id) .limit(LOGS_LEN) .all() ) stages_apprentissages = ( db.session.query(EntrepriseStageApprentissage, Identite) .order_by(EntrepriseStageApprentissage.date_debut.desc()) - .filter(EntrepriseStageApprentissage.entreprise_id == id) + .filter(EntrepriseStageApprentissage.entreprise_id == entreprise.id) .join(Identite, Identite.id == EntrepriseStageApprentissage.etudid) .all() ) taxes = ( - EntrepriseTaxeApprentissage.query.filter_by(entreprise_id=id) + EntrepriseTaxeApprentissage.query.filter_by(entreprise_id=entreprise.id) .order_by(EntrepriseTaxeApprentissage.annee.desc()) .all() ) @@ -213,16 +213,16 @@ def fiche_entreprise(id): ) -@bp.route("/fiche_entreprise//logs", methods=["GET"]) +@bp.route("/fiche_entreprise//logs", methods=["GET"]) @permission_required(Permission.RelationsEntreprisesView) -def logs_entreprise(id): +def logs_entreprise(entreprise_id): """ Permet d'afficher les logs d'une entreprise """ page = request.args.get("page", 1, type=int) - entreprise = Entreprise.query.filter_by(id=id, visible=True).first_or_404( - description=f"logs fiche entreprise {id} inconnu" - ) + entreprise = Entreprise.query.filter_by( + id=entreprise_id, visible=True + ).first_or_404(description=f"logs fiche entreprise {entreprise_id} inconnu") logs = ( EntrepriseHistorique.query.order_by(EntrepriseHistorique.date.desc()) .filter(EntrepriseHistorique.entreprise_id == entreprise.id) @@ -236,14 +236,16 @@ def logs_entreprise(id): ) -@bp.route("/fiche_entreprise_validation/", methods=["GET"]) +@bp.route("/fiche_entreprise_validation/", methods=["GET"]) @permission_required(Permission.RelationsEntreprisesValidate) -def fiche_entreprise_validation(id): +def fiche_entreprise_validation(entreprise_id): """ Permet d'afficher la fiche entreprise d'une entreprise a valider """ - entreprise = Entreprise.query.filter_by(id=id, visible=False).first_or_404( - description=f"fiche entreprise (validation) {id} inconnue" + entreprise = Entreprise.query.filter_by( + id=entreprise_id, visible=False + ).first_or_404( + description=f"fiche entreprise (validation) {entreprise_id} inconnue" ) sites = entreprise.sites[:] return render_template( @@ -293,15 +295,15 @@ def offres_recues(): ) -@bp.route("/fiche_entreprise//offres_expirees") +@bp.route("/fiche_entreprise//offres_expirees") @permission_required(Permission.RelationsEntreprisesView) -def offres_expirees(id): +def offres_expirees(entreprise_id): """ Permet d'afficher la liste des offres expirés d'une entreprise """ - entreprise = Entreprise.query.filter_by(id=id, visible=True).first_or_404( - description=f"fiche entreprise {id} inconnue" - ) + entreprise = Entreprise.query.filter_by( + id=entreprise_id, visible=True + ).first_or_404(description=f"fiche entreprise {entreprise_id} inconnue") offres_expirees_with_files = [] depts = are.get_depts() for offre in entreprise.offres: @@ -387,21 +389,23 @@ def add_entreprise(): flash("L'entreprise a été ajouté à la liste pour la validation.") return redirect(url_for("entreprises.index")) return render_template( - "entreprises/ajout_entreprise.html", + "entreprises/form_ajout_entreprise.html", title="Ajout entreprise avec correspondant", form=form, ) -@bp.route("/fiche_entreprise/edit_entreprise/", methods=["GET", "POST"]) +@bp.route( + "/fiche_entreprise//edit_entreprise", methods=["GET", "POST"] +) @permission_required(Permission.RelationsEntreprisesChange) -def edit_entreprise(id): +def edit_entreprise(entreprise_id): """ Permet de modifier une entreprise de la base avec un formulaire """ - entreprise = Entreprise.query.filter_by(id=id, visible=True).first_or_404( - description=f"entreprise {id} inconnue" - ) + entreprise = Entreprise.query.filter_by( + id=entreprise_id, visible=True + ).first_or_404(description=f"entreprise {entreprise_id} inconnue") form = EntrepriseModificationForm(siret=entreprise.siret) if form.validate_on_submit(): lien_entreprise = f"{form.nom.data.strip()}" @@ -449,7 +453,9 @@ def edit_entreprise(id): ) db.session.commit() flash("L'entreprise a été modifié.") - return redirect(url_for("entreprises.fiche_entreprise", id=entreprise.id)) + return redirect( + url_for("entreprises.fiche_entreprise", entreprise_id=entreprise.id) + ) elif request.method == "GET": form.siret.data = entreprise.siret form.nom.data = entreprise.nom @@ -465,62 +471,69 @@ def edit_entreprise(id): ) -@bp.route("/fiche_entreprise/desactiver/", methods=["GET", "POST"]) +@bp.route("/fiche_entreprise//desactiver", methods=["GET", "POST"]) @permission_required(Permission.RelationsEntreprisesChange) -def fiche_entreprise_desactiver(id): +def fiche_entreprise_desactiver(entreprise_id): """ Permet de désactiver une entreprise """ entreprise = Entreprise.query.filter_by( - id=id, visible=True, active=True - ).first_or_404(description=f"entreprise {id} inconnue") + id=entreprise_id, visible=True, active=True + ).first_or_404(description=f"entreprise {entreprise_id} inconnue") form = DesactivationConfirmationForm() if form.validate_on_submit(): entreprise.notes_active = form.notes_active.data.strip() entreprise.active = False db.session.commit() flash("L'entreprise a été désactivé.") - return redirect(url_for("entreprises.fiche_entreprise", id=entreprise.id)) + return redirect( + url_for("entreprises.fiche_entreprise", entreprise_id=entreprise.id) + ) return render_template( - "entreprises/confirmation_form.html", + "entreprises/form_confirmation.html", title="Désactiver entreprise", form=form, info_message="Cliquez sur le bouton Modifier pour confirmer la désactivation", ) -@bp.route("/fiche_entreprise/activer/", methods=["GET", "POST"]) +@bp.route("/fiche_entreprise//activer", methods=["GET", "POST"]) @permission_required(Permission.RelationsEntreprisesChange) -def fiche_entreprise_activer(id): +def fiche_entreprise_activer(entreprise_id): """ Permet d'activer une entreprise """ entreprise = Entreprise.query.filter_by( - id=id, visible=True, active=False - ).first_or_404(description=f"entreprise {id} inconnue") + id=entreprise_id, visible=True, active=False + ).first_or_404(description=f"entreprise {entreprise_id} inconnue") form = ActivationConfirmationForm() if form.validate_on_submit(): entreprise.active = True db.session.commit() flash("L'entreprise a été activé.") - return redirect(url_for("entreprises.fiche_entreprise", id=entreprise.id)) + return redirect( + url_for("entreprises.fiche_entreprise", entreprise_id=entreprise.id) + ) return render_template( - "entreprises/confirmation_form.html", + "entreprises/form_confirmation.html", title="Activer entreprise", form=form, info_message="Cliquez sur le bouton Modifier pour confirmer l'activaction", ) -@bp.route("/fiche_entreprise//add_taxe_apprentissage", methods=["GET", "POST"]) -def add_taxe_apprentissage(id): +@bp.route( + "/fiche_entreprise//add_taxe_apprentissage", + methods=["GET", "POST"], +) +def add_taxe_apprentissage(entreprise_id): """ - Permet d'ajouter une taxe d'apprentissage sur un fiche entreprise + Permet d'ajouter une taxe d'apprentissage sur une fiche entreprise """ - entreprise = Entreprise.query.filter_by(id=id, visible=True).first_or_404( - description=f"entreprise {id} inconnue" - ) - form = TaxeApprentissageForm(hidden_entreprise_id=id) + entreprise = Entreprise.query.filter_by( + id=entreprise_id, visible=True + ).first_or_404(description=f"entreprise {entreprise_id} inconnue") + form = TaxeApprentissageForm(hidden_entreprise_id=entreprise.id) if form.validate_on_submit(): taxe = EntrepriseTaxeApprentissage( entreprise_id=entreprise.id, @@ -530,7 +543,9 @@ def add_taxe_apprentissage(id): ) db.session.add(taxe) db.session.commit() - return redirect(url_for("entreprises.fiche_entreprise", id=entreprise.id)) + return redirect( + url_for("entreprises.fiche_entreprise", entreprise_id=entreprise.id) + ) return render_template( "entreprises/form.html", title="Ajout taxe apprentissage", @@ -539,25 +554,26 @@ def add_taxe_apprentissage(id): @bp.route( - "/fiche_entreprise//edit_taxe_apprentissage/", + "/fiche_entreprise//edit_taxe_apprentissage/", methods=["GET", "POST"], ) -def edit_taxe_apprentissage(id_entreprise, id_taxe): +def edit_taxe_apprentissage(entreprise_id, taxe_id): """ - Permet de modifier une taxe d'apprentissage sur un fiche entreprise + Permet de modifier une taxe d'apprentissage sur une fiche entreprise """ - entreprise = Entreprise.query.filter_by( - id=id_entreprise, visible=True - ).first_or_404(description=f"entreprise {id_entreprise} inconnue") - taxe = EntrepriseTaxeApprentissage.query.filter_by(id=id_taxe).first_or_404( - description=f"taxe d'apprentissage {id_taxe} inconnue" + taxe = EntrepriseTaxeApprentissage.query.filter_by( + id=taxe_id, entreprise_id=entreprise_id + ).first_or_404( + description=f"taxe d'apprentissage {taxe_id} inconnue pour l'entreprise {entreprise_id}" ) form = TaxeApprentissageModificationForm(annee=taxe.annee) if form.validate_on_submit(): taxe.montant = form.montant.data taxe.notes = form.notes.data.strip() db.session.commit() - return redirect(url_for("entreprises.fiche_entreprise", id=entreprise.id)) + return redirect( + url_for("entreprises.fiche_entreprise", entreprise_id=taxe.entreprise_id) + ) elif request.method == "GET": form.montant.data = taxe.montant form.notes.data = taxe.notes @@ -569,27 +585,28 @@ def edit_taxe_apprentissage(id_entreprise, id_taxe): @bp.route( - "/fiche_entreprise//delete_taxe_apprentissage/", + "/fiche_entreprise//delete_taxe_apprentissage/", methods=["GET", "POST"], ) -def delete_taxe_apprentissage(id_entreprise, id_taxe): +def delete_taxe_apprentissage(entreprise_id, taxe_id): """ - Permet de modifier une taxe d'apprentissage sur un fiche entreprise + Permet de modifier une taxe d'apprentissage sur une fiche entreprise """ - entreprise = Entreprise.query.filter_by( - id=id_entreprise, visible=True - ).first_or_404(description=f"entreprise {id_entreprise} inconnue") - taxe = EntrepriseTaxeApprentissage.query.filter_by(id=id_taxe).first_or_404( - description=f"taxe d'apprentissage {id_taxe} inconnue" + taxe = EntrepriseTaxeApprentissage.query.filter_by( + id=taxe_id, entreprise_id=entreprise_id + ).first_or_404( + description=f"taxe d'apprentissage {taxe_id} inconnue pour l'entreprise {entreprise_id}" ) form = SuppressionConfirmationForm() if form.validate_on_submit(): db.session.delete(taxe) db.session.commit() flash("La taxe d'apprentissage a été supprimé de la liste.") - return redirect(url_for("entreprises.fiche_entreprise", id=entreprise.id)) + return redirect( + url_for("entreprises.fiche_entreprise", entreprise_id=taxe.entreprise_id) + ) return render_template( - "entreprises/confirmation_form.html", + "entreprises/form_confirmation.html", title="Supprimer taxe apprentissage", form=form, info_message="Cliquez sur le bouton Supprimer pour confirmer votre supression", @@ -597,17 +614,18 @@ def delete_taxe_apprentissage(id_entreprise, id_taxe): @bp.route( - "/fiche_entreprise_validation//validate_entreprise", methods=["GET", "POST"] + "/fiche_entreprise_validation//validate_entreprise", + methods=["GET", "POST"], ) @permission_required(Permission.RelationsEntreprisesValidate) -def validate_entreprise(id): +def validate_entreprise(entreprise_id): """ Permet de valider une entreprise """ form = ValidationConfirmationForm() - entreprise = Entreprise.query.filter_by(id=id, visible=False).first_or_404( - description=f"entreprise (validation) {id} inconnue" - ) + entreprise = Entreprise.query.filter_by( + id=entreprise_id, visible=False + ).first_or_404(description=f"entreprise (validation) {entreprise_id} inconnue") if form.validate_on_submit(): entreprise.visible = True nom_entreprise = f"{entreprise.nom}" @@ -619,26 +637,26 @@ def validate_entreprise(id): db.session.add(log) db.session.commit() flash("L'entreprise a été validé et ajouté à la liste.") - return redirect(url_for("entreprises.index")) + return redirect(url_for("entreprises.validation")) return render_template( - "entreprises/validate_confirmation.html", + "entreprises/form_validate_confirmation.html", title="Validation entreprise", form=form, ) @bp.route( - "/fiche_entreprise_validation//delete_validation_entreprise", + "/fiche_entreprise_validation//delete_validation_entreprise", methods=["GET", "POST"], ) @permission_required(Permission.RelationsEntreprisesValidate) -def delete_validation_entreprise(id): +def delete_validation_entreprise(entreprise_id): """ Permet de supprimer une entreprise en attente de validation avec une formulaire de validation """ - entreprise = Entreprise.query.filter_by(id=id, visible=False).first_or_404( - description=f"entreprise (validation) {id} inconnue" - ) + entreprise = Entreprise.query.filter_by( + id=entreprise_id, visible=False + ).first_or_404(description=f"entreprise (validation) {entreprise_id} inconnue") form = SuppressionConfirmationForm() if form.validate_on_submit(): db.session.delete(entreprise) @@ -646,23 +664,23 @@ def delete_validation_entreprise(id): flash("L'entreprise a été supprimé de la liste des entreprise à valider.") return redirect(url_for("entreprises.validation")) return render_template( - "entreprises/confirmation_form.html", + "entreprises/form_confirmation.html", title="Supression entreprise", form=form, info_message="Cliquez sur le bouton Supprimer pour confirmer votre supression", ) -@bp.route("/fiche_entreprise//add_offre", methods=["GET", "POST"]) +@bp.route("/fiche_entreprise//add_offre", methods=["GET", "POST"]) @permission_required(Permission.RelationsEntreprisesChange) -def add_offre(id): +def add_offre(entreprise_id): """ Permet d'ajouter une offre a une entreprise """ - entreprise = Entreprise.query.filter_by(id=id, visible=True).first_or_404( - description=f"entreprise {id} inconnue" - ) - form = OffreCreationForm(hidden_entreprise_id=id) + entreprise = Entreprise.query.filter_by( + id=entreprise_id, visible=True + ).first_or_404(description=f"entreprise {entreprise_id} inconnue") + form = OffreCreationForm(hidden_entreprise_id=entreprise.id) if form.validate_on_submit(): offre = EntrepriseOffre( entreprise_id=entreprise.id, @@ -708,7 +726,9 @@ def add_offre(id): db.session.add(log) db.session.commit() flash("L'offre a été ajouté à la fiche entreprise.") - return redirect(url_for("entreprises.fiche_entreprise", id=entreprise.id)) + return redirect( + url_for("entreprises.fiche_entreprise", entreprise_id=entreprise.id) + ) return render_template( "entreprises/form.html", title="Ajout offre", @@ -716,14 +736,19 @@ def add_offre(id): ) -@bp.route("/fiche_entreprise/edit_offre/", methods=["GET", "POST"]) +@bp.route( + "/fiche_entreprise//edit_offre/", + methods=["GET", "POST"], +) @permission_required(Permission.RelationsEntreprisesChange) -def edit_offre(id): +def edit_offre(entreprise_id, offre_id): """ Permet de modifier une offre """ - offre = EntrepriseOffre.query.filter_by(id=id).first_or_404( - description=f"offre {id} inconnue" + offre = EntrepriseOffre.query.filter_by( + id=offre_id, entreprise_id=entreprise_id + ).first_or_404( + description=f"offre {offre_id} inconnue pour l'entreprise {entreprise_id}" ) offre_depts = EntrepriseOffreDepartement.query.filter_by(offre_id=offre.id).all() form = OffreModificationForm( @@ -765,7 +790,9 @@ def edit_offre(id): db.session.add(log) db.session.commit() flash("L'offre a été modifié.") - return redirect(url_for("entreprises.fiche_entreprise", id=offre.entreprise.id)) + return redirect( + url_for("entreprises.fiche_entreprise", entreprise_id=offre.entreprise_id) + ) elif request.method == "GET": form.intitule.data = offre.intitule form.description.data = offre.description @@ -781,14 +808,19 @@ def edit_offre(id): ) -@bp.route("/fiche_entreprise/delete_offre/", methods=["GET", "POST"]) +@bp.route( + "/fiche_entreprise//delete_offre/", + methods=["GET", "POST"], +) @permission_required(Permission.RelationsEntreprisesChange) -def delete_offre(id): +def delete_offre(entreprise_id, offre_id): """ Permet de supprimer une offre """ - offre = EntrepriseOffre.query.filter_by(id=id).first_or_404( - description=f"offre {id} inconnue" + offre = EntrepriseOffre.query.filter_by( + id=offre_id, entreprise_id=entreprise_id + ).first_or_404( + description=f"offre {offre_id} inconnue pour l'entreprise {entreprise_id}" ) entreprise_id = offre.entreprise.id form = SuppressionConfirmationForm() @@ -812,37 +844,46 @@ def delete_offre(id): db.session.add(log) db.session.commit() flash("L'offre a été supprimé de la fiche entreprise.") - return redirect(url_for("entreprises.fiche_entreprise", id=entreprise_id)) + return redirect( + url_for("entreprises.fiche_entreprise", entreprise_id=offre.entreprise_id) + ) return render_template( - "entreprises/confirmation_form.html", + "entreprises/form_confirmation.html", title="Supression offre", form=form, info_message="Cliquez sur le bouton Supprimer pour confirmer votre supression", ) -@bp.route("/offres_recues/delete_offre_recue/", methods=["GET", "POST"]) +@bp.route( + "/offres_recues/delete_offre_recue/", methods=["GET", "POST"] +) @permission_required(Permission.RelationsEntreprisesView) -def delete_offre_recue(id): +def delete_offre_recue(envoi_offre_id): """ Permet de supprimer une offre reçue """ offre_recue = EntrepriseEnvoiOffre.query.filter_by( - id=id, receiver_id=current_user.id - ).first_or_404(description=f"offre recu {id} inconnue") + id=envoi_offre_id, receiver_id=current_user.id + ).first_or_404(description=f"offre recu {envoi_offre_id} inconnue") db.session.delete(offre_recue) db.session.commit() return redirect(url_for("entreprises.offres_recues")) -@bp.route("/fiche_entreprise/expired/", methods=["GET", "POST"]) +@bp.route( + "/fiche_entreprise//expired/", + methods=["GET", "POST"], +) @permission_required(Permission.RelationsEntreprisesChange) -def expired(id): +def expired(entreprise_id, offre_id): """ Permet de rendre expirée et non expirée une offre """ - offre = EntrepriseOffre.query.filter_by(id=id).first_or_404( - description=f"offre {id} inconnue" + offre = EntrepriseOffre.query.filter_by( + id=offre_id, entreprise_id=entreprise_id + ).first_or_404( + description=f"offre {offre_id} inconnue pour l'entreprise {entreprise_id}" ) offre.expired = not offre.expired db.session.commit() @@ -850,19 +891,21 @@ def expired(id): flash("L'offre a été rendu expirée") else: flash("L'offre a été rendu non expirée") - return redirect(url_for("entreprises.fiche_entreprise", id=offre.entreprise_id)) + return redirect( + url_for("entreprises.fiche_entreprise", entreprise_id=offre.entreprise_id) + ) @bp.route( - "/fiche_entreprise//add_site", + "/fiche_entreprise//add_site", methods=["GET", "POST"], ) @permission_required(Permission.RelationsEntreprisesChange) -def add_site(id): - entreprise = Entreprise.query.filter_by(id=id, visible=True).first_or_404( - description=f"entreprise {id} inconnue" - ) - form = SiteCreationForm(hidden_entreprise_id=id) +def add_site(entreprise_id): + entreprise = Entreprise.query.filter_by( + id=entreprise_id, visible=True + ).first_or_404(description=f"entreprise {entreprise_id} inconnue") + form = SiteCreationForm(hidden_entreprise_id=entreprise.id) if form.validate_on_submit(): lien_entreprise = f"{entreprise.nom} - {form.nom.data.strip()}" site = EntrepriseSite( @@ -886,7 +929,9 @@ def add_site(id): db.session.add(log) db.session.commit() flash("Le site a été créé et ajouté à la fiche entreprise") - return redirect(url_for("entreprises.fiche_entreprise", id=entreprise.id)) + return redirect( + url_for("entreprises.fiche_entreprise", entreprise_id=entreprise.id) + ) return render_template( "entreprises/form.html", title="Ajout site", @@ -895,18 +940,17 @@ def add_site(id): @bp.route( - "/fiche_entreprise//edit_site/", + "/fiche_entreprise//edit_site/", methods=["GET", "POST"], ) -def edit_site(id_entreprise, id_site): - entreprise = Entreprise.query.filter_by( - id=id_entreprise, visible=True - ).first_or_404(description=f"entreprise {id_entreprise} inconnue") +def edit_site(entreprise_id, site_id): site = EntrepriseSite.query.filter_by( - id=id_site, entreprise_id=entreprise.id - ).first_or_404(description=f"site {id_site} inconnu") + id=site_id, entreprise_id=entreprise_id + ).first_or_404( + description=f"site {site_id} inconnu pour l'entreprise {entreprise_id}" + ) form = SiteModificationForm( - hidden_entreprise_id=id_entreprise, hidden_site_id=id_site + hidden_entreprise_id=site.entreprise_id, hidden_site_id=site.id ) if form.validate_on_submit(): site.nom = form.nom.data.strip() @@ -915,7 +959,9 @@ def edit_site(id_entreprise, id_site): site.ville = form.ville.data.strip() site.pays = (form.pays.data.strip() if form.pays.data.strip() else "FRANCE",) db.session.commit() - return redirect(url_for("entreprises.fiche_entreprise", id=site.entreprise_id)) + return redirect( + url_for("entreprises.fiche_entreprise", entreprise_id=site.entreprise_id) + ) elif request.method == "GET": form.nom.data = site.nom form.adresse.data = site.adresse @@ -930,25 +976,24 @@ def edit_site(id_entreprise, id_site): @bp.route( - "/fiche_entreprise//add_correspondant/", + "/fiche_entreprise//site//add_correspondant", methods=["GET", "POST"], ) @permission_required(Permission.RelationsEntreprisesChange) -def add_correspondant(id_entreprise, id_site): +def add_correspondant(entreprise_id, site_id): """ Permet d'ajouter un correspondant a une entreprise """ - entreprise = Entreprise.query.filter_by( - id=id_entreprise, visible=True - ).first_or_404(description=f"entreprise {id_entreprise} inconnue") - site = EntrepriseSite.query.filter_by(id=id_site).first_or_404( - description=f"site {id_site} inconnue" + site = EntrepriseSite.query.filter_by( + id=site_id, entreprise_id=entreprise_id + ).first_or_404( + description=f"site {site_id} inconnue pour l'entreprise {entreprise_id}" ) - form = CorrespondantsCreationForm(hidden_entreprise_id=entreprise.id) + form = CorrespondantsCreationForm(hidden_entreprise_id=site.entreprise_id) if form.validate_on_submit(): for correspondant_entry in form.correspondants.entries: correspondant = EntrepriseCorrespondant( - entreprise_id=entreprise.id, + entreprise_id=site.entreprise_id, site_id=site.id, civilite=correspondant_entry.civilite.data, nom=correspondant_entry.nom.data.strip(), @@ -973,25 +1018,32 @@ def add_correspondant(id_entreprise, id_site): db.session.add(log) db.session.commit() flash("Le correspondant a été ajouté à la fiche entreprise.") - return redirect(url_for("entreprises.fiche_entreprise", id=entreprise.id)) + return redirect( + url_for("entreprises.fiche_entreprise", entreprise_id=site.entreprise_id) + ) return render_template( - "entreprises/ajout_correspondants.html", + "entreprises/form_ajout_correspondants.html", title="Ajout correspondant", form=form, ) -@bp.route("/fiche_entreprise/edit_correspondant/", methods=["GET", "POST"]) +@bp.route( + "/fiche_entreprise//site//edit_correspondant/", + methods=["GET", "POST"], +) @permission_required(Permission.RelationsEntreprisesChange) -def edit_correspondant(id): +def edit_correspondant(entreprise_id, site_id, correspondant_id): """ Permet de modifier un correspondant """ correspondant = ( db.session.query(EntrepriseCorrespondant) .join(Entreprise, EntrepriseCorrespondant.entreprise_id == Entreprise.id) - .filter(EntrepriseCorrespondant.id == id, Entreprise.visible == True) - .first_or_404(description=f"correspondant {id} inconnu") + .filter( + EntrepriseCorrespondant.id == correspondant_id, Entreprise.visible == True + ) + .first_or_404(description=f"correspondant {correspondant_id} inconnu") ) form = CorrespondantModificationForm( hidden_entreprise_id=correspondant.entreprise_id, @@ -1018,7 +1070,10 @@ def edit_correspondant(id): db.session.commit() flash("Le correspondant a été modifié.") return redirect( - url_for("entreprises.fiche_entreprise", id=correspondant.entreprise_id) + url_for( + "entreprises.fiche_entreprise", + entreprise_id=correspondant.entreprise_id, + ) ) elif request.method == "GET": form.civilite.data = correspondant.civilite @@ -1037,17 +1092,22 @@ def edit_correspondant(id): ) -@bp.route("/fiche_entreprise/delete_correspondant/", methods=["GET", "POST"]) +@bp.route( + "/fiche_entreprise//site//delete_correspondant/", + methods=["GET", "POST"], +) @permission_required(Permission.RelationsEntreprisesChange) -def delete_correspondant(id): +def delete_correspondant(entreprise_id, site_id, correspondant_id): """ Permet de supprimer un correspondant """ correspondant = ( db.session.query(EntrepriseCorrespondant) .join(Entreprise, EntrepriseCorrespondant.entreprise_id == Entreprise.id) - .filter(EntrepriseCorrespondant.id == id, Entreprise.visible == True) - .first_or_404(description=f"correspondant {id} inconnu") + .filter( + EntrepriseCorrespondant.id == correspondant_id, Entreprise.visible == True + ) + .first_or_404(description=f"correspondant {correspondant_id} inconnu") ) form = SuppressionConfirmationForm() if form.validate_on_submit(): @@ -1063,25 +1123,31 @@ def delete_correspondant(id): db.session.commit() flash("Le correspondant a été supprimé de la fiche entreprise.") return redirect( - url_for("entreprises.fiche_entreprise", id=correspondant.entreprise_id) + url_for( + "entreprises.fiche_entreprise", + entreprise_id=correspondant.entreprise_id, + ) ) return render_template( - "entreprises/confirmation_form.html", + "entreprises/form_confirmation.html", title="Supression correspondant", form=form, info_message="Cliquez sur le bouton Supprimer pour confirmer votre supression", ) -@bp.route("/fiche_entreprise//add_contact", methods=["GET", "POST"]) +@bp.route( + "/fiche_entreprise//contacts/add_contact", + methods=["GET", "POST"], +) @permission_required(Permission.RelationsEntreprisesChange) -def add_contact(id): +def add_contact(entreprise_id): """ Permet d'ajouter un contact avec une entreprise """ - entreprise = Entreprise.query.filter_by(id=id, visible=True).first_or_404( - description=f"entreprise {id} inconnue" - ) + entreprise = Entreprise.query.filter_by( + id=entreprise_id, visible=True + ).first_or_404(description=f"entreprise {entreprise_id} inconnue") form = ContactCreationForm( date=f"{datetime.now().strftime('%Y-%m-%dT%H:%M')}", utilisateur=f"{current_user.nom} {current_user.prenom} ({current_user.user_name})" @@ -1106,7 +1172,7 @@ def add_contact(id): ) db.session.add(contact) db.session.commit() - return redirect(url_for("entreprises.contacts", id=entreprise.id)) + return redirect(url_for("entreprises.contacts", entreprise_id=entreprise.id)) return render_template( "entreprises/form.html", title="Ajout contact", @@ -1114,14 +1180,19 @@ def add_contact(id): ) -@bp.route("/fiche_entreprise/edit_contact/", methods=["GET", "POST"]) +@bp.route( + "/fiche_entreprise//contacts/edit_contact/", + methods=["GET", "POST"], +) @permission_required(Permission.RelationsEntreprisesChange) -def edit_contact(id): +def edit_contact(entreprise_id, contact_id): """ Permet d'editer un contact avec une entreprise """ - contact = EntrepriseContact.query.filter_by(id=id).first_or_404( - description=f"contact {id} inconnu" + contact = EntrepriseContact.query.filter_by( + id=contact_id, entreprise=entreprise_id + ).first_or_404( + description=f"contact {contact_id} inconnu pour l'entreprise {entreprise_id}" ) form = ContactModificationForm() if form.validate_on_submit(): @@ -1138,7 +1209,9 @@ def edit_contact(id): contact.user = utilisateur.id contact.notes = form.notes.data db.session.commit() - return redirect(url_for("entreprises.contacts", id=contact.entreprise)) + return redirect( + url_for("entreprises.contacts", entreprise_id=contact.entreprise) + ) elif request.method == "GET": utilisateur = User.query.filter_by(id=contact.user).first() form.date.data = contact.date.strftime("%Y-%m-%dT%H:%M") @@ -1153,15 +1226,15 @@ def edit_contact(id): ) -@bp.route("/fiche_entreprise//contacts") +@bp.route("/fiche_entreprise//contacts") @permission_required(Permission.RelationsEntreprisesView) -def contacts(id): +def contacts(entreprise_id): """ Permet d'afficher une page avec la liste des contacts d'une entreprise """ - entreprise = Entreprise.query.filter_by(id=id, visible=True).first_or_404( - description=f"entreprise {id} inconnue" - ) + entreprise = Entreprise.query.filter_by( + id=entreprise_id, visible=True + ).first_or_404(description=f"entreprise {entreprise_id} inconnue") contacts = EntrepriseContact.query.filter_by(entreprise=entreprise.id).all() return render_template( "entreprises/contacts.html", @@ -1171,15 +1244,18 @@ def contacts(id): ) -@bp.route("/fiche_entreprise//add_stage_apprentissage", methods=["GET", "POST"]) +@bp.route( + "/fiche_entreprise//add_stage_apprentissage", + methods=["GET", "POST"], +) @permission_required(Permission.RelationsEntreprisesChange) -def add_stage_apprentissage(id): +def add_stage_apprentissage(entreprise_id): """ Permet d'ajouter un étudiant ayant réalisé un stage ou une alternance sur la fiche entreprise de l'entreprise """ - entreprise = Entreprise.query.filter_by(id=id, visible=True).first_or_404( - description=f"entreprise {id} inconnue" - ) + entreprise = Entreprise.query.filter_by( + id=entreprise_id, visible=True + ).first_or_404(description=f"entreprise {entreprise_id} inconnue") form = StageApprentissageCreationForm() if form.validate_on_submit(): etudiant_nomcomplet = form.etudiant.data.upper().strip() @@ -1209,27 +1285,32 @@ def add_stage_apprentissage(id): db.session.add(stage_apprentissage) db.session.commit() flash("L'étudiant a été ajouté sur la fiche entreprise.") - return redirect(url_for("entreprises.fiche_entreprise", id=entreprise.id)) + return redirect( + url_for("entreprises.fiche_entreprise", entreprise_id=entreprise.id) + ) return render_template( - "entreprises/ajout_stage_apprentissage.html", + "entreprises/form_ajout_stage_apprentissage.html", title="Ajout stage / apprentissage", form=form, ) @bp.route( - "/fiche_entreprise/edit_stage_apprentissage/", methods=["GET", "POST"] + "/fiche_entreprise//edit_stage_apprentissage/", + methods=["GET", "POST"], ) @permission_required(Permission.RelationsEntreprisesChange) -def edit_stage_apprentissage(id): +def edit_stage_apprentissage(entreprise_id, stage_apprentissage_id): """ Permet de modifier un étudiant ayant réalisé un stage ou une alternance sur la fiche entreprise de l'entreprise """ stage_apprentissage = EntrepriseStageApprentissage.query.filter_by( - id=id - ).first_or_404(description=f"stage_apprentissage {id} inconnue") + id=stage_apprentissage_id, entreprise_id=entreprise_id + ).first_or_404( + description=f"stage_apprentissage {stage_apprentissage_id} inconnue pour l'entreprise {entreprise_id}" + ) etudiant = Identite.query.filter_by(id=stage_apprentissage.etudid).first_or_404( - description=f"etudiant {id} inconnue" + description=f"etudiant {stage_apprentissage.etudid} inconnue" ) form = StageApprentissageModificationForm() if form.validate_on_submit(): @@ -1259,7 +1340,8 @@ def edit_stage_apprentissage(id): db.session.commit() return redirect( url_for( - "entreprises.fiche_entreprise", id=stage_apprentissage.entreprise_id + "entreprises.fiche_entreprise", + entreprise_id=stage_apprentissage.entreprise_id, ) ) elif request.method == "GET": @@ -1269,48 +1351,55 @@ def edit_stage_apprentissage(id): form.date_fin.data = stage_apprentissage.date_fin form.notes.data = stage_apprentissage.notes return render_template( - "entreprises/ajout_stage_apprentissage.html", + "entreprises/form_ajout_stage_apprentissage.html", title="Modification stage / apprentissage", form=form, ) @bp.route( - "/fiche_entreprise/delete_stage_apprentissage/", methods=["GET", "POST"] + "/fiche_entreprise//delete_stage_apprentissage/", + methods=["GET", "POST"], ) @permission_required(Permission.RelationsEntreprisesChange) -def delete_stage_apprentissage(id): +def delete_stage_apprentissage(entreprise_id, stage_apprentissage_id): """ Permet de supprimer un étudiant ayant réalisé un stage ou une alternance sur la fiche entreprise de l'entreprise """ stage_apprentissage = EntrepriseStageApprentissage.query.filter_by( - id=id - ).first_or_404(description=f"stage_apprentissage {id} inconnu") + id=stage_apprentissage_id, entreprise_id=entreprise_id + ).first_or_404(description=f"stage_apprentissage {stage_apprentissage_id} inconnu") form = SuppressionConfirmationForm() if form.validate_on_submit(): db.session.delete(stage_apprentissage) db.session.commit() return redirect( url_for( - "entreprises.fiche_entreprise", id=stage_apprentissage.entreprise_id + "entreprises.fiche_entreprise", + entreprise_id=stage_apprentissage.entreprise_id, ) ) return render_template( - "entreprises/confirmation_form.html", + "entreprises/form_confirmation.html", title="Supression stage/apprentissage", form=form, info_message="Cliquez sur le bouton Supprimer pour confirmer votre supression", ) -@bp.route("/fiche_entreprise/envoyer_offre/", methods=["GET", "POST"]) +@bp.route( + "/fiche_entreprise//envoyer_offre/", + methods=["GET", "POST"], +) @permission_required(Permission.RelationsEntreprisesSend) -def envoyer_offre(id): +def envoyer_offre(entreprise_id, offre_id): """ Permet d'envoyer une offre à un utilisateur ScoDoc """ - offre = EntrepriseOffre.query.filter_by(id=id).first_or_404( - description=f"offre {id} inconnue" + offre = EntrepriseOffre.query.filter_by( + id=offre_id, entreprise_id=entreprise_id + ).first_or_404( + description=f"offre {offre_id} inconnue pour l'entreprise {entreprise_id}" ) form = EnvoiOffreForm() if form.validate_on_submit(): @@ -1333,9 +1422,11 @@ def envoyer_offre(id): db.session.add(envoi_offre) db.session.commit() flash(f"L'offre a été envoyé à {responsable.get_nomplogin()}.") - return redirect(url_for("entreprises.fiche_entreprise", id=offre.entreprise_id)) + return redirect( + url_for("entreprises.fiche_entreprise", entreprise_id=offre.entreprise_id) + ) return render_template( - "entreprises/envoi_offre_form.html", + "entreprises/form_envoi_offre.html", title="Envoyer une offre", form=form, ) @@ -1494,7 +1585,7 @@ def import_donnees(): @bp.route( - "/get_offre_file////" + "/fiche_entreprise//offre//get_offre_file//" ) @permission_required(Permission.RelationsEntreprisesView) def get_offre_file(entreprise_id, offre_id, filedir, filename): @@ -1526,14 +1617,19 @@ def get_offre_file(entreprise_id, offre_id, filedir, filename): abort(404, description=f"fichier {filename} inconnu") -@bp.route("/fiche_entreprise/add_offre_file/", methods=["GET", "POST"]) +@bp.route( + "/fiche_entreprise//offre//add_offre_file", + methods=["GET", "POST"], +) @permission_required(Permission.RelationsEntreprisesChange) -def add_offre_file(offre_id): +def add_offre_file(entreprise_id, offre_id): """ Permet d'ajouter un fichier à une offre """ - offre = EntrepriseOffre.query.filter_by(id=offre_id).first_or_404( - description=f"offre {offre_id} inconnue" + offre = EntrepriseOffre.query.filter_by( + id=offre_id, entreprise_id=entreprise_id + ).first_or_404( + description=f"offre {offre_id} inconnue pour l'entreprise {entreprise_id}" ) form = AjoutFichierForm() if form.validate_on_submit(): @@ -1550,7 +1646,9 @@ def add_offre_file(offre_id): filename = secure_filename(file.filename) file.save(os.path.join(path, filename)) flash("Le fichier a été ajouté a l'offre.") - return redirect(url_for("entreprises.fiche_entreprise", id=offre.entreprise_id)) + return redirect( + url_for("entreprises.fiche_entreprise", entreprise_id=offre.entreprise_id) + ) return render_template( "entreprises/form.html", title="Ajout fichier à une offre", @@ -1559,16 +1657,18 @@ def add_offre_file(offre_id): @bp.route( - "/fiche_entreprise/delete_offre_file//", + "/fiche_entreprise//offre//delete_offre_file/", methods=["GET", "POST"], ) @permission_required(Permission.RelationsEntreprisesChange) -def delete_offre_file(offre_id, filedir): +def delete_offre_file(entreprise_id, offre_id, filedir): """ Permet de supprimer un fichier d'une offre """ - offre = EntrepriseOffre.query.filter_by(id=offre_id).first_or_404( - description=f"offre {offre_id} inconnue" + offre = EntrepriseOffre.query.filter_by( + id=offre_id, entreprise_id=entreprise_id + ).first_or_404( + description=f"offre {offre_id} inconnue pour l'entreprise {entreprise_id}" ) form = SuppressionConfirmationForm() if form.validate_on_submit(): @@ -1583,10 +1683,12 @@ def delete_offre_file(offre_id, filedir): shutil.rmtree(path) flash("Le fichier relié à l'offre a été supprimé.") return redirect( - url_for("entreprises.fiche_entreprise", id=offre.entreprise_id) + url_for( + "entreprises.fiche_entreprise", entreprise_id=offre.entreprise_id + ) ) return render_template( - "entreprises/confirmation_form.html", + "entreprises/form_confirmation.html", title="Suppression fichier d'une offre", form=form, info_message="Cliquez sur le bouton Supprimer pour confirmer votre supression", diff --git a/app/templates/entreprises/_correspondant.html b/app/templates/entreprises/_correspondant.html index d8c0084606..549c40f09a 100644 --- a/app/templates/entreprises/_correspondant.html +++ b/app/templates/entreprises/_correspondant.html @@ -26,8 +26,8 @@ {% if current_user.has_permission(current_user.Permission.RelationsEntreprisesChange, None) %} {% endif %} \ No newline at end of file diff --git a/app/templates/entreprises/_offre.html b/app/templates/entreprises/_offre.html index c799b6f420..4ae08232d2 100644 --- a/app/templates/entreprises/_offre.html +++ b/app/templates/entreprises/_offre.html @@ -23,31 +23,31 @@ {% for fichier in offre[1] %} {{ fichier[1] }} {% if current_user.has_permission(current_user.Permission.RelationsEntreprisesChange, None) %} - supprimer + supprimer {% endif %}
{% endfor %} {% if current_user.has_permission(current_user.Permission.RelationsEntreprisesChange, None) %} - Ajoutez un fichier + Ajoutez un fichier {% endif %}
{% if current_user.has_permission(current_user.Permission.RelationsEntreprisesChange, None) %} - Modifier l'offre - Supprimer l'offre + Modifier l'offre + Supprimer l'offre {% endif %} {% if current_user.has_permission(current_user.Permission.RelationsEntreprisesSend, None) %} - Envoyer l'offre + Envoyer l'offre {% endif %} {% if current_user.has_permission(current_user.Permission.RelationsEntreprisesChange, None) %} {% if not offre[0].expired %} - Rendre expirée + Rendre expirée {% else %} - Rendre non expirée + Rendre non expirée {% endif %} {% endif %}
diff --git a/app/templates/entreprises/contacts.html b/app/templates/entreprises/contacts.html index df2cbc4724..e79f214b97 100644 --- a/app/templates/entreprises/contacts.html +++ b/app/templates/entreprises/contacts.html @@ -15,7 +15,7 @@ Entreprises