[Plugin Tiers][Sujet Principal] Plugin Veolia eau

[damien916]

Ah tiens, je jetterai un œil aux logs demain, tu as l'erreur depuis quand ?

[Spilarix]

Je viens de voir que c'est spécifique ToutSurMonEau. Le fichier excel ne doit pas lui plaire.
Au passage, je viens de voir qu'il n'est désormais plus possible de mettre un fake id pour récupérer le fichier avec les données.
Attends toi à voir arriver des PR

[damien916]

Ok
Pour les pr je te conseil un plugin qui s'appelle outilsdev pour envoyer des fichiers et tester en local, ça sera surement plus rapide pour toi que d'attendre le merge
Bon courage

[Spilarix]

Le bug provient à priori du nom de la feuille Excel lors de l'exécution de cette ligne :
$sheetData = $objPHPExcel->getActiveSheet()->toArray(null,true,true,true);
Il faut que je trouve comment nettoyer les caractères refusés.

Pour le token devenu obligatoire, il va falloir que je scrap un lien pour en extraire cet id.

Je vais regarder pour ce plugin qui simplifierait beaucoup oui

[damien916]

Si tu pouvais m'envoyer en dm un fichier je pourrais comparer à ceux de chez Veolia, l'encodage n'est peut être pas le même

[Spilarix]

Bonjour,

Le problème des caractères sera réglé avec la PR que je viens de faire.
Il ne reste plus que le token de téléchargement à gérer.

[damien916]

Hello, ok top, je vais merger dès que j'aurais le temps

[Naboleo]

Pas de précipitation de mon coté, pour l'instant les mises à jour semblent difficiles (et c'est pas lié au plugin, c'est pareil avec celui des volets)
https://www.jeedom.com/market/core/api/api.php

Code : Tout sélectionner

[2017-12-15 09:07:11][ALERT] : [START UPDATE]
[2017-12-15 09:07:11][ALERT] : Début de la mise à jour de : veolia_eau
[2017-12-15 09:07:11][ALERT] : Action de pre update...
[2017-12-15 09:07:12][ALERT] : OK
[2017-12-15 09:07:12][ALERT] : Téléchargement du plugin...
[2017-12-15 09:07:22][ALERT] : Erreur curl sur : https://www.jeedom.com/market/core/api/api.php. Détail :Operation timed out after 10001 milliseconds with 0 bytes received
[2017-12-15 09:07:22][ALERT] : [END UPDATE ERROR]hp. Détail :Operation timed out after 10001 milliseconds with 0 bytes received

[damien916]

Salut @spilarix, je viens de merger et de pousser en bêta.
Bon week-end

[Spilarix]

Bonjour Damien,

"outilsdev" m'aide beaucoup, merci pour l'info.
En gérant le token de ToutPourMonEau et en voulant mutualiser du code, j'ai remarqué une incohérence potentielle pour le cas "eau-services" :
Dans sa config (méthode getConso, case 2), il est question que l'export du site soit un csv et dans TraiteConso, on traite un fichier html.
Est-ce bien la config qui n'est plus d'actualité ? (Si oui, merci de ne pas modifier, je corrigerai dans ma prochaine PR).

En fonction de ta réponse, j'adapterai le code.

Merci

[Spilarix]

Je viens de faire une PR qui devrait être la dernière.
Bien que j'ai testé et que ce soit fonctionnel, je retesterai dès que tu l'auras passé en bêta, afin d'être certain de ne pas avoir fait de betises lors du merge.

Bonne soirée !

[damien916]

Salut Spilarix,
Concernant "eau-services" (Veolia méditerranée), je ne suis pas à l'origine de cette partie, c'est @frederic34 qui avait fait une PR pour ajouter ce service, du coup je n'ai aucune idée du fichier exporté.

Quelqu'un qui utilise ce plugin pour Veolia Méditerranée peut-il confirmer que ça fonctionne actuellement ? et nous informer sur le type de fichier exporté ? Merci d'avance.

[Spilarix]

J'ai traité ToutSurMonEau comme un cas particulier qui n'écrit pas la page de consommation, afin d'éviter toute régression sur les autres services.
Dans les faits, je pense que seul eau-service est peut-être un cas particulier et que les autres pourraient s'éviter également l'écriture inutile.

[puppet]

Bonjour,

depuis quelques jours, sur le plugin j'ai l'erreur

Code : Tout sélectionner

[2017-12-17 06:00:05][ERROR] : Erreur sur la fonction cronHourly du plugin : Call to undefined function mb_convert_encoding()

j'interroge tout sur mon eau... J'ai vu que d'autre avais des soucis, si je peut fournir d'autres infos, dites moi quoi...

Merci pour le dev.

Puppet

[Spilarix]

ToutSurMonEau a changé de site internet.
La nouvelle version du plugin pour prendre en compte les changements est actuellement en beta.
J'ai proposé a Damien les dernières modifications techniques. Dès qu'elles seront intégrées, je retesterai que c'est tout bon et ça passera ensuite en stable pour tout le monde.

Patience donc

Bonne soirée.

[damien916]

Bonjour @puppet,
Sur un post précédant, quelqu'un avait eu un soucis avec une fonction php "mb_", cette personne avait résolu le problème un installant un paquet supplémentaire il me semble, il faudrait retrouver le post.

[cadavor]

Pour eau-services.com après plusieurs d'erreurs : "date invalide - impossible de trouver 2 slash :"

Je me suis rendu sur le site pour vérifier les accès et le fichier CSV téléchargé.
Je me suis alors rendu compte qu'il me demandait de changer mon mot de passe.

Après la chose faite et la mise à jour du plugin, tout fonctionne à nouveau chez moi.

Merci.

[Spilarix]

@Damien Peux-tu passer ma PR afin que je la valide ?

[damien916]

Hello,
@spilarix J'ai mergé ta PR (en bêta)
@puppet J'ai ajouté un polyfill, qui pourra peut-être fixer le problème (en bêta également).

Je passerai en stable dès que j'aurais au moins un retour positif sur chaque service.

Bonne soirée à tous

[damien916]

Bonjour à tous,
Le télé relevé Veolia ne fonctionne pas chez moi ce matin (bêta), je vais regarder pourquoi.
Bonne journée