From bb331e31c135be5d8048a22ab1936c685126c7de Mon Sep 17 00:00:00 2001
From: Emmanuel Viennet
Date: Wed, 8 Nov 2023 23:19:58 +0100
Subject: [PATCH] =?UTF-8?q?Am=C3=A9liore=20form=20config=20assiduite=20+?=
=?UTF-8?q?=20edt?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
---
app/forms/main/config_assiduites.py | 44 +++++++++++++--
app/forms/main/config_cas.py | 2 +
app/forms/main/config_personalized_links.py | 6 +-
app/models/moduleimpls.py | 3 -
app/scodoc/sco_edit_module.py | 11 ++--
app/scodoc/sco_formsemestre.py | 3 +-
app/scodoc/sco_formsemestre_edit.py | 55 ++++++++++++-------
app/scodoc/sco_groups.py | 4 +-
app/static/js/assiduites.js | 18 +++---
.../assiduites/pages/config_assiduites.j2 | 44 +++++++++------
app/templates/configuration.j2 | 4 +-
app/views/scodoc.py | 25 ++++-----
12 files changed, 138 insertions(+), 81 deletions(-)
diff --git a/app/forms/main/config_assiduites.py b/app/forms/main/config_assiduites.py
index ba7198b82..da62b8b4d 100644
--- a/app/forms/main/config_assiduites.py
+++ b/app/forms/main/config_assiduites.py
@@ -28,12 +28,14 @@
"""
Formulaire configuration Module Assiduités
"""
+import datetime
from flask_wtf import FlaskForm
-from wtforms import SubmitField, DecimalField
+from wtforms import DecimalField, SubmitField, ValidationError
from wtforms.fields.simple import StringField
+from wtforms.validators import Optional
+
from wtforms.widgets import TimeInput
-import datetime
class TimeField(StringField):
@@ -72,9 +74,28 @@ class TimeField(StringField):
else:
raise ValueError
self.data = datetime.time(hour, minutes, seconds)
- except ValueError:
+ except ValueError as exc:
self.data = None
- raise ValueError(self.gettext("Not a valid time string"))
+ raise ValueError(self.gettext("Not a valid time string")) from exc
+
+
+def check_tick_time(form, field):
+ """Le tick_time doit être entre 0 et 60 minutes"""
+ if field.data < 1 or field.data > 59:
+ raise ValidationError("Valeur de granularité invalide (entre 1 et 59)")
+
+
+def check_ics_path(form, field):
+ """Vérifie que le chemin est bien un chemin absolu
+ et qu'il contient edt_id
+ """
+ data = field.data.strip()
+ if not data:
+ return
+ if not data.startswith("/"):
+ raise ValidationError("Le chemin vers les ics doit commencer par /")
+ if not "{edt_id}" in data:
+ raise ValidationError("Le chemin vers les ics doit utiliser {edt_id}")
class ConfigAssiduitesForm(FlaskForm):
@@ -84,7 +105,20 @@ class ConfigAssiduitesForm(FlaskForm):
lunch_time = TimeField("Heure de midi (date pivot entre Matin et Après Midi)")
afternoon_time = TimeField("Fin de la journée")
- tick_time = DecimalField("Granularité de la Time Line (temps en minutes)", places=0)
+ tick_time = DecimalField(
+ "Granularité de la timeline (temps en minutes)",
+ places=0,
+ validators=[check_tick_time],
+ )
+
+ edt_ics_path = StringField(
+ label="Chemin vers les ics",
+ description="""Chemin absolu unix sur le serveur vers le fichier ics donnant l'emploi
+ du temps d'un semestre. La balise {edt_id} sera remplacée par l'edt_id du
+ semestre (par défaut, son code étape Apogée).
+ Si ce champ n'est pas renseigné, les emplois du temps ne seront pas utilisés.""",
+ validators=[Optional(), check_ics_path],
+ )
submit = SubmitField("Valider")
cancel = SubmitField("Annuler", render_kw={"formnovalidate": True})
diff --git a/app/forms/main/config_cas.py b/app/forms/main/config_cas.py
index 978b84cc8..3a31b47fe 100644
--- a/app/forms/main/config_cas.py
+++ b/app/forms/main/config_cas.py
@@ -42,11 +42,13 @@ def check_cas_uid_from_mail_regexp(form, field):
if not ScoDocSiteConfig.cas_uid_from_mail_regexp_is_valid(field.data):
raise ValidationError("expression régulière invalide")
+
def check_cas_edt_id_from_xml_regexp(form, field):
"Vérifie la regexp fournie pour l'extraction du CAS id"
if not ScoDocSiteConfig.cas_edt_id_from_xml_regexp_is_valid(field.data):
raise ValidationError("expression régulière pour edt_id invalide")
+
class ConfigCASForm(FlaskForm):
"Formulaire paramétrage CAS"
cas_enable = BooleanField("Activer le CAS")
diff --git a/app/forms/main/config_personalized_links.py b/app/forms/main/config_personalized_links.py
index 1aed31302..b2293a2c2 100644
--- a/app/forms/main/config_personalized_links.py
+++ b/app/forms/main/config_personalized_links.py
@@ -29,7 +29,7 @@ def PersonalizedLinksForm() -> _PersonalizedLinksForm:
F,
f"link_{idx}",
StringField(
- f"Titre",
+ "Titre",
validators=[
validators.Optional(),
validators.Length(min=1, max=80),
@@ -42,7 +42,7 @@ def PersonalizedLinksForm() -> _PersonalizedLinksForm:
F,
f"link_url_{idx}",
StringField(
- f"URL",
+ "URL",
description="adresse, incluant le http.",
validators=[
validators.Optional(),
@@ -56,7 +56,7 @@ def PersonalizedLinksForm() -> _PersonalizedLinksForm:
F,
f"link_with_args_{idx}",
BooleanField(
- f"ajouter arguments",
+ "ajouter arguments",
description="query string avec ids",
),
)
diff --git a/app/models/moduleimpls.py b/app/models/moduleimpls.py
index 3ccff4cf8..75672c1b9 100644
--- a/app/models/moduleimpls.py
+++ b/app/models/moduleimpls.py
@@ -42,9 +42,6 @@ class ModuleImpl(db.Model):
viewonly=True,
)
- def __init__(self, **kwargs):
- super(ModuleImpl, self).__init__(**kwargs)
-
def __repr__(self):
return f"<{self.__class__.__name__} {self.id} module={repr(self.module)}>"
diff --git a/app/scodoc/sco_edit_module.py b/app/scodoc/sco_edit_module.py
index ea0ca9fb7..4f5b9235e 100644
--- a/app/scodoc/sco_edit_module.py
+++ b/app/scodoc/sco_edit_module.py
@@ -74,7 +74,8 @@ _moduleEditor = ndb.EditableTable(
"semestre_id",
"numero",
"code_apogee",
- "module_type"
+ "module_type",
+ "edt_id",
#'ects'
),
sortkey="numero, code, titre",
@@ -171,7 +172,7 @@ def do_module_delete(oid):
d'en créer une nouvelle version pour la modifier sans affecter
les semestres déjà en place.
- reprendre
"""
raise ScoGenError(err_page)
@@ -645,7 +646,7 @@ def module_edit(
"title": "Code Apogée",
"size": 25,
"explanation": """(optionnel) code élément pédagogique Apogée ou liste de codes ELP
- séparés par des virgules (ce code est propre à chaque établissement, se rapprocher
+ séparés par des virgules (ce code est propre à chaque établissement, se rapprocher
du référent Apogée).
""",
"validator": lambda val, _: len(val) < APO_CODE_STR_LEN,
@@ -682,7 +683,7 @@ def module_edit(
]
+ ["-1"],
"explanation": """Parcours dans lesquels est utilisé ce module.
- Attention: si le module ne doit pas avoir les mêmes coefficients suivant le parcours,
+ Attention: si le module ne doit pas avoir les mêmes coefficients suivant le parcours,
il faut en créer plusieurs versions, car dans ScoDoc chaque module a ses coefficients.""",
},
)
@@ -746,7 +747,7 @@ def module_edit(
"input_type": "separator",
"title": f"""{scu.EMO_WARNING }
Pas de parcours:
- associer un référentiel de compétence""",
diff --git a/app/scodoc/sco_formsemestre.py b/app/scodoc/sco_formsemestre.py
index 8c77ed6a3..d01c7af82 100644
--- a/app/scodoc/sco_formsemestre.py
+++ b/app/scodoc/sco_formsemestre.py
@@ -68,6 +68,7 @@ _formsemestreEditor = ndb.EditableTable(
"ens_can_edit_eval",
"elt_sem_apo",
"elt_annee_apo",
+ "edt_id",
),
filter_dept=True,
sortkey="date_debut",
@@ -571,7 +572,7 @@ def view_formsemestre_by_etape(etape_apo=None, fmt="html"):
),
html_title=html_title,
html_next_section="""""",
)
tab.base_url = "%s?etape_apo=%s" % (request.base_url, etape_apo or "")
diff --git a/app/scodoc/sco_formsemestre_edit.py b/app/scodoc/sco_formsemestre_edit.py
index 1aedb43e7..4f5ae61de 100644
--- a/app/scodoc/sco_formsemestre_edit.py
+++ b/app/scodoc/sco_formsemestre_edit.py
@@ -40,6 +40,7 @@ from app.models import (
ModuleImpl,
Evaluation,
UniteEns,
+ ScoDocSiteConfig,
ScolarFormSemestreValidation,
ScolarAutorisationInscription,
ApcValidationAnnee,
@@ -321,7 +322,7 @@ def do_formsemestre_createwithmodules(edit=False, formsemestre: FormSemestre = N
"size": 40,
"title": "Nom de ce semestre",
"explanation": f"""n'indiquez pas les dates, ni le semestre, ni la modalité dans
- le titre: ils seront automatiquement ajoutés """,
"allow_null": False,
@@ -445,13 +446,25 @@ def do_formsemestre_createwithmodules(edit=False, formsemestre: FormSemestre = N
},
)
)
+ if ScoDocSiteConfig.get("edt_ics_path"):
+ modform.append(
+ (
+ "edt_id",
+ {
+ "size": 32,
+ "title": "Identifiant EDT",
+ "explanation": "optionnel, identifiant sur le logiciel emploi du temps (par défaut, utilise la première étape Apogée).",
+ "allow_null": True,
+ },
+ )
+ )
if edit:
formtit = f"""
-
Sélectionner les modules, leurs responsables et les étudiants
+
Sélectionner les modules, leurs responsables et les étudiants
à inscrire:
"""
else:
@@ -510,7 +523,7 @@ def do_formsemestre_createwithmodules(edit=False, formsemestre: FormSemestre = N
{
"input_type": "boolcheckbox",
"title": "",
- "explanation": """Autoriser tous les enseignants associés
+ "explanation": """Autoriser tous les enseignants associés
à un module à y créer des évaluations""",
},
),
@@ -585,8 +598,8 @@ def do_formsemestre_createwithmodules(edit=False, formsemestre: FormSemestre = N
{
"input_type": "separator",
"title": f"""{scu.EMO_WARNING }
- Pas de parcours:
- vérifier la formation""",
@@ -784,7 +797,7 @@ def do_formsemestre_createwithmodules(edit=False, formsemestre: FormSemestre = N
if tf[0] == 0 or msg:
return f"""
"
}
retour au tableau de bord
"""
@@ -1309,11 +1322,11 @@ def formsemestre_delete(formsemestre_id):
html_sco_header.html_sem_header("Suppression du semestre"),
"""
Attention !
A n'utiliser qu'en cas d'erreur lors de la saisie d'une formation. Normalement,
-un semestre ne doit jamais être supprimé
+un semestre ne doit jamais être supprimé
(on perd la mémoire des notes et de tous les événements liés à ce semestre !).
-
Tous les modules de ce semestre seront supprimés.
+
Tous les modules de ce semestre seront supprimés.
Ceci n'est possible que si :
@@ -1497,24 +1510,24 @@ def do_formsemestre_delete(formsemestre_id):
req = "DELETE FROM sco_prefs WHERE formsemestre_id=%(formsemestre_id)s"
cursor.execute(req, {"formsemestre_id": formsemestre_id})
# --- Suppression des groupes et partitions
- req = """DELETE FROM group_membership
- WHERE group_id IN
+ req = """DELETE FROM group_membership
+ WHERE group_id IN
(SELECT gm.group_id FROM group_membership gm, partition p, group_descr gd
- WHERE gm.group_id = gd.id AND gd.partition_id = p.id
+ WHERE gm.group_id = gd.id AND gd.partition_id = p.id
AND p.formsemestre_id=%(formsemestre_id)s)
"""
cursor.execute(req, {"formsemestre_id": formsemestre_id})
- req = """DELETE FROM group_descr
- WHERE id IN
- (SELECT gd.id FROM group_descr gd, partition p
- WHERE gd.partition_id = p.id
+ req = """DELETE FROM group_descr
+ WHERE id IN
+ (SELECT gd.id FROM group_descr gd, partition p
+ WHERE gd.partition_id = p.id
AND p.formsemestre_id=%(formsemestre_id)s)
"""
cursor.execute(req, {"formsemestre_id": formsemestre_id})
req = "DELETE FROM partition WHERE formsemestre_id=%(formsemestre_id)s"
cursor.execute(req, {"formsemestre_id": formsemestre_id})
# --- Responsables
- req = """DELETE FROM notes_formsemestre_responsables
+ req = """DELETE FROM notes_formsemestre_responsables
WHERE formsemestre_id=%(formsemestre_id)s"""
cursor.execute(req, {"formsemestre_id": formsemestre_id})
# --- Etapes
@@ -1606,7 +1619,7 @@ def formsemestre_edit_uecoefs(formsemestre_id, err_ue_id=None):
Indiquez "auto" (ou laisser vide) pour que ScoDoc calcule automatiquement le coefficient,
ou bien entrez une valeur (nombre réel).
-
Dans le doute, si le mode auto n'est pas applicable et que tous les étudiants sont inscrits aux mêmes modules de ce semestre, prenez comme coefficient la somme indiquée.
+
Dans le doute, si le mode auto n'est pas applicable et que tous les étudiants sont inscrits aux mêmes modules de ce semestre, prenez comme coefficient la somme indiquée.
Sinon, référez vous au programme pédagogique. Les lignes en rouge
sont à changer.
@@ -1734,7 +1747,7 @@ def formsemestre_edit_uecoefs(formsemestre_id, err_ue_id=None):
return f"""{html_sco_header.html_sem_header("Coefficients des UE du semestre")}
{" ".join(message)}
-
@@ -1347,12 +1347,12 @@ function generateEtudRow(
-
-
+
+
`;
return HTML;
diff --git a/app/templates/assiduites/pages/config_assiduites.j2 b/app/templates/assiduites/pages/config_assiduites.j2
index 05c6a4390..eb6bec567 100644
--- a/app/templates/assiduites/pages/config_assiduites.j2
+++ b/app/templates/assiduites/pages/config_assiduites.j2
@@ -13,26 +13,36 @@ affectent notamment les comptages d'absences de tous les bulletins des
-
-
+
+
+
Emplois du temps
+
ScoDoc peut récupérer les emplois du temps de chaque session.