php - mail(); får en fejl når jeg prøver at lave en query inde i $message - Udvikleren.dk

Tags: php mail

<< < 1 2 > >>

Bruger #16555 @ 22.03.12 17:28

Jeg får en fejl når jeg forsøger at lave en query inde i den variabel der indeholder selve beskeden som sendes via mail() funktionen.

Jeg stopper nærmest scriptet, og vil jo gerne have mulighed for at emaile kunden vare oplysningerne, og ikke kun hvilket beløb der er betalt samt leverings adresse.

Der går galt her:

$get_cart_info...

Den query giver mig følgende fejl i browseren og mailen sendes ikke:

Catchable fatal error: Object of class mysqli_result could not be converted to string in /home/www/xxx.dk/views/betaling_accepteret.php on line 129 (som er ved $get_cart_info)

Jeg går ud fra det er en syntax fejl i sql der stopper scriptet? Det ser nu bare rigtigt ud i mine øjne, men jeg tror der skal et par nye til at se på det, da jeg vist har stirret mig blind på det..

Så hvordan kan jeg lave denne mail, uden at knække den og ødelægge den ved at "stoppe" den ved $get_cart_info??

PHP kode

 $order_mail_comfirmation = '<div style="width:80%;">
 <p>Tak fordi du handlede hos xxx</p>
 <small>Der er registreret flg. detaljer vedrørende dit køb:</small><br /><br />
 <table>
 <tr>
     <td><small><b>Transaktions id:</b></small></td>
     <td style="padding-left:20px;">
         <small>'.$_SESSION['transaktions_id'].'</small>
         </td>
 </tr>
 <tr>
     <td><small><b>Ordre id:</b></small></td>
     <td style="padding-left:20px;"><small>'.$_SESSION['order_id'].'</small>                 
         </td>
 </tr>
 <tr>
     <td><small><b>Beløb hævet på kort:</b></small></td>
     <td style="padding-left:20px;">
         <small>'.number_format($samlet_pris,2).'</small></td>
 </tr>
 <tr>
     <td><small><b>Dato og tid:</b></small></td>
     <td style="padding-left:20px;"><small>'.$added_time.'</small></td>
 </tr>
 </table>
 <br />
 <small>
 Følgende vare(r) vil blive <b>afsendt</b> indenfor 1-2 hverdage, såfremt det ikke er en bestillingsvare.</small><br /><br />
 
 <table>
 '.$get_cart_info = mysqli_query($connection, "
 SELECT 
 produkt_id, navn, vare_type, model, antal, pris, str 
 FROM cart_indloest 
 WHERE user_id = '".$user_id."' AND ordre_id = '".$_SESSION['order_id']."'") or die(mysqli_error($connection));
 
 while ($cit = mysqli_fetch_array($get_cart_info)) // $cit = cart items
 {
 // Hent billed extension fra produkter så billedet af det købte produkt kan vises på kvitteringen:
 $get_img_info = mysqli_query($connection, "SELECT img FROM produkter WHERE id = '".$cit['produkt_id']."'");
 while ($img_ext = mysqli_fetch_array($get_img_info))
 {
 $img = $img_ext['img'];
 }
         
 '<tr>
         
 <td style="border:1px solid #666;">
 <img src="../backend/produkt_img/backend_img/'.$cit['produkt_id'].'.'.$img.'" width="60" height="90" />
 </td>
         
 <td style="padding-left:15px; padding-right:15px; background-color:#EEE; border:1px solid #666;">
 <small style="color:#333; font-weight:bold;"><b>&bull; '.$cit['antal'].' par '
               .ucfirst($cit['navn']).' '
               .ucfirst($cit['vare_type']).' - Model '
               .ucfirst($cit['model']).' - Stk. pris: '
               .number_format($cit['pris'],2).' DKK. - Størrelse: '
               .$cit['str'].'</b></small>
 </td>
 </tr>';
     }
 '</table>
 <br />
 <small>
 xxx forventer at varerne vil blive <b>leveret</b> indenfor 2-3 hverdage:
 </small>
 <br /><br />
 <hr />
 <br />
 <small><b>Modtager og adresse er registreret til:</b></small>
 <br />
 <small> '.ucfirst($_SESSION['fornavn']).' '.ucfirst($_SESSION['efternavn']).' </small>
 <br />
 <small> '.ucfirst($_SESSION['gade']).' '.$_SESSION['hus_nr'].'<br />'.$_SESSION['post_nr'].' '.$_SESSION['by'].' </small>
 <br /><br />
 <small><b>Der er afsendt en kvittering til:</b></small><br />
 <small> '. $_SESSION['email'].' </small><br /><br />
 </div>';
 
 // Send email til kunden her:
     $to         = $_SESSION['email'];
     $subject     = 'Ordrebekræftelse fra xxx';    
     $message     = $order_mail_comfirmation;
     
     $headers     = "Content-type: text/html; charset=utf-8\r\n";                    
     $headers     .= '<div style="width:500px; height:40px; padding-top:7px; background-color:#ccc; border:1px solid #666;">
     <p style="color:#666; margin-left:100px;">
     <b>Ordrebekræftelse</b>
     </p></div>';    
     $headers .= '<hr style="width:80%;" align="left"/>';        
     mail($to, $subject, $message, $headers);

Indlæg senest redigeret d. 22.03.2012 17:47 af Bruger #16555

16 svar postet i denne tråd vises herunder
0 indlæg har modtaget i alt 0 karma

Sorter efter stemmer Sorter efter dato

Bruger #16555 @ 23.03.12 13:52

Hej Michael,

Tak for den shortcut, jeg har ændret den lidt så den passede til det hele.

Jeg får fint alle rækker ud via den join, men mit problem er stadig det samme!

Jeg får lavet en fejl når jeg vil integrere netop de resultater fra den query i den email (ordre bekræftelse) jeg gerne vil sende til kunderne. Det er her det går galt.

Før fik jeg også informationerne ud, bare uden en join - Men jeg får dog ødelagt mailen når jeg prøver at integrere resultatet i mail beskeden, som ser således ud:

PHP kode

 $order_mail_confirmation = '<div style="width:80%;">
 <p>Tak fordi du handlede hos XXX.</p>
 <small>Der er registreret flg. detaljer vedrørende dit køb:</small><br /><br />
 <table>
 <tr>
 <td><small><b>Transaktions id:</b></small></td>
 <td style="padding-left:20px;"><small>'.$_SESSION['transaktions_id'].'</small></td>
 </tr>
 <tr>
 <td><small><b>Ordre id:</b></small></td>
 <td style="padding-left:20px;"><small>'.$_SESSION['order_id'].'</small></td>
 </tr>
 <tr>
 <td><small><b>Beløb hævet på kort:</b></small></td>
 <td style="padding-left:20px;"><small>'.number_format($samlet_pris,2).'</small></td>
 </tr>
 
 <tr>
 <td><small><b>Dato og tid:</b></small></td>
 <td style="padding-left:20px;"><small>'.$added_time.'</small></td>
 </tr>
 </table>
 <br />
 <small>
 Følgende vare(r) vil blive <b>afsendt</b> indenfor 1-2 hverdage, 
 såfremt det ikke er en bestillingsvare.
 </small><br /><br />
 <table>
 ........................................................
 [b]LIGE HER VIL JEG GERNE UDSKRIVE RESULTATERNE FRA DEN QUERY DER JOINER
  PRODUKT INFO OG IMG FRA HHV: CART_INDLOEST OG PRODUKTER[/b]
 ........................................................
 
 
 </table>
 <br />
 <small>
 XXX forventer at varerne vil blive <b>leveret</b> indenfor 2-3 hverdage:
 </small>
 <br /><br />
 <hr />
 <br />
 <small><b>Modtager og adresse er registreret til:</b></small>
 <br />
 <small> '.ucfirst($_SESSION['fornavn']).' '.ucfirst($_SESSION['efternavn']).' </small>
 <br />
 <small> '.ucfirst($_SESSION['gade']).' '.$_SESSION['hus_nr'].'<br />'.$_SESSION['post_nr'].' '.$_SESSION['by'].' </small>
 <br /><br />
 <small><b>Der er afsendt en kvittering til:</b></small><br />
 <small> '. $_SESSION['email'].' </small><br /><br />
 </div>';
 // Send email til kunden her:
     $to         = $_SESSION['email'];
     $subject     = 'Ordrebekræftelse fra XXX';    
     $message     = $order_mail_confirmation;
     
     $headers     = "Content-type: text/html; charset=utf-8\r\n";                    
     $headers     .= '<div style="width:500px; height:40px; padding-top:7px; background-color:#ccc; border:1px solid #666;">
     <p style="color:#666; margin-left:100px;">
     <b>Ordrebekræftelse</b>
     </p></div>';    
     $headers .= '<hr style="width:80%;" align="left"/>';        
     mail($to, $subject, $message, $headers);

Indlæg senest redigeret d. 23.03.2012 13:55 af Bruger #16555

Bruger #10216 @ 23.03.12 14:07

4.283

Cirka sådan her burde det kunne gøres:

PHP kode

 <?php
 // SQL - de to SQL statements kan kombineres med JOIN
 $sql = "SELECT
             cart_indloest.produkt_id,
             cart_indloest.navn,
             cart_indloest.vare_type,
             cart_indloest.model,
             cart_indloest.antal,
             cart_indloest.pris,
             cart_indloest.str,
             product.img
         FROM cart_indloest
         JOIN produkter ON produkter.id = product_id
         WHERE user_id = '".$user_id."' AND ordre_id = '".$_SESSION['order_id']."'
         GROUP BY cart_indloest.produkt_id"; 
  
 // Udfør SQL
 $resource = mysqli_query($connection, $sql);
  
 // hent data fra result
 while ($row = mysqli_fetch_assoc($resource)) {// $cit = cart items
      $result[] = $row;
 }
  
 // Loop vores data
 $order_mail_product_list = '';
 foreach($result as $row) {
  $order_mail_product_list .= '<tr><td style="border:1px solid #666;">
          <img src="../backend/produkt_img/backend_img/'.$cit['produkt_id'].'.'.$img.'" width="60" height="90" />
   </td>
   <td style="padding-left:15px; padding-right:15px; background-color:#EEE; border:1px solid #666;">
   <small style="color:#333; font-weight:bold;"><b>&bull;
            <img src="'.$row['img'].'" alt="" /> 
           '.$row['antal'].' par '
           .ucfirst($row['navn']).' '
           .ucfirst($row['vare_type']).' - Model '
           .ucfirst($row['model']).' - Stk. pris: '
           .number_format($row['pris'],2).' DKK. - Størrelse: '
           .$row['str'].'</b></small>
           </td>
           </tr>';
 }
 
   
 $order_mail_confirmation = '<div style="width:80%;">
  <p>Tak fordi du handlede hos XXX.</p>
  <small>Der er registreret flg. detaljer vedrørende dit køb:</small><br /><br />
  <table>
  <tr>
  <td><small><b>Transaktions id:</b></small></td>
  <td style="padding-left:20px;"><small>'.$_SESSION['transaktions_id'].'</small></td>
  </tr>
  <tr>
  <td><small><b>Ordre id:</b></small></td>
  <td style="padding-left:20px;"><small>'.$_SESSION['order_id'].'</small></td>
  </tr>
  <tr>
  <td><small><b>Beløb hævet på kort:</b></small></td>
  <td style="padding-left:20px;"><small>'.number_format($samlet_pris,2).'</small></td>
  </tr>
  
  <tr>
  <td><small><b>Dato og tid:</b></small></td>
  <td style="padding-left:20px;"><small>'.$added_time.'</small></td>
  </tr>
  </table>
  <br />
  <small>
  Følgende vare(r) vil blive <b>afsendt</b> indenfor 1-2 hverdage, 
  såfremt det ikke er en bestillingsvare.
  </small><br /><br />
  <table>
  '.$order_mail_product_list /*** HER INDSÆTTES PRODUKTLISTEN ***/.'
  </table>
  <br />
  <small>
  XXX forventer at varerne vil blive <b>leveret</b> indenfor 2-3 hverdage:
  </small>
  <br /><br />
  <hr />
  <br />
  <small><b>Modtager og adresse er registreret til:</b></small>
  <br />
  <small> '.ucfirst($_SESSION['fornavn']).' '.ucfirst($_SESSION['efternavn']).' </small>
  <br />
  <small> '.ucfirst($_SESSION['gade']).' '.$_SESSION['hus_nr'].'<br />'.$_SESSION['post_nr'].' '.$_SESSION['by'].' </small>
  <br /><br />
  <small><b>Der er afsendt en kvittering til:</b></small><br />
  <small> '. $_SESSION['email'].' </small><br /><br />
  </div>';
  // Send email til kunden her:
      $to         = $_SESSION['email'];
      $subject     = 'Ordrebekræftelse fra XXX';    
      $message     = $order_mail_confirmation;
      
      $headers     = "Content-type: text/html; charset=utf-8\r\n";                    
      $headers     .= '<div style="width:500px; height:40px; padding-top:7px; background-color:#ccc; border:1px solid #666;">
      <p style="color:#666; margin-left:100px;">
      <b>Ordrebekræftelse</b>
      </p></div>';    
      $headers .= '<hr style="width:80%;" align="left"/>';        
      mail($to, $subject, $message, $headers);

Men det er langt fra den optimale løsning. Til sådanne ting vil jeg anbefale at du går i gang med at kigge på nogle template-mekanismer.

Indlæg senest redigeret d. 23.03.2012 14:07 af Bruger #10216

Bruger #16555 @ 23.03.12 14:26

Ja det er en skidt løsning, og havde ikke forventet at det at sende en simpel ordre bekræftelse skulle ende med at være det stykke kode der skulle tage længst tid på en hel shop..

Jeg må lære mere om mail, måder at sende/integrere potentielt større data sæt deri.

Hvordan mener du præcis at du vil benytte din fremgangsmåde som skrevet ovenfor?

'.$order_mail_product_list /*** HER INDSÆTTES PRODUKTLISTEN ***/.'??

Når jeg begynder at fifle med (forkerte, red..) måder at få resultaterne derind på, så ja, sker der ikke så meget, og mailen kommer enten ikke frem eller der er blot blankt der hvor varerne skulle have været skrevet op på ordre bekræftelsen.

Bruger #10216 @ 23.03.12 14:44

4.283

Kig efter variablen $order_mail_product_list - dens værdi sættes i loopet.

Email headers må ikke indeholde HTML - her skal du læse dokumentationen http://dk2.php.net/manual/en/function.mail.php for hvad der må være i $headers. Der er masser af eksempler for hvordan du skriver mail eksekveringen.

Alternativt kan du f.eks. benytte PHPMailer til den slags - men det er en helt anden boldgade.

Bruger #16555 @ 23.03.12 17:14

Jeg havde overset at du havde indsat den der :-)

Jeg fik dog dobbelt antal varer ind i mailen som kunne ses på kvitteringen. Men det er nu helt iorden.

Michael: Det var dette der fik det til at virke, som jeg ikke selv havde set:
$order_mail_product_list = '';
foreach($result as $row) {
$order_mail_product_list .= etc etc..

Så blev data gemt fint, fra 1 vare til 100 - og det var det jeg var på udkig efter.

Så mange tak for løsningen!

Jeg er lidt en novice ud i mail, men vil du anbefale PHPMailer fremfor mail()?

Hvad er forskellen i måden man kan sende data på?

Og i hvilke tilfælde ville det være en bedre fremgangsmåde?

Så har jeg det i baghovedet til næste gang :-)

Indlæg senest redigeret d. 23.03.2012 17:14 af Bruger #16555

Bruger #10216 @ 23.03.12 22:08

4.283

mail() er php's egen mail funktion...

PHPMailer er et API der pakker mail() ind, og tilføjer en rækker mere logiske metoder og funktioner.
Se evt. mere her: http://code.google.com/a/apache-extras.org/p/phpmailer/wiki/UsefulTutorial#First_time

Forskellen i at sende data er ikke svært anderledes i det ene vs. det andet. Med mail() får du kun en funktion og så skal du vide resten, hvilke relevante headers du skal sende med oma. Med PHPMailer får du en OOP tilgang hvor du indstiller nogle parametrer, og så vil APIet sørge for at generere de korrekte headers til din email.

Den ene tilgang er ikke mere korrekt end den anden. Afhængig af vane, kan den ene dog være nemmere end den anden.

<< < 1 2 > >>

mail(); får en fejl når jeg prøver at lave en query inde i $message

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler