Problem with display of Order Confirmation emails

[djbeatnik]

Hiya,

Any help would be greatly appreciated.

The styling is out on some Order Confirmation emails and random code is being inserted.

I have recently moved cart to a vps and had to amend files to get it going with higher php.

Thank you

Any ideas Dr Byte?

Zen Cart 1.3.7

[kobra]

Zen Cart 1.3.7 really!!!!

It is time to upgrade from code that is vulnerable and 8 years old

[djbeatnik]

I have so many addons otherwise I would, cart is only for customers with accounts, so no card transactions etc. But the notification emails are looking messy since php upgrade.

Zen Cart 1.3.7
PHP Version 5.4.45
MySQL 5.0.95

[carlwhat]

it hardly looks random....

can i safely assume that this is an html email?

you have some html tags that are appearing as text. i would guess that you have a quoting problem going on...

[djbeatnik]

Yeah, not random, from order.php, randomly displayed, and not in all emails, some are displayed correctly.
Quoting problem?
Thanks

[carlwhat]

djbeatnik,
sorry if i was not being clear... looking at the order class i have something like this:

 $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'] . '&nbsp;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>' .

i'm not sure if this is from 3.7 or not, but it looks close....

it looks like you may have a quote (either ' or ") in a product description (or some other field), which is then screwing up the variable: $this->products_ordered_html

i hope that makes sense and is helpful to you.

best.

[djbeatnik]

Thanks, there are some single and double quotes in some of the descriptions, any idea on how to fix this without removing them?
Thanks again.

[carlwhat]

djb,
i just want to summarize before we go:

• you are having some "random" html code in order confirmation emails.
• these emails are html emails.
• some item descriptions contain single quotes and double quotes.
• we think that the email problem is related to these quotes in the item descriptions.

the only reason why i reiterate this information, is that i'm still not 100% that is your problem it is entirely possible that the quotes are your problem and you are more sure than i am. i would hate to address something that is not, in fact, the source of your issue... and really it looks like the field $this->products_ordered_attributes is actually the problem field. but lets give it a whirl!

i would replace the following:

 $this->products_ordered_html .=
	'<tr>' .
	'<td class="product-details" align="right" valign="top" width="30">' . $this->products[$i]['qty'] . '&nbsp;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>' .

with

 $this->products_ordered_html = htmlspecialcharacters($this->products_ordered_html) .
	'<tr>' .
	'<td class="product-details" align="right" valign="top" width="30">' . $this->products[$i]['qty'] . '&nbsp;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>' .

you can read about the php function here:

http://php.net/manual/en/function.htmlspecialchars.php

the key is executing that function at the right place.

i hope that helps! good luck!

[kobra]

we think that the email problem is related to these quotes in the item descriptions.

Or one can use escape characters for special characters in their html mark up
for double quotes it would be

" or  "

Others can be seen here
https://en.wikipedia.org/wiki/List_o...ity_references