Skip to content

extract rdv#to_query to wizards objects

Vincent Agnano requested to merge refactor/extract-rdv-to-query into master

Created by: adipasquale

encore un mini refacto preliminaire pour https://github.com/betagouv/rdv-solidarites.fr/pull/987 !

Je m'apprete a rajouter des methodes dans le modèle RDV et ça fait dépasser la taille raisonnable acceptée par le linter. Plutôt que déplacer mes nouvelles méthodes dans un concern pour éviter ce problème, je propose de sortir une fonction qui je pense n'a pas sa place dans ce modèle : to_query.

C'est une méthode utilisée pour générer des liens avec des query params correspondant à un RDV. C'est donc plus une méthode de présentation que de modèle. Elle est exclusivement appelée et surchargée depuis les form models user rdv wizard et agent rdv wizard. J'ai inliné et simplifié cette génération de query params, par ex côté user il n'y a pas besoin (et c'est un peu dangereux) de passer les paramètres agent_ids ou context.

Merge request reports