From e34ae9b3ecff5141743d4bb55cbee4741de115ef Mon Sep 17 00:00:00 2001 From: Arthur ZHU Date: Fri, 6 May 2022 18:50:12 +0200 Subject: [PATCH] changement historique (log) --- app/entreprises/forms.py | 5 +- app/entreprises/models.py | 4 +- app/entreprises/routes.py | 46 +++++++++++++------ app/static/css/entreprises.css | 4 ++ app/templates/entreprises/entreprises.html | 6 +-- .../entreprises/fiche_entreprise.html | 2 +- 6 files changed, 44 insertions(+), 23 deletions(-) diff --git a/app/entreprises/forms.py b/app/entreprises/forms.py index 0ab88a86e..65d503cca 100644 --- a/app/entreprises/forms.py +++ b/app/entreprises/forms.py @@ -81,8 +81,8 @@ def _build_string_field(label, required=True, render_kw=None): class EntreprisesFilterForm(FlaskForm): - active = BooleanField("Afficher les entreprises désactivés") - association = BooleanField("Afficher les associations partenaires") + active = BooleanField("Toutes les entreprises") + association = BooleanField("Seulement les associations partenaires") class EntrepriseCreationForm(FlaskForm): @@ -662,7 +662,6 @@ class TaxeApprentissageForm(FlaskForm): class TaxeApprentissageModificationForm(FlaskForm): - hidden_annee = HiddenField() annee = IntegerField("Année (*)") montant = IntegerField( "Montant (*)", diff --git a/app/entreprises/models.py b/app/entreprises/models.py index 72975a642..b61ecaec7 100644 --- a/app/entreprises/models.py +++ b/app/entreprises/models.py @@ -137,7 +137,9 @@ class EntrepriseLog(db.Model): id = db.Column(db.Integer, primary_key=True) date = db.Column(db.DateTime(timezone=True), server_default=db.func.now()) authenticated_user = db.Column(db.Text) - object = db.Column(db.Integer) + entreprise_id = db.Column(db.Integer) + object = db.Column(db.Text) + object_id = db.Column(db.Integer) text = db.Column(db.Text) diff --git a/app/entreprises/routes.py b/app/entreprises/routes.py index be88e2ad0..a57c77e13 100644 --- a/app/entreprises/routes.py +++ b/app/entreprises/routes.py @@ -176,7 +176,7 @@ def fiche_entreprise(id): sites = entreprise.sites[:] logs = ( EntrepriseLog.query.order_by(EntrepriseLog.date.desc()) - .filter_by(object=id) + .filter(EntrepriseLog.entreprise_id == id) .limit(LOGS_LEN) .all() ) @@ -216,7 +216,7 @@ def logs_entreprise(id): ) logs = ( EntrepriseLog.query.order_by(EntrepriseLog.date.desc()) - .filter_by(object=id) + .filter(EntrepriseLog.entreprise_id == id) .paginate(page=page, per_page=20) ) return render_template( @@ -362,6 +362,7 @@ def add_entreprise(): log = EntrepriseLog( authenticated_user=current_user.user_name, text=f"{nom_entreprise} - Création de la fiche entreprise ({entreprise.nom})", + entreprise_id=entreprise.id, ) db.session.add(log) db.session.commit() @@ -398,7 +399,7 @@ def edit_entreprise(id): if entreprise.nom != form.nom.data.strip(): log = EntrepriseLog( authenticated_user=current_user.user_name, - object=entreprise.id, + entreprise_id=entreprise.id, text=f"{nom_entreprise} - Modification du nom (ancien nom: {entreprise.nom})", ) entreprise.nom = form.nom.data.strip() @@ -406,7 +407,7 @@ def edit_entreprise(id): if entreprise.adresse != form.adresse.data.strip(): log = EntrepriseLog( authenticated_user=current_user.user_name, - object=entreprise.id, + entreprise_id=entreprise.id, text=f"{nom_entreprise} - Modification de l'adresse (ancienne adresse: {entreprise.adresse})", ) entreprise.adresse = form.adresse.data.strip() @@ -414,7 +415,7 @@ def edit_entreprise(id): if entreprise.codepostal != form.codepostal.data.strip(): log = EntrepriseLog( authenticated_user=current_user.user_name, - object=entreprise.id, + entreprise_id=entreprise.id, text=f"{nom_entreprise} - Modification du code postal (ancien code postal: {entreprise.codepostal})", ) entreprise.codepostal = form.codepostal.data.strip() @@ -422,7 +423,7 @@ def edit_entreprise(id): if entreprise.ville != form.ville.data.strip(): log = EntrepriseLog( authenticated_user=current_user.user_name, - object=entreprise.id, + entreprise_id=entreprise.id, text=f"{nom_entreprise} - Modification de la ville (ancienne ville: {entreprise.ville})", ) entreprise.ville = form.ville.data.strip() @@ -430,7 +431,7 @@ def edit_entreprise(id): if entreprise.pays != form.pays.data.strip() or not form.pays.data.strip(): log = EntrepriseLog( authenticated_user=current_user.user_name, - object=entreprise.id, + entreprise_id=entreprise.id, text=f"{nom_entreprise} - Modification du pays (ancien pays: {entreprise.pays})", ) entreprise.pays = ( @@ -604,7 +605,8 @@ def validate_entreprise(id): nom_entreprise = f"{entreprise.nom}" log = EntrepriseLog( authenticated_user=current_user.user_name, - text=f"{nom_entreprise} - Validation de la fiche entreprise ({entreprise.nom}) avec un correspondant", + entreprise_id=entreprise.id, + text=f"{nom_entreprise} - Validation de la fiche entreprise ({entreprise.nom})", ) db.session.add(log) db.session.commit() @@ -688,7 +690,9 @@ def add_offre(id): file.save(os.path.join(path, filename)) log = EntrepriseLog( authenticated_user=current_user.user_name, - object=entreprise.id, + entreprise_id=entreprise.id, + object="offre", + object_id=offre.id, text="Création d'une offre", ) db.session.add(log) @@ -740,7 +744,9 @@ def edit_offre(id): db.session.delete(offre_dept) log = EntrepriseLog( authenticated_user=current_user.user_name, - object=offre.entreprise_id, + entreprise_id=offre.entreprise_id, + object="offre", + object_id=offre.id, text="Modification d'une offre", ) db.session.add(log) @@ -785,7 +791,9 @@ def delete_offre(id): shutil.rmtree(path) log = EntrepriseLog( authenticated_user=current_user.user_name, - object=offre.entreprise_id, + entreprise_id=offre.entreprise_id, + object="offre", + object_id=offre.id, text="Suppression d'une offre", ) db.session.add(log) @@ -925,13 +933,17 @@ def add_correspondant(id_entreprise, id_site): origine=correspondant_entry.origine.data.strip(), notes=correspondant_entry.notes.data.strip(), ) + db.session.add(correspondant) + db.session.commit() + db.session.refresh(correspondant) log = EntrepriseLog( authenticated_user=current_user.user_name, - object=entreprise.id, + entreprise_id=correspondant.entreprise_id, + object="correspondant", + object_id=correspondant.id, text="Création d'un correspondant", ) db.session.add(log) - db.session.add(correspondant) db.session.commit() flash("Le correspondant a été ajouté à la fiche entreprise.") return redirect(url_for("entreprises.fiche_entreprise", id=entreprise.id)) @@ -967,7 +979,9 @@ def edit_correspondant(id): correspondant.notes = form.notes.data.strip() log = EntrepriseLog( authenticated_user=current_user.user_name, - object=correspondant.entreprise_id, + entreprise_id=correspondant.entreprise_id, + object="correspondant", + object_id=correspondant.id, text="Modification d'un correspondant", ) db.session.add(log) @@ -1007,7 +1021,9 @@ def delete_correspondant(id): db.session.delete(correspondant) log = EntrepriseLog( authenticated_user=current_user.user_name, - object=correspondant.entreprise_id, + entreprise_id=correspondant.entreprise_id, + object="correspondant", + object_id=correspondant.id, text="Suppression d'un correspondant", ) db.session.add(log) diff --git a/app/static/css/entreprises.css b/app/static/css/entreprises.css index 702cee93b..997bc1cba 100644 --- a/app/static/css/entreprises.css +++ b/app/static/css/entreprises.css @@ -120,4 +120,8 @@ #liste-taxes-apprentissages { list-style: none; padding-left: 0; +} + +#form-entreprise-filter > label { + margin-right: 20px; } \ No newline at end of file diff --git a/app/templates/entreprises/entreprises.html b/app/templates/entreprises/entreprises.html index e755d1d83..5931bae12 100644 --- a/app/templates/entreprises/entreprises.html +++ b/app/templates/entreprises/entreprises.html @@ -37,10 +37,10 @@

Liste des entreprises

{% if form %} -
+ {{ form.hidden_tag() }} - - + {{ form.active.label }} + {{ form.association.label }}
{% endif %} diff --git a/app/templates/entreprises/fiche_entreprise.html b/app/templates/entreprises/fiche_entreprise.html index 74fa32394..35c9365f1 100644 --- a/app/templates/entreprises/fiche_entreprise.html +++ b/app/templates/entreprises/fiche_entreprise.html @@ -51,7 +51,7 @@ {% for taxe in taxes %}
  • supprimer - {{ taxe.annee }} : {{ taxe.montant }} euros + {{ taxe.annee }} : {{ taxe.montant }} euros {% if taxe.notes %}- {{ taxe.notes}} {% endif %}
    • {% endfor %}