From 90bf31fc03c3b6a8772c8d134a941390da443bdf Mon Sep 17 00:00:00 2001 From: Emmanuel Viennet Date: Tue, 30 Jan 2024 22:12:55 +0100 Subject: [PATCH] Tag module formation selon leur type --- app/models/modules.py | 23 ++++++- app/scodoc/sco_edit_ue.py | 13 +++- app/scodoc/sco_formation_recap.py | 3 + app/scodoc/sco_tag_module.py | 96 ++++++++---------------------- app/static/js/module_tag_editor.js | 39 +++++++++--- app/views/notes.py | 23 +++++++ 6 files changed, 117 insertions(+), 80 deletions(-) diff --git a/app/models/modules.py b/app/models/modules.py index 430835a9..2c3a9131 100644 --- a/app/models/modules.py +++ b/app/models/modules.py @@ -1,6 +1,6 @@ """ScoDoc 9 models : Modules """ -from flask import current_app +from flask import current_app, g from app import db from app.models import APO_CODE_STR_LEN @@ -310,6 +310,14 @@ class Module(db.Model): return [] return self.parcours + def add_tag(self, tag: "NotesTag"): + """Add tag to module. Check if already has it.""" + if tag.id in {t.id for t in self.tags}: + return + self.tags.append(tag) + db.session.add(self) + db.session.flush() + class ModuleUECoef(db.Model): """Coefficients des modules vers les UE (APC, BUT) @@ -372,6 +380,19 @@ class NotesTag(db.Model): dept_id = db.Column(db.Integer, db.ForeignKey("departement.id"), index=True) title = db.Column(db.Text(), nullable=False) + @classmethod + def get_or_create(cls, title: str, dept_id: int | None = None) -> "NotesTag": + """Get tag, or create it if it doesn't yet exists. + If dept_id unspecified, use current dept. + """ + dept_id = dept_id if dept_id is not None else g.scodoc_dept_id + tag = NotesTag.query.filter_by(dept_id=dept_id, title=title).first() + if tag is None: + tag = NotesTag(dept_id=dept_id, title=title) + db.session.add(tag) + db.session.flush() + return tag + # Association tag <-> module notes_modules_tags = db.Table( diff --git a/app/scodoc/sco_edit_ue.py b/app/scodoc/sco_edit_ue.py index f33eeacd..dd95dfd3 100644 --- a/app/scodoc/sco_edit_ue.py +++ b/app/scodoc/sco_edit_ue.py @@ -670,6 +670,7 @@ def ue_table(formation_id=None, semestre_idx=1, msg=""): # was ue_list semestre_idx = None else: semestre_idx = int(semestre_idx) + show_tags = scu.to_bool(request.args.get("show_tags", 0)) locked = formation.has_locked_sems(semestre_idx) semestre_ids = range(1, parcours.NB_SEM + 1) # transition: on requete ici via l'ORM mais on utilise les fonctions ScoDoc7 @@ -875,11 +876,13 @@ du programme" (menu "Semestre") si vous avez un semestre en cours); ) # Description des UE/matières/modules H.append( - """ + f"""
Programme pédagogique:
- montrer les tags des modules + montrer les tags des modules
""" ) @@ -978,6 +981,11 @@ du programme" (menu "Semestre") si vous avez un semestre en cours); formation_id=formation_id) }">Table récapitulative de la formation +
  • Tagguer tous les modules par leur type (tag res, sae). +
  • + """ return htm diff --git a/app/scodoc/sco_formation_recap.py b/app/scodoc/sco_formation_recap.py index fbd72b54..3b5a923e 100644 --- a/app/scodoc/sco_formation_recap.py +++ b/app/scodoc/sco_formation_recap.py @@ -103,6 +103,7 @@ def formation_table_recap(formation_id, fmt="html") -> Response: "heures_cours": mod.heures_cours, "heures_td": mod.heures_td, "heures_tp": mod.heures_tp, + "tags": ", ".join(t.title for t in mod.tags if t.title), "_css_row_class": f"mod {mod.type_abbrv()}", } ) @@ -117,6 +118,7 @@ def formation_table_recap(formation_id, fmt="html") -> Response: "heures_cours", "heures_td", "heures_tp", + "tags", ] if not formation.is_apc(): columns_ids.insert(columns_ids.index("ects"), "coef") @@ -132,6 +134,7 @@ def formation_table_recap(formation_id, fmt="html") -> Response: "heures_cours": "Cours (h)", "heures_td": "TD (h)", "heures_tp": "TP (h)", + "tags": "Tags", "ects": "ECTS", } diff --git a/app/scodoc/sco_tag_module.py b/app/scodoc/sco_tag_module.py index 3c62fd55..e5f6a5d8 100644 --- a/app/scodoc/sco_tag_module.py +++ b/app/scodoc/sco_tag_module.py @@ -38,17 +38,14 @@ import re from flask import g -from app.comp import res_sem -from app.comp.res_compat import NotesTableCompat -from app.models import FormSemestre +from app import db, log +from app.models import Formation, NotesTag +from app.scodoc import sco_edit_module import app.scodoc.sco_utils as scu import app.scodoc.notesdb as ndb -from app import log -from app.scodoc import sco_edit_module -from app.scodoc import sco_etud from app.scodoc.sco_exceptions import ScoValueError -# Opérations à implementer: +# Opérations implementées: # + liste des modules des formations de code donné (formation_code) avec ce tag # + liste de tous les noms de tag # + tag pour un nom @@ -62,6 +59,7 @@ from app.scodoc.sco_exceptions import ScoValueError # module_tag_set( module_id, taglist ) -> modifie les tags +# NOTA: ancien code, n'utile pas de modèles SQLAlchemy class ScoTag(object): """Generic tags for ScoDoc""" @@ -232,7 +230,7 @@ def module_tag_search(term: str | int): return scu.sendJSON(data) -def module_tag_list(module_id=""): +def module_tag_list(module_id="") -> list[str]: """les noms de tags associés à ce module""" r = ndb.SimpleDictFetch( """SELECT t.title @@ -249,6 +247,7 @@ def module_tag_set(module_id="", taglist=None): """taglist may either be: a string with tag names separated by commas ("un,deux") or a list of strings (["un", "deux"]) + Remplace les tags existants """ if not taglist: taglist = [] @@ -284,34 +283,6 @@ def module_tag_set(module_id="", taglist=None): return "", http.HTTPStatus.NO_CONTENT -def get_etud_tagged_modules(etudid, tagname): - """Liste d'infos sur les modules de ce semestre avec ce tag. - Cherche dans tous les semestres dans lesquel l'étudiant est ou a été inscrit. - Construit la liste des modules avec le tag donné par tagname - """ - etud = sco_etud.get_etud_info(etudid=etudid, filled=True)[0] - R = [] - for sem in etud["sems"]: - formsemestre = FormSemestre.query.get_or_404(sem["formsemestre_id"]) - nt: NotesTableCompat = res_sem.load_formsemestre_results(formsemestre) - modimpls = nt.get_modimpls_dict() - for modimpl in modimpls: - tags = module_tag_list(module_id=modimpl["module_id"]) - if tagname in tags: - moy = nt.get_etud_mod_moy( - modimpl["moduleimpl_id"], etudid - ) # ou NI si non inscrit - R.append( - { - "sem": sem, - "moy": moy, # valeur réelle, ou NI (non inscrit au module ou NA (pas de note) - "moduleimpl": modimpl, - "tags": tags, - } - ) - return R - - def split_tagname_coeff(tag: str, separateur=":") -> tuple[str, float]: """Découpage d'un tag, tel que saisi par un utilisateur dans le programme, pour en extraire : @@ -335,42 +306,27 @@ def split_tagname_coeff(tag: str, separateur=":") -> tuple[str, float]: try: pond = float(temp[1]) return (temp[0], pond) - except: - """Renvoie tout le tag si le découpage à échouer""" + except (IndexError, ValueError, TypeError): + # Renvoie tout le tag si le découpage a échoué return (tag, 1.0) else: - """initialise le coeff de pondération à 1 lorsqu'aucun coeff de pondération n'est indiqué dans le tag""" + # initialise le coeff de pondération à 1 lorsqu'aucun coeff de pondération + # n'ait indiqué dans le tag return (tag, 1.0) -"""Tests: -from debug import * -from app.scodoc.sco_tag_module import * -_ = go_dept(app, 'RT').Notes - -t = ModuleTag( 'essai') -t.tag_module('totoro') # error (module invalide) -t.tag_module('MOD21460') -t.delete() # detruit tag et assoc -t = ModuleTag( 'essai2') -t.tag_module('MOD21460') -t.tag_module('MOD21464') -t.list_modules() -t.list_modules(formation_code='ccc') # empty list -t.list_modules(formation_code='FCOD2') - - -Un essai de get_etud_tagged_modules: -from debug import * -from app.scodoc.sco_tag_module import * -_ = go_dept(app, 'GEA').Notes - -etudid='GEAEID80687' -etud = sco_etud.get_etud_info( etudid=etudid, filled=True)[0] -sem = etud['sems'][0] - -[ tm['moy'] for tm in get_etud_tagged_modules( etudid, 'allo') ] - -# si besoin après modif par le Web: -# sco_cache.invalidate_formsemestre() -""" +def formation_tag_modules_by_type(formation: Formation): + """Taggue tous les modules de la formation en fonction de leur type : 'res', 'sae', 'malus' + Ne taggue pas les modules standards. + """ + tag_titles = { + m.type_abbrv() for m in formation.modules + } # usually {'res', 'mod', 'sae'} + tag_by_type = { + tag_title: NotesTag.get_or_create(title=tag_title, dept_id=formation.dept_id) + for tag_title in tag_titles + } + for module in formation.modules: + if module.module_type != scu.ModuleType.STANDARD: + module.add_tag(tag_by_type[module.type_abbrv()]) + db.session.commit() diff --git a/app/static/js/module_tag_editor.js b/app/static/js/module_tag_editor.js index e7927fa7..89484c26 100644 --- a/app/static/js/module_tag_editor.js +++ b/app/static/js/module_tag_editor.js @@ -23,11 +23,36 @@ $(function () { // version readonly readOnlyTags($(".module_tag_editor_ro")); - $(".sco_tag_checkbox").click(function () { - if ($(this).is(":checked")) { - $(".sco_tag_edit").show(); - } else { - $(".sco_tag_edit").hide(); - } - }); + // $(".sco_tag_checkbox").click(function () { + // if ($(this).is(":checked")) { + // $(".sco_tag_edit").show(); + // } else { + // $(".sco_tag_edit").hide(); + // } + // }); }); + +// tags +function toggleEditDisplay(checkbox) { + const isChecked = checkbox.checked; + document.querySelectorAll('.sco_tag_edit').forEach(el => { + el.style.display = isChecked ? 'block' : 'none'; + }); + // form semection de semestres: + const showTagsInput = document.querySelector('input[name="show_tags"]'); + if (showTagsInput) { + showTagsInput.value = isChecked ? '1' : '0'; + } +} + +document.addEventListener('DOMContentLoaded', function() { + document.querySelectorAll('.sco_tag_checkbox').forEach(checkbox => { + // Set up initial state for each checkbox + toggleEditDisplay(checkbox); + + // Add click event listener to each checkbox + checkbox.addEventListener('click', function() { + toggleEditDisplay(this); + }); + }); +}); \ No newline at end of file diff --git a/app/views/notes.py b/app/views/notes.py index b29fd965..0572353b 100644 --- a/app/views/notes.py +++ b/app/views/notes.py @@ -623,6 +623,29 @@ sco_publish("/module_list", sco_edit_module.module_table, Permission.ScoView) sco_publish("/module_tag_search", sco_tag_module.module_tag_search, Permission.ScoView) +@bp.route("/formation_tag_modules_by_type//") +@scodoc +@permission_required(Permission.EditFormationTags) +def formation_tag_modules_by_type(formation_id: int, semestre_idx: int): + """Taggue tous les modules de la formation en fonction de leur type : 'res', 'sae', 'malus' + Ne taggue pas les modules standards. + """ + formation = Formation.query.filter_by( + id=formation_id, dept_id=g.scodoc_dept_id + ).first_or_404() + sco_tag_module.formation_tag_modules_by_type(formation) + flash("Formation tagguée") + return flask.redirect( + url_for( + "notes.ue_table", + scodoc_dept=g.scodoc_dept, + semestre_idx=semestre_idx, + formation_id=formation.id, + show_tags=1, + ) + ) + + @bp.route("/module_tag_set", methods=["POST"]) @scodoc @permission_required(Permission.EditFormationTags)