Merge branch 'main96' into sans_moment

app/auth/cas.py

@@ -39,6 +39,15 @@ def after_cas_login():
                             "scodoc_cas_login_date"
                         ] = datetime.datetime.now().isoformat()
                         user.cas_last_login = datetime.datetime.utcnow()
+                        if flask.session.get("CAS_EDT_ID"):
+                            # essaie de récupérer l'edt_id s'il est présent
+                            # cet ID peut être renvoyé par le CAS et extrait par ScoDoc
+                            # via l'expression `cas_edt_id_from_xml_regexp`
+                            # voir flask_cas.routing
+                            edt_id = flask.session.get("CAS_EDT_ID")
+                            current_app.logger.info(f"""after_cas_login: storing edt_id for {
+                                user.user_name}: '{edt_id}'""")
+                            user.edt_id = edt_id
                         db.session.add(user)
                         db.session.commit()
                         return flask.redirect(url_for("scodoc.index"))

app/auth/models.py

@@ -185,18 +185,7 @@ class User(UserMixin, db.Model):
                 return self._migrate_scodoc7_password(password)
             return False
 
-        password_ok = check_password_hash(self.password_hash, password)
+        return check_password_hash(self.password_hash, password)
-        if password_ok and cas_enabled and flask.session.get("CAS_EDT_ID"):
-            # essaie de récupérer l'edt_id s'il est présent
-            # cet ID peut être renvoyé par le CAS et extrait par ScoDoc
-            # via l'expression `cas_edt_id_from_xml_regexp`
-            # voir flask_cas.routing
-            edt_id = flask.session.get("CAS_EDT_ID")
-            log(f"Storing edt_id for {self.user_name}: '{edt_id}'")
-            self.edt_id = edt_id
-            db.session.add(self)
-            db.session.commit()
-        return password_ok
 
     def _migrate_scodoc7_password(self, password) -> bool:
         """After migration, rehash password."""

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 <tt>{edt_id}</tt> 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})

app/forms/main/config_cas.py

@@ -38,11 +38,17 @@ from app.models import ScoDocSiteConfig
 
 
 def check_cas_uid_from_mail_regexp(form, field):
-    "Vérifie la regexp fournie pur l'extraction du CAS id"
+    "Vérifie la regexp fournie pour l'extraction du CAS id"
     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")
@@ -58,18 +64,18 @@ class ConfigCASForm(FlaskForm):
         description="""url complète. Commence en général par <tt>https://</tt>.""",
     )
     cas_login_route = StringField(
-        label="Route du login CAS",
+        label="Optionnel: route du login CAS",
         description="""ajouté à l'URL du serveur: exemple <tt>/cas</tt>
             (si commence par <tt>/</tt>, part de la racine)""",
         default="/cas",
     )
     cas_logout_route = StringField(
-        label="Route du logout CAS",
+        label="Optionnel: route du logout CAS",
         description="""ajouté à l'URL du serveur: exemple <tt>/cas/logout</tt>""",
         default="/cas/logout",
     )
     cas_validate_route = StringField(
-        label="Route de validation CAS",
+        label="Optionnel: route de validation CAS",
         description="""ajouté à l'URL du serveur: exemple <tt>/cas/serviceValidate</tt>""",
         default="/cas/serviceValidate",
     )
@@ -81,7 +87,7 @@ class ConfigCASForm(FlaskForm):
     )
 
     cas_uid_from_mail_regexp = StringField(
-        label="Expression pour extraire l'identifiant utilisateur",
+        label="Optionnel: expression pour extraire l'identifiant utilisateur",
         description="""regexp python appliquée au mail institutionnel de l'utilisateur,
         dont le premier groupe doit donner l'identifiant CAS.
         Si non fournie, le super-admin devra saisir cet identifiant pour chaque compte.
@@ -92,6 +98,17 @@ class ConfigCASForm(FlaskForm):
         validators=[Optional(), check_cas_uid_from_mail_regexp],
     )
 
+    cas_edt_id_from_xml_regexp = StringField(
+        label="Optionnel: expression pour extraire l'identifiant edt",
+        description="""regexp python appliquée à la réponse XML du serveur CAS pour
+        retrouver l'id de l'utilisateur sur le SI de l'institution, et notamment sur les
+        calendrier d'emploi du temps. Par exemple, si cet id est renvoyé dans le champ
+        <b>supannEmpId</b>, utiliser:
+        <tt>&lt;cas:supannEmpId&gt;(.*?)&lt;/cas:supannEmpId&gt;</tt>
+        """,
+        validators=[Optional(), check_cas_edt_id_from_xml_regexp],
+    )
+
     cas_ssl_verify = BooleanField("Vérification du certificat SSL")
     cas_ssl_certificate_file = FileField(
         label="Certificat (PEM)",

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",
             ),
         )

app/models/config.py

@@ -286,7 +286,7 @@ class ScoDocSiteConfig(db.Model):
     @classmethod
     def set(cls, name: str, value: str) -> bool:
         "Set parameter, returns True if change. Commit session."
-        value_str = str(value or "")
+        value_str = str(value or "").strip()
         if (cls.get(name) or "") != value_str:
             cfg = ScoDocSiteConfig.query.filter_by(name=name).first()
             if cfg is None:
@@ -429,7 +429,17 @@ class ScoDocSiteConfig(db.Model):
             return False
         # and returns at least one group on a simple cannonical address
         match = pattern.search("emmanuel@exemple.fr")
-        return len(match.groups()) > 0
+        return match is not None and len(match.groups()) > 0
+
+    @classmethod
+    def cas_edt_id_from_xml_regexp_is_valid(cls, exp: str) -> bool:
+        "True si l'expression régulière semble valide"
+        # check that it compiles
+        try:
+            _ = re.compile(exp)
+        except re.error:
+            return False
+        return True
 
     @classmethod
     def assi_get_rounded_time(cls, label: str, default: str) -> float:

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)}>"
 

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",

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",

app/scodoc/sco_formsemestre_edit.py

@@ -40,6 +40,7 @@ from app.models import (
     ModuleImpl,
     Evaluation,
     UniteEns,
+    ScoDocSiteConfig,
     ScolarFormSemestreValidation,
     ScolarAutorisationInscription,
     ApcValidationAnnee,
@@ -445,6 +446,18 @@ 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"""
         <p><a class="stdlink" href="{url_for("notes.formsemestre_edit_uecoefs",

app/scodoc/sco_groups.py

@@ -79,7 +79,9 @@ partitionEditor = ndb.EditableTable(
 )
 
 groupEditor = ndb.EditableTable(
-    "group_descr", "group_id", ("group_id", "partition_id", "group_name", "numero")
+    "group_descr",
+    "group_id",
+    ("group_id", "partition_id", "group_name", "numero", "edt_id"),
 )
 
 group_list = groupEditor.list

app/static/js/assiduites.js

@@ -1408,7 +1408,7 @@ function editAssiduite(assiduite_id, etat, assi) {
 }
 
 /**
- * Récupération des assiduités conflictuelles avec la période de la time line
+ * Récupération des assiduités conflictuelles avec la période de la timeline
  * @param {String | Number} etudid identifiant de l'étudiant
  * @returns {Array[Assiduité]} un tableau d'assiduité
  */

app/templates/assiduites/pages/config_assiduites.j2

@@ -13,24 +13,34 @@ affectent notamment les comptages d'absences de tous les bulletins des
 
 </div>
 
-<div class="row">
+<form class="form form-horizontal" method="post" enctype="multipart/form-data" role="form">
-    <div class="col-md-8">
+    <div class="row">
+        <div class="col-md-8">
+                {{ form.hidden_tag() }}
+                {{ wtf.form_errors(form, hiddens="only") }}
 
-        <form class="form form-horizontal" method="post" enctype="multipart/form-data" role="form">
+                {{ wtf.form_field(form.morning_time) }}
-            {{ form.hidden_tag() }}
+                {{ wtf.form_field(form.lunch_time) }}
-            {{ wtf.form_errors(form, hiddens="only") }}
+                {{ wtf.form_field(form.afternoon_time) }}
-
+                {{ wtf.form_field(form.tick_time) }}
-            {{ wtf.form_field(form.morning_time) }}
+        </div>
-            {{ wtf.form_field(form.lunch_time) }}
-            {{ wtf.form_field(form.afternoon_time) }}
-            {{ wtf.form_field(form.tick_time) }}
-            <div class="form-group">
-                {{ wtf.form_field(form.submit) }}
-                {{ wtf.form_field(form.cancel) }}
-            </div>
-        </form>
     </div>
-</div>
+    <div class="row">
+        <h1>Emplois du temps</h1>
+        <div class="help">ScoDoc peut récupérer les emplois du temps de chaque session.</div>
+        <div class="col-md-8">
+                <div class="config-edt">
+                    {{ wtf.form_field(form.edt_ics_path) }}
+                </div>
+
+                <div class="form-group">
+                    {{ wtf.form_field(form.submit) }}
+                    {{ wtf.form_field(form.cancel) }}
+                </div>
+        </div>
+    </div>
+</form>
+
 
 
 

app/templates/config_cas.j2

@@ -24,6 +24,7 @@
         {{ wtf.form_field(form.cas_validate_route) }}
         {{ wtf.form_field(form.cas_attribute_id) }}
         {{ wtf.form_field(form.cas_uid_from_mail_regexp) }}
+        {{ wtf.form_field(form.cas_edt_id_from_xml_regexp) }}
         <div class="cas_settings">
             {{ wtf.form_field(form.cas_ssl_verify) }}
             {{ wtf.form_field(form.cas_ssl_certificate_file) }}

app/templates/configuration.j2

@@ -73,8 +73,8 @@ Heure: <b><tt>{{ time.strftime("%d/%m/%Y %H:%M") }}</tt></b>
     </p>
 </section>
 <section>
-    <h2>Assiduité</h2>
+    <h2>Assiduité et emplois du temps</h2>
-    <p><a class="stdlink" href="{{url_for('scodoc.config_assiduites')}}">Configuration du suivi de l'assiduité</a>
+    <p><a class="stdlink" href="{{url_for('scodoc.config_assiduites')}}">Configuration du suivi de l'assiduité et accès aux emplois du temps</a>
     </p>
 </section>
 

app/views/scodoc.py

@@ -273,6 +273,10 @@ def config_cas():
             "cas_uid_from_mail_regexp", form.data["cas_uid_from_mail_regexp"]
         ):
             flash("Expression extraction identifiant CAS enregistrée")
+        if ScoDocSiteConfig.set(
+            "cas_edt_id_from_xml_regexp", form.data["cas_edt_id_from_xml_regexp"]
+        ):
+            flash("Expression extraction identifiant edt enregistrée")
         if ScoDocSiteConfig.set("cas_ssl_verify", form.data["cas_ssl_verify"]):
             flash("Vérification SSL modifiée")
         if form.cas_ssl_certificate_file.data:
@@ -300,6 +304,9 @@ def config_cas():
         form.cas_uid_from_mail_regexp.data = ScoDocSiteConfig.get(
             "cas_uid_from_mail_regexp"
         )
+        form.cas_edt_id_from_xml_regexp.data = ScoDocSiteConfig.get(
+            "cas_edt_id_from_xml_regexp"
+        )
         form.cas_ssl_verify.data = ScoDocSiteConfig.get("cas_ssl_verify")
     return render_template(
         "config_cas.j2",
@@ -316,6 +323,7 @@ def config_assiduites():
     form = ConfigAssiduitesForm()
     if request.method == "POST" and form.cancel.data:  # cancel button
         return redirect(url_for("scodoc.index"))
+
     if form.validate_on_submit():
         if ScoDocSiteConfig.set("assi_morning_time", form.data["morning_time"]):
             flash("Heure du début de la journée enregistrée")
@@ -323,18 +331,13 @@ def config_assiduites():
             flash("Heure de midi enregistrée")
         if ScoDocSiteConfig.set("assi_afternoon_time", form.data["afternoon_time"]):
             flash("Heure de fin de la journée enregistrée")
-        if (
+        if ScoDocSiteConfig.set("assi_tick_time", float(form.data["tick_time"])):
-            form.data["tick_time"] > 0
-            and form.data["tick_time"] < 60
-            and ScoDocSiteConfig.set("assi_tick_time", float(form.data["tick_time"]))
-        ):
             flash("Granularité de la timeline enregistrée")
-        else:
+        if ScoDocSiteConfig.set("edt_ics_path", form.data["edt_ics_path"]):
-            flash("Erreur : Granularité invalide ou identique")
+            flash("Chemin vers les calendriers ics enregistré")
-
         return redirect(url_for("scodoc.configuration"))
 
-    elif request.method == "GET":
+    if request.method == "GET":
         form.morning_time.data = ScoDocSiteConfig.get(
             "assi_morning_time", datetime.time(8, 0, 0)
         )
@@ -349,12 +352,13 @@ def config_assiduites():
         except ValueError:
             form.tick_time.data = 15.0
             ScoDocSiteConfig.set("assi_tick_time", 15.0)
+        form.edt_ics_path.data = ScoDocSiteConfig.get("edt_ics_path")
 
-        return render_template(
+    return render_template(
-            "assiduites/pages/config_assiduites.j2",
+        "assiduites/pages/config_assiduites.j2",
-            form=form,
+        form=form,
-            title="Configuration du module Assiduité",
+        title="Configuration du module Assiduité",
-        )
+    )
 
 
 @bp.route("/ScoDoc/config_codes_decisions", methods=["GET", "POST"])