From b74d525c28de5cd5bdae6d877ba3513ccca02180 Mon Sep 17 00:00:00 2001
From: Iziram <matthias.hartmann@iziram.fr>
Date: Fri, 15 Mar 2024 15:56:33 +0100
Subject: [PATCH] =?UTF-8?q?Assiduit=C3=A9:=20signal=5Fassiduites=5Fdiff=20?=
 =?UTF-8?q?OK?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 app/scodoc/sco_moduleimpl_status.py           |   9 +-
 app/static/css/assiduites.css                 |  15 +-
 app/static/js/assiduites.js                   |  10 +
 app/static/js/date_utils.js                   |   9 +
 .../pages/signal_assiduites_diff.j2           | 476 +++++++++++++-----
 app/views/assiduites.py                       |  10 +-
 6 files changed, 388 insertions(+), 141 deletions(-)

diff --git a/app/scodoc/sco_moduleimpl_status.py b/app/scodoc/sco_moduleimpl_status.py
index c45d1b84f..aee1c91ec 100644
--- a/app/scodoc/sco_moduleimpl_status.py
+++ b/app/scodoc/sco_moduleimpl_status.py
@@ -25,8 +25,8 @@
 #
 ##############################################################################
 
-"""Tableau de bord module
-"""
+"""Tableau de bord module"""
+
 import math
 import time
 import datetime
@@ -329,8 +329,6 @@ def moduleimpl_status(moduleimpl_id=None, partition_id=None):
         >Saisie Absences journée</a></span>
         """
         )
-        year, week, day = datetime.date.today().isocalendar()
-        semaine: str = f"{year}-W{week}"
         H.append(
             f"""
                 <span class="moduleimpl_abs_link"><a class="stdlink" href="{
@@ -338,11 +336,10 @@ def moduleimpl_status(moduleimpl_id=None, partition_id=None):
                     "assiduites.signal_assiduites_diff",
                     scodoc_dept=g.scodoc_dept,
                     group_ids=group_id,
-                    semaine=semaine,
                     formsemestre_id=formsemestre.id,
                     moduleimpl_id="" if moduleimpl_id is None else moduleimpl_id
             )}"
-        >Saisie Absences hebdo</a></span>
+        >Saisie Absences Différée</a></span>
         """
         )
 
diff --git a/app/static/css/assiduites.css b/app/static/css/assiduites.css
index 02730449c..f4d1ee4f7 100644
--- a/app/static/css/assiduites.css
+++ b/app/static/css/assiduites.css
@@ -440,12 +440,6 @@
   position: absolute;
   z-index: 5;
   border: 5px solid var(--color-primary);
-  /* background-color: rgba(36, 36, 36, 0.25);
-  background-image: repeating-linear-gradient(135deg,
-      transparent,
-      transparent 5px,
-      rgba(81, 81, 81, 0.61) 5px,
-      rgba(81, 81, 81, 0.61) 10px); */
   border-radius: 5px;
 }
 
@@ -455,6 +449,15 @@
   transform: translateX(-50%);
 }
 
+.assiduite-infos {
+  position: absolute;
+  right: 0;
+  margin: 5px;
+  top: 0;
+  font-size: 16px;
+  cursor: pointer;
+}
+
 .action-buttons {
   display: flex;
   justify-content: center;
diff --git a/app/static/js/assiduites.js b/app/static/js/assiduites.js
index a29c2f25e..8054f3de3 100644
--- a/app/static/js/assiduites.js
+++ b/app/static/js/assiduites.js
@@ -855,6 +855,16 @@ function setupAssiduiteBubble(el, assiduite) {
   bubble.className = "assiduite-bubble";
   bubble.classList.add(assiduite.etat.toLowerCase());
 
+  // Ajout d'un lien pour plus d'informations
+  const infos = document.createElement("a");
+  infos.className = "assiduite-infos";
+  infos.textContent = `ℹ️`;
+  infos.title = "Cliquez pour plus d'informations";
+  infos.target = "_blank";
+  infos.href = `tableau_assiduite_actions?type=assiduite&action=details&obj_id=${assiduite.assiduite_id}`;
+
+  bubble.appendChild(infos);
+
   const idDiv = document.createElement("div");
   idDiv.className = "assiduite-id";
   getModuleImpl(assiduite).then((modImpl) => {
diff --git a/app/static/js/date_utils.js b/app/static/js/date_utils.js
index 57925cf98..0db9d3b0f 100644
--- a/app/static/js/date_utils.js
+++ b/app/static/js/date_utils.js
@@ -255,6 +255,13 @@ Object.defineProperty(Date.prototype, "format", {
   value: function (formatString) {
     let iso = this.toIsoUtcString();
     switch (formatString) {
+      case "DD/MM/YYYY":
+        return this.toLocaleString("fr-FR", {
+          day: "2-digit",
+          month: "2-digit",
+          year: "numeric",
+          timeZone: SCO_TIMEZONE,
+        });
       case "DD/MM/Y HH:mm":
         return this.toLocaleString("fr-FR", {
           day: "2-digit",
@@ -275,6 +282,8 @@ Object.defineProperty(Date.prototype, "format", {
           hour12: false,
           timeZone: SCO_TIMEZONE,
         });
+      case "HH:mm":
+        return iso.slice(11, 16);
 
       case "YYYY-MM-DDTHH:mm":
         // slice : YYYY-MM-DDTHH
diff --git a/app/templates/assiduites/pages/signal_assiduites_diff.j2 b/app/templates/assiduites/pages/signal_assiduites_diff.j2
index e7b016a94..f19e59acb 100644
--- a/app/templates/assiduites/pages/signal_assiduites_diff.j2
+++ b/app/templates/assiduites/pages/signal_assiduites_diff.j2
@@ -97,19 +97,12 @@
         z-index: 0;
     }
     
+    .pointer{
+        cursor: pointer;
+    }
 
 </style>
 
-{# Temporaire #}
-<style>
-.wip::before{
-    content: "WIP 🚧";
-    font-size: 1.5em;
-    margin: 2px;
-    color: red;
-}
-</style>
-
 {% endblock styles %}
 
 {% block scripts %}
@@ -121,142 +114,375 @@
 
 <script>
 
-    function afficherPDP(checked) {
-        if (checked) {
-            gtrcontent.setAttribute("data-pdp", "true");
-        } else {
-            gtrcontent.removeAttribute("data-pdp");
-        }
+    /**
+ * Permet d'afficher ou non les photos des étudiants
+ * @param {boolean} checked
+ */
+function afficherPDP(checked) {
+  if (checked) {
+    gtrcontent.setAttribute("data-pdp", "true");
+  } else {
+    gtrcontent.removeAttribute("data-pdp");
+  }
+}
+
+/**
+ * Permet d'ajouter une nouvelle période au tableau
+ * Par défaut la période est générèe avec les valeurs des inputs
+ * Si une période est passée en paramètre, alors on utilise ses valeurs
+ * @param {Object} period  - La période à ajouter
+ */
+async function nouvellePeriode(period = null) {
+  // On récupère l'id de la période
+  let periodId;
+  if (period) {
+    periodId = period.periodId;
+  } else {
+    periodId = currentPeriodId++;
+  }
+
+  // On récupère les valeurs des inputs
+  let date = document.getElementById("date").value;
+  let debut = document.getElementById("debut").value;
+  let fin = document.getElementById("fin").value;
+  let moduleimpl_id = document.getElementById("moduleimpl_select").value;
+  const moduleimpl = await getModuleImpl({ moduleimpl_id: moduleimpl_id });
+
+  // Si une période est passée en paramètre, on utilise ses valeurs
+  if (period) {
+    date = period.date_debut.format("DD/MM/YYYY");
+    debut = period.date_debut.format("HH:mm");
+    fin = period.date_fin.format("HH:mm");
+    moduleimpl_id = period.moduleimpl_id;
+  }else{
+    //Sinon on vérifie qu'on a bien des valeurs
+    const text = document.createTextNode("Veuillez remplir tous les champs pour ajouter une période.")
+    if (date == "" || debut == "" || fin == "" || moduleimpl_id == "") {
+      openAlertModal(
+        "Erreur",
+        text
+      );
+      return;
     }
+  }
 
-    async function nouvellePeriode(){
-        periodeId++;
+  // On ajoute la nouvelle période au tableau
+  let periodeDiv = document.createElement("div");
+  periodeDiv.classList.add("cell", "sticky");
+  periodeDiv.id = `periode-${periodId}`;
+  const periodP = document.createElement("p");
+  periodP.textContent = `Période du ${date} de ${debut} à ${fin}`;
 
-        // On récupère les valeurs des inputs
-        let date = document.getElementById("date").value;
-        let debut = document.getElementById("debut").value;
-        let fin = document.getElementById("fin").value;
-        let moduleimpl_id = document.getElementById("moduleimpl_select").value;
-        const moduleimpl = await getModuleImpl({moduleimpl_id: moduleimpl_id});
+  // On ajoute le moduleimpl
+  const modP = document.createElement("p");
+  modP.textContent = moduleimpl;
 
-        // On ajoute la nouvelle période au tableau
-        let periodeDiv = document.createElement("div");
-        periodeDiv.classList.add("cell", "sticky");
-        const periodP = document.createElement("p");
-        periodP.textContent = `Période du ${date} de ${debut} à ${fin}`;
+  // On ajoute le bouton pour supprimer la période
+  const close = document.createElement("button");
+  close.textContent = "❌";
+  close.addEventListener("click", () => {
+    // On supprime toutes les cases du tableau correspondant à cette période
+    document
+      .querySelectorAll(
+        `.cell[data-periodeid="${periodeDiv.getAttribute("data-periodeid")}"]`
+      )
+      .forEach((e) => e.remove());
+    // On supprime la période de la Map periodes
+    periodes.delete(Number(periodeDiv.getAttribute("data-periodeid")));
+  });
+  //On ajoute les éléments au DOM
+  periodeDiv.appendChild(periodP);
+  periodeDiv.appendChild(modP);
+  periodeDiv.appendChild(close);
+  periodeDiv.setAttribute("data-periodeid", periodId);
+  document.getElementById("tableau-periode").appendChild(periodeDiv);
 
-        const modP = document.createElement("p");
-        modP.textContent = moduleimpl;
+  // On récupère les étudiants (etudids)
+  let etudids = [
+    ...document.querySelectorAll("#tableau-periode .header[data-etudid]"),
+  ].map((e) => e.getAttribute("data-etudid"));
 
-        const close = document.createElement("button");
-        close.textContent = "❌";
-        close.addEventListener("click", (event)=>{
-            document.querySelectorAll(`.cell[data-periodeid="${periodeDiv.getAttribute('data-periodeid')}"]`).forEach((e)=>e.remove());
-        });
+  // On génère une date de début et de fin de la période
+  const date_debut = new Date(
+    $("#date").datepicker("getDate").format("YYYY-MM-DD") + "T" + debut
+  );
+  const date_fin = new Date(
+    $("#date").datepicker("getDate").format("YYYY-MM-DD") + "T" + fin
+  );
+  date_debut.add(1, "seconds");
 
-        periodeDiv.appendChild(periodP);
-        periodeDiv.appendChild(modP);
-        periodeDiv.appendChild(close);
-        periodeDiv.setAttribute("data-periodeid", periodeId);
-        document.getElementById("tableau-periode").appendChild(periodeDiv);
-        
-        let etudids = [...document
-        .querySelectorAll("#tableau-periode .header[data-etudid]")]
-        .map((e)=>e.getAttribute("data-etudid"));
-        
-        const date_debut = new Date($("#date").datepicker("getDate").format("YYYY-MM-DD") + "T" + debut);
-        const date_fin = new Date($("#date").datepicker("getDate").format("YYYY-MM-DD") + "T" + fin);
-        date_debut.add(1, "seconds")
-        const url =
-        `../../api/assiduites/group/query?date_debut=${date_debut.toFakeIso()}` +
-        `&date_fin=${date_fin.toFakeIso()}&etudids=${etudids.join(',')}&with_justifs`;
-        
+  // Préparation de la requête
+  const url =
+    `../../api/assiduites/group/query?date_debut=${date_debut.toFakeIso()}` +
+    `&date_fin=${date_fin.toFakeIso()}&etudids=${etudids.join(
+      ","
+    )}&with_justifs`;
 
-        await fetch(url)
-        .then((res) => {
-        if (!res.ok) {
-            throw new Error("Network response was not ok");
+  //Si la période n'existait pas, alors on l'ajoute à la Map
+  if (!period) {
+    periodes.set(periodId, {
+      date_debut: date_debut.clone().add(-1, "seconds"),
+      date_fin: date_fin,
+      moduleimpl_id: moduleimpl_id,
+      periodId: periodId,
+    });
+  }
+
+  // On récupère les incriptions au module
+  const inscriptions = await getInscriptionModule(moduleimpl_id);
+
+  // On récupère les assiduités
+  await fetch(url)
+    // On convertit la réponse en JSON
+    .then((res) => {
+      if (!res.ok) {
+        throw new Error("Network response was not ok");
+      }
+      return res.json();
+    })
+    // On traite les données
+    .then((data) => {
+      for (let etudid of etudids) {
+        // On crée une case pour chaque étudiant
+        let cell = document.createElement("div");
+        cell.classList.add("cell");
+        cell.setAttribute("data-etudid", etudid);
+        cell.setAttribute("data-periodeid", periodId);
+        cell.id = `cell-${etudid}-${periodId}`;
+        document.getElementById("tableau-periode").appendChild(cell);
+
+        //Vérification inscription au module
+        // Si l'étudiant n'est pas inscrit, on le notifie et on passe à l'étudiant suivant
+        const inscrit =
+          inscriptions == null ? true : inscriptions.find((e) => e == etudid);
+        if (!inscrit) {
+          cell.textContent = "Non inscrit";
+          cell.classList.add("non-inscrit");
+          continue;
         }
-        return res.json();
-        })
-        .then((data)=>{
-            Object.entries(data).forEach(([etudid, assiduites]) => {
-                let cell = document.createElement("div");
-                cell.classList.add("cell");
-                cell.setAttribute("data-etudid", etudid);
-                cell.setAttribute("data-periodeid", periodeId);
 
-                if (assiduites.length == 0) {
-                    ["present", "retard", "absent"].forEach((value) => {
-                        const cbox = document.createElement('input');
-                        cbox.type = "checkbox";
-                        cbox.value = value;
-                        cbox.name = `rbtn_${etudid}_${periodeId}`
-                            cbox.classList.add("rbtn", value);
+        //Gestion des assiduités déjà existantes
+        const assiduites = data[etudid];
+        // Si l'étudiant n'a pas d'assiduité, on crée les boutons assiduité
+        if (assiduites.length == 0) {
+          ["present", "retard", "absent"].forEach((value) => {
+            const cbox = document.createElement("input");
+            cbox.type = "checkbox";
+            cbox.value = value;
+            cbox.name = `rbtn_${etudid}_${periodId}`;
+            cbox.classList.add("rbtn", value);
 
-                        cbox.addEventListener('click', (event)=>{
-                            const parent = event.target.parentElement;
-                            parent.querySelectorAll(".rbtn").forEach((ele)=>{
-                                if(ele.value != value){
-                                    ele.checked = false;
-                                }
-                            })
-                        })
-
-                        cbox.checked = etatDef.value == value
-
-                        cell.appendChild(cbox);
-                    });
-                }else{
-                    setupAssiduiteBubble(cell, assiduites[0])
+            // Event pour être sur qu'un seul bouton est coché à la fois
+            cbox.addEventListener("click", (event) => {
+              const parent = event.target.parentElement;
+              parent.querySelectorAll(".rbtn").forEach((ele) => {
+                if (ele.value != value) {
+                  ele.checked = false;
                 }
-
-                document.getElementById("tableau-periode").appendChild(cell);
+              });
             });
-        }).catch((error)=>{
-            console.error('Error:', error);
+            // Si une valeur par défaut est donnée alors on l'applique
+            cbox.checked = etatDef.value == value;
+
+            cell.appendChild(cbox);
+          });
+        } else {
+          // Si une (ou plus) assiduité sont trouvée pour la période
+          // alors on affiche les informations  de la première assiduité
+          setupAssiduiteBubble(cell, assiduites[0]);
+        }
+      }
+    })
+    //Si jamais la requête échoue, on affiche un message d'erreur dans la console
+    .catch((error) => {
+      console.error("Error:", error);
+    });
+}
+/**
+ * Permet de récupérer la saisie puis créer les assiduités grâce à l'api
+ */
+function sauvegarderAssiduites() {
+  // Initialisation de la liste des assiduités à créer
+  let assiduitesData = [];
+  // Pour chaque période, on récupère les assiduités saisies
+  for (let [periodeId, periode] of periodes.entries()) {
+    // On prend chaque cellule correspondant à la période
+    const cells = document.querySelectorAll(
+      `.cell[data-periodeid="${periodeId}"][data-etudid]`
+    );
+    // Pour chaque cellule, on récupère l'état de l'assiduité
+    cells.forEach((cell) => {
+      const etudid = cell.getAttribute("data-etudid");
+      const etat = cell.querySelector(".rbtn:checked")?.value;
+      // Il est possible que l'état soit null
+      // - Cas où l'étudiant n'est pas inscrit
+      // - Cas où l'étudiant avait déjà une assiduité
+      if (etat) {
+        // On génère un objet "assiduité"
+        /*
+        {
+            etudid: <int>,
+            etat: <string>,
+            date_debut: <string>,
+            date_fin: <string>,
+            moduleimpl_id: <int>,
+            periodId: <int>
+        }
+        */
+        assiduitesData.push({
+          etudid: etudid,
+          etat: etat,
+          ...periode,
+        });
+      }
+    });
+  }
+  // Une fois les assiduités générées, on les envoie à l'api
+  async_post(
+    "../../api/assiduites/create",
+    assiduitesData,
+    // Si la requête passe
+    async (data) => {
+      // On supprime toutes les cases du tableau pour le mettre à jour
+      document.querySelectorAll(".cell").forEach((e) => e.remove());
+
+      // On recrée les périodes
+      // (cela permet de redemander les assiduités, donc mettre à jour les cases)
+      for (let periode of periodes.values()) {
+        await nouvellePeriode(periode);
+      }
+
+      // Si il y n'a pas d'erreur, on affiche un message de succès
+      if (data.errors.length == 0) {
+        const span = document.createElement("span");
+        span.textContent = "Les assiduités ont bien été sauvegardées.";
+        openAlertModal(
+          "Sauvegarde des assiduités",
+          span,
+          null,
+          "var(--color-present)"
+        );
+        return;
+      }
+      // Si il y a des erreurs, on les affiche
+      if (data.errors.length > 0) {
+        // On crée une map pour regrouper les erreurs par période
+        const erreurs = new Map();
+        data.errors.forEach((err) => {
+          // Pour chaque période on créer une liste d'erreurs
+          // format : [message, etudid]
+          const assi = assiduitesData[err.indice];
+          const msg = err.message;
+          const periodErrors = erreurs.get(assi.periodId) || [];
+
+          // Récupération du nom de l'étudiant
+          const etud = document.querySelector(
+            `#head-${assi.etudid} span`
+          ).textContent;
+          periodErrors.push([`Erreur pour ${etud} : ${msg}`, assi.etudid]);
+          erreurs.set(assi.periodId, periodErrors);
         });
 
-    }
+        // Création du DOM
+        /*
+        <ul>
+            <li>
+                Période du ... de ... à ...
+                <ul>
+                    <li>Erreur pour ...</li>
+                    <li>Erreur pour ...</li>
+                </ul>
+            /li>
+        </ul>
+        */
 
+        const ul = document.createElement("ul");
+        //Pour chaque période on créer un titre "periode du ... de ... à ..."
+        for (let [periodeId, periodErrors] of erreurs.entries()) {
+          const period = periodes.get(periodeId);
+          const li = document.createElement("li");
+          // On affiche la période
+          li.textContent = `Période du ${period.date_debut.format(
+            "DD/MM/YYYY HH:mm"
+          )} à ${period.date_fin.format("HH:mm")}`;
 
+          // Nous emmène à la période lorsqu'on clique dessus
+          li.addEventListener("click", () => {
+            location.href = `#periode-${periodeId}`;
+          });
+          li.classList.add("pointer");
 
-    let periodeId = 0;
-    const moduleimpls = new Map();
-    const nonWorkDays = [{{ nonworkdays| safe }}];
-
-
-    window.forceModule = "{{ forcer_module }}" == "True"
-    if (window.forceModule) {
-        if (moduleimpl_select.value == "") {
-            document.getElementById('forcemodule').style.display = "block";
-                add_periode.disabled = true;
+          // Pour chaque erreur, on créer un élément de liste
+          const ul2 = document.createElement("ul");
+          periodErrors.forEach((err) => {
+            const li2 = document.createElement("li");
+            li2.textContent = err[0];
+            li2.classList.add("pointer");
 
+            // Nous emmène à la case de l'étudiant lorsqu'on clique dessus
+            li2.addEventListener("click", () => {
+              location.href = `#cell-${err[1]}-${periodeId}`;
+            });
+            ul2.appendChild(li2);
+          });
+          li.appendChild(ul2);
+          ul.appendChild(li);
         }
 
-        moduleimpl_select?.addEventListener('change', (e) => {
-            if (e.target.value != "") {
-                document.getElementById('forcemodule').style.display = "none";
-                add_periode.disabled = false;
-
-            } else {
-                document.getElementById('forcemodule').style.display = "block";
-                add_periode.disabled = true;
-
-            }
-        });
+        openAlertModal(
+          "Erreurs lors de la sauvegarde des assiduités",
+          ul,
+          "Les autres assiduités ont bien été sauvegardées."
+        );
+      }
+    },
+    (e) => {
+      console.error("Erreur lors de la création des assiduités", e);
     }
+  );
+}
 
-    async function main(){
-        afficherPDP(pdp.checked);
-        $('#date').on('change', async function(d) {
-        // On vérifie si la date est un jour travaillé
-        dateCouranteEstTravaillee();
-        });
+// Mis en place des variables globales
+let currentPeriodId = 0;
+const periodes = new Map();
+const moduleimpls = new Map();
+const inscriptionsModules = new Map();
+const nonWorkDays = [{{ nonworkdays| safe }}];
+
+// Vérification du forçage de module
+window.forceModule = "{{ forcer_module }}" == "True";
+if (window.forceModule) {
+  if (moduleimpl_select.value == "") {
+    document.getElementById("forcemodule").style.display = "block";
+    add_periode.disabled = true;
+  }
+  // Désactivation du bouton d'ajout de période si aucun module n'est sélectionné
+  // et affichage du message de forçage de module
+  moduleimpl_select?.addEventListener("change", (e) => {
+    if (e.target.value != "") {
+      document.getElementById("forcemodule").style.display = "none";
+      add_periode.disabled = false;
+    } else {
+      document.getElementById("forcemodule").style.display = "block";
+      add_periode.disabled = true;
     }
+  });
+}
+/**
+ * Fonction exécutée au lancement de la page
+ * - On affiche ou non les photos des étudiants
+ * - On vérifie si la date est un jour travaillé
+ */
+async function main() {
+  afficherPDP(pdp.checked);
+  $("#date").on("change", async function (d) {
+    // On vérifie si la date est un jour travaillé
+    dateCouranteEstTravaillee();
+  });
+}
 
+main();
 
-    main();
 </script>
 
 {% endblock scripts %}
@@ -311,8 +537,6 @@
         - Sauvegarder
         - Afficher la photo de profil
         - Assiduité par défaut (aucune, present, retard, absent)
-        ? - Import Excel (fournie un fichier excel 
-            avec les étudiants et les périodes préremplis)
     --->
 
     <div id="actions" class="box">
@@ -330,12 +554,8 @@
             <option value="absent">Absence</option>
         </select>
         </label>
-
-        <label for="excel" class="wip">
-            Importer Excel :
-            <input type="file" name="excel" id="excel" accept=".xlsx, .xls, .csv">
-        </label>
-        <button class="wip" id="save wip">Sauvegarder l'assiduité</button>
+        
+        <button id="save" onclick="sauvegarderAssiduites()">Sauvegarder l'assiduité</button>
     </div>
 </div>
 
diff --git a/app/views/assiduites.py b/app/views/assiduites.py
index d77729fc9..347b140cf 100644
--- a/app/views/assiduites.py
+++ b/app/views/assiduites.py
@@ -1848,10 +1848,18 @@ def signal_assiduites_diff():
             grp + ' <span class="fontred">' + groups_infos.groups_titles + "</span>"
         )
 
+    moduleimpl_id = request.args.get("moduleimpl_id", -1)
+    try:
+        moduleimpl_id = int(moduleimpl_id)
+    except ValueError:
+        moduleimpl_id = -1
+
     return render_template(
         "assiduites/pages/signal_assiduites_diff.j2",
         etudiants=etudiants,
-        moduleimpl_select=_module_selector(formsemestre=formsemestre),
+        moduleimpl_select=_module_selector(
+            formsemestre=formsemestre, moduleimpl_id=moduleimpl_id
+        ),
         gr=gr_tit,
         nonworkdays=_non_work_days(),
         sco=ScoData(formsemestre=formsemestre),