ajout site pour les entreprises, corrections de bugs

app/entreprises/forms.py

@@ -123,12 +123,9 @@ class EntrepriseCreationForm(FlaskForm):
                 self.prenom_correspondant.errors.append("Ce champ est requis")
                 validate = False
             if not self.telephone.data.strip() and not self.mail.data.strip():
-                self.telephone.errors.append(
+                msg = "Saisir un moyen de contact (mail ou téléphone)"
-                    "Saisir un moyen de contact (mail ou téléphone)"
+                self.telephone.errors.append(msg)
-                )
+                self.mail.errors.append(msg)
-                self.mail.errors.append(
-                    "Saisir un moyen de contact (mail ou téléphone)"
-                )
                 validate = False
 
         return validate
@@ -173,6 +170,15 @@ class EntrepriseModificationForm(FlaskForm):
         }
 
 
+class SiteCreationForm(FlaskForm):
+    nom = _build_string_field("Nom du site (*)")
+    adresse = _build_string_field("Adresse (*)")
+    codepostal = _build_string_field("Code postal (*)")
+    ville = _build_string_field("Ville (*)")
+    pays = _build_string_field("Pays", required=False)
+    submit = SubmitField("Envoyer", render_kw=SUBMIT_MARGE)
+
+
 class MultiCheckboxField(SelectMultipleField):
     widget = ListWidget(prefix_label=False)
     option_widget = CheckboxInput()
@@ -313,10 +319,9 @@ class CorrespondantCreationForm(FlaskForm):
             validate = False
 
         if not self.telephone.data and not self.mail.data:
-            self.telephone.errors.append(
+            msg = "Saisir un moyen de contact (mail ou téléphone)"
-                "Saisir un moyen de contact (mail ou téléphone)"
+            self.telephone.errors.append(msg)
-            )
+            self.mail.errors.append(msg)
-            self.mail.errors.append("Saisir un moyen de contact (mail ou téléphone)")
             validate = False
 
         return validate
@@ -399,10 +404,9 @@ class CorrespondantModificationForm(FlaskForm):
             validate = False
 
         if not self.telephone.data and not self.mail.data:
-            self.telephone.errors.append(
+            msg = "Saisir un moyen de contact (mail ou téléphone)"
-                "Saisir un moyen de contact (mail ou téléphone)"
+            self.telephone.errors.append(msg)
-            )
+            self.mail.errors.append(msg)
-            self.mail.errors.append("Saisir un moyen de contact (mail ou téléphone)")
             validate = False
 
         return validate

app/entreprises/models.py

@@ -9,16 +9,18 @@ class Entreprise(db.Model):
     adresse = db.Column(db.Text)
     codepostal = db.Column(db.Text)
     ville = db.Column(db.Text)
-    pays = db.Column(db.Text, default="FRANCE")
+    pays = db.Column(db.Text)
     visible = db.Column(db.Boolean, default=False)
     active = db.Column(db.Boolean, default=True)
     notes_active = db.Column(db.Text)
-    correspondants = db.relationship(
+
-        "EntrepriseCorrespondant",
+    sites = db.relationship(
+        "EntrepriseSite",
         backref="entreprise",
         lazy="dynamic",
         cascade="all, delete-orphan",
     )
+
     offres = db.relationship(
         "EntrepriseOffre",
         backref="entreprise",
@@ -37,13 +39,24 @@ class Entreprise(db.Model):
         }
 
 
-# class EntrepriseSite(db.Model):
+class EntrepriseSite(db.Model):
-#     __tablename__ = "are_sites"
+    __tablename__ = "are_sites"
-#     id = db.Column(db.Integer, primary_key=True)
+    id = db.Column(db.Integer, primary_key=True)
-#     entreprise_id = db.Column(
+    entreprise_id = db.Column(
-#         db.Integer, db.ForeignKey("are_entreprises.id", ondelete="cascade")
+        db.Integer, db.ForeignKey("are_entreprises.id", ondelete="cascade")
-#     )
+    )
-#     nom = db.Column(db.Text)
+    nom = db.Column(db.Text)
+    adresse = db.Column(db.Text)
+    codepostal = db.Column(db.Text)
+    ville = db.Column(db.Text)
+    pays = db.Column(db.Text)
+
+    correspondants = db.relationship(
+        "EntrepriseCorrespondant",
+        backref="site",
+        lazy="dynamic",
+        cascade="all, delete-orphan",
+    )
 
 
 class EntrepriseCorrespondant(db.Model):
@@ -52,7 +65,7 @@ class EntrepriseCorrespondant(db.Model):
     entreprise_id = db.Column(
         db.Integer, db.ForeignKey("are_entreprises.id", ondelete="cascade")
     )
-    # site_id = db.Column(db.Integer, db.ForeignKey("are_sites.id", ondelete="cascade"))
+    site_id = db.Column(db.Integer, db.ForeignKey("are_sites.id", ondelete="cascade"))
     civilite = db.Column(db.String(1))
     nom = db.Column(db.Text)
     prenom = db.Column(db.Text)

app/entreprises/routes.py

@@ -16,6 +16,7 @@ from app.entreprises.forms import (
     DesactivationConfirmationForm,
     EntrepriseCreationForm,
     EntrepriseModificationForm,
+    SiteCreationForm,
     SuppressionConfirmationForm,
     OffreCreationForm,
     OffreModificationForm,
@@ -37,6 +38,7 @@ from app.entreprises.models import (
     EntrepriseCorrespondant,
     EntrepriseLog,
     EntrepriseContact,
+    EntrepriseSite,
     EntrepriseStageApprentissage,
     EntrepriseEnvoiOffre,
     EntrepriseOffreDepartement,
@@ -146,9 +148,7 @@ def fiche_entreprise(id):
             offre_with_files = are.get_offre_files_and_depts(offre, depts)
             if offre_with_files is not None:
                 offres_with_files.append(offre_with_files)
-    correspondants = []
+    sites = entreprise.sites[:]
-    if current_user.has_permission(Permission.RelationsEntreprisesCorrespondants, None):
-        correspondants = entreprise.correspondants[:]
     logs = (
         EntrepriseLog.query.order_by(EntrepriseLog.date.desc())
         .filter_by(object=id)
@@ -166,7 +166,7 @@ def fiche_entreprise(id):
         "entreprises/fiche_entreprise.html",
         title="Fiche entreprise",
         entreprise=entreprise,
-        correspondants=correspondants,
+        sites=sites,
         offres=offres_with_files,
         logs=logs,
         stages_apprentissages=stages_apprentissages,
@@ -297,10 +297,22 @@ def add_entreprise():
         )
         db.session.add(entreprise)
         db.session.commit()
-        if form.nom_correspondant.data.strip():
         db.session.refresh(entreprise)
+        site = EntrepriseSite(
+            entreprise_id=entreprise.id,
+            nom=form.nom_entreprise.data.strip(),
+            adresse=form.adresse.data.strip(),
+            codepostal=form.codepostal.data.strip(),
+            ville=form.ville.data.strip(),
+            pays=form.pays.data.strip() if form.pays.data.strip() else "FRANCE",
+        )
+        db.session.add(site)
+        db.session.commit()
+        if form.nom_correspondant.data.strip():
+            db.session.refresh(site)
             correspondant = EntrepriseCorrespondant(
                 entreprise_id=entreprise.id,
+                site_id=site.id,
                 civilite=form.civilite.data,
                 nom=form.nom_correspondant.data.strip(),
                 prenom=form.prenom_correspondant.data.strip(),
@@ -716,20 +728,57 @@ def expired(id):
     return redirect(url_for("entreprises.fiche_entreprise", id=offre.entreprise_id))
 
 
-@bp.route("/fiche_entreprise/<int:id>/add_correspondant", methods=["GET", "POST"])
+@bp.route(
+    "/fiche_entreprise/<int:id>/add_site",
+    methods=["GET", "POST"],
+)
 @permission_required(Permission.RelationsEntreprisesChange)
-def add_correspondant(id):
+def add_site(id):
+    entreprise = Entreprise.query.filter_by(id=id, visible=True).first_or_404(
+        description=f"entreprise {id} inconnue"
+    )
+    form = SiteCreationForm()
+    if form.validate_on_submit():
+        site = EntrepriseSite(
+            entreprise_id=entreprise.id,
+            nom=form.nom.data.strip(),
+            adresse=form.adresse.data.strip(),
+            codepostal=form.codepostal.data.strip(),
+            ville=form.ville.data.strip(),
+            pays=form.pays.data.strip() if form.pays.data.strip() else "FRANCE",
+        )
+        db.session.add(site)
+        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 render_template(
+        "entreprises/form.html",
+        title="Ajout site",
+        form=form,
+    )
+
+
+@bp.route(
+    "/fiche_entreprise/<int:id_entreprise>/add_correspondant/<int:id_site>",
+    methods=["GET", "POST"],
+)
+@permission_required(Permission.RelationsEntreprisesChange)
+def add_correspondant(id_entreprise, id_site):
     """
     Permet d'ajouter un correspondant a une entreprise
     """
-    entreprise = Entreprise.query.filter_by(id=id, visible=True).first_or_404(
+    entreprise = Entreprise.query.filter_by(
-        description=f"entreprise {id} inconnue"
+        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"
     )
     form = CorrespondantsCreationForm(hidden_entreprise_id=entreprise.id)
     if form.validate_on_submit():
         for correspondant_entry in form.correspondants.entries:
             correspondant = EntrepriseCorrespondant(
                 entreprise_id=entreprise.id,
+                site_id=site.id,
                 civilite=correspondant_entry.civilite.data,
                 nom=correspondant_entry.nom.data.strip(),
                 prenom=correspondant_entry.prenom.data.strip(),
@@ -789,7 +838,7 @@ 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", id=correspondant.entreprise_id)
         )
     elif request.method == "GET":
         form.civilite.data = correspondant.civilite

app/static/css/entreprises.css

@@ -54,23 +54,24 @@
     margin-bottom: -5px;
 }
 
-.entreprise, .correspondant, .offre {
+.entreprise, .correspondant, .offre, .site{
     border: solid 2px;
     border-radius: 10px;
     padding: 10px;
     margin-bottom: 10px;
+    margin-top: 10px;
 }
 
-.correspondants-et-offres {
+.sites-et-offres {
     display: flex;
     justify-content: space-between;
 }
 
-.correspondants-et-offres > div {
+.sites-et-offres > div {
     flex: 1 0 0;
 }
 
-.correspondants-et-offres > div:nth-child(2) {
+.sites-et-offres > div:nth-child(2) {
     margin-left: 20px;
 }
 

app/templates/entreprises/fiche_entreprise.html

@@ -37,25 +37,40 @@
             </div>
         </div>
 
-        {% if current_user.has_permission(current_user.Permission.RelationsEntreprisesChange, None) %}
         <div>
+        {% if current_user.has_permission(current_user.Permission.RelationsEntreprisesChange, None) %}
             <a class="btn btn-primary" href="{{ url_for('entreprises.edit_entreprise', id=entreprise.id) }}">Modifier</a>
+            <a class="btn btn-danger" href="{{ url_for('entreprises.fiche_entreprise_desactiver', id=entreprise.id) }}">Désactiver</a>
             <a class="btn btn-danger" href="{{ url_for('entreprises.delete_entreprise', id=entreprise.id) }}">Supprimer</a>
+            <a class="btn btn-primary" href="{{ url_for('entreprises.add_site', id=entreprise.id) }}">Ajouter site</a>
             <a class="btn btn-primary" href="{{ url_for('entreprises.add_offre', id=entreprise.id) }}">Ajouter offre</a>
-            <a class="btn btn-primary" href="{{ url_for('entreprises.add_correspondant', id=entreprise.id) }}">Ajouter correspondant</a>
         {% endif %}
             <a class="btn btn-primary" href="{{ url_for('entreprises.contacts', id=entreprise.id) }}">Liste contacts</a>
             <a class="btn btn-primary" href="{{ url_for('entreprises.offres_expirees', id=entreprise.id) }}">Voir les offres expirées</a>
-        <div>
+        </div>
 
 
-        <div class="correspondants-et-offres">
+        <div class="sites-et-offres">
-            {% if correspondants %}
+            {% if sites %}
             <div>
-                <h3>Correspondants</h3>
+                <h3>Sites</h3>
-                {% for correspondant in correspondants %}
+                {% for site in sites %}
+                <div class="site">
+                    Nom : {{ site.nom }}<br>
+                    Adresse : {{ site.adresse }}<br>
+                    Code postal : {{ site.codepostal }}<br>
+                    Ville : {{ site.ville }}<br>
+                    Pays : {{ site.pays }}
+                    {% if current_user.has_permission(current_user.Permission.RelationsEntreprisesChange, None) %}
+                        <br><a class="btn btn-primary" href="{{ url_for('entreprises.add_correspondant', id_entreprise=entreprise.id, id_site=site.id) }}">Ajouter correspondant</a>
+                    {% endif %}
+                    {% if current_user.has_permission(current_user.Permission.RelationsEntreprisesCorrespondants, None) %}
+                        {% for correspondant in site.correspondants %}
                             {% include 'entreprises/_correspondant.html' %}
                         {% endfor %}
+                    {% endif %}
+                </div>
+                {% endfor %}
             </div>
             {% endif %}
 

migrations/versions/01d13209021e_ajout_colonnes_tables_relations_.py

@@ -1,40 +0,0 @@
-"""ajout colonnes tables relations entreprises
-
-Revision ID: 01d13209021e
-Revises: e97b2a10f86c
-Create Date: 2022-04-21 21:02:01.319876
-
-"""
-from alembic import op
-import sqlalchemy as sa
-
-
-# revision identifiers, used by Alembic.
-revision = "01d13209021e"
-down_revision = "e97b2a10f86c"
-branch_labels = None
-depends_on = None
-
-
-def upgrade():
-    # ### commands auto generated by Alembic - please adjust! ###
-    op.add_column(
-        "are_correspondants", sa.Column("civilite", sa.String(length=1), nullable=True)
-    )
-    op.add_column("are_correspondants", sa.Column("origine", sa.Text(), nullable=True))
-    op.add_column("are_correspondants", sa.Column("notes", sa.Text(), nullable=True))
-    op.add_column("are_entreprises", sa.Column("active", sa.Boolean(), nullable=True))
-    op.add_column(
-        "are_entreprises", sa.Column("notes_active", sa.Text(), nullable=True)
-    )
-    # ### end Alembic commands ###
-
-
-def downgrade():
-    # ### commands auto generated by Alembic - please adjust! ###
-    op.drop_column("are_entreprises", "notes_active")
-    op.drop_column("are_entreprises", "active")
-    op.drop_column("are_correspondants", "notes")
-    op.drop_column("are_correspondants", "origine")
-    op.drop_column("are_correspondants", "civilite")
-    # ### end Alembic commands ###

migrations/versions/d5b3bdd1d622_ajout_colonnes_tables_relations_.py

@@ -0,0 +1,65 @@
+"""ajout colonnes tables relations entreprises, sites
+
+Revision ID: d5b3bdd1d622
+Revises: e97b2a10f86c
+Create Date: 2022-04-25 18:29:50.841280
+
+"""
+from alembic import op
+import sqlalchemy as sa
+
+
+# revision identifiers, used by Alembic.
+revision = "d5b3bdd1d622"
+down_revision = "e97b2a10f86c"
+branch_labels = None
+depends_on = None
+
+
+def upgrade():
+    # ### commands auto generated by Alembic - please adjust! ###
+    op.create_table(
+        "are_sites",
+        sa.Column("id", sa.Integer(), nullable=False),
+        sa.Column("entreprise_id", sa.Integer(), nullable=True),
+        sa.Column("nom", sa.Text(), nullable=True),
+        sa.Column("adresse", sa.Text(), nullable=True),
+        sa.Column("codepostal", sa.Text(), nullable=True),
+        sa.Column("ville", sa.Text(), nullable=True),
+        sa.Column("pays", sa.Text(), nullable=True),
+        sa.ForeignKeyConstraint(
+            ["entreprise_id"], ["are_entreprises.id"], ondelete="cascade"
+        ),
+        sa.PrimaryKeyConstraint("id"),
+    )
+    op.add_column(
+        "are_correspondants", sa.Column("site_id", sa.Integer(), nullable=True)
+    )
+    op.add_column(
+        "are_correspondants", sa.Column("civilite", sa.String(length=1), nullable=True)
+    )
+    op.add_column("are_correspondants", sa.Column("origine", sa.Text(), nullable=True))
+    op.add_column("are_correspondants", sa.Column("notes", sa.Text(), nullable=True))
+    op.create_foreign_key(
+        None, "are_correspondants", "are_sites", ["site_id"], ["id"], ondelete="cascade"
+    )
+    op.add_column("are_entreprises", sa.Column("active", sa.Boolean(), nullable=True))
+    op.add_column(
+        "are_entreprises", sa.Column("notes_active", sa.Text(), nullable=True)
+    )
+    # ### end Alembic commands ###
+
+
+def downgrade():
+    # ### commands auto generated by Alembic - please adjust! ###
+    op.drop_column("are_entreprises", "notes_active")
+    op.drop_column("are_entreprises", "active")
+    op.drop_constraint(
+        "are_correspondants_site_id_fkey", "are_correspondants", type_="foreignkey"
+    )
+    op.drop_column("are_correspondants", "notes")
+    op.drop_column("are_correspondants", "origine")
+    op.drop_column("are_correspondants", "civilite")
+    op.drop_column("are_correspondants", "site_id")
+    op.drop_table("are_sites")
+    # ### end Alembic commands ###