Produits Entr'ouvert » Hobo

welco est déprécié aussi non ?

Benjamin Dauvergne a écrit :

J'ai ajouté un test explicite de ce qui est actuellement actif et j'ai désactivé welco (en plus de ce qui est demandé dans la description).

Et là je me demande si je ne devrais pas aussi le faire pour la commande cook...

+# List of service class names visible in the create service menu.
+HOBO_SERVICES_ENABLED = [
+    'Authentic',
+    'BiJoe',

Ça m'a l'air casse-gueule d'avoir à suivre ainsi la casse;

+        return cls.__name__ in settings.HOBO_SERVICES_ENABLED

du coup je verrais bien tout en minuscules dans les settings et un .lower() ici.

J'aurais également plutôt été à préférer l'option inverse, disabled plutôt que enabled. (dans l'idée de ne pas avoir à chercher les sites où cette option aurait été modifiée quand on décidera d'avoir lingo par défaut).

Frédéric Péters a écrit :

J'aurais également plutôt été à préférer l'option inverse, disabled plutôt que enabled. (dans l'idée de ne pas avoir à chercher les sites où cette option aurait été modifiée quand on décidera d'avoir lingo par défaut).

Pas vraiment d'accord, vu qu'on a pas de quoi retirer un élément à la liste dans config.json on peut juste ajouter ou écraser, si on fait "HOBO_SERVICES_ENABLED.extend": ["lingo"] ça n'aura pas tellement de conséquence quand on l'activera, par contre "HOBO_SERVICES_DISABLED": ["fargo", "welco", "bijoe"], si on désactive autre chose dans le futur il faudra chercher où c'est utilisé.

Du problème originel vu, c'est au niveau du cook qu'on veut (aussi) fermer la possibilité. Je crois pas que ça joue ici, cook ne semble rien vérifier.

Thomas Noël a écrit :

Du problème originel vu, c'est au niveau du cook qu'on veut (aussi) fermer la possibilité. Je crois pas que ça joue ici, cook ne semble rien vérifier.

Ok j'ajoute le check sur _enabled, mais tu veux une erreur ou juste ignorer ?

Benjamin Dauvergne a écrit :

Thomas Noël a écrit :

Du problème originel vu, c'est au niveau du cook qu'on veut (aussi) fermer la possibilité. Je crois pas que ça joue ici, cook ne semble rien vérifier.

Ok j'ajoute le check sur _enabled, mais tu veux une erreur ou juste ignorer ?

Idéalement, comme pour les vérifications DNS et certif, ça serait bien quelque chose qui stoppe tout dès la début ; même si ça me semble pas si facile dans ce cas. Au pire on ignore avec un petit warning et hop.

0001 :

Petit un soucis de commentaire :

# List of service class names visible in the create service menu.  <- c'est l'inverse
HOBO_SERVICES_DISABLED = [ ...

Et en profiter pour ajouter « HOBO_SERVICES_ENABLED = [] », histoire de le "documenter" (dire que ça a priorité), et ainsi te permettre de virer le getattr dans le code.

0002 et 0003, jolis.

0004 : je mettrais le test des commandes create_X avant les vérifications d'URL ; histoire de ne pas vérifier les URL des services désactivés.

Le reste me semble tout ok.

Automatic expiration