Projet

Général

Profil

Bug #55743

Caluire Axel: ne pas retourner de réservation sur les ativités fermées

Ajouté par Nicolas Roche il y a plus de 2 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
21 juillet 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Exploiter l'attribut FERME disponible dans les données retournées par GETAGENDA,
pour ne pas afficher les activités périscolaires pendant les vacances scolaires.


Fichiers

Révisions associées

Révision 7adab0d8 (diff)
Ajouté par Lauréline Guérin il y a plus de 2 ans

caluire-axel: hide closed days (#55743)

Historique

#1

Mis à jour par Nicolas Roche il y a plus de 2 ans

#3

Mis à jour par Thomas Noël il y a plus de 2 ans

« Voici un patch naïf que me semble fonctionner. » c'est vraiment pas rassurant :-) Serait-ce d'abord testable «à l'arrache» sur la recette (patch manuel du code + reload de passerelle sur les deux nodes) ?

Par ailleurs, dans le XML je vois <FERME>O</FERME>, mais quid de <FERME>N</FERME> ? Peut-être que « if day.get('FERME') == 'O': continue » serait plus raisonnable.

#4

Mis à jour par Nicolas Roche il y a plus de 2 ans

Merci, cet attribut est effectivement optionnel :

'{urn:AllAxelTypes}ONEmptyType': 'bool_optional',

(par contre il est bien traduit en booléen par le schéma XSD, j'ai précisé "is True" comme tu me l'as conseillé.)

#5

Mis à jour par Thomas Noël il y a plus de 2 ans

Nicolas Roche a écrit :

(...j'ai précisé "is True" comme tu me l'as conseillé.)

Et en fait faut pas écouter toujours mes conseils :) Maintenant que j'ai relu l'ensemble je comprends bien que c'est un booléen qu'on va recevoir et qu'il ne sera vrai que si c'est "O" et donc, tout roule, if day.get('FERME'): continue sera plus propre. (désolé je change trop d'avis trop souvent trop rapidement).

Je me pose quand même une question, parce que tout ça me semble un peu bizarre quand même : c'est sûr et certain qu'on ne veut pas plutôt faire un item avec disabled=True dans ce cas, comme on le voit plus bas dans le code ? Ca sonne plus logique : c'est fermé = c'est désactivé.

#7

Mis à jour par Stéphane Guiet il y a plus de 2 ans

si une activité "vacances" ne remonte QUE dans sa plage calendaire (du premier jour des vacances au dernier jour)

Nous avons demandé à Caluire d'inscrire Enfant 1 aux vacances d'automne et de Noël pour vérifier l'hypothèse
Et si les vacances sont bien bornées, ta moulinette est une bonne idée.

Le plus simple je crois c'est d'accepter de voir les mercredi et les activités perisco grisées pendant les vacances ...

Bof ... pour ma part je préfère que le férié ou un jour de fermeture exceptionnel soit absent plutôt que de voir un affichage temps périscolaires et journée d'accueil de loisirs tout le mois de juillet.

#8

Mis à jour par Nicolas Roche il y a plus de 2 ans

Patch corrigé au cas où, mais bon, on s'oriente dans une autre voie.

#9

Mis à jour par Lauréline Guérin il y a plus de 2 ans

J'étais partie dans une autre branche à carrément ignorer tout ce qui est calculé 'grey' (calcul dans lequel FERME n'intervient pas, car on a une corrélation entre FERME = True et une valeur de MATIN qui donnera grey), puis après discussion avec Stéphane, marche arrière: on veut que les jours en MATIN=F/S/T (grey) remontent (ils seront en FERME = False)

J'ai modifié le test et l'ai déplacé dans les tests get_agenda

#10

Mis à jour par Nicolas Roche il y a plus de 2 ans

  • Statut changé de Solution proposée à Solution validée
#11

Mis à jour par Lauréline Guérin il y a plus de 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 7adab0d8c829d2b5e23aab70236134a11f33d2c0
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date:   Mon Jul 26 15:54:17 2021 +0200

    caluire-axel: hide closed days (#55743)
#12

Mis à jour par Frédéric Péters il y a plus de 2 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF