Toilettage du module TransportWeb.API.StatsController

[vdegove]

Remplace la PR #4429 : les mêmes modifications de code sont appliquées, sans bouger le code de module.

Les modifications :

• On extrait une vraie fonction (au lieu d’une fonction anonyme) qui est appelée par le cache, pour pouvoir par la suite réaliser un cache préemptif dessus
• Cette fonction prend en argument un atom qui désigne la fonctionnalité au lieu de prendre une query Ecto
• J’ai bougé dans cette fonction l’appel à la base de données DB.Repo.all() au lieu de l’avoir plus loin dans les sous-fonctions.
• Le fonctionnement des stats de vélo libre service est unifié avec celui des autres stats, avec des étapes similaires dans leur ordre

Pour le coup ça peut se lire commit par commit.