diff --git a/app/models/assiduites.py b/app/models/assiduites.py index f645f5c5a..241e44aa6 100644 --- a/app/models/assiduites.py +++ b/app/models/assiduites.py @@ -353,12 +353,12 @@ class Assiduite(ScoDocModel): elif self.external_data is not None and "module" in self.external_data: return ( - "Tout module" + "Autre module (pas dans la liste)" if self.external_data["module"] == "Autre" else self.external_data["module"] ) - return "Non spécifié" if traduire else None + return "Module non spécifié" if traduire else None def get_saisie(self) -> str: """ diff --git a/app/scodoc/html_sco_header.py b/app/scodoc/html_sco_header.py index 6362d4db3..5d42e3d9d 100644 --- a/app/scodoc/html_sco_header.py +++ b/app/scodoc/html_sco_header.py @@ -25,8 +25,7 @@ # ############################################################################## -"""HTML Header/Footer for ScoDoc pages -""" +"""HTML Header/Footer for ScoDoc pages""" import html @@ -101,7 +100,7 @@ _HTML_BEGIN = f""" @@ -218,7 +217,7 @@ def sco_header( -{% endblock %} - - -{% block app_content %} -
- -

Liste de l'assiduité et des justificatifs de {{sco.etud.html_link_fiche()|safe}}

- {{tableau | safe }} -
- -{% include "assiduites/explication_etats_justifs.j2" %} - -{% endblock app_content %} diff --git a/app/templates/assiduites/pages/signal_assiduites_diff.j2 b/app/templates/assiduites/pages/signal_assiduites_diff.j2 index 0c5c800ec..2d940fd9b 100644 --- a/app/templates/assiduites/pages/signal_assiduites_diff.j2 +++ b/app/templates/assiduites/pages/signal_assiduites_diff.j2 @@ -570,6 +570,13 @@ window.addEventListener("load", main);

Signalement différé de l'assiduité {{gr |safe}}

+
+Attention, cette page est en cours de fusion avec la page de Saisie journalière. +
+Vous pouvez dès à présent cliquer sur ce lien pour accéder à la nouvelle version. + +
+
-
+ {# /etud-insidebar #} diff --git a/app/views/assiduites.py b/app/views/assiduites.py index e9a32454c..f5ffe223f 100644 --- a/app/views/assiduites.py +++ b/app/views/assiduites.py @@ -511,51 +511,6 @@ def _record_assiduite_etud( return False -@bp.route("/liste_assiduites_etud") -@scodoc -@permission_required(Permission.ScoView) -def liste_assiduites_etud(): - """ - liste_assiduites_etud Affichage de toutes les assiduites et justificatifs d'un etudiant - Args: - etudid (int): l'identifiant de l'étudiant - - Returns: - str: l'html généré - """ - - # Récupération de l'étudiant concerné - etudid = request.args.get("etudid", -1) - etud: Identite = Identite.query.get_or_404(etudid) - if etud.dept_id != g.scodoc_dept_id: - abort(404, "étudiant inexistant dans ce département") - - # Gestion d'une assiduité unique (redirigé depuis le calendrier) TODO-Assiduites - assiduite_id: int = request.args.get("assiduite_id", -1) - - # Préparation de la page - tableau = _prepare_tableau( - liste_assi.AssiJustifData.from_etudiants( - etud, - ), - filename=f"assiduites-justificatifs-{etud.id}", - afficher_etu=False, - filtre=liste_assi.AssiFiltre(type_obj=0), - options=liste_assi.AssiDisplayOptions(show_module=True), - cache_key=f"tableau-etud-{etud.id}", - ) - if not tableau[0]: - return tableau[1] - # Page HTML: - return render_template( - "assiduites/pages/liste_assiduites.j2", - assi_id=assiduite_id, - etud=etud, - tableau=tableau[1], - sco=ScoData(etud), - ) - - @bp.route("/bilan_etud") @scodoc @permission_required(Permission.ScoView) @@ -583,28 +538,19 @@ def bilan_etud(): sco_preferences.get_preference("assi_metrique", dept_id=g.scodoc_dept_id), ) - # Récupération des assiduités et justificatifs de l'étudiant - data = liste_assi.AssiJustifData( - etud.assiduites.filter( - Assiduite.etat != scu.EtatAssiduite.PRESENT, Assiduite.est_just == False + # Préparation de la page + tableau = _prepare_tableau( + liste_assi.AssiJustifData.from_etudiants( + etud, ), - etud.justificatifs.filter( - Justificatif.etat.in_( - [scu.EtatJustificatif.ATTENTE, scu.EtatJustificatif.MODIFIE] - ) - ), - ) - - table = _prepare_tableau( - data, + filename=f"assiduites-justificatifs-{etud.id}", afficher_etu=False, - filename=f"Bilan assiduité {etud.nomprenom}", - titre="Bilan de l'assiduité de l'étudiant", - cache_key=f"tableau-etud-{etud.id}-bilan", + filtre=liste_assi.AssiFiltre(type_obj=0), + options=liste_assi.AssiDisplayOptions(show_module=True), + cache_key=f"tableau-etud-{etud.id}", ) - - if not table[0]: - return table[1] + if not tableau[0]: + return tableau[1] # Génération de la page HTML return render_template( @@ -614,7 +560,7 @@ def bilan_etud(): date_debut=date_debut, date_fin=date_fin, sco=ScoData(etud), - tableau=table[1], + tableau=tableau[1], ) @@ -654,7 +600,7 @@ def edit_justificatif_etud(justif_id: int): back_url = request.args.get("back_url", None) redirect_url = back_url or url_for( - "assiduites.liste_assiduites_etud", + "assiduites.bilan_etud", scodoc_dept=g.scodoc_dept, etudid=justif.etudiant.id, ) @@ -958,8 +904,6 @@ def choix_date() -> str: Route utilisée uniquement si la date courante n'est pas dans le semestre concerné par la requête vers une des pages suivantes : - saisie_assiduites_group - - visu_assiduites_group - """ formsemestre_id = request.args.get("formsemestre_id") formsemestre: FormSemestre = FormSemestre.query.get_or_404(formsemestre_id) @@ -989,11 +933,7 @@ def choix_date() -> str: if ok: return redirect( url_for( - ( - "assiduites.signal_assiduites_group" - if request.args.get("readonly") is None - else "assiduites.visu_assiduites_group" - ), + "assiduites.signal_assiduites_group", scodoc_dept=g.scodoc_dept, formsemestre_id=formsemestre_id, group_ids=group_ids, @@ -1065,7 +1005,7 @@ def signal_assiduites_group(): ) if not groups_infos.members: return ( - html_sco_header.sco_header(page_title="Saisie journalière de l'assiduité") + html_sco_header.sco_header(page_title="Saisie de l'assiduité") + "

Aucun étudiant !

" + html_sco_header.sco_footer() ) @@ -1156,137 +1096,7 @@ def signal_assiduites_group(): sco=ScoData(formsemestre=formsemestre), sem=sem["titre_num"], timeline=_timeline(heures=",".join([f"'{s}'" for s in heures])), - title="Saisie journalière des assiduités", - ) - - -@bp.route("/visu_assiduites_group") -@scodoc -@permission_required(Permission.ScoView) -def visu_assiduites_group(): - """ - Visualisation des assiduités des groupes pour le jour donné - dans le formsemestre_id et le moduleimpl_id - Returns: - str: l'html généré - """ - - # Récupération des paramètres de la requête - formsemestre_id: int = request.args.get("formsemestre_id", -1) - moduleimpl_id: int = request.args.get("moduleimpl_id") - date: str = request.args.get("jour", datetime.date.today().isoformat()) - group_ids: list[int] = request.args.get("group_ids", None) - if group_ids is None: - group_ids = [] - else: - group_ids = group_ids.split(",") - map(str, group_ids) - - # Vérification du moduleimpl_id - if moduleimpl_id is not None: - try: - moduleimpl_id = int(moduleimpl_id) - except (TypeError, ValueError) as exc: - raise ScoValueError("identifiant de moduleimpl invalide") from exc - # Vérification du formsemestre_id - if formsemestre_id is not None: - try: - formsemestre_id = int(formsemestre_id) - except (TypeError, ValueError) as exc: - raise ScoValueError("identifiant de formsemestre invalide") from exc - - # Récupérations des/du groupe(s) - groups_infos = sco_groups_view.DisplayedGroupsInfos( - group_ids, moduleimpl_id=moduleimpl_id, formsemestre_id=formsemestre_id - ) - if not groups_infos.members: - return ( - html_sco_header.sco_header(page_title="Saisie journalière de l'assiduité") - + "

Aucun étudiant !

" - + html_sco_header.sco_footer() - ) - - # --- Filtrage par formsemestre --- - formsemestre_id = groups_infos.formsemestre_id - - formsemestre: FormSemestre = FormSemestre.query.get_or_404(formsemestre_id) - if formsemestre.dept_id != g.scodoc_dept_id: - abort(404, "groupes inexistants dans ce département") - - # Récupération des étudiants du/des groupe(s) - etuds = [ - sco_etud.get_etud_info(etudid=m["etudid"], filled=True)[0] - for m in groups_infos.members - ] - - # --- Vérification de la date --- - real_date = scu.is_iso_formated(date, True).date() - if real_date < formsemestre.date_debut or real_date > formsemestre.date_fin: - # Si le jour est hors semestre, renvoyer vers choix date - return redirect( - url_for( - "assiduites.choix_date", - formsemestre_id=formsemestre_id, - group_ids=group_ids, - moduleimpl_id=moduleimpl_id, - scodoc_dept=g.scodoc_dept, - readonly="true", - ) - ) - - # --- Restriction en fonction du moduleimpl_id --- - if moduleimpl_id: - mod_inscrits = { - x["etudid"] - for x in sco_moduleimpl.do_moduleimpl_inscription_list( - moduleimpl_id=moduleimpl_id - ) - } - etuds_inscrits_module = [e for e in etuds if e["etudid"] in mod_inscrits] - if etuds_inscrits_module: - etuds = etuds_inscrits_module - else: - # Si aucun etudiant n'est inscrit au module choisi... - moduleimpl_id = None - - # --- Génération du HTML --- - - if groups_infos.tous_les_etuds_du_sem: - gr_tit = "en" - else: - if len(groups_infos.group_ids) > 1: - grp = "des groupes" - else: - grp = "du groupe" - gr_tit = ( - grp + ' ' + groups_infos.groups_titles + "" - ) - - # Récupération du semestre en dictionnaire - sem = formsemestre.to_dict() - - return render_template( - "assiduites/pages/signal_assiduites_group.j2", - date=_dateiso_to_datefr(date), - defdem=_get_etuds_dem_def(formsemestre), - forcer_module=sco_preferences.get_preference( - "forcer_module", - formsemestre_id=formsemestre_id, - dept_id=g.scodoc_dept_id, - ), - formsemestre_date_debut=str(formsemestre.date_debut), - formsemestre_date_fin=str(formsemestre.date_fin), - formsemestre_id=formsemestre_id, - gr_tit=gr_tit, - grp=sco_groups_view.menu_groups_choice(groups_infos), - minitimeline=_mini_timeline(), - moduleimpl_select=_module_selector(formsemestre, moduleimpl_id), - nonworkdays=_non_work_days(), - sem=sem["titre_num"], - timeline=_timeline(), - readonly="true", - sco=ScoData(formsemestre=formsemestre), - title="Saisie journalière de l'assiduité", + title="Saisie de l'assiduité", ) @@ -2028,6 +1838,8 @@ def signal_assiduites_diff(): dept_id=g.scodoc_dept_id, ), nouv_plage=nouv_plage, + formsemestre_id=formsemestre_id, + group_ids=group_ids, ) @@ -2298,89 +2110,6 @@ def _get_date_str(deb: datetime.datetime, fin: datetime.datetime) -> str: return f'du {deb.strftime("%d/%m/%Y %H:%M")} au {fin.strftime("%d/%m/%Y %H:%M")}' -def _get_days_between_dates(deb: str, fin: str) -> str: - """ - _get_days_between_dates récupère tous les jours entre deux dates - - Args: - deb (str): date de début - fin (str): date de fin - - Returns: - str: une chaine json représentant une liste des jours - ['date_iso','date_iso2', ...] - """ - if deb is None or fin is None: - return "null" - try: - if isinstance(deb, str) and isinstance(fin, str): - date_deb: datetime.date = datetime.date.fromisoformat(deb) - date_fin: datetime.date = datetime.date.fromisoformat(fin) - else: - date_deb, date_fin = deb.date(), fin.date() - except ValueError: - return "null" - dates: list[str] = [] - while date_deb <= date_fin: - dates.append(f'"{date_deb.isoformat()}"') - date_deb = date_deb + datetime.timedelta(days=1) - - return f"[{','.join(dates)}]" - - -def _differee( - etudiants: list[dict], - moduleimpl_select: str, - date: str = None, - periode: dict[str, str] = None, - formsemestre_id: int = None, -) -> str: - """ - _differee Génère un tableau de saisie différé - - Args: - etudiants (list[dict]): la liste des étudiants (représentés par des dictionnaires) - moduleimpl_select (str): l'html représentant le selecteur de module - date (str, optional): la première date à afficher. Defaults to None. - periode (dict[str, str], optional):La période par défaut de la première colonne. - formsemestre_id (int, optional): l'id du semestre pour le selecteur de module. - - Returns: - str: le widget (html/css/js) - """ - if date is None: - date = datetime.date.today().isoformat() - - forcer_module = sco_preferences.get_preference( - "forcer_module", - formsemestre_id=formsemestre_id, - dept_id=g.scodoc_dept_id, - ) - - assi_etat_defaut = sco_preferences.get_preference( - "assi_etat_defaut", - formsemestre_id=formsemestre_id, - dept_id=g.scodoc_dept_id, - ) - - periode_defaut = sco_preferences.get_preference( - "periode_defaut", - formsemestre_id=formsemestre_id, - dept_id=g.scodoc_dept_id, - ) - - return render_template( - "assiduites/widgets/differee.j2", - etudiants=etudiants, - assi_etat_defaut=assi_etat_defaut, - periode_defaut=periode_defaut, - forcer_module=forcer_module, - moduleimpl_select=moduleimpl_select, - date=date, - periode=periode, - ) - - def _module_selector(formsemestre: FormSemestre, moduleimpl_id: int = None) -> str: """ _module_selector Génère un HTMLSelectElement à partir des moduleimpl du formsemestre @@ -2450,7 +2179,7 @@ def _module_selector_multiple( ) -def _timeline(formsemestre_id: int = None, heures=None) -> str: +def _timeline(heures=None) -> str: """ _timeline retourne l'html de la timeline @@ -2465,11 +2194,9 @@ def _timeline(formsemestre_id: int = None, heures=None) -> str: return render_template( "assiduites/widgets/timeline.j2", t_start=ScoDocSiteConfig.assi_get_rounded_time("assi_morning_time", "08:00:00"), + t_mid=ScoDocSiteConfig.assi_get_rounded_time("assi_lunch_time", "13:00:00"), t_end=ScoDocSiteConfig.assi_get_rounded_time("assi_afternoon_time", "18:00:00"), tick_time=ScoDocSiteConfig.get("assi_tick_time", 15), - periode_defaut=sco_preferences.get_preference( - "periode_defaut", formsemestre_id - ), heures=heures, ) @@ -2854,14 +2581,26 @@ def _generate_assiduite_bubble(assiduite: Assiduite) -> str: # Récupérer informations saisie saisie: str = assiduite.get_saisie() - motif: str = assiduite.description if assiduite.description else "" + motif: str = assiduite.description or "Non spécifié" + + # Récupérer date + + if assiduite.date_debut.date() == assiduite.date_fin.date(): + jour = assiduite.date_debut.strftime("%d/%m/%Y") + heure_deb: str = assiduite.date_debut.strftime("%H:%M") + heure_fin: str = assiduite.date_fin.strftime("%H:%M") + date: str = f"{jour} de {heure_deb} à {heure_fin}" + else: + date: str = ( + f"du {assiduite.date_debut.strftime('%d/%m/%Y')} " + + f"au {assiduite.date_fin.strftime('%d/%m/%Y')}" + ) return render_template( "assiduites/widgets/assiduite_bubble.j2", moduleimpl=moduleimpl_infos, etat=scu.EtatAssiduite(assiduite.etat).name.lower(), - date_debut=assiduite.date_debut.strftime("%d/%m/%Y %H:%M"), - date_fin=assiduite.date_fin.strftime("%d/%m/%Y %H:%M"), + date=date, saisie=saisie, motif=motif, )