Plugin Dayinfo et core Jeedom

Salon des Bêta-Testeurs Plugins Tiers (V4)
1

Salut @lunarok

J’ai regardé rapidement ton code et je pense qu’il y quelques cas qui passent à la trappe avec les informations du core (coordonnées, pays etc)

Chez moi j’ai mis le pays (stateCode) à « FR »…
image
Dans ton code, c’est en minuscule (et tu fais un strtolower)

    public function isNotWorkable(){
            if ($stateCode == 'fr') {
                $country = "france";

Par ailleurs, on trouve de temps en temps des majuscules

            } else if ($stateCode == 'CA') {
                $country = "canada";

Dans tous les cas, canada ne doit pas bien fonctionner…

Il me semble que la norme est en majuscule, non ?

La question se pose aussi coté @Loic ne faut-il pas forcer les majuscules et se mettre d’accord ?
A vous lire

2

Tu l’as dit toi même, je fais un strtolower avant le test. Pour éviter de chercher justement.
Par contre oui, CA du coup ca marche pas, je corrige.
Et le problème remonté sur le forum (par les end user après une release en stable) vient qu’il en manque un sur le pays pour les geotrav

3

Parfait, ça fera des soucis de moins.