From 688e0657b4ad2406dc437f839dacaa0cb92e6b84 Mon Sep 17 00:00:00 2001
From: Arthur ZHU <arthur.zhu38@gmail.com>
Date: Tue, 16 Aug 2022 18:34:08 +0200
Subject: [PATCH] refactoring

---
 app/entreprises/routes.py                     | 10 ++--
 app/templates/entreprises/_offre.html         | 46 +++++++++----------
 app/templates/entreprises/correspondants.html | 16 +++----
 .../entreprises/fiche_entreprise.html         | 22 ++++-----
 .../entreprises/offres_expirees.html          |  2 +-
 app/templates/entreprises/offres_recues.html  | 40 ++++++++--------
 6 files changed, 68 insertions(+), 68 deletions(-)

diff --git a/app/entreprises/routes.py b/app/entreprises/routes.py
index b2f888321..41b8febba 100644
--- a/app/entreprises/routes.py
+++ b/app/entreprises/routes.py
@@ -251,16 +251,16 @@ def offres_recues():
         .all()
     )
     offres_recues_with_files = []
-    for offre in offres_recues:
+    for envoi_offre, offre in offres_recues:
         correspondant = EntrepriseCorrespondant.query.filter_by(
-            id=offre[1].correspondant_id
+            id=offre.correspondant_id
         ).first()
         files = []
         path = os.path.join(
             Config.SCODOC_VAR_DIR,
             "entreprises",
-            f"{offre[1].entreprise_id}",
-            f"{offre[1].id}",
+            f"{offre.entreprise_id}",
+            f"{offre.id}",
         )
         if os.path.exists(path):
             for dir in glob.glob(
@@ -269,7 +269,7 @@ def offres_recues():
                 for file in glob.glob(f"{dir}/*"):
                     file = [os.path.basename(dir), os.path.basename(file)]
                     files.append(file)
-        offres_recues_with_files.append([offre[0], offre[1], files, correspondant])
+        offres_recues_with_files.append([envoi_offre, offre, files, correspondant])
     return render_template(
         "entreprises/offres_recues.html",
         title="Offres reçues",
diff --git a/app/templates/entreprises/_offre.html b/app/templates/entreprises/_offre.html
index 4ae08232d..188cb692d 100644
--- a/app/templates/entreprises/_offre.html
+++ b/app/templates/entreprises/_offre.html
@@ -1,53 +1,53 @@
 {# -*- mode: jinja-html -*- #}
 <div class="offre">
     <div style="word-break:break-all; text-align: justify;">
-        Ajouté le {{ offre[0].date_ajout.strftime('%d/%m/%y') }} à {{ offre[0].date_ajout.strftime('%Hh%M') }}<br>
-        Intitulé : {{ offre[0].intitule }}<br>
-        Description : {{ offre[0].description }}<br>
-        Type de l'offre : {{ offre[0].type_offre }}<br>
-        Missions : {{ offre[0].missions }}<br>
-        Durée : {{ offre[0].duree }}<br>
-        {% if offre[2] %}
-        Département(s) : {% for offre_dept in offre[2] %} <div class="offre-depts">{{ offre_dept.dept_id|get_dept_acronym }}</div> {% endfor %}<br>
+        Ajouté le {{ offre.date_ajout.strftime('%d/%m/%y') }} à {{ offre.date_ajout.strftime('%Hh%M') }}<br>
+        Intitulé : {{ offre.intitule }}<br>
+        Description : {{ offre.description }}<br>
+        Type de l'offre : {{ offre.type_offre }}<br>
+        Missions : {{ offre.missions }}<br>
+        Durée : {{ offre.duree }}<br>
+        {% if offre_depts %}
+        Département(s) : {% for offre_dept in offre_depts %} <div class="offre-depts">{{ offre_dept.dept_id|get_dept_acronym }}</div> {% endfor %}<br>
         {% endif %}
 
-        {% if offre[0].correspondant_id %}
-        Contacté {{ offre[3].nom }} {{ offre[3].prenom }}
-        {% if offre[3].mail and offre[3].telephone %} 
-        ({{ offre[3].mail }} - {{ offre[3].telephone }})<br>
+        {% if offre.correspondant_id %}
+        Contacté {{ correspondant.nom }} {{ correspondant.prenom }}
+        {% if correspondant.mail and correspondant.telephone %} 
+        ({{ correspondant.mail }} - {{ correspondant.telephone }})<br>
         {% else %}
-        ({{ offre[3].mail }}{{offre[3].telephone}})<br>
+        ({{ correspondant.mail }}{{ correspondant.telephone }})<br>
         {% endif %}
         {% endif %}
 
-        {% for fichier in offre[1] %}
-            <a href="{{ url_for('entreprises.get_offre_file', entreprise_id=entreprise.id, offre_id=offre[0].id, filedir=fichier[0], filename=fichier[1] )}}">{{ fichier[1] }}</a>
+        {% for filedir, filename in files %}
+            <a href="{{ url_for('entreprises.get_offre_file', entreprise_id=entreprise.id, offre_id=offre.id, filedir=filedir, filename=filename )}}">{{ filename }}</a>
             {% if current_user.has_permission(current_user.Permission.RelationsEntreprisesChange, None) %}
-            <a href="{{ url_for('entreprises.delete_offre_file', entreprise_id=entreprise.id, offre_id=offre[0].id, filedir=fichier[0] )}}" style="margin-left: 5px;"><img title="Supprimer fichier" alt="supprimer" width="10" height="9" border="0" src="/ScoDoc/static/icons/delete_small_img.png" /></a>
+            <a href="{{ url_for('entreprises.delete_offre_file', entreprise_id=entreprise.id, offre_id=offre.id, filedir=filedir )}}" style="margin-left: 5px;"><img title="Supprimer fichier" alt="supprimer" width="10" height="9" border="0" src="/ScoDoc/static/icons/delete_small_img.png" /></a>
             {% endif %}
             <br>
         {% endfor %}
 
         {% if current_user.has_permission(current_user.Permission.RelationsEntreprisesChange, None) %}
-        <a href="{{ url_for('entreprises.add_offre_file', entreprise_id=entreprise.id, offre_id=offre[0].id) }}">Ajoutez un fichier</a>
+        <a href="{{ url_for('entreprises.add_offre_file', entreprise_id=entreprise.id, offre_id=offre.id) }}">Ajoutez un fichier</a>
         {% endif %}
     </div>
     
     <div class="parent-btn">
         {% if current_user.has_permission(current_user.Permission.RelationsEntreprisesChange, None) %}
-        <a class="btn btn-primary" href="{{ url_for('entreprises.edit_offre', entreprise_id=offre[0].entreprise_id, offre_id=offre[0].id) }}">Modifier l'offre</a>
-        <a class="btn btn-danger" href="{{ url_for('entreprises.delete_offre', entreprise_id=offre[0].entreprise_id, offre_id=offre[0].id) }}">Supprimer l'offre</a>
+        <a class="btn btn-primary" href="{{ url_for('entreprises.edit_offre', entreprise_id=offre.entreprise_id, offre_id=offre.id) }}">Modifier l'offre</a>
+        <a class="btn btn-danger" href="{{ url_for('entreprises.delete_offre', entreprise_id=offre.entreprise_id, offre_id=offre.id) }}">Supprimer l'offre</a>
         {% endif %}
 
         {% if current_user.has_permission(current_user.Permission.RelationsEntreprisesSend, None) %}
-        <a class="btn btn-primary" href="{{ url_for('entreprises.envoyer_offre', entreprise_id=entreprise.id, offre_id=offre[0].id) }}">Envoyer l'offre</a>
+        <a class="btn btn-primary" href="{{ url_for('entreprises.envoyer_offre', entreprise_id=entreprise.id, offre_id=offre.id) }}">Envoyer l'offre</a>
         {% endif %}
 
         {% if current_user.has_permission(current_user.Permission.RelationsEntreprisesChange, None) %}
-            {% if not offre[0].expired %}
-                <a class="btn btn-danger" href="{{ url_for('entreprises.expired', entreprise_id=offre[0].entreprise_id, offre_id=offre[0].id) }}">Rendre expirée</a>
+            {% if not offre.expired %}
+                <a class="btn btn-danger" href="{{ url_for('entreprises.expired', entreprise_id=offre.entreprise_id, offre_id=offre.id) }}">Rendre expirée</a>
             {% else %}
-                <a class="btn btn-success" href="{{ url_for('entreprises.expired', entreprise_id=offre[0].entreprise_id, offre_id=offre[0].id) }}">Rendre non expirée</a>
+                <a class="btn btn-success" href="{{ url_for('entreprises.expired', entreprise_id=offre.entreprise_id, offre_id=offre.id) }}">Rendre non expirée</a>
             {% endif %}
         {% endif %}
     </div>
diff --git a/app/templates/entreprises/correspondants.html b/app/templates/entreprises/correspondants.html
index 0f89ffc67..d4ce63225 100644
--- a/app/templates/entreprises/correspondants.html
+++ b/app/templates/entreprises/correspondants.html
@@ -37,15 +37,15 @@
                 </tr>
             </thead>
             <tbody>
-            {% for correspondant in correspondants %}
+            {% for correspondant, site in correspondants %}
                 <tr>
-                    <td>{{ correspondant[0].nom }}</td>
-                    <td>{{ correspondant[0].prenom }}</td>
-                    <td>{{ correspondant[0].telephone }}</td>
-                    <td>{{ correspondant[0].mail }}</td>
-                    <td>{{ correspondant[0].poste}}</td>
-                    <td>{{ correspondant[0].service}}</td>
-                    <td><a href="{{ url_for('entreprises.fiche_entreprise', entreprise_id=correspondant[1].entreprise.id) }}">{{ correspondant[1].nom }}</a></td>
+                    <td>{{ correspondant.nom }}</td>
+                    <td>{{ correspondant.prenom }}</td>
+                    <td>{{ correspondant.telephone }}</td>
+                    <td>{{ correspondant.mail }}</td>
+                    <td>{{ correspondant.poste}}</td>
+                    <td>{{ correspondant.service}}</td>
+                    <td><a href="{{ url_for('entreprises.fiche_entreprise', entreprise_id=site.entreprise.id) }}">{{ site.nom }}</a></td>
                 </tr>
             {% endfor %}
             </tbody>
diff --git a/app/templates/entreprises/fiche_entreprise.html b/app/templates/entreprises/fiche_entreprise.html
index cd3ccc8a6..c895348ab 100644
--- a/app/templates/entreprises/fiche_entreprise.html
+++ b/app/templates/entreprises/fiche_entreprise.html
@@ -126,7 +126,7 @@
             {% if offres %}
             <div>
                 <h3>Offres - <a href="{{ url_for('entreprises.offres_expirees', entreprise_id=entreprise.id) }}">Voir les offres expirées</a></h3>
-                {% for offre in offres %}
+                {% for offre, files, offre_depts, correspondant in offres %}
                     {% include 'entreprises/_offre.html' %}
                 {% endfor %}
             </div>
@@ -155,15 +155,15 @@
                 </tr>
             </thead>
             <tbody>
-                {% for data in stages_apprentissages %}
+                {% for stage_apprentissage, etudiant in stages_apprentissages %}
                 <tr>
-                    <td>{{ data[0].date_debut.strftime('%d/%m/%Y') }}</td>
-                    <td>{{ data[0].date_fin.strftime('%d/%m/%Y') }}</td>
-                    <td>{{ (data[0].date_fin-data[0].date_debut).days//7 }} semaines</td>
-                    <td>{{ data[0].type_offre }}</td>
-                    <td><a href="{{ url_for('scolar.ficheEtud', scodoc_dept=data[1].dept_id|get_dept_acronym, etudid=data[0].etudid) }}">{{ data[1].nom|format_nom }} {{ data[1].prenom|format_prenom }}</a></td>
-                    <td>{% if data[0].formation_text %}{{ data[0].formation_text }}{% endif %}</td>
-                    <td>{{ data[0].notes }}</td>
+                    <td>{{ stage_apprentissage.date_debut.strftime('%d/%m/%Y') }}</td>
+                    <td>{{ stage_apprentissage.date_fin.strftime('%d/%m/%Y') }}</td>
+                    <td>{{ (stage_apprentissage.date_fin-stage_apprentissage.date_debut).days//7 }} semaines</td>
+                    <td>{{ stage_apprentissage.type_offre }}</td>
+                    <td><a href="{{ url_for('scolar.ficheEtud', scodoc_dept=etudiant.dept_id|get_dept_acronym, etudid=stage_apprentissage.etudid) }}">{{ etudiant.nom|format_nom }} {{ etudiant.prenom|format_prenom }}</a></td>
+                    <td>{% if stage_apprentissage.formation_text %}{{ stage_apprentissage.formation_text }}{% endif %}</td>
+                    <td>{{ stage_apprentissage.notes }}</td>
                     {% if current_user.has_permission(current_user.Permission.RelationsEntreprisesChange, None) %}
                     <td>
                         <div class="btn-group">
@@ -171,8 +171,8 @@
                                 <span class="caret"></span>
                             </a>
                             <ul class="dropdown-menu pull-left">
-                                <li><a href="{{ url_for('entreprises.edit_stage_apprentissage', entreprise_id=entreprise.id, stage_apprentissage_id=data[0].id) }}">Modifier</a></li>
-                                <li><a href="{{ url_for('entreprises.delete_stage_apprentissage', entreprise_id=entreprise.id, stage_apprentissage_id=data[0].id) }}" style="color:red">Supprimer</a></li>
+                                <li><a href="{{ url_for('entreprises.edit_stage_apprentissage', entreprise_id=entreprise.id, stage_apprentissage_id=stage_apprentissage.id) }}">Modifier</a></li>
+                                <li><a href="{{ url_for('entreprises.delete_stage_apprentissage', entreprise_id=entreprise.id, stage_apprentissage_id=stage_apprentissage.id) }}" style="color:red">Supprimer</a></li>
                             </ul>
                         </div>
                     </td>
diff --git a/app/templates/entreprises/offres_expirees.html b/app/templates/entreprises/offres_expirees.html
index 52736dff8..06142d412 100644
--- a/app/templates/entreprises/offres_expirees.html
+++ b/app/templates/entreprises/offres_expirees.html
@@ -19,7 +19,7 @@
     <div class="container">
         <h1>Offres expirées - {{ entreprise.nom }}</h1>
         {% if offres_expirees %}
-            {% for offre in offres_expirees %}
+            {% for offre, files, offre_depts, correspondant in offres_expirees %}
                 {% include 'entreprises/_offre.html' %}
             {% endfor %}
         {% else %}
diff --git a/app/templates/entreprises/offres_recues.html b/app/templates/entreprises/offres_recues.html
index 779de74f1..ce7719b9e 100644
--- a/app/templates/entreprises/offres_recues.html
+++ b/app/templates/entreprises/offres_recues.html
@@ -7,41 +7,41 @@
     <div class="container">
         <h1>Offres reçues</h1>
         {% if offres_recues %}
-            {% for offre in offres_recues %}
+            {% for envoi_offre, offre, files, correspondant in offres_recues %}
             <div class="offre offre-recue">
                 <div style="word-break:break-all; text-align: justify;">
-                    Envoyé le {{ offre[0].date_envoi.strftime('%d/%m/%Y') }} à {{ offre[0].date_envoi.strftime('%Hh%M') }} par {{ offre[0].sender_id|get_nomcomplet_by_id }}<br>
-                    Intitulé : {{ offre[1].intitule }}<br>
-                    Description : {{ offre[1].description }}<br>
-                    Type de l'offre : {{ offre[1].type_offre }}<br>
-                    Missions : {{ offre[1].missions }}<br>
-                    Durée : {{ offre[1].duree }}<br>
+                    Envoyé le {{ envoi_offre.date_envoi.strftime('%d/%m/%Y') }} à {{ envoi_offre.date_envoi.strftime('%Hh%M') }} par {{ envoi_offre.sender_id|get_nomcomplet_by_id }}<br>
+                    Intitulé : {{ offre.intitule }}<br>
+                    Description : {{ offre.description }}<br>
+                    Type de l'offre : {{ offre.type_offre }}<br>
+                    Missions : {{ offre.missions }}<br>
+                    Durée : {{ offre.duree }}<br>
 
-                    {% if offre[1].correspondant_id %}
-                    Contacté {{ offre[3].nom }} {{ offre[3].prenom }}
-                        {% if offre[3].mail and offre[3].telephone %} 
-                        ({{ offre[3].mail }} - {{ offre[3].telephone }})
+                    {% if offre.correspondant_id %}
+                    Contacté {{ correspondant.nom }} {{ correspondant.prenom }}
+                        {% if correspondant.mail and correspondant.telephone %} 
+                        ({{ correspondant.mail }} - {{ correspondant.telephone }})
                         {% else %}
-                        ({{ offre[3].mail }}{{ offre[3].telephone }})
+                        ({{ correspondant.mail }}{{ correspondant.telephone }})
                         {% endif %}
                     {% endif %}
 
-                    {% if offre[3].poste %}
-                    , poste : {{ offre[3].poste }}
+                    {% if correspondant.poste %}
+                    - poste : {{ correspondant.poste }}
                     {% endif %}
-                    {% if offre[3].service %}
-                    , service : {{ offre[3].service }}
+                    {% if correspondant.service %}
+                    - service : {{ correspondant.service }}
                     {% endif %}
                     <br>
 
-                    <a href="{{ url_for('entreprises.fiche_entreprise', entreprise_id=offre[1].entreprise_id) }}">lien vers l'entreprise</a><br>
+                    <a href="{{ url_for('entreprises.fiche_entreprise', entreprise_id=offre.entreprise_id) }}">lien vers l'entreprise</a><br>
 
-                    {% for fichier in offre[2] %}
-                        <a href="{{ url_for('entreprises.get_offre_file', entreprise_id=offre[1].entreprise_id, offre_id=offre[1].id, filedir=fichier[0], filename=fichier[1]) }}">{{ fichier[1] }}</a><br>
+                    {% for filedir, filename in files %}
+                        <a href="{{ url_for('entreprises.get_offre_file', entreprise_id=offre.entreprise_id, offre_id=offre.id, filedir=filedir, filename=filename) }}">{{ filename }}</a><br>
                     {% endfor %}
                 </div>
                 <div>
-                    <a href="{{ url_for('entreprises.delete_offre_recue', envoi_offre_id=offre[0].id) }}" style="margin-left: 5px;"><img title="Supprimer" alt="supprimer" width="16" height="16" border="0" src="/ScoDoc/static/icons/delete_small_img.png" /></a>
+                    <a href="{{ url_for('entreprises.delete_offre_recue', envoi_offre_id=envoi_offre.id) }}" style="margin-left: 5px;"><img title="Supprimer" alt="supprimer" width="16" height="16" border="0" src="/ScoDoc/static/icons/delete_small_img.png" /></a>
                 </div>
             </div>
             {% endfor %}