From 21b2e0f582f438e9d7eaab4930db76f4eed4cbd5 Mon Sep 17 00:00:00 2001
From: Iziram <matthias.hartmann@iziram.fr>
Date: Thu, 29 Feb 2024 08:47:03 +0100
Subject: [PATCH] =?UTF-8?q?Assiduit=C3=A9=20:=20fix=20bug=20module=20selec?=
 =?UTF-8?q?tor=20signal=5Fassiduites=5Fgroup?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../pages/signal_assiduites_group.j2          | 51 ++++++++++---------
 app/views/assiduites.py                       |  6 +--
 2 files changed, 28 insertions(+), 29 deletions(-)

diff --git a/app/templates/assiduites/pages/signal_assiduites_group.j2 b/app/templates/assiduites/pages/signal_assiduites_group.j2
index 12380afaa..4eb21e413 100644
--- a/app/templates/assiduites/pages/signal_assiduites_group.j2
+++ b/app/templates/assiduites/pages/signal_assiduites_group.j2
@@ -1,23 +1,23 @@
 {% extends "sco_page.j2" %}
 
 {% block title %}
-    {{title}}
+{{title}}
 {% endblock title %}
 
 
 {% block scripts %}
-    {{ super() }}
-    <script src="{{scu.STATIC_DIR}}/libjs/bootstrap-multiselect-1.1.2/bootstrap-multiselect.min.js"></script>
-    <script src="{{scu.STATIC_DIR}}/libjs/purl.js"></script>
-    <script src="{{scu.STATIC_DIR}}/js/etud_info.js"></script>
-    <script src="{{scu.STATIC_DIR}}/js/groups_view.js"></script>
-    <script src="{{scu.STATIC_DIR}}/js/date_utils.js"></script>
-    <script src="{{scu.STATIC_DIR}}/js/assiduites.js"></script>
+{{ super() }}
+<script src="{{scu.STATIC_DIR}}/libjs/bootstrap-multiselect-1.1.2/bootstrap-multiselect.min.js"></script>
+<script src="{{scu.STATIC_DIR}}/libjs/purl.js"></script>
+<script src="{{scu.STATIC_DIR}}/js/etud_info.js"></script>
+<script src="{{scu.STATIC_DIR}}/js/groups_view.js"></script>
+<script src="{{scu.STATIC_DIR}}/js/date_utils.js"></script>
+<script src="{{scu.STATIC_DIR}}/js/assiduites.js"></script>
 
-    <script>
+<script>
 
     {% if readonly != "false" %}
-    function getPeriodValues(){
+    function getPeriodValues() {
         return [0, 23]
     }
     {% endif %}
@@ -30,9 +30,9 @@
 
     setupDate();
     updateDate();
-    if (!readOnly){
-        setupTimeLine(()=>{
-                generateAllEtudRow();
+    if (!readOnly) {
+        setupTimeLine(() => {
+            generateAllEtudRow();
         });
     }
 
@@ -63,16 +63,16 @@
             }
         });
     }
-    </script>
+</script>
 {% endblock scripts %}
 
 {% block styles %}
-    {{ super() }}
-    <link rel="stylesheet" href="{{scu.STATIC_DIR}}/libjs/bootstrap/css/bootstrap.min.css">
-    <link rel="stylesheet" href="{{scu.STATIC_DIR}}/libjs/bootstrap/css/bootstrap-theme.min.css">
-    <link rel="stylesheet" href="{{scu.STATIC_DIR}}/libjs/bootstrap-multiselect-1.1.2/bootstrap-multiselect.min.css">
-    <link rel="stylesheet" href="{{scu.STATIC_DIR}}/css/assiduites.css">
-    <link rel="stylesheet" href="{{scu.STATIC_DIR}}/css/minitimeline.css">
+{{ super() }}
+<link rel="stylesheet" href="{{scu.STATIC_DIR}}/libjs/bootstrap/css/bootstrap.min.css">
+<link rel="stylesheet" href="{{scu.STATIC_DIR}}/libjs/bootstrap/css/bootstrap-theme.min.css">
+<link rel="stylesheet" href="{{scu.STATIC_DIR}}/libjs/bootstrap-multiselect-1.1.2/bootstrap-multiselect.min.css">
+<link rel="stylesheet" href="{{scu.STATIC_DIR}}/css/assiduites.css">
+<link rel="stylesheet" href="{{scu.STATIC_DIR}}/css/minitimeline.css">
 {% endblock styles %}
 
 
@@ -80,7 +80,11 @@
 {% include "assiduites/widgets/toast.j2" %}
 
 {{ minitimeline|safe }}
-
+<style>
+    #moduleimpl_select {
+        max-width: 200px;
+    }
+</style>
 <section id="content">
 
     <div class="no-display">
@@ -102,8 +106,7 @@
             <div class="infos-button">Groupes&nbsp;: {{grp|safe}}</div>
             <div class="infos-button" style="margin-left: 24px;">Date&nbsp;: <span style="margin-left: 8px;"
                     id="datestr"></span>
-                <input type="text" name="tl_date" id="tl_date" value="{{ date }}"
-                    onchange="updateDate()">
+                <input type="text" name="tl_date" id="tl_date" value="{{ date }}" onchange="updateDate()">
             </div>
         </div>
     </fieldset>
@@ -162,4 +165,4 @@
 
 </section>
 
-{% endblock app_content %}
+{% endblock app_content %}
\ No newline at end of file
diff --git a/app/views/assiduites.py b/app/views/assiduites.py
index f892edc6d..16d5c9a06 100644
--- a/app/views/assiduites.py
+++ b/app/views/assiduites.py
@@ -2181,12 +2181,8 @@ def _module_selector(formsemestre: FormSemestre, moduleimpl_id: int = None) -> s
     """
     # récupération des ues du semestre
     ntc: NotesTableCompat = res_sem.load_formsemestre_results(formsemestre)
-    ues = ntc.get_ues_stat_dict()
 
-    modimpls_list: list[dict] = []
-    for ue in ues:
-        # Ajout des moduleimpl de chaque ue dans la liste des moduleimpls
-        modimpls_list += ntc.get_modimpls_dict(ue_id=ue["ue_id"])
+    modimpls_list: list[dict] = ntc.get_modimpls_dict()
 
     # prévoie la sélection par défaut d'un moduleimpl s'il a été passé en paramètre
     selected = "" if moduleimpl_id is not None else "selected"