Le 22/03/2018 à 19:55, spip-zone-commit@rezo.net a écrit :
Author: nicolas.dorigny@gmail.com
Date: 2018-03-22 19:55:54 +0100 (Thu, 22 Mar 2018)
New Revision: 109634Modified:
_plugins_/formidable/trunk/paquet.xml
_plugins_/formidable/trunk/traiter/email.php
Log:
Ça ne pouvait pas marcher, vu que le tableau $corps est défini plus bas.Details: Connexion · GitLab
Salut,
juste oour l'avenir : c'est possible d'avoir des informations plus précise que "ca" 
?
++
Maïeul
Le 23/03/2018 à 13:20, Maïeul a écrit :
juste oour l'avenir : c'est possible d'avoir des informations plus précise que "ca"
Ha ben tu tombes bien toi 
C'est justement sur ton commit que je suis revenu ^^
Tu déclarais $corps['repondre_a'] = $courriel_envoyeur_accuse;
Ce qui ne pouvait pas marcher, car le tableau $corps était initialisé en dessous :
Ça merdait dans le cas ou tu déclarais un nom et un email pour l'expéditeur de l'accusé de réception, mais que tu ne cochais pas la case "Insérer le courriel de l’envoyeur dans le champ From"
Ça n'ajoutait pas l'email de l'expéditeur en reply-to, comme c'était censé le faire.
C'est assez pervers d'ailleurs, cette case est placée beaucoup plus haut, les utilisateurs n'avaient pas compris que c'était lié (moi non plus, j'ai du décortiquer pour comprendre).
--
nicod_
Le vendredi 23 mars 2018 à 21:52 +0100, nicod_ a écrit :
Le 23/03/2018 à 13:20, Maïeul a écrit :
>
> juste oour l'avenir : c'est possible d'avoir des informations plus
> précise que "ca"
Ha ben tu tombes bien toiC'est justement sur ton commit que je suis revenu ^^
Tu déclarais $corps['repondre_a'] = $courriel_envoyeur_accuse;
Connexion · GitLab
k/traiter/email.php?rev=109568#L276Ce qui ne pouvait pas marcher, car le tableau $corps était initialisé
en
dessous :
Connexion · GitLab
k/traiter/email.php?rev=109568#L306
Ça merdait dans le cas ou tu déclarais un nom et un email pour
l'expéditeur de l'accusé de réception, mais que tu ne cochais pas la
case "Insérer le courriel de l’envoyeur dans le champ From"
Ça n'ajoutait pas l'email de l'expéditeur en reply-to, comme c'était
censé le faire.C'est assez pervers d'ailleurs, cette case est placée beaucoup plus
haut, les utilisateurs n'avaient pas compris que c'était lié (moi non
plus, j'ai du décortiquer pour comprendre).
Ah non désolé dans mon commit je ne déclarais pas
$corps['repondre_a']
cela devait être un bug plus ancien,
c
mais effectivement c'est pervers et bien caché. Merci pour les
corrections
Maïeul
Le 24/03/2018 à 08:33, Maïeul Rouquette a écrit :
Ah non désolé dans mon commit je ne déclarais pas
$corps['repondre_a']cela devait être un bug plus ancien,
Au temps pour moi, me suis trompé de n° de commit, c'était il y a 3 ans
--
nicod_
Le samedi 24 mars 2018 à 11:09 +0100, nicod_ a écrit :
Le 24/03/2018 à 08:33, Maïeul Rouquette a écrit :
>
> Ah non désolé dans mon commit je ne déclarais pas
> $corps['repondre_a']
>
> Connexion · GitLab
>
> cela devait être un bug plus ancien,
Au temps pour moi, me suis trompé de n° de commit, c'était il y a 3
ans
ok
merci en tous cas