@@ -50,7 +53,7 @@
@@ -95,7 +98,17 @@
}
{% endblock %}
+
+{% macro render_field_errors(form, field_name) %}
+ {% if form[field_name].errors %}
+
+ {% for error in form[field_name].errors %}
+ {{ error }}
+ {% endfor %}
+
+ {% endif %}
+{% endmacro %}
diff --git a/app/templates/sidebar.j2 b/app/templates/sidebar.j2
index 95bb93cd67..73050751af 100755
--- a/app/templates/sidebar.j2
+++ b/app/templates/sidebar.j2
@@ -62,20 +62,20 @@
{% endif %}
{% endif %}
@@ -83,7 +83,7 @@
{# LOGO #}
-
\ No newline at end of file
+
diff --git a/app/views/assiduites.py b/app/views/assiduites.py
index b08c2ebf6b..ba5aa61831 100644
--- a/app/views/assiduites.py
+++ b/app/views/assiduites.py
@@ -32,13 +32,14 @@ from flask import abort, url_for, redirect, Response
from flask_login import current_user
from app import db
-
+from app.api.assiduites import create_one_assiduite
from app.comp import res_sem
from app.comp.res_compat import NotesTableCompat
from app.decorators import (
scodoc,
permission_required,
)
+from app.forms.assiduite.ajout_assiduite_etud import AjoutAssiduiteEtudForm
from app.models import (
FormSemestre,
Identite,
@@ -48,6 +49,7 @@ from app.models import (
Departement,
Evaluation,
)
+from app.scodoc.codes_cursus import UE_STANDARD
from app.auth.models import User
from app.models.assiduites import get_assiduites_justif, compute_assiduites_justified
import app.tables.liste_assiduites as liste_assi
@@ -246,19 +248,19 @@ def bilan_dept():
return "\n".join(H)
-@bp.route("/SignaleAssiduiteEtud")
+@bp.route("/ajout_assiduite_etud", methods=["GEt", "POST"])
@scodoc
@permission_required(Permission.AbsChange)
-def signal_assiduites_etud():
+def ajout_assiduite_etud():
"""
- signal_assiduites_etud Saisie de l'assiduité d'un étudiant
+ ajout_assiduite_etud Saisie d'une assiduité d'un étudiant
Args:
etudid (int): l'identifiant de l'étudiant
date_deb, date_fin: heures début et fin (ISO sans timezone)
moduleimpl_id
- evaluation_id
- saisie_eval : si présent, mode "évaluation"
+ evaluation_id : si présent, mode "évaluation"
+ fmt: si xls, renvoie le tableau des assiduités enregistrées
Returns:
str: l'html généré
"""
@@ -266,11 +268,13 @@ def signal_assiduites_etud():
etud = Identite.get_etud(etudid)
# Gestion évaluations (appel à la page depuis les évaluations)
- saisie_eval: bool = request.args.get("saisie_eval") is not None
+ evaluation_id: int = request.args.get("evaluation_id")
+ saisie_eval = evaluation_id is not None
+
date_deb: str = request.args.get("date_deb")
date_fin: str = request.args.get("date_fin")
moduleimpl_id: int = request.args.get("moduleimpl_id", "")
- evaluation_id: int = request.args.get("evaluation_id")
+
redirect_url: str = (
"#"
if not saisie_eval
@@ -281,21 +285,32 @@ def signal_assiduites_etud():
)
)
- # Préparation de la page (Header)
- header: str = html_sco_header.sco_header(
- page_title="Saisie assiduité",
- init_qtip=True,
- javascripts=[
- "js/assiduites.js",
- "js/date_utils.js",
- "js/etud_info.js",
- ],
- cssstyles=CSSSTYLES
- + [
- "css/assiduites.css",
- ],
- )
+ form = AjoutAssiduiteEtudForm(request.form)
+ # On dresse la liste des modules de l'année scolaire en cours
+ # auxquels est inscrit l'étudiant pour peupler le menu "module"
+ modimpls_by_formsemestre = etud.get_modimpls_by_formsemestre(scu.annee_scolaire())
+ choices = {
+ "": [("", "Non spécifié"), ("autre", "Autre module (pas dans la liste)")]
+ }
+ for formsemestre_id in modimpls_by_formsemestre:
+ formsemestre: FormSemestre = FormSemestre.query.get(formsemestre_id)
+ # indique le nom du semestre dans le menu (optgroup)
+ choices[formsemestre.titre_annee()] = [
+ (m.id, m.module.code)
+ for m in modimpls_by_formsemestre[formsemestre_id]
+ if m.module.ue.type == UE_STANDARD
+ ]
+ form.modimpl.choices = choices
+ if form.validate_on_submit():
+ if form.cancel.data: # cancel button
+ return redirect(redirect_url)
+ ok = _record_assiduite_etud(etud, form)
+ if ok:
+ flash("enregistré")
+ return redirect(redirect_url)
+
+ # Le tableau des assiduités+justificatifs déjà en base:
is_html, tableau = _prepare_tableau(
liste_assi.AssiJustifData.from_etudiants(
etud,
@@ -305,52 +320,123 @@ def signal_assiduites_etud():
filtre=liste_assi.AssiFiltre(type_obj=1),
options=liste_assi.AssiDisplayOptions(show_module=True),
)
+ #
if not is_html:
return tableau
- # Génération de la page
- return HTMLBuilder(
- header,
- _mini_timeline(),
- render_template(
- "assiduites/pages/ajout_assiduites.j2",
- sco=ScoData(etud),
- assi_limit_annee=sco_preferences.get_preference(
- "assi_limit_annee",
- dept_id=g.scodoc_dept_id,
- ),
- assi_morning=ScoDocSiteConfig.get("assi_morning_time", "08:00"),
- assi_evening=ScoDocSiteConfig.get("assi_afternoon_time", "18:00"),
- saisie_eval=saisie_eval,
- date_deb=date_deb,
- date_fin=date_fin,
- etud=etud,
- redirect_url=redirect_url,
- moduleimpl_id=moduleimpl_id,
- tableau=tableau,
- ),
- # render_template(
- # "assiduites/pages/signal_assiduites_etud.j2",
- # sco=ScoData(etud),
- # date=_dateiso_to_datefr(date),
- # morning=morning,
- # lunch=lunch,
- # timeline=_timeline(heures=",".join([f"'{s}'" for s in heures])),
- # afternoon=afternoon,
- # nonworkdays=_non_work_days(),
- # forcer_module=sco_preferences.get_preference(
- # "forcer_module", dept_id=g.scodoc_dept_id
- # ),
- # diff=_differee(
- # etudiants=[sco_etud.get_etud_info(etudid=etud.etudid, filled=True)[0]],
- # moduleimpl_select=select,
- # ),
- # saisie_eval=saisie_eval,
- # date_deb=date_deb,
- # date_fin=date_fin,
- # redirect_url=redirect_url,
- # moduleimpl_id=moduleimpl_id,
- # ),
- ).build()
+
+ return render_template(
+ "assiduites/pages/ajout_assiduite_etud.j2",
+ etud=etud,
+ form=form,
+ moduleimpl_id=moduleimpl_id,
+ redirect_url=redirect_url,
+ sco=ScoData(etud),
+ tableau=tableau,
+ scu=scu,
+ )
+
+
+def _record_assiduite_etud(
+ etud: Identite,
+ form: AjoutAssiduiteEtudForm,
+) -> bool:
+ """Enregistre les données du formulaire de saisie assiduité.
+ Returns ok if successfully recorded, else put error info in the form.
+ Format attendu des données du formulaire:
+ form.assi_etat.data : 'absent'
+ form.date_debut.data : '05/12/2023'
+ form.heure_debut.data : '09:06' (heure locale du serveur)
+ """
+ ok = True
+ debut_jour = "00:00"
+ fin_jour = "23:59:59"
+ # On commence par convertir individuellement tous les champs
+ try:
+ date_debut = datetime.datetime.strptime(form.date_debut.data, "%d/%m/%Y")
+ except ValueError:
+ form.date_debut.errors.append("date début invalide")
+ ok = False
+ try:
+ date_fin = (
+ datetime.datetime.strptime(form.date_fin.data, "%d/%m/%Y")
+ if form.date_fin.data
+ else None
+ )
+ except ValueError:
+ form.date_fin.errors.append("date fin invalide")
+ ok = False
+
+ if date_fin:
+ # ignore les heures si plusieurs jours
+ heure_debut = datetime.time.fromisoformat(debut_jour) # 0h
+ heure_fin = datetime.time.fromisoformat(fin_jour) # minuit
+ else:
+ try:
+ heure_debut = datetime.time.fromisoformat(
+ form.heure_debut.data or debut_jour
+ )
+ except ValueError:
+ form.heure_debut.errors.append("heure début invalide")
+ ok = False
+ try:
+ heure_fin = datetime.time.fromisoformat(form.heure_fin.data or fin_jour)
+ except ValueError:
+ form.heure_fin.errors.append("heure fin invalide")
+ ok = False
+ # Le module (avec "autre")
+ mod_data = form.modimpl.data
+ if mod_data:
+ if mod_data == "autre":
+ moduleimpl_id = "autre"
+ else:
+ try:
+ moduleimpl_id = int(mod_data)
+ except ValueError:
+ form.modimpl.error("choix de module invalide")
+ ok = False
+ else:
+ moduleimpl_id = None
+ if not ok:
+ return False
+ # Vérifie cohérence des dates/heures
+ dt_debut = datetime.datetime.combine(date_debut, heure_debut)
+ dt_fin = datetime.datetime.combine(date_fin or date_debut, heure_fin)
+ if dt_fin <= dt_debut:
+ form.errors["general_errors"] = ["Erreur: dates début/fin incohérentes"]
+ return False
+ data = {
+ "date_debut": dt_debut.isoformat(),
+ "date_fin": dt_fin.isoformat(),
+ "etat": form.assi_etat.data,
+ "moduleimpl_id": moduleimpl_id,
+ }
+ ok, result = create_one_assiduite(data, etud)
+ if ok == 200:
+ # assiduite_id = result["assiduite_id"]
+ return True
+ form.errors["general_errors"] = [f"Erreur: {result}"]
+ return False
+
+ # # Génération de la page
+ # return HTMLBuilder(
+ # header,
+ # _mini_timeline(),
+ # render_template(
+ # "assiduites/pages/ajout_assiduites.j2",
+ # sco=ScoData(etud),
+ # assi_limit_annee=sco_preferences.get_preference(
+ # "assi_limit_annee",
+ # dept_id=g.scodoc_dept_id,
+ # ),
+ # saisie_eval=saisie_eval,
+ # date_deb=date_deb,
+ # date_fin=date_fin,
+ # etud=etud,
+ # redirect_url=redirect_url,
+ # moduleimpl_id=moduleimpl_id,
+ # tableau=tableau,
+ # scu=scu,
+ # ),
@bp.route("/ListeAssiduitesEtud")
@@ -1513,10 +1599,12 @@ def signal_evaluation_abs(etudid: int = None, evaluation_id: int = None):
# rediriger vers page saisie
return redirect(
url_for(
- "assiduites.signal_assiduites_etud",
+ "assiduites.ajout_assiduite_etud",
etudid=etudid,
evaluation_id=evaluation.id,
- date_deb=evaluation.date_debut.strftime("%Y-%m-%dT%H:%M:%S"),
+ date_deb=evaluation.date_debut.strftime(
+ "%Y-%m-%dT%H:%M:%S"
+ ), # XXX TODO
date_fin=evaluation.date_fin.strftime("%Y-%m-%dT%H:%M:%S"),
moduleimpl_id=evaluation.moduleimpl.id,
saisie_eval="true",
@@ -1540,12 +1628,14 @@ def signal_evaluation_abs(etudid: int = None, evaluation_id: int = None):
if "Duplication" in msg:
msg = """Une autre saisie concerne déjà cette période.
En cliquant sur continuer vous serez redirigé vers la page de
- saisie des assiduités de l'étudiant."""
+ saisie de l'assiduité de l'étudiant."""
dest: str = url_for(
- "assiduites.signal_assiduites_etud",
+ "assiduites.ajout_assiduite_etud",
etudid=etudid,
evaluation_id=evaluation.id,
- date_deb=evaluation.date_debut.strftime("%Y-%m-%dT%H:%M:%S"),
+ date_deb=evaluation.date_debut.strftime(
+ "%Y-%m-%dT%H:%M:%S"
+ ), # XXX TODO
date_fin=evaluation.date_fin.strftime("%Y-%m-%dT%H:%M:%S"),
moduleimpl_id=evaluation.moduleimpl.id,
saisie_eval="true",
diff --git a/app/views/users.py b/app/views/users.py
index 143a5e4c78..3c0c11972b 100644
--- a/app/views/users.py
+++ b/app/views/users.py
@@ -535,7 +535,7 @@ def create_user_form(user_name=None, edit=0, all_roles=True):
"d",
{
"input_type": "separator",
- "title": f"L'utilisateur sera crée dans le département {auth_dept}",
+ "title": f"L'utilisateur sera créé dans le département {auth_dept}",
},
)
)