Page 1 sur 1

Message d'erreur

Posté : 17 janv. 2010, 17:00
par mars77
Bonjour a tous,

J'ai récupéré un code que je veux adapter au goût du jour pour envoyer un mail avec une pièce jointe.

Code : Tout sélectionner

<?php /* PARAMETRAGE DU SCRIPT */ /* ENTREZ VOTRE ADRESSE EMAIL ENTRE LES GUILLEMETS*/ $dest="[email protected]"; $reponse=StripSlashes("Entrez ici la réponse quand une personne envoie le formulaire"); /* FIN DU PARAMETRAGE */ DESCRIPTION this class encapsulates the PHP mail() function. implements CC, Bcc, Priority headers */ class Mail { var $sendto= array(); var $from, $msubject; var $acc= array(); var $abcc= array(); var $aattach= array(); var $priorities= array( '1 (Highest)', '2 (High)', '3 (Normal)', '4 (Low)', '5 (Lowest)' ); // Mail contructor function Mail() { $this->autoCheck( true ); } /* autoCheck( $boolean ) * activate or desactivate the email addresses validator * ex: autoCheck( true ) turn the validator on * by default autoCheck feature is on */ function autoCheck( $bool ) { if( $bool ) $this->checkAddress = true; else $this->checkAddress = false; } /* Subject( $subject ) * define the subject line of the email * $subject: any valid mono-line string */ function Subject( $subject ) { $this->msubject = strtr( $subject, "\r\n" , " " ); } /* From( $from ) * set the sender of the mail * $from should be an email address */ function From( $from ) { if( ! is_string($from) ) { echo "Class Mail: error, From is not a string"; exit; } $this->from= $from; } /* To( $to ) * set the To ( recipient ) * $to : email address, accept both a single address or an array of addresses */ function To( $to ) { // TODO : test validité sur to if( is_array( $to ) ) $this->sendto= $to; else $this->sendto[] = $to; if( $this->checkAddress == true ) $this->CheckAdresses( $this->sendto ); } /* Cc() * set the CC headers ( carbon copy ) * $cc : email address(es), accept both array and string */ function Cc( $cc ) { if( is_array($cc) ) $this->acc= $cc; else $this->acc[]= $cc; if( $this->checkAddress == true ) $this->CheckAdresses( $this->acc ); } /* Bcc() * set the Bcc headers ( blank carbon copy ). * $bcc : email address(es), accept both array and string */ function Bcc( $bcc ) { if( is_array($bcc) ) { $this->abcc = $bcc; } else { $this->abcc[]= $bcc; } if( $this->checkAddress == true ) $this->CheckAdresses( $this->abcc ); } /* Body() * set the body of the mail ( message ) */ function Body( $body ) { $this->body= $body; } /* Send() * fornat and send the mail */ function Send() { // build the headers $this->_build_headers(); // include attached files if( sizeof( $this->aattach > 0 ) ) { $this->_build_attachement(); $body = $this->fullBody . $this->attachment; } // envoie du mail aux destinataires principal for( $i=0; $i< sizeof($this->sendto); $i++ ) { $res = mail($this->sendto[$i], $this->msubject,$body, $this->headers); // TODO : trmt res } } /* Organization( $org ) * set the Organisation header */ function Organization( $org ) { if( trim( $org != "" ) ) $this->organization= $org; } /* Priority( $priority ) * set the mail priority * $priority : integer taken between 1 (highest) and 5 ( lowest ) * ex: $m->Priority(1) ; => Highest */ function Priority( $priority ) { if( ! intval( $priority ) ) return false; if( ! isset( $this->priorities[$priority-1]) ) return false; $this->priority= $this->priorities[$priority-1]; return true; } /* Attach( $filename, $filetype ) * attach a file to the mail * $filename : path of the file to attach * $filetype : MIME-type of the file. default to 'application/x-unknown-content-type' * $disposition : instruct the Mailclient to display the file if possible ("inline") or always as a link ("attachment") * possible values are "inline", "attachment" */ function Attach( $filename, $filetype='application/x-unknown-content-type', $disposition = "inline" ) { // TODO : si filetype="", alors chercher dans un tablo de MT connus / extension du fichier $this->aattach[] = $filename; $this->actype[] = $filetype; $this->adispo[] = $disposition; } /* Get() * return the whole e-mail , headers + message * can be used for displaying the message in plain text or logging it */ function Get() { $this->_build_headers(); if( sizeof( $this->aattach > 0 ) ) { $this->_build_attachement(); $this->body= $this->body . $this->attachment; } $mail = $this->headers; $mail .= "\n$this->body"; return $mail; } /* ValidEmail( $email ) * return true if email adress is ok - regex from Manuel Lemos ([email protected]) * $address : email address to check */ function ValidEmail($address) { if( ereg( ".*<(.+)>", $address, $regs ) ) { $address = $regs[1]; } if(ereg( "^[^@ ]+@([a-zA-Z0-9\-]+\.)+([a-zA-Z0-9\-]{2}|net|com|gov|mil|org|edu|int)\$",$address) ) return true; else return false; } /* CheckAdresses() * check validity of email addresses * if unvalid, output an error message and exit, this may be customized * $aad : array of emails addresses */ function CheckAdresses( $aad ) { for($i=0;$i< sizeof( $aad); $i++ ) { if( ! $this->ValidEmail( $aad[$i]) ) { echo "Class Mail, method Mail : invalid address $aad[$i]"; exit; } } } /********************** PRIVATE METHODS BELOW **********************************/ /* _build_headers() * [INTERNAL] build the mail headers */ function _build_headers() { // creation du header mail $this->headers= "From: $this->from\n"; $this->to= implode( ", ", $this->sendto ); if( count($this->acc) > 0 ) { $this->cc= implode( ", ", $this->acc ); $this->headers .= "CC: $this->cc\n"; } if( count($this->abcc) > 0 ) { $this->bcc= implode( ", ", $this->abcc ); $this->headers .= "BCC: $this->bcc\n"; } if( $this->organization != "" ) $this->headers .= "Organization: $this->organization\n"; if( $this->priority != "" ) $this->headers .= "X-Priority: $this->priority\n"; } /* * _build_attachement() * internal use only - check and encode attach file(s) */ function _build_attachement() { $this->boundary= "------------" . md5( uniqid("myboundary") ); // TODO : variable bound $this->headers .= "MIME-Version: 1.0\nContent-Type: multipart/mixed;\n boundary=\"$this->boundary\"\n\n"; $this->fullBody = "This is a multi-part message in MIME format.\n--$this->boundary\nContent-Type: text/plain; charset=us-ascii\nContent-Transfer-Encoding: 7bit\n\n" . $this->body ."\n"; $sep= chr(13) . chr(10); $ata= array(); $k=0; // for each attached file, do... for( $i=0; $i < sizeof( $this->aattach); $i++ ) { $filename = $this->aattach[$i]; $basename = basename($filename); $ctype = $this->actype[$i]; // content-type $disposition = $this->adispo[$i]; if( ! file_exists( $filename) ) { echo "Class Mail, method attach : file $filename can't be found"; exit; } $subhdr= "--$this->boundary\nContent-type: $ctype;\n name=\"$basename\"\nContent-Transfer-Encoding: base64\nContent-Disposition: $disposition;\n filename=\"$basename\"\n"; $ata[$k++] = $subhdr; // non encoded line length $linesz= filesize( $filename)+1; $fp= fopen( $filename, 'r' ); $data= base64_encode(fread( $fp, $linesz)); fclose($fp); $ata[$k++] = chunk_split( $data ); /* // OLD version - used in php < 3.0.6 - replaced by chunk_split() $deb=0; $len=76; $data_len= strlen($data); do { $ata[$k++]= substr($data,$deb,$len); $deb += $len; } while($deb < $data_len ); */ } $this->attachment= implode($sep, $ata); } } // class Mail $NomFichier_name= $_FILES["NomFichier"]["name"]; $NomFichier= $_FILES["NomFichier"]["name"]; $subject = $_POST['subject']; $email = $_POST['email']; $msg= $_POST['msg']; $priority = $_POST['priority']; $subject=StripSlashes($subject); $msg=StripSlashes($msg); $msg="Message depuis votre site web: $msg"; $m= new Mail; // create the mail $m->From( "$email" ); $m->To( "$dest"); $m->Subject( "$subject" ); $m->Body( $msg); // set the body if ($email1!="") { $m->Cc( "$email1"); } $m->Priority($priority) ; if ("$NomFichier_name"!="") { copy("$NomFichier","../upload/$NomFichier_name"); $m->Attach( "../upload/$NomFichier_name", "application/octet-stream" ); } $m->Send(); if ("$NomFichier_name"!="") { Unlink("../upload/$NomFichier_name"); } echo "$reponse";
J'ai eu plusieurs messages d'erreurs que j'ai pu corriger, mais il en reste encore que je ne sais plus quoi faire.

Voilà les messages d'erreur:


Deprecated: Function ereg() is deprecated in E:\wamp\www\Communparapente\teste\formmail.php on line 258

Deprecated: Function ereg() is deprecated in E:\wamp\www\Communparapente\teste\formmail.php on line 261

Notice: Undefined variable: email1 in E:\wamp\www\Communparapente\teste\formmail.php on line 387

Warning: copy(59.jpg) [function.copy]: failed to open stream: No such file or directory in E:\wamp\www\Communparapente\teste\formmail.php on line 392

Notice: Undefined property: Mail::$organization in E:\wamp\www\Communparapente\teste\formmail.php on line 312
Class Mail, method attach : file ../upload/59.jpg can't be found

Qui peu m'aider à trouver d' ou vient ce message. merci beaucoup pour votre aide.

Re: Message d'erreur

Posté : 17 janv. 2010, 19:25
par Ryle
Modération :
Afin d'améliorer la lisibilité de ton message,
pense à utiliser les balises [code] ou [php] (selon le langage utilisé).

Elles sont disponibles au-dessus de la zone de saisie de ton message
lorsque tu postes un nouveau message.

Des indications sont disponibles sur la manière de mettre en forme vos messages dans la FAQ


Concernant les messages d'erreur :
- "Deprecated: Function ereg() is deprecated" t'indique que la fonction ereg() est déprécié dans la version de php que tu utilises et qu'il vaut mieux utiliser une autre fonction (ereg pouvant être amené à disparaitre). C'est uniquement un avertissement, ton code fonctionnera quand même, mais autant anticiper et pas avoir à revenir sur ton code :) Tu peux utiliser preg_match() à la place.
- "Notice: Undefined variable: email1" indique que tu fais appel à une variable nommée $email1, alors que cette variable n'a pas été définie. PHP va donc considérer que sa valeur est nulle, mais il t'informe de cette liberté qu'il s'octroi. Pour éviter cela, il te suffit de déclarer et/ou initialiser ta variable avant d'y faire appel :)
- "Warning: copy(59.jpg) [function.copy]: failed to open stream: No such file or directory" : tu essayes de copier un fichier ou un dossier, mais PHP ne parvient pas à le trouver à l'endroit indiqué... vérifies que le chemin est bon, et si c'est le cas, que php a les droits d'accès pour lire et copier le fichier

Re: Message d'erreur

Posté : 17 janv. 2010, 23:09
par mars77
Hello,

Merci de m'avoir répondu et désolé de n'avoir pas écrit juste le message. Je vais essayer de faire mieux cette fois.

Maintenant j'ai modifier mon code et remplacé par la fonction preg_match.
Le problème c'est que j'ai un autre message d'erreur.

Warning: preg_match() [function.preg-match]: Unknown modifier '+' in E:\wamp\www\Communparapente\teste\formmail.php on line 258

Warning: preg_match() [function.preg-match]: Unknown modifier '@' in E:\wamp\www\Communparapente\teste\formmail.php on line 261
Class Mail, method Mail : invalid address [email protected]

Code : Tout sélectionner

function ValidEmail($address) { if( preg_match( ".*<(.+)>", $address, $regs ) ) { $address = $regs[1]; } if(preg_match( "^[^@ ]+@([a-zA-Z0-9\-]+\.)+([a-zA-Z0-9\-]{2}|net|com|gov|mil|org|edu|int)\$",$address) ) return true; else return false; }
Cela fait 2 heures que je cherche l'erreur dans ma fonction mais ne trouve pas. Est ce que j'ai oublié quelque chose ?

Merci pour votre aide.

Re: Message d'erreur

Posté : 18 janv. 2010, 17:11
par dunbar
Salut
Proposition de correction :
 function ValidEmail($address)
{
        if( preg_match( "/.*<(.+)>/", $address, $regs ) ) {
                $address = $regs[1];
        }
         if(preg_match( "/^[^@  ]+@([a-zA-Z0-9\-]+\.)+([a-zA-Z0-9\-]{2}|net|com|gov|mil|org|edu|int)\$/",$address) )
                 return true;
         else
                 return false;
}

Re: Message d'erreur

Posté : 18 janv. 2010, 23:05
par mars77
Merci pour ton aide mon message d'erreur à été remplacé par un autre.

La je deviens fou.

le message :
Warning: mail() [function.mail]: Permission denied: headers injection (empty line) in /home/www/f4e28b24b8dad49a90a589664765c79c/web/mail2/formmail.php on line 176

Mon code:
function Send()
{
        // build the headers
        $this->_build_headers();

        // include attached files
        if( sizeof( $this->aattach > 0 ) ) {
                $this->_build_attachement();
                $body = $this->fullBody . $this->attachment;
        }

        // envoie du mail aux destinataires principal
        for( $i=0; $i< sizeof($this->sendto); $i++ ) {
                $res = mail($this->sendto[$i], $this->msubject,$body, $this->headers);
                // TODO : trmt res
        }

}
Je vois que c'est une problème de permission et je sais pas ce qui bloque.
La je séche encore et ne trouve pas le problème. Merci à celui qui saura m'aider.

Re: Message d'erreur

Posté : 18 janv. 2010, 23:39
par Ryle
Le problème vient des headers que tu envoi à la fonction mail (donc de ta variable $this->headers). Celle-ci contient apparement une ligne vide et php la rejette par sécurité...

A vérifier, mais je pense que ton problème vient d'ici :
$this->headers .= "MIME-Version: 1.0\nContent-Type: multipart/mixed;\n boundary=\"$this->boundary\"\n\n";
Tu as à la fin deux retours à la ligne consécutifs "\n\n", essaye en en supprimant un des deux pour éviter la ligne vide...

Re: Message d'erreur

Posté : 19 janv. 2010, 22:34
par mars77
Hello,

Merci beaucoup pour ton aide. C'était effectivement ça. J'ai enlevé 1 des 2 \n et j'arrive a envoyer des mails.

Maintenant j'aimerai envoyer le mail à plusieurs personnes. Les adresses sont enregistrée dans ma table membres2.

J'ai fait un code pour afficher mes adresses.
$select = "select * FROM membres2 "; 
$result = mysql_query($select,$link)  or die ('Erreur : '.mysql_error() );
while($row = mysql_fetch_array($result)) {
$email1= ''.$row["email"].'';
}
echo $email1;
Le problème c'est que ça m'affiche que ma première adresse mail et non toute les adresses vu que je suis en dehors de ma boucle.

le résultat que j'aimerai c'est :

$emial1 = adressemail1,adressemail2, adresseemail3, ect.

merci encore pour votre aide

Re: Message d'erreur

Posté : 19 janv. 2010, 22:43
par Ryle
En réalité, ce n'est pas la première, mais la dernière adresse qu'il t'affiche, car à chaque passage dans ton while, il écrase le précédent contenu de ta variable $email. Ce qu'il te faut c'est concaténer les adresses et compléter ta chaine au fur et à mesure (ou faire un tableau de valeur, ça peut être sympa aussi :))
$email1 = ''; // on déclare la variable
while($row = mysql_fetch_assoc($result)) {
  if ($email1!='') // si la chaine n'est pas vide, on ajoute une virgule entre les adresses
    $email1.= ', ';
  $email1.= $row["email"]; // pour concaténer on utilise l'opérateur ".=" au lieu du "=". 
}
echo $email1;
Le même avec un tableau :
$email1 = array(); // on déclare la variable
while($row = mysql_fetch_assoc($result)) {
  $email1[] = $row["email"]; // on ajoute l'email dans le tableau
}
echo implode(',', $email1); // avec implode on insère une virgule entre chaque email et on obtient le même résultat 

Re: Message d'erreur

Posté : 19 janv. 2010, 23:15
par mars77
Merci beaucoup pour ton aide.

Cela fonctionne très bien. Je te remercie encore.

A+