Avertissement + sortie des tableaux des modules APC en formation classiques.

2023-12-15 01:07:08 +01:00 · 2023-12-15 01:07:08 +01:00 · 2e6ac8e60a
parent 170c7ce4b6
commit 2e6ac8e60a
6 changed files with 53 additions and 21 deletions
--- a/app/api/evaluations.py
+++ b/app/api/evaluations.py
@ -148,7 +148,7 @@ def evaluation_notes(evaluation_id: int):
@scodoc
@permission_required(Permission.EnsView)
@as_json
-def evaluation_set_notes(evaluation_id: int):
+def evaluation_set_notes(evaluation_id: int):  # evaluation-notes-set
    """Écriture de notes dans une évaluation.
    The request content type should be "application/json",
    and contains:
--- a/app/models/formsemestre.py
+++ b/app/models/formsemestre.py
@ -400,6 +400,7 @@ class FormSemestre(db.Model):
        """Liste des modimpls du semestre (y compris bonus)
        - triée par type/numéro/code en APC
        - triée par numéros d'UE/matières/modules pour les formations standard.
+        Hors APC, élimine les modules de type ressources et SAEs.
        """
        modimpls = self.modimpls.all()
        if self.formation.is_apc():
@ -411,6 +412,14 @@ class FormSemestre(db.Model):
                )
            )
        else:
+            modimpls = [
+                mi
+                for mi in modimpls
+                if (
+                    mi.module.module_type
+                    not in (scu.ModuleType.RESSOURCE, scu.ModuleType.SAE)
+                )
+            ]
            modimpls.sort(
                key=lambda m: (
                    m.module.ue.numero or 0,
--- a/app/scodoc/sco_edit_ue.py
+++ b/app/scodoc/sco_edit_ue.py
@ -117,7 +117,7 @@ def do_ue_create(args):
    ues = ue_list({"formation_id": args["formation_id"], "acronyme": args["acronyme"]})
    if ues:
        raise ScoValueError(
-            f"""Acronyme d'UE "{args['acronyme']}" déjà utilisé ! 
+            f"""Acronyme d'UE "{args['acronyme']}" déjà utilisé !
                (chaque UE doit avoir un acronyme unique dans la formation)"""
        )
    if (
@ -420,8 +420,8 @@ def ue_edit(ue_id=None, create=False, formation_id=None, default_semestre_idx=No
                "size": 12,
                "title": "Code UE",
                "max_length": SHORT_STR_LEN,
-                "explanation": """code interne (non vide). Toutes les UE partageant le même code 
-                (et le même code de formation) sont compatibles (compensation de semestres, capitalisation d'UE). 
+                "explanation": """code interne (non vide). Toutes les UE partageant le même code
+                (et le même code de formation) sont compatibles (compensation de semestres, capitalisation d'UE).
                Voir liste ci-dessous.""",
                "allow_null": False,
            },
@ -764,7 +764,7 @@ def ue_table(formation_id=None, semestre_idx=1, msg=""):  # was ue_list
        H.append(
            """<p class="help">Cette formation est verrouillée car
 des semestres verrouillés s'y réferent.
-Si vous souhaitez modifier cette formation (par exemple pour y ajouter un module), 
+Si vous souhaitez modifier cette formation (par exemple pour y ajouter un module),
 vous devez:
 </p>
 <ul class="help">
@ -827,7 +827,7 @@ du programme" (menu "Semestre") si vous avez un semestre en cours);
            msg_refcomp = "associer à un référentiel de compétences"
        else:
            descr_refcomp = f"""Référentiel de compétences:
-            <a href="{url_for('notes.refcomp_show', 
+            <a href="{url_for('notes.refcomp_show',
            scodoc_dept=g.scodoc_dept, refcomp_id=formation.referentiel_competence.id)}"
            class="stdlink">
            {formation.referentiel_competence.type_titre}
@ -856,7 +856,7 @@ du programme" (menu "Semestre") si vous avez un semestre en cours);
            )
            for parc in formation.referentiel_competence.parcours:
                H.append(
-                    f"""<div><a href="{url_for("notes.parcour_formation", 
+                    f"""<div><a href="{url_for("notes.parcour_formation",
                    scodoc_dept=g.scodoc_dept, formation_id=formation.id, parcour_id=parc.id )
                }">{parc.code}</a></div>"""
                )
@ -865,7 +865,7 @@ du programme" (menu "Semestre") si vous avez un semestre en cours);
        H.append(
            f"""
        <li> <a class="stdlink" href="{
-            url_for('notes.edit_modules_ue_coefs', 
+            url_for('notes.edit_modules_ue_coefs',
                scodoc_dept=g.scodoc_dept, formation_id=formation_id, semestre_idx=semestre_idx)
        }">{'Visualiser' if locked else 'Éditer'} les coefficients des ressources et SAÉs</a>
        </li>
@ -915,7 +915,7 @@ du programme" (menu "Semestre") si vous avez un semestre en cours);
                    }">Ajouter une UE</a>
                </li>
                <li><a href="{
-                    url_for('notes.formation_add_malus_modules', 
+                    url_for('notes.formation_add_malus_modules',
                    scodoc_dept=g.scodoc_dept, formation_id=formation_id)
                }" class="stdlink">Ajouter des modules de malus dans chaque UE</a>
                </li>
@ -956,14 +956,14 @@ du programme" (menu "Semestre") si vous avez un semestre en cours);
            )
            }">Créer une nouvelle version de la formation</a> (copie non verrouillée)
            </li>
-            
+
        """
        )
        if not len(formsemestres):
            H.append(
                f"""
            <li><a class="stdlink" href="{
-            url_for('notes.formation_delete', 
+            url_for('notes.formation_delete',
            scodoc_dept=g.scodoc_dept, formation_id=formation_id
            )
            }">Supprimer cette formation</a> (pas encore utilisée par des semestres)
@ -976,7 +976,7 @@ du programme" (menu "Semestre") si vous avez un semestre en cours);
            url_for('notes.formation_table_recap', scodoc_dept=g.scodoc_dept,
                formation_id=formation_id)
            }">Table récapitulative de la formation</a>
-        </li>    
+        </li>
        <li><a class="stdlink" href="{
            url_for('notes.formation_export', scodoc_dept=g.scodoc_dept,
                formation_id=formation_id, fmt='xml')
@ -1050,9 +1050,9 @@ def _html_select_semestre_idx(formation_id, semestre_ids, semestre_idx):
        else:
            label = f"S{i}"
        htm += f"""<option value="{i}" {
-            'selected' 
-            if (semestre_idx == i) 
-            or (i == "all" and semestre_idx is None) 
+            'selected'
+            if (semestre_idx == i)
+            or (i == "all" and semestre_idx is None)
            else ''
            }>{label}</option>
        """
@ -1172,7 +1172,7 @@ def _ue_table_ues(
        )
        if (iue >= len(ues) - 1) or ue["semestre_id"] != ues[iue + 1]["semestre_id"]:
            H.append(
-                f"""</ul><ul><li><a href="{url_for('notes.ue_create', scodoc_dept=g.scodoc_dept, 
+                f"""</ul><ul><li><a href="{url_for('notes.ue_create', scodoc_dept=g.scodoc_dept,
                formation_id=ue['formation_id'], semestre_idx=ue['semestre_id'])
            }">Ajouter une UE dans le semestre {ue['semestre_id'] or ''}</a></li></ul>
            </div>
@ -1205,7 +1205,7 @@ def _ue_table_matieres(
            if editable and not sco_edit_matiere.matiere_is_locked(mat["matiere_id"]):
                H.append(
                    f"""<a class="stdlink" href="{
-                        url_for("notes.matiere_edit", 
+                        url_for("notes.matiere_edit",
                        scodoc_dept=g.scodoc_dept, matiere_id=mat["matiere_id"])
                        }">
                    """
@ -1314,6 +1314,10 @@ def _ue_table_modules(
                '<a class="discretelink" title="Modifier le module numéro %(numero)s, utilisé par %(nb_moduleimpls)d sessions" href="module_edit?module_id=%(module_id)s">'
                % mod
            )
+        if mod["module_type"] not in (scu.ModuleType.STANDARD, scu.ModuleType.MALUS):
+            H.append(
+                f"""<span class="invalid-module-type">{scu.EMO_WARNING} type incompatible </span>"""
+            )
        H.append(
            '<span class="formation_module_tit">%s</span>'
            % scu.join_words(mod["code"], mod["titre"])
@ -1358,7 +1362,7 @@ def _ue_table_modules(
        if editable and add_suppress_link:
            H.append(
                f"""<a class="stdlink" href="{
-                    url_for("notes.matiere_delete", 
+                    url_for("notes.matiere_delete",
                    scodoc_dept=g.scodoc_dept, matiere_id=mat["matiere_id"])}"
                >la supprimer</a>
                """
@ -1367,7 +1371,7 @@ def _ue_table_modules(
    if editable:  # and ((not parcours.UE_IS_MODULE) or len(Modlist) == 0):
        H.append(
            f"""<li> <a class="stdlink" href="{
-                    url_for("notes.module_create", 
+                    url_for("notes.module_create",
                    scodoc_dept=g.scodoc_dept, matiere_id=mat["matiere_id"])}"
            >{create_element_msg}</a></li>
            """
@ -1451,7 +1455,7 @@ def do_ue_edit(args, bypass_lock=False, dont_invalidate_cache=False):
        ues = ue_list({"formation_id": ue["formation_id"], "acronyme": new_acro})
        if ues and ues[0]["ue_id"] != ue_id:
            raise ScoValueError(
-                f"""Acronyme d'UE "{args['acronyme']}" déjà utilisé ! 
+                f"""Acronyme d'UE "{args['acronyme']}" déjà utilisé !
                (chaque UE doit avoir un acronyme unique dans la formation.)"""
            )
    # On ne peut pas supprimer le code UE:
--- a/app/scodoc/sco_formsemestre_edit.py
+++ b/app/scodoc/sco_formsemestre_edit.py
@ -257,6 +257,13 @@ def do_formsemestre_createwithmodules(edit=False, formsemestre: FormSemestre = N
            .order_by(Module.module_type, UniteEns.numero, Module.numero)
            .all()
        )
+        # Elimine les ressources et SAE sauf si déjà dans le semestre
+        modules = [
+            m
+            for m in modules
+            if (m.module_type not in (scu.ModuleType.RESSOURCE, scu.ModuleType.SAE))
+            or m.id in module_ids_set
+        ]
    # Pour regroupement des modules par semestres:
    semestre_ids = {}
    for mod in modules:
--- a/app/scodoc/sco_formsemestre_status.py
+++ b/app/scodoc/sco_formsemestre_status.py
@ -1176,11 +1176,18 @@ def formsemestre_status(formsemestre_id=None, check_parcours=True):
        H += [_TABLEAU_MODULES_FOOT, "</div>"]
    else:
        # formations classiques: groupe par UE
+        # élimine les modules BUT qui aurait pu se glisser là suite à un
+        # changement de type de formation par exemple
+        modimpls_classic = [
+            m
+            for m in modimpls
+            if m["module"]["module_type"] not in (ModuleType.RESSOURCE, ModuleType.SAE)
+        ]
        H += [
            "<p>",
            _TABLEAU_MODULES_HEAD,
            formsemestre_tableau_modules(
-                modimpls,
+                modimpls_classic,
                nt,
                formsemestre,
                can_edit=can_edit,
--- a/app/static/css/scodoc.css
+++ b/app/static/css/scodoc.css
@ -2464,6 +2464,11 @@ li.module_malus span.formation_module_tit {
  text-decoration: underline;
 }

+span.invalid-module-type {
+  color: red;
+  font-style: italic;
+}
+
 span.formation_module_ue {
  color: #6e7d92;
  font-size: 75%;