Странный глюк с str_replace в моем PHP-скрипте - PullRequest
Новая
       1

Странный глюк с str_replace в моем PHP-скрипте

0 голосов
/ 11 января 2012

У меня странная проблема с str_replace в моем php-коде ниже. Предполагается, что в каждом цикле он должен заменить {имя} на имя человека, извлеченное из базы данных. Что я на самом деле делаю, так это если я отправляю по почте двум людям, первый из которых заменяет их имя, чтобы они получали электронное письмо dear, bob bla bla bla. Второй всегда кажется dear , {name} bla bla bla. Это как во втором цикле что-то не так? 

            <?php

                 $formid = mysql_real_escape_string($_GET[token]);
            $templatequery = mysql_query("SELECT * FROM hqfjt_chronoforms_data_addmailinglistmessage WHERE cf_id = '$formid'") or die(mysql_error());
            $templateData = mysql_fetch_object($templatequery);

            $gasoiluserTemplate = $templateData->gasoilusers;
            $dervuserTemplate = $templateData->dervusers;
            $kerouserTemplate = $templateData->kerousers;
            $templateMessage = $templateData->mailinglistgroupmessage;
            $templatename = $templateData->mailinglistgroupname;
                                ?>  
                    <?php
            require_once('./send/class.phpmailer.php');
            //include("class.smtp.php"); // optional, gets called from within class.phpmailer.php if not already loaded

            $mail = new PHPMailer(true); //defaults to using php "mail()"; the true param means it will throw exceptions on errors, which we need to catch

            // $body                = file_get_contents('contents.html');

            $body = '<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
            <html xmlns="http://www.w3.org/1999/xhtml">
            <head>
            <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
            <style>#title {padding-left:120px;padding-top:10px;font-family:"Times New Roman", Times, serif; font-size:22px; font-weight:bold; color:#fff;}</style>
            </head>

            <body>
            <div style="background:
                                              none repeat scroll 0% 0% rgb(6, 38,
                                              97); width:780px;">
            <img id="_x0000_i1030" style="padding-left:100px;padding-right:100px;"
                                                  src="http://www.chandlersoil.com/images/newsletter/header.gif"
                                                  alt="Chandlers Oil and Gas"
                                                  border="0" height="112"
                                                  width="580">
                                                  <div id="title">{message}</div>

                                                  </div>
            </body>
            </html>
            ';

                            // $body = preg_replace('/\\\\/i', $body);

                            $mail->SetFrom('crea@cruiseit.co.uk', 'Chandlers Oil & Gas');
                            $mail->AddReplyTo('crea@cruiseit.co.uk', 'Chandlers Oil & Gas');

                            $mail->Subject       = "Your Fuel Prices From Chandlers Oil & Gas";

                            $query  = "SELECT leadname,businessname,email FROM hqfjt_chronoforms_data_addupdatelead WHERE keromailinglist='$kerouserTemplate' AND dervmailinglist='$dervuserTemplate' AND gasoilmailinglist='$gasoiluserTemplate'";
                            $result = mysql_query($query);

                            // Bail out on error 
            if (!$result)  
              { 
                trigger_error("Database error: ".mysql_error()." Query used was: ".htmlentities($query), E_USER_ERROR); 
                die();
               }                                    

            while ($row = mysql_fetch_array ($result)) {
              $mail->AltBody    = "To view the message, please use an HTML compatible email viewer!";

               // THIS PULLS THE CLIENTS FIRST NAME OUT ON EACH LOOP
               $firstname = $row["leadname"];

               //THIS PULLS THE CLIENTS BUSSINESS NAME OUT ON EACH LOOP
               $businessname = $row["businessname"];

               // IF THE FIRST NAME FIELD IS BLANK USE THE BUSINESS NAME INSTEAD
               if ($firstname = '')
               {$name = $row["businessname"];}
               else 
               {$name = $row["leadname"];}

               // THIS REPLACES THE {NAME} IN THE PULLED IN TEMPLATE MESSAGE WITH THE CLIENTS NAME DEFINED IN $name
               $body = str_replace('{name}', $name, $body);

               // THIS REPLACES {fuel} IN THE PULLED IN TEMPLATE WITH THE TEMPLATE NAME (WHICH IS THE TYPE OF FUEL)
               $body = str_replace('{fuel}', $templatename, $body); 

               // THIS REPLACES THE {message} IN THE $body ARRAY WITH THE TEMPLATE MESSAGE HELD IN $templateMessage
               $body = str_replace('{message}', $templateMessage, $body);


              $mail->MsgHTML($body);
              $mail->AddAddress($row["email"], $row["leadname"]);




              if(!$mail->Send()) {
                echo "Mailer Error (" . str_replace("@", "&#64;", $row["email"]) . ') ' . $mail->ErrorInfo . '<br>';
              } else {
                echo "Message sent to :" . $row["full_name"] . ' (' . str_replace("@", "&#64;", $row["email"]) . ')<br>';
              }
              // Clear all addresses and attachments for next loop
              $mail->ClearAddresses();
              $mail->ClearAttachments();
            }
            ?>

Ответы [ 2 ]

2 голосов
/ 11 января 2012

$body содержит шаблон письма.В вашем цикле вы присваиваете этой переменной возвращаемое значение из str_replace().После этого вы не можете ожидать, что он будет содержать оригинальный шаблон при следующей итерации.Для этого вам понадобится временная переменная: 

while (...) {
   $bodyTemp = str_replace('{name}', $name, $body);
   $bodyTemp = str_replace('{fuel}', $templatename, $bodyTemp);
   // ...
}

Кроме того, чтобы сделать ваш код немного более читабельным, я мог бы предложить вместо этого использовать strtr(): 

while (...) {
   $bodyTemp = strtr($body, array(
      '{name}' => $name,
      '{fuel}' => $templatename,
      // ...
   ));
}

Это спасает вас от повторяющихся вызовов str_replace().

0 голосов
/ 11 января 2012

Перед циклом $ body может содержать Dear {name}.Затем вы цикл один раз, тогда он содержит Dear Iain.Затем во втором цикле нет {name} для замены.

...