Pages saisies absence + refonte tableau bord semestre. Close #342

app/models/groups.py

@@ -74,6 +74,10 @@ class GroupDescr(db.Model):
             f"""<{self.__class__.__name__} {self.id} "{self.group_name or '(tous)'}">"""
         )
 
+    def get_nom_with_part(self) -> str:
+        "Nom avec partition: 'TD A'"
+        return f"{self.partition.partition_name or ''} {self.group_name or '-'}"
+
 
 group_membership = db.Table(
     "group_membership",

app/scodoc/html_sidebar.py

@@ -87,8 +87,8 @@ def sidebar():
         { sidebar_common() }
         <div class="box-chercheetud">Chercher étudiant:<br/>
         <form method="get" id="form-chercheetud"
-              action="{ url_for('scolar.search_etud_in_dept', scodoc_dept=g.scodoc_dept) }">
+            action="{url_for('scolar.search_etud_in_dept', scodoc_dept=g.scodoc_dept) }">
-        <div><input type="text" size="12" id="in-expnom" name="expnom" spellcheck="false"></input></div>
+        <div><input type="text" size="12" class="in-expnom" name="expnom" spellcheck="false"></input></div>
         </form></div>
         <div class="etud-insidebar">
         """

app/scodoc/sco_abs.py

@@ -31,9 +31,7 @@
 import calendar
 import datetime
 import html
-import string
 import time
-import types
 
 from app.scodoc import notesdb as ndb
 from app import log
@@ -42,9 +40,9 @@ from app.scodoc.sco_exceptions import ScoValueError, ScoInvalidDateError
 from app.scodoc import sco_abs_notification
 from app.scodoc import sco_cache
 from app.scodoc import sco_etud
-from app.scodoc import sco_formsemestre
 from app.scodoc import sco_formsemestre_inscriptions
 from app.scodoc import sco_preferences
+import app.scodoc.sco_utils as scu
 
 # --- Misc tools.... ------------------
 
@@ -247,9 +245,9 @@ def day_names():
     If work_saturday property is set, include saturday
     """
     if is_work_saturday():
-        return ["Lundi", "Mardi", "Mercredi", "Jeudi", "Vendredi", "Samedi"]
+        return scu.DAY_NAMES[:-1]
     else:
-        return ["Lundi", "Mardi", "Mercredi", "Jeudi", "Vendredi"]
+        return scu.DAY_NAMES[:-2]
 
 
 def next_iso_day(date):

app/scodoc/sco_find_etud.py

@@ -53,12 +53,10 @@ def form_search_etud(
 ):
     "form recherche par nom"
     H = []
-    if title:
-        H.append("<h2>%s</h2>" % title)
     H.append(
         f"""<form action="{ url_for("scolar.search_etud_in_dept", scodoc_dept=g.scodoc_dept) }" method="POST">
     <b>{title}</b>
-    <input type="text" name="expnom" width="12" spellcheck="false" value="">
+    <input type="text" name="expnom" class="in-expnom" width="12" spellcheck="false" value="">
     <input type="submit" value="Chercher">
     <br/>(entrer une partie du nom)
     """

app/scodoc/sco_formsemestre_status.py

@@ -761,22 +761,29 @@ def _make_listes_sem(sem, with_absences=True):
         if with_absences:
             first_monday = sco_abs.ddmmyyyy(sem["date_debut"]).prev_monday()
             form_abs_tmpl = f"""
-            <td><form action="{url_for(
+            <td>
+                <a href="%(url_etat)s">absences</a>
+            </td>
+            <td>
+            <form action="{url_for(
                         "absences.SignaleAbsenceGrSemestre", scodoc_dept=g.scodoc_dept
                     )}" method="get">
                 <input type="hidden" name="datefin" value="{sem['date_fin']}"/>
                 <input type="hidden" name="group_ids" value="%(group_id)s"/>
                 <input type="hidden" name="destination" value="{destination}"/>
-                <input type="submit" value="Saisir absences du" />
+                <input type="submit" value="Saisir abs des" />
                 <select name="datedebut" class="noprint">
             """
             date = first_monday
             for jour in sco_abs.day_names():
-                form_abs_tmpl += '<option value="%s">%s</option>' % (date, jour)
+                form_abs_tmpl += f'<option value="{date}">{jour}s</option>'
                 date = date.next_day()
-            form_abs_tmpl += """
+            form_abs_tmpl += f"""
                 </select>
-                <a href="%(url_etat)s">état</a>
+
+                <a href="{
+                    url_for("absences.choix_semaine", scodoc_dept=g.scodoc_dept) 
+                }?group_id=%(group_id)s">saisie par semaine</a>
                 </form></td>
             """
         else:
@@ -788,7 +795,7 @@ def _make_listes_sem(sem, with_absences=True):
     # Genere liste pour chaque partition (categorie de groupes)
     for partition in sco_groups.get_partitions_list(sem["formsemestre_id"]):
         if not partition["partition_name"]:
-            H.append("<h4>Tous les étudiants</h4>" % partition)
+            H.append("<h4>Tous les étudiants</h4>")
         else:
             H.append("<h4>Groupes de %(partition_name)s</h4>" % partition)
         groups = sco_groups.get_partition_groups(partition)
@@ -818,20 +825,6 @@ def _make_listes_sem(sem, with_absences=True):
                         )
                     }">{group["label"]}</a>
                     </td><td>
-                    (<a href="{
-                        url_for("scolar.groups_view",
-                            group_ids=group["group_id"],
-                            format="xls",
-                            scodoc_dept=g.scodoc_dept,
-                        )
-                    }">tableur</a>)
-                    <a href="{
-                        url_for("scolar.groups_view",
-                            curtab="tab-photos",
-                            group_ids=group["group_id"],
-                            scodoc_dept=g.scodoc_dept,
-                        )
-                    }">Photos</a>
                     </td>
                     <td>({n_members} étudiants)</td>
                     """

app/scodoc/sco_utils.py

@@ -178,6 +178,7 @@ MONTH_NAMES = (
     "novembre",
     "décembre",
 )
+DAY_NAMES = ("lundi", "mardi", "mercredi", "jeudi", "vendredi", "samedi", "dimanche")
 
 
 def fmt_note(val, note_max=None, keep_numeric=False):

app/static/js/scodoc.js

@@ -3,14 +3,14 @@
 
 $(function () {
     // Autocomplete recherche etudiants par nom
-    $("#in-expnom").autocomplete(
+    $(".in-expnom").autocomplete(
         {
             delay: 300, // wait 300ms before suggestions
             minLength: 2, // min nb of chars before suggest
             position: { collision: 'flip' }, // automatic menu position up/down
-            source: "search_etud_by_name",
+            source: SCO_URL + "/search_etud_by_name",
             select: function (event, ui) {
-                $("#in-expnom").val(ui.item.value);
+                $(".in-expnom").val(ui.item.value);
                 $("#form-chercheetud").submit();
             }
         });

app/templates/sidebar.html

@@ -15,7 +15,7 @@
     <h2 class="insidebar">Dépt. {{ sco.prefs["DeptName"] }}</h2>
     <a href="{{ url_for('scolar.index_html', scodoc_dept=g.scodoc_dept) }}" class="sidebar">Accueil</a> <br />
     {% if sco.prefs["DeptIntranetURL"] %}
-    <a href="{{ sco.prefs["DeptIntranetURL"] }}" class="sidebar">
+    <a href="{{ sco.prefs[" DeptIntranetURL"] }}" class="sidebar">
         {{ sco.prefs["DeptIntranetTitle"] }}</a>
     {% endif %}
     <br>
@@ -41,7 +41,7 @@
         <form method="get" id="form-chercheetud"
             action="{{ url_for('scolar.search_etud_in_dept', scodoc_dept=g.scodoc_dept) }}">
             <div>
-                <input type="text" size="12" id="in-expnom" name="expnom" spellcheck="false" />
+                <input type="text" size="12" class="in-expnom" name="expnom" spellcheck="false" />
             </div>
         </form>
     </div>
@@ -49,7 +49,7 @@
     <div class="etud-insidebar">
         {% if sco.etud %}
         <h2 id="insidebar-etud"><a href="{{url_for(
-            "scolar.ficheEtud", scodoc_dept=g.scodoc_dept, etudid=sco.etud.id )}}" class="sidebar">
+            'scolar.ficheEtud', scodoc_dept=g.scodoc_dept, etudid=sco.etud.id )}}" class="sidebar">
                 <span class="fontred">{{sco.etud.nomprenom}}</span></a>
         </h2>
         <b>Absences</b>

app/views/absences.py

@@ -69,7 +69,7 @@ from app.decorators import (
     permission_required,
     permission_required_compat_scodoc7,
 )
-from app.models import FormSemestre
+from app.models import FormSemestre, GroupDescr
 from app.models.absences import BilletAbsence
 from app.views import absences_bp as bp
 
@@ -119,63 +119,71 @@ def sco_publish(route, function, permission, methods=["GET"]):
 @scodoc7func
 def index_html():
     """Gestionnaire absences, page principale"""
-    # crude portage from 1999 DTML
-    sems = sco_formsemestre.do_formsemestre_list()
-    authuser = current_user
 
     H = [
         html_sco_header.sco_header(
-            page_title="Gestion des absences",
+            page_title="Saisie des absences",
             cssstyles=["css/calabs.css"],
             javascripts=["js/calabs.js"],
         ),
-        """<h2>Gestion des Absences</h2>""",
+        """<h2>Traitement des absences</h2>
+        <p class="help">
+        Pour saisir des absences ou consulter les états, il est recommandé par passer par 
+        le semestre concerné (saisie par jours nommés ou par semaines).
+        </p>
+        """,
     ]
-    if not sems:
+    H.append(
+        """<p class="help">Pour signaler, annuler ou justifier une absence pour un seul étudiant, 
+        choisissez d'abord concerné:</p>"""
+    )
+    H.append(sco_find_etud.form_search_etud())
+    if current_user.has_permission(
+        Permission.ScoAbsChange
+    ) and sco_preferences.get_preference("handle_billets_abs"):
         H.append(
-            """<p class="warning">Aucun semestre défini (ou aucun groupe d'étudiant)</p>"""
-        )
-    else:
-        H.append(
-            """<ul><li><a href="EtatAbsences">Afficher l'état des absences (pour tout un groupe)</a></li>"""
-        )
-        if sco_preferences.get_preference("handle_billets_abs"):
-            H.append(
-                """<li><a href="listeBillets">Traitement des billets d'absence en attente</a></li>"""
-            )
-        H.append(
-            """<p>Pour signaler, annuler ou justifier une absence, choisissez d'abord l'étudiant concerné:</p>"""
-        )
-        H.append(sco_find_etud.form_search_etud())
-        if authuser.has_permission(Permission.ScoAbsChange):
-            H.extend(
-                (
-                    """<hr/>
-<form action="SignaleAbsenceGrHebdo" id="formw">
-<input type="hidden" name="destination" value="%s"/>
-<p>
-<span  style="font-weight: bold; font-size:120%%;">
- Saisie par semaine </span> - Choix du groupe:
- <input name="datelundi" type="hidden" value="x"/>
             """
-                    % request.base_url,
+            <h2 style="margin-top: 30px;">Billets d'absence</h2>
-                    sco_abs_views.formChoixSemestreGroupe(),
+            <ul><li><a href="listeBillets">Traitement des billets d'absence en attente</a></li></ul>
-                    "</p>",
+            """
-                    cal_select_week(),
+        )
-                    """<p class="help">Sélectionner le groupe d'étudiants, puis cliquez sur une semaine pour
-saisir les absences de toute cette semaine.</p>
-                      </form>""",
-                )
-            )
-        else:
-            H.append(
-                """<p class="scoinfo">Vous n'avez pas l'autorisation d'ajouter, justifier ou supprimer des absences.</p>"""
-            )
-
     H.append(html_sco_header.sco_footer())
     return "\n".join(H)
 
 
+@bp.route("/choix_semaine")
+@scodoc
+@permission_required(Permission.ScoAbsChange)
+@scodoc7func
+def choix_semaine(group_id):
+    """Page choix semaine sur calendrier pour saisie absences d'un groupe"""
+    group = GroupDescr.query.get_or_404(group_id)
+    H = [
+        html_sco_header.sco_header(
+            page_title="Saisie des absences",
+            cssstyles=["css/calabs.css"],
+            javascripts=["js/calabs.js"],
+        ),
+        f"""
+        <h2>Saisie des Absences</h2>
+        <form action="SignaleAbsenceGrHebdo" id="formw">
+        <p>
+        <span  style="font-weight: bold; font-size:120%;">
+        Saisie par semaine </span> - Groupe: {group.get_nom_with_part()}
+        <input name="datelundi" type="hidden" value="x"/>
+        <input name="group_ids" type="hidden" value="{group_id}"/>
+        </p>
+        """,
+        cal_select_week(),
+        """<p class="help">Sélectionner le groupe d'étudiants, puis cliquez sur une semaine pour
+        saisir les absences de toute cette semaine.</p>
+        </form>
+        """,
+        html_sco_header.sco_footer(),
+    ]
+    return "\n".join(H)
+
+
 def cal_select_week(year=None):
     "display calendar allowing week selection"
     if not year:
@@ -479,7 +487,7 @@ def SignaleAbsenceGrSemestre(
     datedebut,
     datefin,
     destination="",
-    group_ids=[],  # list of groups to display
+    group_ids=(),  # list of groups to display
     nbweeks=4,  # ne montre que les nbweeks dernieres semaines
     moduleimpl_id=None,
 ):
@@ -566,7 +574,7 @@ def SignaleAbsenceGrSemestre(
         url_link_semaines += "&moduleimpl_id=" + str(moduleimpl_id)
     #
     dates = [x.ISO() for x in dates]
-    dayname = sco_abs.day_names()[jourdebut.weekday]
+    day_name = sco_abs.day_names()[jourdebut.weekday]
 
     if groups_infos.tous_les_etuds_du_sem:
         gr_tit = "en"
@@ -579,19 +587,18 @@ def SignaleAbsenceGrSemestre(
 
     H = [
         html_sco_header.sco_header(
-            page_title="Saisie des absences",
+            page_title=f"Saisie des absences du {day_name}",
             init_qtip=True,
             javascripts=["js/etud_info.js", "js/abs_ajax.js"],
             no_side_bar=1,
         ),
-        """<table border="0" cellspacing="16"><tr><td>
+        f"""<table border="0" cellspacing="16"><tr><td>
-            <h2>Saisie des absences %s %s, 
+            <h2>Saisie des absences {gr_tit} {sem["titre_num"]}, 
-            les <span class="fontred">%s</span></h2>
+            les <span class="fontred">{day_name}s</span></h2>
             <p>
-            <a href="%s">%s</a>
+            <a href="{url_link_semaines}">{msg}</a>
             <form id="abs_form" action="doSignaleAbsenceGrSemestre" method="post">
-            """
+            """,
-        % (gr_tit, sem["titre_num"], dayname, url_link_semaines, msg),
     ]
     #
     if etuds:
@@ -820,7 +827,6 @@ def _gen_form_saisie_groupe(
     # version pour formulaire avec AJAX (Yann LB)
     H.append(
         """
-        <p><input type="button" value="Retour" onClick="window.location='%s'"/>
         </p>
         </form>
         </p>
@@ -831,8 +837,7 @@ def _gen_form_saisie_groupe(
         </p><p class="help">Si vous "décochez" une case,  l'absence correspondante sera supprimée.
         Attention, les modifications sont automatiquement entregistrées au fur et à mesure.
         </p>
-    """
+        """
-        % destination
     )
     return H
 

sco_version.py

@@ -1,7 +1,7 @@
 # -*- mode: python -*-
 # -*- coding: utf-8 -*-
 
-SCOVERSION = "9.1.92"
+SCOVERSION = "9.2.0a"
 
 SCONAME = "ScoDoc"