Produits Entr'ouvert » Passerelle

Pas encore prêt mais ça commence à ressembler à quelque chose.

Si quelqu'un peut déjà jeter un œil ce serait super (parce déjà 700 lignes ... joie).

Lecture rapide,

# passerelle - uniform access to multiple data sources and services
# Copyright (C) 2018 Entr'ouvert

En début de fichier. (et un jour faudra corriger la quantité de fichiers qui ne suivent pas ça).

+TZ = timezone('Europe/Paris')  # FIXME, configuration ?

Au cas où le debian/settings.py pose déjà TIME_ZONE = 'Europe/Paris'.

+        raise APIError("Invalid time format : %s" % time_str, log_error=True)

Pas d'espace avant : en anglais.

Tu fais systématiquement log_error=True. Est-ce parce que tu as une utilisation particulière de cette exception, ou parce que tu penses que ça devrait être le cas partout ? (je serais alors pour discuter de l'adaptation du comportement par défaut).

+def _parse_time(time_str):
...
+def build_createreservation_params(data):
+    date_str = data['date']
+    start_date = parse_date(date_str)
+    if start_date is None:
+        raise APIError("Invalid date string: %s" % date_str, log_error=True)

J'adopterais la même pratique pour parse_date et parse_time, plutôt que factoriser parse_time mais répéter la gestion d'erreur du parse_date.

+    keep_cookies = models.BooleanField(
+        verbose_name=_('Keep HTTP cookies'), default=True)

Il y a des raisons pour parfois être mis à False ?

+    verify_cert = models.BooleanField(
+        default=True, verbose_name=_('Check HTTPS Certificate validity'))

Si toutes les instances rencontrées avaient un certificat valide, je zapperais ce paramètre.

+    class Meta:
+        verbose_name = _('Planitech connector')

Ne répétons pas "connector" dans les noms des connecteurs. (oui ici aussi il y aurait quelques connecteurs à corriger)

+    def _login(self):
...
+        response = self.requests.get(auth_url, headers={'MH-PASSWORD': hash_pass})

Je trouve curieux de prendre la response puis que la fonction se termine sans rien en faire.

+    def createreservation(self, request):
+        data = validate_json(request, CREATE_RESERVATION_SCHEMA)

C'est voyant ça que j'ai attaché un patch rapide à #25590. Si on décide d'adopter json schema, faisons-le officiellement, encourageons la pratique, partageons le code.

+    def daily(self):

Ça m'ennuie d'avoir ce code là-dedans; je préférerais une méthode au nom explicite, genre cache_places_info, et le daily qui l'appelle, si ça a un sens.

+    @endpoint(methods=['get'], perm='can_access')
+    def updateplaces(self, request):
+        self.daily()

Ou updateplaces comme nom explicite, même si j'aime moins.

À noter aussi que w.c.s. dispose (désormais) d'un cache sur les datasources, peut-être que ça rend inutile de l'avoir côté Passerelle.

+++ b/passerelle/contrib/planitech/mste.py
@@ -0,0 +1,267 @@

no comment sur ce contenu, mais quand même tapé notre entête passerelle/licence en haut de fichier.

+    def getfreegaps(self, request):

Je ne sais pas quel va être l'usage du connecteur, ni précisément ce que retourne cet endpoint, mais si jamais c'est proche de l'idée "créneau libre" de chrono, pour profiter automatiquement du widget adapté côté w.c.s., le contenu pourrait être transformé pour être équivalent à ce que produit chrono.

(revu avec Emmanuel, simplification drastique de l'affaire à faire, un nouveau plan est en cours qui devrait rendre la lecture du code plus claire)

1. passerelle - uniform access to multiple data sources and services

(attention au s final, absent de la branche)

Je marquerais pour traduction les descriptions des endpoints.

Quid des FIXME qui restent dans le code ?