Bug #55743
Caluire Axel: ne pas retourner de réservation sur les ativités fermées
0%
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
Historique
Mis à jour par Nicolas Roche il y a plus de 2 ans
- Fichier 0001-caluire-axel-hide-closed-activities-on-get_agenda-55.patch 0001-caluire-axel-hide-closed-activities-on-get_agenda-55.patch ajouté
- Tracker changé de Support à Bug
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
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.
Mis à jour par Nicolas Roche il y a plus de 2 ans
- Fichier 0001-caluire-axel-hide-closed-activities-on-get_agenda-55.patch 0001-caluire-axel-hide-closed-activities-on-get_agenda-55.patch ajouté
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é.)
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é.
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.
Mis à jour par Nicolas Roche il y a plus de 2 ans
- Fichier 0001-caluire-axel-hide-closed-activities-on-get_agenda-55.patch 0001-caluire-axel-hide-closed-activities-on-get_agenda-55.patch ajouté
Patch corrigé au cas où, mais bon, on s'oriente dans une autre voie.
Mis à jour par Lauréline Guérin il y a plus de 2 ans
- Fichier 0001-caluire-axel-hide-closed-days-55743.patch 0001-caluire-axel-hide-closed-days-55743.patch ajouté
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
Mis à jour par Nicolas Roche il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
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)
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
caluire-axel: hide closed days (#55743)