phpMyNewsletter 2 Questions

**Nicolas-37000** · 19/02/2009, 20h14

Bonsoir, j'ai installer aujourd'hui ce script et je suis très satisfait.

J'ai toute fois deux petits problèmes

Je veux envoyer des news en HTML ça fonctionne très bien mais en bas dans le mail reçu, j'ai le lien pour la désinscription qui ne fait pas joli du tout.
Comment l'enlever ?

L'autre problème plus gênant le sujet de la news ne s'affiche pas dans le sujet du mail comment faire ?

Merci pour vos pistes

**rider74** · 09/03/2009, 21h32

Pour enlever le lien de désinscription, il faut éditer le fichier send.php qui se trouve dans le dossier admin.
Poste ce fichier en pièce jointe que je le regarde à nouveau.

Attention tout de même la loi et notamment la CNIL impose la présence de ce lien. (après faut voir la nature de ta news, si c'est sur un site perso ça passera si tu l'enlèves)

Le sujet de la news et le sujet du mail sont 2 choses différentes il me semble.
Pourquoi ne pas copier le sujet de la news dans le sujet du mail à la main?
Tu dois le faire uniquement une fois, c'est pas si terrible.

**Nicolas-37000** · 10/03/2009, 15h05

Merci Rider, pour le sujet de la news ça roule.

Pour le lien de désinscription je cherche toujours...je repasse ici pour dire où j'en suis. Réussite ou pas.

Bonne journée

**RideKick** · 10/03/2009, 15h12

Je confirme , tu as une ligne a commenter , je l'ai fais chez moi pour le remplacer par mon propre lien .
Mais me souvient plus ou , si tu as le fichier on dis pas non lol

**rider74** · 10/03/2009, 21h01

poste le fichier en pièce jointe, il y en a pour 2 sec quand on sait où chercher

**Nicolas-37000** · 11/03/2009, 13h14

J'ai une grosse crève la et je ne suis pas au top pour mettre le nez dans le code. Par contre promis dès que j'ai quelque chose je reviens ici pour la suite.

**Nicolas-37000** · 13/03/2009, 16h25

Donc effectivement tout se passe dans le fichier send.php

voici la partie du code que j'ai mis en commentaire pour ne plus e^tre embeter par ce lien de désinscription.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
 //NE PLUS AFFICHER LE MESSAGE DE DESINSCRIPTION
 
 //$message.= translate("SEND_UNSUBSCRIPTION_LINK");
 
// $mail->WordWrap = 50;    
 if($format=="html")  $mail->IsHTML(true);                  // send as HTML
 
 for($i=0; $i<count($addr) ; $i++){
   $unsubLink = "";
   $mail->to = array();
   $mail->AddAddress($addr[$i]);
 
   //NE LUS AFFICHER LE LIEN DE DESINSCRIPTION
 
   //if($format=="html") $unsubLink = "<br /><a href=\"".$conf->base_url.$conf->path."subscription.php?list_id=$list_id&op=leave&email_addr=".$addr[$i]."\">";
   //$unsubLink.= $conf->base_url.$conf->path."subscription.php?list_id=$list_id&op=leave&email_addr=".urlencode($addr[$i]);
   //if($format=="html") $unsubLink.="</a></body></html>";

J'ai testé maintenant j'espère que ça ne va pas mettre la zone ailleurs ?

Il faut savoir que j'utilise ce script uniquement avec des listes d'email
importés et non suite à une inscription, et ensuite je préfère gérer mes
désinscriptions à la main. (j'inclus pour cela mon propre lien de désinscription)

Un grand merci pour Rider

Je suis à l'écoute de tous commentaires je classe toutes fois le message en
résolu (c'est la procédure je crois

)

**rider74** · 13/03/2009, 22h15

je trouve ta solution compliqué

il me semble que j'avais fait plus simple, envoie le fichier que je jette un oeil si tu veux

**Nicolas-37000** · 14/03/2009, 08h27

Salut Rider avec plaisir, surtout que je ne suis pas un as en PHP, je bricole plus qu'autre chose. Alors si tu peux me faire faire les choses de façon plus simple évidement je suis preneur

!

Donc j'ai remis le code d'origine sans modifications.

Merci à toi, et très bon weekend !

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
 
<?
include("../include/lib/class.phpmailer.php");
include("../include/config.php");
include("../include/lib/libconfig.php");
include("../include/lib/libnewsletter.php");
include("include/function.php");
include("include/iconv.php");
include("../include/db/db_".$db_type.".inc.php");
$conf = new config();
$conf->getConfig($hostname,$login ,$pass,$database, $table_global_config);
if(empty($conf->language)) $conf->language="english";
include("../include/lang/".$conf->language.".php");
 
//check authentification
$form_pass = (empty($_POST['form_pass']) ? "" : $_POST['form_pass']);
if(!checkAdminAccess($conf->admin_pass, $form_pass)){
  header("Location:index.php");
  exit();
}
 
 
$step = (empty($_GET['step']) ? "" : $_GET['step']);
 
$subject = (!empty($_POST['subject'])) ? $_POST['subject'] : '';
$message = (!empty($_POST['message'])) ? $_POST['message'] : '';
$format = (!empty($_POST['format'])) ? $_POST['format'] : '';
 
$list_id = (!empty($_POST['list_id'])) ? $_POST['list_id'] : '';
$list_id = (!empty($_GET['list_id']) && empty($list_id)) ? $_GET['list_id'] : $list_id;
 
$begin = (!empty($_POST['begin'])) ? $_POST['begin'] : '';
$begin = (!empty($_GET['begin']) && empty($begin)) ? $_GET['begin'] : 0;
 
$msg_id = (!empty($_GET['msg_id'])) ? $_GET['msg_id'] : '';
$sn = (!empty($_GET['sn'])) ? $_GET['sn'] : '';
$error = (!empty($_GET['error'])) ? $_GET['error'] : '';
 
 
switch($step){
 
 case "send":
 // open log file
 $dontlog = 0;
 if ( !$handler = @fopen('./logs/'.date("Ymd").'-list'.$list_id.'-msg'.$msg_id.'.log', 'a+'))
	$dontlog = 1;
 
 
 $limit = $conf->sending_limit;
 $mail = new PHPMailer();
 $mail->CharSet= $conf->charset;;
 $mail->PluginDir= "../include/lib/";
 
 switch($conf->sending_method){
 case "smtp":
 $mail->IsSMTP();	
 $mail->Host = $conf->smtp_host;
 if($conf->smtp_auth){
   $mail->SMTPAuth = true;
   $mail->Username = $conf->smtp_login;
   $mail->Password = $conf->smtp_pass;
 }
 
 break;
 
 case "php_mail":
   $mail->IsMail();
   break;
 
 case "online_mail":
   $mail->IsOnlineEMail();
   break;
 default:
   break;
 }
 
 $newsletter = new Newsletter();
 $newsletter->getConfig($hostname,$login ,$pass,$database, $list_id,$conf->table_listsconfig);
 
 $mail->From     = $newsletter->from;
 $mail->FromName = (strtoupper($conf->charset) == "UTF-8" ? $newsletter->from_name : 
		    iconv( "UTF-8", $conf->charset , $newsletter->from_name));
 
 //get address
 $addr = $newsletter->getAddress($conf->table_email,$begin,$limit);
 $msg = get_message($hostname,$login ,$pass,$database, $conf->table_archives,$msg_id);
 
 $format = $msg[0];
 $subject = stripslashes($msg[1]);
 
 
 $message = stripslashes($msg[2]);
 if($format == "html") $message .="<br />";	
 $message.= translate("SEND_UNSUBSCRIPTION_LINK");
 
// $mail->WordWrap = 50;    
 if($format=="html")  $mail->IsHTML(true);                  // send as HTML
 
 for($i=0; $i<count($addr) ; $i++){
   $unsubLink = "";
   $mail->to = array();
   $mail->AddAddress($addr[$i]);
 
   if($format=="html") $unsubLink = "<br /><a href=\"".$conf->base_url.$conf->path."subscription.php?list_id=$list_id&op=leave&email_addr=".$addr[$i]."\">";
   $unsubLink.= $conf->base_url.$conf->path."subscription.php?list_id=$list_id&op=leave&email_addr=".urlencode($addr[$i]);
   if($format=="html") $unsubLink.="</a></body></html>";
 
 
   $body = "";
   if($format == "html") $body.= "<html><head></head><body>";
   $body.= $message.$unsubLink;
 
 
   $subj = (strtoupper($conf->charset) == "UTF-8" ? $subject : iconv( "UTF-8", $conf->charset , $subject));
   $body = (strtoupper($conf->charset) == "UTF-8" ? $body : iconv( "UTF-8", $conf->charset , $body));
 
   $mail->Subject  = $subj;
   $mail->Body = $body;
 
   @set_time_limit(150); 
   if(!$mail->Send()){
     $errstr = ($begin+$i+1)."\t".date("H:i:s")." \t".
       $addr[$i]."\t".$mail->ErrorInfo."\r\n";	
   }else {
     $errstr = ($begin+$i+1)."\t".date("H:i:s")." \t".
       $addr[$i]."\t OK \r\n";	
   }
   if(!$dontlog) fwrite($handler, $errstr, strlen($errstr));
 } 
 
 
 $begin+=$limit;
 if($begin<$sn){
   header("location:send.php?step=send&error=$error&begin=$begin&list_id=$list_id&msg_id=$msg_id&sn=$sn");
 } else {
   $errstr = "------------------------------------------------------------\r\n";
   $errstr.= "Finished at ".date("H:i:s")."\r\n";
   $errstr.= "============================================================\r\n";
   if(!$dontlog) fwrite($handler, $errstr, strlen($errstr));
   if(!$dontlog) fclose($handler);
 
   header("location:index.php?page=compose&op=done&error=$error&list_id=$list_id&errorlog=$dontlog");
 }
 break;
 
 default:
   $conf = new config();
   $conf->getConfig($hostname,$login ,$pass,$database, $table_global_config);
 
   $message =urldecode($message);
   // save the message in the database
   $date=date("Y-m-d H:i:s");
   $msg_id = save_message($hostname,$login ,$pass,$database, $conf->table_archives, addslashes($subject), $format, addslashes($message), $date, $list_id);
 
 
   // open log file
   $dontlog = 0;
   if ( !$handler = @fopen('./logs/'.date("Ymd").'-list'.$list_id.'-msg'.$msg_id.'.log', 'a+'))
     $dontlog = 1;
 
 
 
   $newsletter = new Newsletter();
   $newsletter->getConfig($hostname,$login ,$pass,$database, $list_id,$conf->table_listsconfig);
 
   $num = $newsletter->getSubscribersNumbers($conf->table_email);
   $errstr = "============================================================\r\n";
   $errstr.= date("d M Y")."\r\n";
   $errstr.= "Started at ".date("H:i:s")."\r\n";
   $errstr.= "NÂ° \t Date \t\t Recipient \t\t Status \r\n";	
   $errstr.= "------------------------------------------------------------\r\n";
   if(!$dontlog) fwrite($handler, $errstr, strlen($errstr));
   if(!$dontlog) fclose($handler);
 
   header("location:send.php?step=send&begin=0&list_id=$list_id&msg_id=$msg_id&sn=$num&error=0");
   break;
 
 
 }
 
 
 
 
 
 
 
?>

**rider74** · 14/03/2009, 22h53

Autant pour moi, tu as fait la modif au bon endroit mais tu en as trop supprimé.

Il faut commenter seulement ces lignes là:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
if($format=="html") 
   		$unsubLink = "<br /><a href=\"".$conf->base_url.$conf->path."subscription.php?list_id=$list_id&op=leave&email_addr=".$addr[$i]."\">";
   		$unsubLink.= $conf->base_url.$conf->path."subscription.php?list_id=$list_id&op=leave&email_addr=".urlencode($addr[$i]);

Du coup, il faut corriger celles-ci:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
  if($format=="html") 
   		$unsubLink.="</a></body></html>";

devient:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
  if($format=="html") 
   		$unsubLink.="</body></html>";

car elles permettent de fermer le corps du message (le </a> étant celui du lien de désinscription, tu dois le virer)

Si tu veux rajouter ton propre lien de désinscription, il te suffit de rajouter une ligne et des accolades dans le bloc précédent:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
  if($format=="html") 
             {   
                $unsubLink="ce_que_tu_veux";
   		$unsubLink.="</body></html>";
             }

Voilà

**Nicolas-37000** · 17/03/2009, 13h14

Merci Rider,

la correction et les commentaires sont très constructifs pour moi.

@très bientôt

**rider74** · 19/03/2009, 11h41

pas de quoi,

bon courage pour le développement du tracking

phpMyNewsletter 2 Questions

EDI, CMS, Outils, Scripts et API PHP

Discussions similaires

Partager

Partager