Product images on order confirmation email

Hi,

I am trying to get product images to show on the order confirmation email. However because I don't know much about php, I will need some guidance. I located the code that draws the table of ordered product details in /includes/classes/order.php.

      // build output for email notification
      $this->products_ordered .=  $this->products[$i]['qty'] . ' x ' . $this->products[$i]['name'] . ($this->products[$i]['model'] != '' ? ' (' . $this->products[$i]['model'] . ') ' : '') . ' = ' .
      $currencies->display_price($this->products[$i]['final_price'], $this->products[$i]['tax'], $this->products[$i]['qty']) .
      ($this->products[$i]['onetime_charges'] !=0 ? "\n" . TEXT_ONETIME_CHARGES_EMAIL . $currencies->display_price($this->products[$i]['onetime_charges'], $this->products[$i]['tax'], 1) : '') .
      $this->products_ordered_attributes . "\n";
      $this->products_ordered_html .=
      '<tr>' .
      '<td class="product-details" align="right" valign="top" width="30">' . $this->products[$i]['qty'] . ' x</td>' .
      '<td class="product-details" valign="top">' . $this->products[$i]['name'] . ($this->products[$i]['model'] != '' ? ' (' . $this->products[$i]['model'] . ') ' : '') .
      '<nobr><small><em> '. $this->products_ordered_attributes .'</em></small></nobr></td>' .
      '<td class="product-details-num" valign="top" align="right">' .
      $currencies->display_price($this->products[$i]['final_price'], $this->products[$i]['tax'], $this->products[$i]['qty']) .
      ($this->products[$i]['onetime_charges'] !=0 ?
      '</td></tr><tr><td class="product-details">' . TEXT_ONETIME_CHARGES_EMAIL . '</td>' .
      '<td>' . $currencies->display_price($this->products[$i]['onetime_charges'], $this->products[$i]['tax'], 1) : '') .
      '</td></tr>';

I need to somehow integrate the following code in the above code to show product images on the order confirmation email.

          $products = $db->Execute("SELECT products_image
                            FROM " . TABLE_PRODUCTS . "
                            WHERE products_id ='" . $order->products[$i]['id'] . "'");
          echo '' . zen_image(DIR_WS_CATALOG . DIR_WS_IMAGES . $products->fields['products_image'] , $order->products[$i]['name'], SMALL_IMAGE_HEIGHT, SMALL_IMAGE_WIDTH) . '</a> ';

I am thinking to join them together this way but I am not sure if it will work. Can someone smarter than me please help check? Changes are highlighted in italic and bold. Thanks in advance :)

// build output for email notification
      $this->products_ordered .=  $this->products[$i]['qty'] . ' x ' [I][B]. zen_image(DIR_WS_CATALOG . DIR_WS_IMAGES . $products->fields['products_image'] , $order->products[$i]['name'], SMALL_IMAGE_HEIGHT, SMALL_IMAGE_WIDTH)[/I][/B] . $this->products[$i]['name'] . ($this->products[$i]['model'] != '' ? ' (' . $this->products[$i]['model'] . ') ' : '') . ' = ' .
      $currencies->display_price($this->products[$i]['final_price'], $this->products[$i]['tax'], $this->products[$i]['qty']) .
      ($this->products[$i]['onetime_charges'] !=0 ? "\n" . TEXT_ONETIME_CHARGES_EMAIL . $currencies->display_price($this->products[$i]['onetime_charges'], $this->products[$i]['tax'], 1) : '') .
      $this->products_ordered_attributes . "\n";
   [I][B]$products = $db->Execute("SELECT products_image FROM " . TABLE_PRODUCTS . " WHERE products_id ='" . $order->products[$i]['id'] . "'");[/I][/B]
      $this->products_ordered_html .=
      '<tr>' .
      '<td class="product-details" align="right" valign="top" width="30">' . $this->products[$i]['qty'] . ' x</td>' .
   [I][B]'<td class="product-details" valign="top">' . zen_image(DIR_WS_CATALOG . DIR_WS_IMAGES . $products->fields['products_image'] , $order->products[$i]['name'], SMALL_IMAGE_HEIGHT, SMALL_IMAGE_WIDTH) .</td>' .[/I][/B]
      '<td class="product-details" valign="top">' . $this->products[$i]['name'] . ($this->products[$i]['model'] != '' ? ' (' . $this->products[$i]['model'] . ') ' : '') .
      '<nobr><small><em> '. $this->products_ordered_attributes .'</em></small></nobr></td>' .
      '<td class="product-details-num" valign="top" align="right">' .
      $currencies->display_price($this->products[$i]['final_price'], $this->products[$i]['tax'], $this->products[$i]['qty']) .
      ($this->products[$i]['onetime_charges'] !=0 ?
      '</td></tr><tr><td class="product-details">' . TEXT_ONETIME_CHARGES_EMAIL . '</td>' .
      '<td>' . $currencies->display_price($this->products[$i]['onetime_charges'], $this->products[$i]['tax'], 1) : '') .
      '</td></tr>';

Tried that, but somehow the images shows up as "no image available". Any suggestion? I thought most people were eager to get this working...

Have you looked at the packing list with images contribution in the archives of the downloads section? It may have a useful hint or two.

afo:

Have you looked at the packing list with images contribution in the archives of the downloads section? It may have a useful hint or two.

I got the hint from packing slip too but it's from Super Order's packing slip. I looked at the packing list with images contribution. The codes are very similar. I could now put these code at various places I want, except the order confirmation email/invoice. I believe I am quite close to getting it to show... but something is not very right still... changes are highlighted in bold and italic. Any suggestion?

      // build output for email notification
      $this->products_ordered .= [B][I]$products->fields['products_image'] .[/B][/I] $this->products[$i]['qty'] . ' x ' . $this->products[$i]['name'] . ($this->products[$i]['model'] != '' ? ' (' . $this->products[$i]['model'] . ') ' : '') . ' = ' .
      $currencies->display_price($this->products[$i]['final_price'], $this->products[$i]['tax'], $this->products[$i]['qty']) .
      ($this->products[$i]['onetime_charges'] !=0 ? "\n" . TEXT_ONETIME_CHARGES_EMAIL . $currencies->display_price($this->products[$i]['onetime_charges'], $this->products[$i]['tax'], 1) : '') .
      $this->products_ordered_attributes . "\n";
      [B][I]$products = $db->Execute("SELECT products_image
                                    FROM " . TABLE_PRODUCTS . "
                                    WHERE products_id ='" . $order->products[$i]['id'] . "'");[/B][/I]
      $this->products_ordered_html .=
      '<tr>' .
      [B][I]'<td class="product-details" align="left" valign="top">' . zen_image(DIR_WS_CATALOG . DIR_WS_IMAGES . $products->fields['products_image'] , $order->products[$i]['name'], IMAGE_SHOPPING_CART_HEIGHT) . '</a> </td>' . [/B][/I]
      '<td class="product-details" align="right" valign="top" width="30">' . $this->products[$i]['qty'] . ' x</td>' .
      '<td class="product-details" valign="top">' . $this->products[$i]['name'] . ($this->products[$i]['model'] != '' ? ' (' . $this->products[$i]['model'] . ') ' : '') .
      '<nobr><small><em> '. $this->products_ordered_attributes .'</em></small></nobr></td>' .
      '<td class="product-details-num" valign="top" align="right">' .
      $currencies->display_price($this->products[$i]['final_price'], $this->products[$i]['tax'], $this->products[$i]['qty']) .
      ($this->products[$i]['onetime_charges'] !=0 ?
      '</td></tr><tr><td class="product-details">' . TEXT_ONETIME_CHARGES_EMAIL . '</td>' .
      '<td>' . $currencies->display_price($this->products[$i]['onetime_charges'], $this->products[$i]['tax'], 1) : '') .
      '</td></tr>';
    }

Try adding the Full path to the image file: http://www.whatever.com/images/whatever.jpg

If you get this going, I would like to see it in the Recover Cart mod's eMail's too!

JOhn ><>

Good news, I got product images to display on order confirmation email after lots of testing but there is a slight problem. When a customer choose to pay using PayPal, my website doesn't generate an order confirmation email to the customer as well as myself. Payments with bank deposit and check/money order seem fine. Can anyone please check and help me figure out why this only happens to PayPal? Thanks.

      // BUILD OUTPUT FOR EMAIL NOTIFICATION
// turned off the following original line
//    $this->products_ordered .= $this->products[$i]['qty'] . ' x ' . $this->products[$i]['name'] . ($this->products[$i]['model'] != '' ? ' (' . $this->products[$i]['model'] . ') ' : '') . ' = ' .
// changed to the following new line to include thumbnail images on order confirmation email
      $this->products_ordered .= zen_image(DIR_WS_IMAGES . $products->fields['products_image'], $this->products[$i]['name'], IMAGE_SHOPPING_CART_HEIGHT) . '' . $this->products[$i]['qty'] . ' x ' . $this->products[$i]['name'] . ($this->products[$i]['model'] != '' ? ' (' . $this->products[$i]['model'] . ') ' : '') . ' = ' .
      $currencies->display_price($this->products[$i]['final_price'], $this->products[$i]['tax'], $this->products[$i]['qty']) .
      ($this->products[$i]['onetime_charges'] !=0 ? "\n" . TEXT_ONETIME_CHARGES_EMAIL . $currencies->display_price($this->products[$i]['onetime_charges'], $this->products[$i]['tax'], 1) : '') .
      $this->products_ordered_attributes . "\n";
      $products = $db->Execute("SELECT products_image
                                    FROM " . TABLE_PRODUCTS . "
                                    WHERE products_id ='" . $this->products[$i]['id'] . "'");
      $this->products_ordered_html .=
      '<tr>' .
// also added the following new line to include thumbnail images on order confirmation email
      '<td class="product-details" align="left" valign="top">' . zen_image(DIR_WS_IMAGES . $products->fields['products_image'] , $this->products[$i]['name'], IMAGE_SHOPPING_CART_HEIGHT) . '</a> </td>' .
      '<td class="product-details" align="right" valign="top" width="30">' . $this->products[$i]['qty'] . ' x</td>' .

dealbyethan.com:

  $products = $db->Execute("SELECT products_image
                                FROM " . TABLE_PRODUCTS . "
                                WHERE products_id ='" . $this->products[$i]['id'] . "'");

these lines should be above your first zen_image call that uses $products->fields

that may or may not help your paypal issue


what version of zen cart is this site using now?

these lines should be above your first zen_image call that uses $products->fields

Sorry, what do you mean by call that uses $products->fields? Do you mean change $products = $db->Execute to $products->fields ?

It's version 1.3.7.

Hi DrByte,

Just tried that (see below) but the order confirmation email is still not generated. Any other suggestion? Thanks.

$products = $db->Execute("SELECT products_image
                                    FROM " . TABLE_PRODUCTS . "
                                    WHERE products_id ='" . $this->products[$i]['id'] . "'");
      $this->products_ordered .= zen_image(DIR_WS_IMAGES . $products->fields['products_image'], $this->products[$i]['name'], IMAGE_SHOPPING_CART_HEIGHT) . '' . $this->products[$i]['qty'] . ' x ' . $this->products[$i]['name'] . ($this->products[$i]['model'] != '' ? ' (' . $this->products[$i]['model'] . ') ' : '') . ' = ' .

----

If you undo the changes you've made in these lines of code that you've been quoting, does it work correctly?

Hi DrByte,

When I removed all the codes used for generating thumbnail product images, the order confirmation email did get generated. But I am really keen to have thumbnail product images to display on the order confirmation email because it will help identify the products when fulfilling orders and is of course visually appealing for customers. Can you help? It did generate the order confirmation email when customers checkout using bank deposit or check/money order but not PayPal. Why is that?

dealbyethan.com:

  // BUILD OUTPUT FOR EMAIL NOTIFICATION

// turned off the following original line
// $this->products_ordered .= $this->products[$i]['qty'] . ' x ' . $this->products[$i]['name'] . ($this->products[$i]['model'] != '' ? ' (' . $this->products[$i]['model'] . ') ' : '') . ' = ' .
// changed to the following new line to include thumbnail images on order confirmation email
$this->products_ordered .= zen_image(DIR_WS_IMAGES . $products->fields['products_image'], $this->products[$i]['name'], IMAGE_SHOPPING_CART_HEIGHT) . '' . $this->products[$i]['qty'] . ' x ' . $this->products[$i]['name'] . ($this->products[$i]['model'] != '' ? ' (' . $this->products[$i]['model'] . ') ' : '') . ' = ' .
$currencies->display_price($this->products[$i]['final_price'], $this->products[$i]['tax'], $this->products[$i]['qty']) .
($this->products[$i]['onetime_charges'] !=0 ? "\n" . TEXT_ONETIME_CHARGES_EMAIL . $currencies->display_price($this->products[$i]['onetime_charges'], $this->products[$i]['tax'], 1) : '') .
$this->products_ordered_attributes . "\n";
$products = $db->Execute("SELECT products_image
FROM " . TABLE_PRODUCTS . "
WHERE products_id ='" . $this->products[$i]['id'] . "'");
$this->products_ordered_html .=
'<tr>' .
// also added the following new line to include thumbnail images on order confirmation email
'<td class="product-details" align="left" valign="top">' . zen_image(DIR_WS_IMAGES . $products->fields['products_image'] , $this->products[$i]['name'], IMAGE_SHOPPING_CART_HEIGHT) . '</a> </td>' .
'<td class="product-details" align="right" valign="top" width="30">' . $this->products[$i]['qty'] . ' x</td>' .

I'd suggest a few changes:

1. Put this change back to the original, because it's irrelevant to have an HTML image link in a text-only email:
```php
      // BUILD OUTPUT FOR EMAIL NOTIFICATION
// turned off the following original line
//    $this->products_ordered .= $this->products[$i]['qty'] . ' x ' . $this->products[$i]['name'] . ($this->products[$i]['model'] != '' ? ' (' . $this->products[$i]['model'] . ') ' : '') . ' = ' .
// changed to the following new line to include thumbnail images on order confirmation email
      $this->products_ordered .= zen_image(DIR_WS_IMAGES . $products->fields['products_image'], $this->products[$i]['name'], IMAGE_SHOPPING_CART_HEIGHT) . '' . $this->products[$i]['qty'] . ' x ' . $this->products[$i]['name'] . ($this->products[$i]['model'] != '' ? ' (' . $this->products[$i]['model'] . ') ' : '') . ' = ' .
      $currencies->display_price($this->products[$i]['final_price'], $this->products[$i]['tax'], $this->products[$i]['qty']) .
      ($this->products[$i]['onetime_charges'] !=0 ? "\n" . TEXT_ONETIME_CHARGES_EMAIL . $currencies->display_price($this->products[$i]['onetime_charges'], $this->products[$i]['tax'], 1) : '') .
      $this->products_ordered_attributes . "\n";

2. This code for the HTML-formatted emails doesn't appear to be wrong; however, I might suggest a couple minor changes:

      $[B]prod_img[/B] = $db->Execute("SELECT products_image
                                    FROM " . TABLE_PRODUCTS . "
                                    WHERE products_id ='" . $this->products[$i]['id'] . "'");
      $this->products_ordered_html .=
      '<tr>' .
// also added the following new line to include thumbnail images on order confirmation email
      '<td class="product-details" align="left" valign="top">' . zen_image(DIR_WS_IMAGES . $[B]prod_img[/B]->fields['products_image'] , $this->products[$i]['name'], IMAGE_SHOPPING_CART_HEIGHT) . ' </td>' .
      '<td class="product-details" align="right" valign="top" width="30">' . $this->products[$i]['qty'] . ' x</td>' .

3. Also note that the </a> was removed from the 2nd-last line, above ... ie from here:```
   IMAGE_SHOPPING_CART_HEIGHT) . '</a> </td>' .

However, none of these is likely to affect PayPal transactions specifically.

DrByte:

1. Put this change back to the original, because it's irrelevant to have an HTML image link in a text-only email:

I thought the order confirmation email can be text-only or HTML? I might have accidentally link the image, are you referring to the </a> ? Putting the code back to the original (no change was made) will looks like this:

      // build output for email notification
      $this->products_ordered .=  $this->products[$i]['qty'] . ' x ' . $this->products[$i]['name'] . ($this->products[$i]['model'] != '' ? ' (' . $this->products[$i]['model'] . ') ' : '') . ' = ' .
      $currencies->display_price($this->products[$i]['final_price'], $this->products[$i]['tax'], $this->products[$i]['qty']) .
      ($this->products[$i]['onetime_charges'] !=0 ? "\n" . TEXT_ONETIME_CHARGES_EMAIL . $currencies->display_price($this->products[$i]['onetime_charges'], $this->products[$i]['tax'], 1) : '') .
      $this->products_ordered_attributes . "\n";
      $this->products_ordered_html .=
      '<tr>' .
      '<td class="product-details" align="right" valign="top" width="30">' . $this->products[$i]['qty'] . ' x</td>' .
      '<td class="product-details" valign="top">' . $this->products[$i]['name'] . ($this->products[$i]['model'] != '' ? ' (' . $this->products[$i]['model'] . ') ' : '') .
      '<nobr><small><em> '. $this->products_ordered_attributes .'</em></small></nobr></td>' .
      '<td class="product-details-num" valign="top" align="right">' .
      $currencies->display_price($this->products[$i]['final_price'], $this->products[$i]['tax'], $this->products[$i]['qty']) .
      ($this->products[$i]['onetime_charges'] !=0 ?
      '</td></tr><tr><td class="product-details">' . TEXT_ONETIME_CHARGES_EMAIL . '</td>' .
      '<td>' . $currencies->display_price($this->products[$i]['onetime_charges'], $this->products[$i]['tax'], 1) : '') .
      '</td></tr>';
    }

Assuming I understand your suggestions correctly, I will change the above code to the following to include the thumbnail product images. Is this what you want me to try? Please help me check. Note that changes are highlighted in red. Thank you.

      // build output for email notification
      $prod_img = $db->Execute("SELECT products_image
                                    FROM " . TABLE_PRODUCTS . "
                                    WHERE products_id ='" . $this->products[$i]['id'] . "'");
      $this->products_ordered .= zen_image(DIR_WS_IMAGES . $products->fields['products_image'], $this->products[$i]['name'], IMAGE_SHOPPING_CART_HEIGHT) . '' . $this->products[$i]['qty'] . ' x ' . $this->products[$i]['name'] . ($this->products[$i]['model'] != '' ? ' (' . $this->products[$i]['model'] . ') ' : '') . ' = ' .
      $currencies->display_price($this->products[$i]['final_price'], $this->products[$i]['tax'], $this->products[$i]['qty']) .
      ($this->products[$i]['onetime_charges'] !=0 ? "\n" . TEXT_ONETIME_CHARGES_EMAIL . $currencies->display_price($this->products[$i]['onetime_charges'], $this->products[$i]['tax'], 1) : '') .
      $this->products_ordered_attributes . "\n";
      $this->products_ordered_html .=
      '<tr>' .
      '<td class="product-details" align="left" valign="top">' . zen_image(DIR_WS_IMAGES . $prod_img->fields['products_image'] , $this->products[$i]['name'], IMAGE_SHOPPING_CART_HEIGHT) . ' </td>' .
      '<td class="product-details" align="right" valign="top" width="30">' . $this->products[$i]['qty'] . ' x</td>' .
      '<td class="product-details" valign="top">' . $this->products[$i]['name'] . ($this->products[$i]['model'] != '' ? ' (' . $this->products[$i]['model'] . ') ' : '') .
      '<nobr><small><em> '. $this->products_ordered_attributes .'</em></small></nobr></td>' .
      '<td class="product-details-num" valign="top" align="right">' .
      $currencies->display_price($this->products[$i]['final_price'], $this->products[$i]['tax'], $this->products[$i]['qty']) .
      ($this->products[$i]['onetime_charges'] !=0 ?
      '</td></tr><tr><td class="product-details">' . TEXT_ONETIME_CHARGES_EMAIL . '</td>' .
      '<td>' . $currencies->display_price($this->products[$i]['onetime_charges'], $this->products[$i]['tax'], 1) : '') .
      '</td></tr>';
    }

If an email is text-only, how does an image make any sense?

DrByte:

If an email is text-only, how does an image make any sense?

The order confirmation email I receive is HTML not text-only. Of course, if customers choose to receive only text-only email then images would not make sense. I am doing this just for those customers who choose to receive HTML email. So was the code I put together correct? Please advise. Thanks.

remove your first reference to zen_image()
the rest is likely okay

It still doesn't work with PayPal. The order confirmation email didn't generate. I tried it with check/money order and did receive the order confirmation email with thumbnail picture.

Which PayPal module?

It's PayPal IPN

Try this instead:

• Remove the $prod_img query altogether.
• Replace the <td> line with the image, with the following:
  

  '<td class="product-details" align="left" valign="top">' . zen_get_products_image($this->products[$i]['id'], IMAGE_SHOPPING_CART_HEIGHT) . ' </td>' .