Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Liste appels dans une seule table #2

Closed
wants to merge 2 commits into from
Closed

Liste appels dans une seule table #2

wants to merge 2 commits into from

Conversation

jpty
Copy link
Collaborator

@jpty jpty commented Dec 12, 2019

Les requêtes à pagesjaunes.fr sont désactivées

@jmvedrine
Copy link
Owner

Un grand merci pour ton PR.
C'est une fonctionnalité très intéressante et qui j'en suis sûr plaira beaucoup.
Je t'en attribuerai la paternité dans dans la doc.
J'avais bien vu dans ton ancien PR à Jeedom le setlocale(LC_TIME,"fr_FR.utf8"); mais je ne l'avais pas mis car j'avais peur de perturber un strftime ailleurs dans le code de Jeedom (j'ai contribué à d'autres projets libres comme Moodle et on m'a appris à ne jamais faire un truc comme un setlocale dans un plugin). Ceci dit le risque est faible et c'est moche sans je te l'accorde.
Pour ce qui es de la recherche sur pages jaunes, je ne suis pas emballé par l'idée d'un fichier CSV, je pense que je vais plutôt faire une table dans la base (d'autres plugins le font par exemple calendar ou suivi conso).
Mon idée est dans le pull de ne chercher que le nom du dernier appel et de faire un cron toutes les 5 ou 15 minutes qui ira chercher quelques noms manquants pour "rattraper le retard".
Comme çà le plugin devrait éviter de se faire blacklister sur le site pages jaunes.

@jmvedrine
Copy link
Owner

Aussi plutôt que d'utiliser une variable comme ton $_PagesJaunes je vais en faire une case à cocher dans la configuration du plugin

@jpty
Copy link
Collaborator Author

jpty commented Dec 13, 2019

Bonjour,

C'est dommage que le setlocale ne soit pas fait par le core de Jeedom avec la bonne valeur!
Ca oblige à faire une bidouille avec des listes de jours de la semaine, des listes de mois, abbréviés ou pas.
C'est ce qui a donné dans le core jui pour juin et jui pour juillet cet été pour le plugin agenda.
Voir : jeedom/core#1348

Pas de souci pour remplacer le fichier CSV par une table. J'ai fait ce que je maitrisais le plus.
Il faut aussi donner l'accés aux utilisateurs pour entrer les noms et tél des personnes connues.

Ce n'est peut-etre pas la peine de chercher à rattraper le retard. Il se rattrapera tout seul. Il n'y a que 60 appels. (= une vingtaine de jours chez moi )

Oui, il faut faire une case à cocher pour configurer l'appel à pagesjaunes.

Merci pour ma nouvelle paternité. ;-)
JP

@jpty
Copy link
Collaborator Author

jpty commented Dec 13, 2019

Pendant que vous etes dans la configuration du plugin, il y faudrait peut-etre ajouter un autre parametre.
Dans ma liste d'appels entrants, j'en ai à peu pres la moitié qui sont des appels qui dure moins de 4secondes. J'en ai meme un avec une durée de 0s.
Tous ces appels correspondent au décrochage par mon répondeur. Entendant que c'est un répondeur, les personnes raccrochent. Pour moi ce sont des appels manqués.
Le paramètre auquel je pensais serait cette durée à partir de laquelle ce serait un vrai appel entrant.
En dessous de cette valeur, c'est un appel manqué.

@jmvedrine
Copy link
Owner

J'ai fait des essais de fonctions pour récupérer un nom avec l'annuaire inverse de pages jaunes en utilisant curl et visiblement je fais une bétise ou je me fait repérer par la protection anti-robot car je reçois des forbidden en réponse.
Une recherche sur Internet n'a pas donné grand chose.
Si tu utilises curl, que met tu dans les headers et dans le corps ? Sinon quelle fonction ou quelle librairie utilises-tu ?
Merci de ton aide.

@jpty
Copy link
Collaborator Author

jpty commented Dec 15, 2019

Voici la partie interrogation de pagesjaunes quand le telephone n'est pas dans le CSV:

Je vous l'envoie par Message Direct sur Community.
Je n'arrive pas à le formatter correctement ici

@jpty
Copy link
Collaborator Author

jpty commented Dec 30, 2019

Résolu sans utiliser ce PR

@jpty jpty closed this Dec 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants