Présentation MeteoFull

Merci :wink:

L’erreur venait du core et doit être corrigée maintenant.

1 « J'aime »

T’étais pas plutôt en 4.1.17 ??
J’ai modifié le numéro ce matin.
Et le bug viendrait d’un PR accepté en début d’aprem :sneezing_face:

Tu peux updater le core et re-tester ?

exact… tu as raison. je venais de passer en 4.1.1.17a
je viens de recharger le core et c’est OK… Merci

Cool merci pour la confirmation.

Ton linux est pas à jour :joy:

J’ai aussi eu un souci de proprio sur le répertoire data ce midi…
Par contre 777, c’est pas une bonne idée… Other a rien à faire d’un accès complet à un répertoire, 770 c’est déjà bien assez

@naboleo

je suis assez d’accord… mais c’est codé comme ça dans le plugin :thinking: :face_with_raised_eyebrow:

Je note merci pour le retour.

1 « J'aime »

Bonjour
Je suis ravis et tu filicite pour la beauté de ton travail :herb: :rainbow: :star2: :star2: :star2:
cordialement

1 « J'aime »

Alors pour information, si je mets 770, cela ne marche pas donc pas le choix.

Je ferai un test demain, mais ça ne me semble pas logique que ça ne marche pas (enfin sauf si les répertoires sont créer en root ce qui est certainement le cas à cause du sudo)

oui il sont bien créé en root et donc il faut les passer en www-data!

Donc autant les créer avec www-data directement (il faut virer le sudo du mkdir)

Et dans ce cas tu fais comment ?

Www-data c’est le user par défaut de jeedom. Donc il faut juste retirer sudo de tes commandes

Désolé mais c’est quelque chose que je ne maîtrise pas trop. Peux-tu donner un exemple ? Merci d’avance.

apres le sudo chmod… tu fais:
sudo chown -R www-data:www-data /var/www/html/plugins/MeteoFull/

Tu as dans ton code :

sudo -s mkdir /var/www/html/plugins/MeteoFull/core/class/…/…/data > /dev/null 2>&1;

Ça doit devenir

mkdir /var/www/html/plugins/MeteoFull/core/class/…/…/data > /dev/null 2>&1;

Et je suis presque sûr que la commande chmod 777 juste en dessous ne sera plus utile
Pas de pc sous la main pour confirmer sur le moment

Merci les gars. Je regarde ça demain et dans ce cas je mettrais ça à jour si cela corrige le souci.

Mise a jour de ce matin

tous les tooltips indiquent depasse le seuil, souci avec icones en bas