Surcharge et plugin

Manu_TJ

Març de 2010

Bonjour

mon hébergeur (Oléane) impose pour l’utilisation de l’envoi de mail, avec la fonction PHP mail(), que le header contienne obligatoirement la valeur Return-Path.
Or SPIP ne renseigne pas cette valeur.

J’ai créé une fonction inc_envoyer_mail(), placée dans mon fichier mes_fonctions.php en adaptant le code.
Quand je veux envoyer la lettre de nouveautés, ça marche : super !

Maintenant, j’utilise aussi le plugin Tickets.
Je vois qu’il appelle la fonction d’envoi de mail par
$envoyer_mail = charger_fonction(’envoyer_mail’,’inc’);

En regardant le site http://programmer.spip.org/charger_fonction, je comprends que la fonction charger_fonction a justement pour rôle de récupérer la nouvelle fonction d’envoi de mail (celle que j’ai
créée) plutôt que la fonction dist existante (dans inc/envoyer_mail.php).

Pourtant le plugin Tickets n’utilise pas ma fonction mais toujours celle de SPIP.

Ma question est la suivante : s’il est possible de surcharger SPIP, est-il possible avec le même mécanisme de surcharger un plugin ?

Merci pour votre aide.

Par note per data Par date chronologique

Indiqueu l’interés que us porta a aquesta resposta

0

Manu_TJ

Març de 2010

J’ai un peu avancé sur ce "problème". A priori, ma nouvelle fonction doit être placée dans mes_options.php (qui est appelé systématiquement) et pas dans mes_fonctions.php (qui est appelé seulement au calcul d’un squelette).

Ma fonction est opérationnelle (l’envoi de la lettre de nouveautés fonctionne grâce à cette fonction).

Mais la présence de cette fonction "plante" le plugin Tickets. Dans ce plugin, on peut affecter une personne à un ticket. Le changement de personne génère l’envoi d’un mail. Or, avec ma fonction , le formulaire plante.

Comment avancer dans le débogage ?
Indiqueu l’interés que us porta a aquesta resposta

0
denisb

Març de 2010
sans avoir ta fonction sous les yeux : impossible (nombre d’arguments ? redéfinition de fonction ? mauvais path ? ...).

mais un simple
if (strpos($headers, "Return-Path:")===FALSE) { $headers .= "Return-Path: $from\n"; }
ajouté à la fonction «_dist» n’est pas suffisant ?

(«$from» ou n’importe quelle adresse en dur)

Manu_TJ

Març de 2010

En fait je voudrais éviter de modifier la fonction "_dist" pour les problèmes de mise à jour.

Ci dessous le code de ma fonction :

function inc_envoyer_mail($email, $sujet, $texte, $from = "", $headers = "") {
	include_spip('inc/charsets');
	include_spip('inc/filtres');
	include_spip('inc/envoyer_mail');

	if (!email_valide($email)) return false;
	if ($email == _T('info_mail_fournisseur')) return false; // tres fort

	// Traiter les headers existants
	if (strlen($headers)) $headers = trim($headers)."\n";

	// Fournir si possible un Message-Id: conforme au RFC1036,
	// sinon SpamAssassin denoncera un MSGID_FROM_MTA_HEADER

	$email_envoi = $GLOBALS['meta']["email_envoi"];
	if (email_valide($email_envoi)) {
		preg_match('/(@\S+)/', $email_envoi, $domain);
		$mid = 'Message-Id: <' . time() . '_' . rand() . '_' . md5($email . $texte) . $domain[1] . ">\n";
	} else {
		spip_log("Meta email_envoi invalide. Le mail sera probablement vu comme spam.");
		$email_envoi = $email;
		$mid = '';
	}
	if (!$from) $from = $email_envoi;

	// ceci est la RegExp NO_REAL_NAME faisant hurler SpamAssassin
	if (preg_match('/^["\s]*\<?\S+\@\S+\>?\s*$/', $from))
		$from .= ' (' . str_replace(')','', translitteration(str_replace('@', ' at ', $from))) . ')';

	// Et maintenant le champ From:
	$headers .= "From: $from\n";
	
	// Et maintenant le champ Return-Path:
	$headers .= "Return-Path: $from\n";

	// indispensable pour les sites qui colle d'office From: serveur-http
	// sauf si deja mis par l'envoyeur
	if (strpos($headers,"Reply-To:")===FALSE)
		$headers .= "Reply-To: $from\n";

	$charset = $GLOBALS['meta']['charset'];

	// Ajouter le Content-Type et consort s'il n'y est pas deja
	if (strpos($headers, "Content-Type: ") === false)
		$headers .=
		"Content-Type: text/plain; charset=$charset\n".
		"Content-Transfer-Encoding: 8bit\n" .
		"MIME-Version: 1.0\n";

	$headers .= $mid;

	// nettoyer les &eacute; &#8217, &emdash; etc...
	// les 'cliquer ici' etc sont a eviter;  voir:
	// http://mta.org.ua/spamassassin-2.55/stuff/wiki.CustomRulesets/20050914/rules/french_rules.cf
	$texte = nettoyer_caracteres_mail($texte);
	$sujet = nettoyer_caracteres_mail($sujet);

	// encoder le sujet si possible selon la RFC
	if (init_mb_string()) {
		# un bug de mb_string casse mb_encode_mimeheader si l'encoding interne
		# est UTF-8 et le charset iso-8859-1 (constate php5-mac ; php4.3-debian)
		mb_internal_encoding($charset);
		$sujet = mb_encode_mimeheader($sujet, $charset, 'Q', "\n");
		mb_internal_encoding('utf-8');
	}

	spip_log("[AI] mail $email\n$sujet\n$headers",'mails');

	// Ajouter le \n final
	if ($headers = trim($headers)) $headers .= "\n";
	if (function_exists('wordwrap'))
		$texte = wordwrap($texte);

	if (_OS_SERVEUR == 'windows') {
		$texte = preg_replace ("@\r*\n@","\r\n", $texte);
		$headers = preg_replace ("@\r*\n@","\r\n", $headers);
		$sujet = preg_replace ("@\r*\n@","\r\n", $sujet);
	}

	return mail($email, $sujet, $texte, $headers);
}

Les modifications sont :
– ajout de include_spip(’inc/envoyer_mail’);
– ajout de $headers .= "Return-Path: $from\n";

denisb

Març de 2010

et avec (plus simplement) dans mes_options.php :

function inc_envoyer_mail($email, $sujet, $texte, $from = "", $headers = "") {
  $headers .= "\nReturn-Path: ".$from;
  include_spip('inc/envoyer_mail');
  inc_envoyer_mail_dist($email, $sujet, $texte, $from, $headers);
}

Indiqueu l’interés que us porta a aquesta resposta

0

Manu_TJ

Març de 2010

Merci pour ton aide.
Mais ça ne change rien :
– l’entête du mail de la lettre de nouveautés contient bien "Return-Path" mais vide
– le plugin Tickets continue d’afficher brièvement la roue ajax, lors du changement de personne assignée, et disparaît. Puis ce morceau de formulaire disparaît (le changement a bien lieu mais pas de mail dans prive_mails.log)

Mystère...
Indiqueu l’interés que us porta a aquesta resposta

0

Manu_TJ

Abril de 2010

Bon j’en perds mon latin et mes maigres connaissances ne me permettent pas d’aller plus loin.
Je vais donc modifier le fichier envoyer_mail.php (dans ecrire/inc) puisque je ne vois pas comment faire autrement.
Je suis un peu frustré parce que je ne comprends pas pourquoi mais bon, c’est comme ça.

Une question encore, malgré tout : est ce que quelqu’un connaît un plugin simple qui envoie aussi des mails ? Histoire de voir si c’est le plugin Tickets qui pose problème, ou si c’est le fait d’ajouter une fonction dans mes_options qui n’est pas bien géré...

Manu, qui n’aime pas ne pas comprendre
Indiqueu l’interés que us porta a aquesta resposta

0

meg

Juliol de 2010

Des nouvelles?
Je suis dans le même cas...