[php] email bouwen probleem

[MrDummy]

Ik ben bezig een systeem te maken waar men emails mét attachments kan sturen naar de klanten.
Meerdere attachments dus.

Ik heb diverse scripts bekeken en dus delen overgenomen. Het zou moeten werken, ik heb met bestaande emails vergeleken en geanalyseerd op juiste opbouw.

Het lijkt te lukken, maar er gebeuren vreemde dingen. Een stukje code dat belangrijk is voor "herkennen" van email body, bleek verdwenen te zijn.

Ik zal stukjes code hier tonen zodat je kan zien of er ergens fout is.

Deze staat in functies.php. De function code werkt prima, gebruik ook op andere sites.

PHP:

function sendmail($mailTo,$mailSubject,$mailBody,$mailFrom,$type='text',$mime_boundary)
    {
    // extra 
    // $mysite = whatsMySite();
    // define("MYSITE",$mysite);
    
    $mailHeader="";

    // header maken en sturen
    $mailHeader .= "From: $mailFrom\n";
    $mailHeader .= "Reply-To: $mailFrom\n";
    $mailHeader .= "X-Mailer: http://www.vdsluisvakanties.nl\n";    
    $mailHeader .= "X-Sender-IP: {$_SERVER['REMOTE_ADDR']}\n";  
    
    // type headers
    if ($type=="multi")
        {
        if ($mime_boundary=="") {
            list($mtime,$time)=explode(" ",microtime());
            $fulltime=$time+$mtime;
            $semi_rand = md5($fulltime);
            $mime_boundary = "x{$semi_rand}x";
            }
        
        $mailHeader .=  "\nMIME-Version: 1.0\n" .
                        "Content-Type: multipart/mixed;\n" .
                        " boundary=\"--------{$mime_boundary}\"\n";

        $mailBody .=    "\nMIME-Version: 1.0\n".
                        "Content-Type: multipart/mixed;\n" .
                        " boundary=\"--------{$mime_boundary}\"\n";
                        "This is a multi-part message in MIME format.\n".
                        "--------{$mime_boundary}\n".
                        "Content-Type: text/html; charset=UTF-8\n".
                        $mailBody . "\n".
                        "--------{$mime_boundary}--\n"; 
        }

    if ($type=="html")
        {
        $mailHeader .= "MIME-Version: 1.0\n";
        $mailHeader .= "Content-Type: text/html; charset=\"UTF-8\"\n";
        $mailHeader .= "Content-Transfer-Encoding: 7bit\n\n";
        }
    if ($type=="text")
        {
        $mailHeader .= "MIME-Version: 1.0\n";
        $mailHeader .= "Content-Type: text/plain; charset=\"UTF-8\"\n";
        $mailHeader .= "Content-Transfer-Encoding: 7bit\n\n";
        }
    
    $mailParams = "-f$mailFrom";
    $mailResult = mail($mailTo,$mailSubject,$mailBody,$mailHeader,$mailParams);
    return $mailResult;
    }

function whatsMySite() {
    // protocol 
    if(isset($_SERVER['HTTPS']) and ("on" == $_SERVER['HTTPS'])) {
        $mysite = "https://";
    }
    else {
        $mysite = "http://";
    }
        
    // host            
    $mysite .= $_SERVER['HTTP_HOST'];
    
    // path
    $path = dirname($_SERVER['SCRIPT_NAME']);
    if("/" != $path) {
        $mysite .= $path;
    }
    
    return($mysite);
}

En in maakemail.php heb ik deze. Dat wordt aangeroepen via de emailformulier.

PHP:

$email=$_POST['email'];
$body=$_POST['ebody'];
$subject=stripslashes($_POST['subject']);
$attachment=$_POST['attachment'];

$filetype="application/pdf";

list($mtime,$time)=explode(" ",microtime());
$fulltime=$time+$mtime;
$semi_rand = md5($fulltime);
$mime_boundary = "x{$semi_rand}x";

$body.="\n\n";
// attachment data collect
$rows=count($attachment);
if ($rows>0)
    {
    $_SESSION['string']=true;
    for ($i=0;$i<$rows;$i++)
        {
        $_SESSION['briefid']=$attachment[$i];
        $adata=include "maakpdf.php";
        $data=chunk_split(base64_encode($adata[pdf]));
        $filename=$adata[file];
        $body .= "\n--------{$mime_boundary}\n" .
        "Content-Type: {$filetype};\n" .
        " name=\"{$filename}\"\n" .
        "Content-Transfer-Encoding: base64\n" .
        "Content-Disposition: attachment;\n" .
        " filename=\"{$filename}\"\n\n" .
        $data;
        }
    }

unset($_SESSION['toonbrief']);
unset($_SESSION['string']);
// finished
$result=sendmail($email,$subject,$body,$mailFrom,'multi',$mime_boundary);

Maakpdf.php is al goed getest. Het werkt met FPDF code en ik heb al zonder problemen PDF mee gemaakt. Er zitten geen fouten in. De string gaat via return in de php en dat heb ik geen fouten mee gehad. Het werkt goed.

Bij tests is deel van te zien, die in alle testemails verschijnt onderaan.

code:

MIME-Version 1.0
Content-type: multipart/mixed;
 boundary="--------xxxxx"

(hier verschijnt stukje emailtekst en daaronder gemaakte attachments in base64)

Maar een stukje tekst ontbreekt steeds in alle mailtests. Dat zit NIET ertussen in alle testemails.

code:

This is a multi-part message in MIME format
--------xxxxx
Content-Type: text/html; charset=UTF-8

Waarom dat eruit is geknipt weet ik niet. Het is gewoon weg bij aankomst. Hierdoor leest de mailprogramma, maar ook online webmail, de geheel verkeerd en laat zien als één groot brok tekst zien met attachment codesoep eronder. Het kan zo niet!

Wie kan me goede tips geven over correct opbouwen van email? Zodat er niks weggelaten wordt tijdens versturen en goed gelezen wordt. Veel sites hebben soortgelijke code die ik gebruik, dus ik kan geen antwoord vinden. Ik weet niet precies wat de regels zijn in $header string en wat in $body moet komen staan.
Het sturen gaat met mail() functie.
Misschien zit het probleem in boundary regels. Ik heb dat overal gelijk gezet. Dat zou niet veel moeten uitmaken, want ik heb werkende emails gezien die overal gelijk is.

Aan je het woord. Ik ben erg benieuwd.

[Cartman!]

Zoals in alle topics over problemen met mailen... gebruik PHPMailer of Zend_Mail...

[MrDummy]

Je doet alsof de bestaande scripts op internet allemaal fout zijn.
Waarom moet je per sé PHPMailer pakken als er maar beetje aangepast hoeft te worden?

Ik heb al eerder zonder problemen HTML emails mee gestuurd, en dit is nieuw geval met meerdere attachments en boundary regels erin. Dat doen alle emailsprogramma's, ik wil graag weten waarom het niet goed gaat.

Als het goed werkt voor een deel, ben ik er bijna. Ik moet alleen even weten waarom stukje code weggehaald is bij aankomst.

[curry684]

Iedereen gebruikt PHPMailer omdat iedere mailclient op deze wereld subtiel anders werkt. Ik heb HTML emails gezien die in Outlook 2003 wel goed werkten en in Outlook 2000 en 2007 niet. Voeg bij dat soort dingen de Thunderbirds, de Eudora's, de Pine's en de Gmail's van deze wereld en je bent, met alle respect, zot als je zelf het wiel uit gaat vinden voor een productieapplicatie.

Wat je wil is 3 regels code met PHPMailer. En als je een keer een klacht krijgt van een user download je een nieuwe PHPMailer en dan werkt het wel. Sommige wielen zijn de moeite van het opnieuw uitvinden niet waard.

[RobIII]

MrDummy schreef op woensdag 28 januari 2009 @ 22:40:
Wie kan me goede tips geven over correct opbouwen van email?

Daar hebben we RFC's voor

[MrDummy]

En ook nogal wat sites met PHPMailer.
http://phpmailer.codeworxtech.com/index.php (loop achter?)
http://sourceforge.net/projects/phpmailer (schijnt nieuwste)

En paar links die niet echt veilig zijn.

Ik ga morgen maar kijken wat PHPMailer doet. Maar je bent geheel vrij om antwoord te geven

Ik heb stukje ontbrekende deel verhuisd naar $header, en dat verschijnt nu wel, maar nog steeds in text mode.
Neem aan dat <html><body> ook in emailbericht moet komen staan...

[TeeDee]

MrDummy schreef op woensdag 28 januari 2009 @ 23:11:
En ook nogal wat sites met PHPMailer.
http://phpmailer.codeworxtech.com/index.php (loop achter?)
http://sourceforge.net/projects/phpmailer (schijnt nieuwste)

En paar links die niet echt veilig zijn.

Waarom zouden die links niet veilig zijn?

Ik heb stukje ontbrekende deel verhuisd naar $header, en dat verschijnt nu wel, maar nog steeds in text mode.
Neem aan dat <html><body> ook in emailbericht moet komen staan...

Als ik naar je sendmail kijk is de $type default geset op 'text'. Mag jij me uitleggen waarom je constant text door krijgt.

[RobIII]

TeeDee schreef op woensdag 28 januari 2009 @ 23:22:
Als ik naar je sendmail kijk is de $type default geset op 'text'. Mag jij me uitleggen waarom je constant text door krijgt.

Ik zie dan wel weer:

$result=sendmail($email,$subject,$body,$mailFrom,'multi',$mime_boundary);

Zonder verder te kijken dan m'n neus lang is ziet 't er wel naar uit dat er 'multi' gepassed wordt naar sendmail

[TeeDee]

RobIII schreef op woensdag 28 januari 2009 @ 23:45:
[...]

Ik zie dan wel weer:

[...]

Zonder verder te kijken dan m'n neus lang is ziet 't er wel naar uit dat er 'multi' gepassed wordt naar sendmail

Als ik het even naar wat pseudo C# 'cast':

C#:

//doe dingen in maakemail.php (goeie naam ;))
sendmail(email,subject,body,mailFrom,"multi",mime_boundary); 
//overhevelen!
private void sendmail(
     string email,string subject,
     string body,string mailFrom,string type,string mime_boundary)
{
  type="text";
  //doe dingen!

  if(type=="text")
  {
     //attach text only headers
  }
}

Dat gebeurt er ongeveer als ik het goed begrijp. Dus of verder dan je neus lang is kijken of ik begrijp echt geen reet van hoe PHP zulke dingen doet.

[ Voor 7% gewijzigd door TeeDee op 28-01-2009 23:57 ]

[2playgames]

Inderdaad, je snapt PHP niet

Het volgende stuk PHP:

PHP:

<?php

function foo($a, $b = 'text') {
}

?>

zou in Java zo gemaakt worden

Java:

public void foo(String a, String b) {
 // bla
}

public void foo(String a) {
  foo(a, "text");
}

[TeeDee]

Blegh....

Sorry RobIII

Poging om het goed te maken

PHP:

       $mailBody .=     "\nMIME-Version: 1.0\n".
                        "Content-Type: multipart/mixed;\n" .
                        " boundary=\"--------{$mime_boundary}\"\n";
                        "This is a multi-part message in MIME format.\n".
                        "--------{$mime_boundary}\n".
                        "Content-Type: text/html; charset=UTF-8\n".
                        $mailBody . "\n".
                        "--------{$mime_boundary}--\n";

Hoort die ; wel op regel 3 en moet dat niet een . zijn?

[ Voor 91% gewijzigd door TeeDee op 29-01-2009 00:14 ]

[RobIII]

TeeDee schreef op woensdag 28 januari 2009 @ 23:55:
of ik begrijp echt geen reet van hoe PHP zulke dingen doet.

Nou weet ik ook weinig van PHP (en wil dat graag zo houden ) maar je moet het zien als een 'optionele parameter' waarbij 'ie, wanneer niet opgegeven, een default value heeft.

TeeDee schreef op donderdag 29 januari 2009 @ 00:08:
Blegh....

Sorry RobIII

Te laat. Je bent nu in mijn achting gedaald

[ Voor 21% gewijzigd door RobIII op 29-01-2009 00:08 ]

[MrDummy]

TeeDee schreef op donderdag 29 januari 2009 @ 00:08:
Hoort die ; wel op regel 3 en moet dat niet een . zijn?

Scherp gezien, dude
Het gaf geen error melding in php, waardoor het niet meteen opvalt.

Even nieuwe test maken....

[TeeDee]

En dat om 00:14.

RobIII schreef op donderdag 29 januari 2009 @ 00:08:
[...]
Te laat. Je bent nu in mijn achting gedaald

And going up...

[MrDummy]

De emailtest laat nog steeds mysterieuze verdwenen eerste 4-5 regels zien die gemaakt is in $mailBody. De $header is in orde. Ik heb <html><body> erbij gezet maar dat heeft geen effect.
Ze zijn gewoon weg. En dat stoort het me veel.

Ik ga morgen even debuggen of het eigen PHP probleem (de string is zelf "stuk" gegaan) is of dat het mail probleem (onderweg "stuk" gegaan) is. De tekst voor mailbericht komt uit TinyMCE, die als HTML opmaakt.

Je hoort het nog van me.

---
Update:

Bij debuggen van output van $mailBody ontdek ik dat er twee keer bericht plus attachment code erin is gezet. De boosdoener blijkt te zitten in ".=" (met puntje) waardoor er op verkeerde manier is samengevoegd.

PHP:

        $mailBody =     "This is a multi-part message in MIME format.\n".
                        "--------{$mime_boundary}\n".
                        "Content-Type: text/html; charset=UTF-8\n".
                        $mailBody . "\n".
                        "--------{$mime_boundary}--\n";

Zo moet het eigenlijk worden zonder puntje.
Ook elders met <html><body> is met .= samengevoegd, wat ook fout is.

Mocht het niet goed uitkomen, dan steek ik maar witte vlag op en installeer phpmailer...

[ Voor 44% gewijzigd door MrDummy op 29-01-2009 10:56 ]

[MrDummy]

De correctie in de code helpt wat, maar de client blijft nog steeds hele email aanzien als groot brok tekst.

Dus heb ik maar phpmailer gepakt en al snel na 15 minuten maken en nachecken is email goed doorgekomen met losse attachments. Inderdaad, nog een keer wiel uitvinden is veel werk. Het zou wel mooi zijn als er kleine oplossing of verklaring is voor het probleem dat email niet goed werkt met huidige code. Misschien moet het multipart/alternative zijn ipv multipart/mixed (want dat doet phpmailer bij html weergave)

En ja, ik heb mijn witte vlag opgestoken.

[curry684]

"He who admits his mistakes is a strong man"

Succes ermee

[Cartman!]

Als je de verschillen nog graag wilt weten pak je de headers van de niet-werkende mail en de wel-werkende mail en vergelijk je ze

[Verwijderd]

Misschien kun je hier wat uit halen, dit is een script wat ik van internet heb geplukt, het is een email script waar je meerdere attachments aan kunt toevoegen

code:

<? 

// store as "mime_mail.inc" 

class mime_mail 
{ 
   var $parts; 
   var $to; 
   var $from; 
   var $headers; 
   var $subject; 
   var $body; 

   /* 
    *     void mime_mail() 
    *     class constructor 
    */ 

   function mime_mail() { 
      $this->parts = array(); 
      $this->to =  ""; 
      $this->from =  ""; 
      $this->subject =  ""; 
      $this->body =  ""; 
      $this->headers =  ""; 
   } 

   /* 
    *     void add_attachment(string message, [string name], [string ctype]) 
    *     Add an attachment to the mail object 
    */ 

   function add_attachment($message, $name =  "", $ctype = "application/octet-stream") { 
      $this->parts[] = array ( 
            "ctype" => $ctype, 
            "message" => $message, 
            "encode" => $encode, 
            "name" => $name 
                           ); 
   } 

   /* 
    *      void build_message(array part= 
    *      Build message parts of an multipart mail 
    */ 

   function build_message($part) { 
      $message = $part[ "message"]; 
      $message = chunk_split(base64_encode($message)); 
      $encoding =  "base64"; 
      return  "Content-Type: ".$part[ "ctype"]. 
         ($part[ "name"]? "; name = \"".$part[ "name"]. 
         "\"" :  ""). 

         "\nContent-Transfer-Encoding: $encoding\n\n$message\n"; 
   } 

   /* 
    *      void build_multipart() 
    *      Build a multipart mail 
    */ 

   function build_multipart() { 
      $boundary =  "b".md5(uniqid(time())); 
      $multipart = 
         "Content-Type: multipart/mixed; boundary = $boundary\n\nThis is a MIME encoded message.\n\n--$boundary"; 

         for($i = sizeof($this->parts)-1; $i >= 0; $i--) 
      { 
         $multipart .=  "\n".$this->build_message($this->parts[$i]). 
            "--$boundary"; 
      } 
      return $multipart.=  "--\n"; 
   } 

   /* 
    *      string get_mail() 
    *      returns the constructed mail 
    */ 

   function get_mail($complete = true) { 
      $mime =  ""; 
      if (!empty($this->from)) 
         $mime .=  "From: ".$this->from. "\n"; 
      if (!empty($this->headers)) 
         $mime .= $this->headers. "\n"; 

      if ($complete) { 
         if (!empty($this->to)) { 
            $mime .= "To: $this->to\n"; 
         } 
         if (!empty($this->subject)) { 
            $mime .= "Subject: $this->subject\n"; 
         } 
      } 

      if (!empty($this->body)) 
         $this->add_attachment($this->body,  "",  "text/plain"); 
      $mime .=  "MIME-Version: 1.0\n".$this->build_multipart(); 

      return $mime; 
   } 

   /* 
    *      void send() 
    *      Send the mail (last class-function to be called) 
    */ 

   function send() { 
      $mime = $this->get_mail(false); 
      mail($this->to, $this->subject,  "", $mime); 
   } 
};  // end of class 

?> 

<? 
include "mime_mail.inc"; 

if (empty($recipient)){ 
?> 

Please ENTER your E-mail adress to receive the files: 
<form> 
<input type="text" name="recipient"> 
<input type="submit"value="ok"> 
</form> 

<? 
exit; 
} 
print "Sending file to $recipient.<p>"; 

$mail = new mime_mail; 
$mail->from ="[e-mail]"; 
$mail->to=$recipient; 
$mail->subject= "Test"; 
$mail->body= "Test test test"; 

$content_type="image/jpeg"; 

$dir = opendir(".."); 

while (($filename = readdir($dir))){ 
if (ereg("\.jpg$", $filename)){ 
$fd = fopen($filename, "r"); 
$data =fread($fd, filesize($filename)); 
fclose($fd); 

$mail->add_attachment($data, $filename, $content_type); 

print "I have added $filename. <br>"; 
} 
} 
closedir($dir); 
$mail->send(); 
print"Ihave send off the email. "; 
print "Veel plezier ermee"; 
?>

[RobIII]

Verwijderd schreef op vrijdag 30 januari 2009 @ 17:42:

Je hebt duidelijk bovenstaande discussie niet gevolgd
En, hoewel goedbedoeld, er zijn ongeveer anderhalf miljoen php email scripts op het web te vinden waarbij in datzelfde aantal - 100 ofzo het wiel niet opnieuw is uitgevonden op een vierkante manier. Als iemand dat zoekt kan 'ie dat prima zelf vinden. Ik zie niet wat jouw 'vondst' beter maakt dan eender welke andere.

[ Voor 53% gewijzigd door RobIII op 30-01-2009 18:15 ]