From 99b182e1c744ee0ae8cc8f1eecc30ee95e4e167d Mon Sep 17 00:00:00 2001
From: Emmanuel Viennet <emmanuel.viennet@gmail.com>
Date: Mon, 11 Dec 2023 15:24:55 +0100
Subject: [PATCH] =?UTF-8?q?Bug=20fix:=20acc=C3=A8s=20aux=20fichiers=20just?=
 =?UTF-8?q?ificatifs?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 app/api/justificatifs.py                           | 10 +++++-----
 .../assiduites/pages/tableau_assiduite_actions.j2  | 14 ++++++++++----
 .../assiduites/widgets/tableau_actions/details.j2  |  8 ++++++--
 .../assiduites/widgets/tableau_actions/modifier.j2 |  2 +-
 sco_version.py                                     |  2 +-
 5 files changed, 23 insertions(+), 13 deletions(-)

diff --git a/app/api/justificatifs.py b/app/api/justificatifs.py
index e90098e5e..57c36b3f2 100644
--- a/app/api/justificatifs.py
+++ b/app/api/justificatifs.py
@@ -24,7 +24,6 @@ from app.models import (
     Justificatif,
     Departement,
     FormSemestre,
-    FormSemestreInscription,
 )
 from app.models.assiduites import (
     compute_assiduites_justified,
@@ -647,15 +646,16 @@ def justif_import(justif_id: int = None):
 
 
 @bp.route("/justificatif/<int:justif_id>/export/<filename>", methods=["GET", "POST"])
-@api_web_bp.route("/justificatif/<int:justif_id>/export/<filename>", methods=["GET", "POST"])
+@api_web_bp.route(
+    "/justificatif/<int:justif_id>/export/<filename>", methods=["GET", "POST"]
+)
 @scodoc
 @login_required
-@permission_required(Permission.AbsChange)
-def justif_export(justif_id: int = None, filename: str = None):
+@permission_required(Permission.AbsJustifView)
+def justif_export(justif_id: int | None = None, filename: str | None = None):
     """
     Retourne un fichier d'une archive d'un justificatif
     """
-
     # On récupère le justificatif concerné
     query: Query = Justificatif.query.filter_by(id=justif_id)
     if g.scodoc_dept:
diff --git a/app/templates/assiduites/pages/tableau_assiduite_actions.j2 b/app/templates/assiduites/pages/tableau_assiduite_actions.j2
index 7ebfe730c..903fceba4 100644
--- a/app/templates/assiduites/pages/tableau_assiduite_actions.j2
+++ b/app/templates/assiduites/pages/tableau_assiduite_actions.j2
@@ -13,11 +13,17 @@
 {% else%}
     {% include "assiduites/widgets/tableau_actions/details.j2" %}
 {% endif %}
-<br>
-<hr>
-<br>
-<a href="" id="lien-retour">Retour</a>
 
+{% if not current_user.has_permission(sco.Permission.AbsJustifView)%}
+<div class="help fontred" style="margin-top: 16px;">
+    Vous n'avez pas la permission d'ouvrir les fichiers justificatifs
+    déposés par d'autres personnes.
+</div>
+{% endif %}
+
+<div style="margin-top: 32px;">
+    <a href="" id="lien-retour">retour</a>
+</div>
 <script>
     window.addEventListener('load', () => {
         document.getElementById("lien-retour").href = document.referrer;
diff --git a/app/templates/assiduites/widgets/tableau_actions/details.j2 b/app/templates/assiduites/widgets/tableau_actions/details.j2
index 9d8193c07..5b47eebbc 100644
--- a/app/templates/assiduites/widgets/tableau_actions/details.j2
+++ b/app/templates/assiduites/widgets/tableau_actions/details.j2
@@ -90,10 +90,14 @@
         <div>Total : {{objet.justification.fichiers.total}} </div>
         <ul>
             {% for filename in objet.justification.fichiers.filenames %}
-            <li><a
-                    href="{{url_for('api.justif_export',justif_id=objet.justif_id,filename=filename, scodoc_dept=g.scodoc_dept)}}">{{filename}}</a>
+            <li>
+                <a
+                    href="{{url_for('apiweb.justif_export',justif_id=objet.justif_id,filename=filename, scodoc_dept=g.scodoc_dept)}}">{{filename}}</a>
             </li>
             {% endfor %}
+            {% if not objet.justification.fichiers.filenames %}
+            <li class="fontred">fichiers non visibles</li>
+            {% endif %}
         </ul>
         {% else %}
         <span class="text">Aucun</span>
diff --git a/app/templates/assiduites/widgets/tableau_actions/modifier.j2 b/app/templates/assiduites/widgets/tableau_actions/modifier.j2
index 18f6305da..73dedb4e5 100644
--- a/app/templates/assiduites/widgets/tableau_actions/modifier.j2
+++ b/app/templates/assiduites/widgets/tableau_actions/modifier.j2
@@ -53,7 +53,7 @@ Actuellement noté{{etud.e}} en <b>{{objet_name|lower()}}</b> du {{objet.date_de
             <li data-id="{{filename}}">
                 <a data-file="{{filename}}">❌</a>
                 <a data-link=""
-                    href="{{url_for('api.justif_export',justif_id=objet.justif_id,filename=filename, scodoc_dept=g.scodoc_dept)}}"><span
+                    href="{{url_for('apiweb.justif_export',justif_id=objet.justif_id,filename=filename, scodoc_dept=g.scodoc_dept)}}"><span
                         data-file="{{filename}}">{{filename}}</span></a>
             </li>
             {% endfor %}
diff --git a/sco_version.py b/sco_version.py
index 678af3349..6bac59e95 100644
--- a/sco_version.py
+++ b/sco_version.py
@@ -1,7 +1,7 @@
 # -*- mode: python -*-
 # -*- coding: utf-8 -*-
 
-SCOVERSION = "9.6.66"
+SCOVERSION = "9.6.67"
 
 SCONAME = "ScoDoc"