203 replies to this topic

[EchoGuns]

The way I have certain specifications set up, an item can have several values for the same specification name. The code I just showed currently shows like this:

<p><strong>Spec Name 1</strong></p>
<ul class=specification_box>
<li> Spec1 Value 1</li>
</ul>
<p><strong>Spec Name 1</strong></p>
<ul class=specification_box>
<li> Spec1 Value 2</li>
</ul>
<p><strong>Spec Name 2</strong></p>
<ul class=specification_box>
<li> Spec2 Value 1 </li>
</ul>
<p><strong>Spec Name 2</strong></p>
<ul class=specification_box>
<li> Spec2 Value 2 </li>
</ul>

Im trying to get it to show the name of the specification once for all the values under that specification name.

[kymation]

Now I see what's happening. You have multiple values assigned to a specification for each product. The code in products_specifications.php doesn't handle that correctly. It's going to be necessary to split the SQL into two parts: One to get the name, and the second to loop through and get all of the values. This is going to take a bit of work. Unfortunately I don't have a lot of time to work on this right now. I'll see what I can do in a day or two.

Regards
Jim

[EchoGuns]

Oh, even more complicated than I had thought. Well time is not an issue with me as Im still far from being live. If you get time to look at it I would appreciate it but don't make it a priority.

Thanks!

[Gazonice]

Another potential fix for blank emails when the script is run by a CRON Job is to change the line:

$text_file = basename ($PHP_SELF, ".php") . '.tpl';

to:

$text_file = 'review_mail.tpl';

Assuming the template file resides in the same folder as the script and the filename has not been changed.

This fixed my problem completely.

Garry.