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

Problème compatibilité PluXml 5.5 #2

Open
Deevad opened this issue Jul 9, 2016 · 11 comments
Open

Problème compatibilité PluXml 5.5 #2

Deevad opened this issue Jul 9, 2016 · 11 comments

Comments

@Deevad
Copy link

Deevad commented Jul 9, 2016

Bonjour,
Deux soucis sont apparus avec la version de PluXml 5.5 rendant le plugin un beu boiteux:

1. 'Unknown sender'
Le mail envoyé n'a plus d'envoyeur ( eg. dans Gmail ci-dessous )
2016-10-07_selection_001

2. Liens cassé
Les liens dans l'email pour répondre au message sont cassés.

@Pluxopolis
Copy link
Owner

Bonjour David
Peux-tu essayer en reportant ce correctif
pluxml/PluXml#174
merci

@Deevad
Copy link
Author

Deevad commented Jul 10, 2016

Bonjour Stéphane,
Je vais essayer ça tout de suite; merci et je donnerai des nouvelles en fin de soirée, le temps de recevoir un paquet de nouveaux commentaires.

@Deevad
Copy link
Author

Deevad commented Jul 11, 2016

J'ai fait la modification en faisant un simple cherrypick de pluxml/PluXml@f7a9459 sur 5.5 ; mais ça n'a pas eu d'effet sur le plugin. Voici encore l'affichage ce matin dans ma boite email:

2016-11-07_selection_001

@pluxml
Copy link
Contributor

pluxml commented Jul 11, 2016

Ha ok, Je n'avais pas compris le problème.
Je vais essayer de reproduire le bug

@pluxml
Copy link
Contributor

pluxml commented Jul 11, 2016

A tout hasard, comme j'utilise aussi ce plugin sur mon site Pluxopolis et que je n'ai pas le bug, n'aurais-tu pas un autre plugin qui viendrait perturber son fonctionnement ?

@pluxml
Copy link
Contributor

pluxml commented Jul 11, 2016

Je viens de refaire un test avec un commentaire bidon. Je reçois bien le mail dans gmail avec un nom de destinataire

@Deevad
Copy link
Author

Deevad commented Jul 11, 2016

Merci pour la confirmation que ça fonctionne bien par chez toi.

Je vais essayer de mettre le plugin en position n°1 dans le chargement des plugins. Une collision de plugin est une bonne piste, merci. Je donnerai des nouvelles.

@Deevad
Copy link
Author

Deevad commented Jul 11, 2016

Je viens de vérifier la 'configuration' du plugin et honte à moi: les champs sont incomplet.

2016-11-07_selection_007

J'ai pourtant bien un data/configuration/plugins/plxMyMailComment.xml mais il est possible que des champs aient disparu dans une mise à jour ( de ma part, ou part des bidouillages automatiques des fichiers *.xml ). Je vais remplir le formulaire correctement; ça devrait résoudre le problème.

@pluxml
Copy link
Contributor

pluxml commented Jul 11, 2016

Ha super; J'espère que ça va fonctionner avec ça

@Deevad
Copy link
Author

Deevad commented Jul 12, 2016

Ca fonctionne parfaitement. Merci!

Dernier soucis: l'URL pour la réponse directe
2016-12-07_selection_009

Le plugin retourne une URL de ce type qui ne fonctionne pas :
http://www.yourdomain.com/core/admin/comment_new.php?c=0377.0377-369

Dans l'admin, c'est ce genre d'URL qui fonctionne avec PluXml 5.5 :
http://www.yourdomain.com/core/admin/comment_new.php?c=0377.1468274242-369

Je vois bien que c'est cette ligne 52 qui semble être responsable dans plxMyMailComment.php:

$eBody .= "<a href=\"".$this->racine."core/admin/comment_new.php?c=".$this->plxRecord_arts->f("numero").".".substr($retour,1)."\">'.$this->getLang("L_EMAIL_DIRECT_ANSWER").'</a>";

Mais je n'ai pas su adapter l'URL.

@Deevad
Copy link
Author

Deevad commented Jul 12, 2016

Note: j'ai finalement changer l'URL par ceci ci-dessous, ce qui pointe vers les commentaires de l'article cible ( http://www.yourdomain.com/core/admin/comment_new.php?a=0377 ) , temporairement plus pratique :)

$eBody .= "<a href=\"".$this->racine."core/admin/comment_new.php?a=".$this->plxRecord_arts->f("numero")."\">'.$this->getLang("L_EMAIL_DIRECT_ANSWER").'</a>";

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants