search

betagouv / rdv-service-public

Prise de RDV pour les services publics
https://rdv.anct.gouv.fr
GNU Affero General Public License v3.0
12 stars 2 forks source link

Ne pas utiliser NotImplementedError #4374

Closed francois-ferrandis closed 1 week ago

francois-ferrandis commented 1 week ago

Pour résumer, cette exception n'hérite pas de StandardError, qui est la classe implicite d'un rescue, et donc il est dangereux de l'utiliser. Plus de détails dans cet article.

francois-ferrandis commented 1 week ago

👍 j’ignorais de le savoir !

y a-t-il une raison de ne pas corriger de la même manière dans les deux cas ? dans un cas tu supprimes la méthode, dans l’autre tu remplace par un raise sans classe.

je n’ai pas d’avis fort dans une direction ou une autre mais je ne vois pas pourquoi c’est géré différemment

Je donne l'explication dans un commentaire, j'aime bien faire ça pour donner du contexte aux explications : https://github.com/betagouv/rdv-service-public/pull/4374/files#r1654497452

adipasquale commented 1 week ago

ah dans un commentaire github ! j’ai mis un moment à chercher le commentaire dans le code

🚀