je développe actuellement un plugin nécessitant SPIP-Listes et je me
suis rendu compte de plusieurs éventuels problèmes. Etant nouveau dans
la communauté je préfère poser la question avant d'envisager tout
commit sur le svn...
Les 3 problèmes que j'ai trouvé :
1) Un auteur qui a été supprimé dans l'administration (marqué
'5poubelle' dans la table spip_auteur) ne peut pas se réinscrire à une
liste.
2) Le champ 'From' des emails envoyés utilisent la valeur de
'stmp_sender' et non pas de 'email_defaut' même si la méthode d'envoi
est réglé sur 'Utiliser la PHP mail() fonction'. Cela oblige d'activer
la méthode d'envoie SMTP, de modifier le champ 'stmp_sender' avant de
sauvegarder et de repasser en 'Utilisé la PHP mail() fonction'.
3) Même chose pour le champ 'Sender' des emails envoyés qui utilisent
la valeur de 'stmp_sender' même si la méthode d'envoi est réglé sur
'Utiliser la PHP mail() fonction'. Pareil cela oblige d'aller activer
la methode SMTP pour modifier le champ avant de revenir en 'PHP mail()
fonction'.
Mes idées :
1) Permettre à un un auteur qui aurait été supprimé dans
l'administration (marqué '5poubelle' dans la table spip_auteur) de se
réinscrire à une liste (je n'ai pas encore regardé le code...).
2) inc/spiplistes_api_globales.php, fonction
spiplistes_email_from_default(), ligne 260, remplacer :
if(
// prendre d'abord celui du sender SMTP si il est activé
(($result = email_valide($ii = trim($GLOBALS['meta']['smtp_sender'])))
// sinon celui par defaut de SPIP-Listes
|| ($result = email_valide($ii = trim($GLOBALS['meta']['email_defaut'])))
// au pire, celui du webmaster
|| ($result = email_valide($ii = trim($GLOBALS['meta']['email_webmaster'])))
)
par
if(
// prendre d'abord celui du defaut de SPIP-Listes
(($result = email_valide($ii = trim($GLOBALS['meta']['email_defaut'])))
// sinon celui du webmaster
|| ($result = email_valide($ii = trim($GLOBALS['meta']['email_webmaster'])))
)
3) Il faudrait créer un champ Sender dans l'administration au même
titre que 'Adresse d'envoi par défaut' et que 'Adresse de retour par
défaut pour les erreurs' afin qu'il ne soit plus dépendant de la
méthode d'envoi SMTP.
Donnez moi vos avis, je suis prêt à ces modifications rapidement.
1) en effet. Un ex-auteur se voit l'abonnement refusé avec la même adresse
mail. Il peut être intéressant de repasser le statut de l'auteur en - au
moins - 'invité' s'il tente de s'abonner à nouveau.
2 et 3) visiblement un bug. Dans mes souvenirs, c'est directement dans la
lib phpMailer ou l'include appelant qu'il faut corriger ça, pas dans
l'api_globales.
Cordialement itou,
--
CP
-----Message d'origine-----
De : Pierre R [mailto:p.rousset@gmail.com]
Envoyé : vendredi 22 mai 2009 08:54
À : spip-zone@rezo.net
Objet : [SPIP Zone] {SPIP-Listes Plugin} 3 problèmes ?!
Bonjour,
je développe actuellement un plugin nécessitant SPIP-Listes et je me
suis rendu compte de plusieurs éventuels problèmes. Etant nouveau dans
la communauté je préfère poser la question avant d'envisager tout
commit sur le svn...
Les 3 problèmes que j'ai trouvé :
1) Un auteur qui a été supprimé dans l'administration (marqué
'5poubelle' dans la table spip_auteur) ne peut pas se réinscrire à une
liste.
2) Le champ 'From' des emails envoyés utilisent la valeur de
'stmp_sender' et non pas de 'email_defaut' même si la méthode d'envoi
est réglé sur 'Utiliser la PHP mail() fonction'. Cela oblige d'activer
la méthode d'envoie SMTP, de modifier le champ 'stmp_sender' avant de
sauvegarder et de repasser en 'Utilisé la PHP mail() fonction'.
3) Même chose pour le champ 'Sender' des emails envoyés qui utilisent
la valeur de 'stmp_sender' même si la méthode d'envoi est réglé sur
'Utiliser la PHP mail() fonction'. Pareil cela oblige d'aller activer
la methode SMTP pour modifier le champ avant de revenir en 'PHP mail()
fonction'.
Mes idées :
1) Permettre à un un auteur qui aurait été supprimé dans
l'administration (marqué '5poubelle' dans la table spip_auteur) de se
réinscrire à une liste (je n'ai pas encore regardé le code...).
2) inc/spiplistes_api_globales.php, fonction
spiplistes_email_from_default(), ligne 260, remplacer :
if(
// prendre d'abord celui du sender SMTP
si il est activé
(($result = email_valide($ii =
trim($GLOBALS['meta']['smtp_sender'])))
// sinon celui par defaut de SPIP-Listes
|| ($result = email_valide($ii =
trim($GLOBALS['meta']['email_defaut'])))
// au pire, celui du webmaster
|| ($result = email_valide($ii =
trim($GLOBALS['meta']['email_webmaster'])))
)
par
if(
// prendre d'abord celui du defaut de
SPIP-Listes
(($result = email_valide($ii =
trim($GLOBALS['meta']['email_defaut'])))
// sinon celui du webmaster
|| ($result = email_valide($ii =
trim($GLOBALS['meta']['email_webmaster'])))
)
3) Il faudrait créer un champ Sender dans l'administration au même
titre que 'Adresse d'envoi par défaut' et que 'Adresse de retour par
défaut pour les erreurs' afin qu'il ne soit plus dépendant de la
méthode d'envoi SMTP.
Donnez moi vos avis, je suis prêt à ces modifications rapidement.