Project

General

Profile

Bug #55743

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

Added by Nicolas Roche 10 days ago. Updated 4 days ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
Target version:
-
Start date:
21 Jul 2021
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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.


Files

Associated revisions

Revision 7adab0d8 (diff)
Added by Lauréline Guérin 5 days ago

caluire-axel: hide closed days (#55743)

History

#1

Updated by Nicolas Roche 10 days ago

#3

Updated by Thomas Noël 10 days ago

« 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

Updated by Nicolas Roche 10 days ago

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

Updated by Thomas Noël 9 days ago

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

Updated by Stéphane Guiet 9 days ago

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

Updated by Nicolas Roche 9 days ago

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

#9

Updated by Lauréline Guerin 5 days ago

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

Updated by Nicolas Roche 5 days ago

  • Status changed from Solution proposée to Solution validée
#11

Updated by Lauréline Guerin 5 days ago

  • Status changed from Solution validée to 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

Updated by Frédéric Péters 4 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF