[zilker]

Well, I fixed it with some bubble gum and duct tape

Here's what I did. I took the tables that created the image border and inserted them into the product_thumbnail.tpl file. Then, I realized that the tables sat in between the url tags that made the image clickable. Since that didn't work out as planned, I was forced to think some more which resulted in another idea. I had to put the url tags inside the tables that I placed in product_thumbnail.tpl.

Since products.tpl and product.tpl both read from the same file and call different variables, the modified code wouldn't pull the image on the product.tpl page. So, I created a new file named product_thumbnail_products.tpl and changed products.tpl to include that new file instead of the original.

Here's what we're looking at:

/product_thumbnail_products.tpl

{* $Id: product_thumbnail.tpl,v 1.14 2004/06/24 09:53:29 max Exp $ *}
{insert name="is_product_image_avail" assign="image_avail" productid="$productid" variantid=""} 
{if $image_avail} 
{if $config.Appearance.show_thumbnails eq "Y"}
<table cellpadding="1" cellspacing="0">
<tr><td class="ImgBorder2">
<table cellpadding="5" cellspacing="0">
<tr><td class="ImgBorder1">

<A href="product.php?productid={$products[product].productid}&cat={$cat}&page={$navigation_page}{if $featured eq 'Y'}&featured{/if}">

[img]{if $tmbn_url}{$tmbn_url}{else}{if $full_url}{$http_location}{else}{$xcart_web_dir}{/if}/image.php?productid={$productid}{if $file_upload_data.file_path}&tmp=y{/if}{/if}[/img]

</A>

</td></tr>
</table>
</td></tr>
</table>

{/if}
{/if}

/product_thumbnail.tpl

{* $Id: product_thumbnail.tpl,v 1.14 2004/06/24 09:53:29 max Exp $ *}
{insert name="is_product_image_avail" assign="image_avail" productid="$productid" variantid=""} 
{if $image_avail} 
{if $config.Appearance.show_thumbnails eq "Y"}

<table cellpadding="1" cellspacing="0">
<tr><td class="ImgBorder2">
<table cellpadding="5" cellspacing="0">
<tr><td class="ImgBorder1">

[img]{if $tmbn_url}{$tmbn_url}{else}{if $full_url}{$http_location}{else}{$xcart_web_dir}{/if}/image.php?productid={$productid}{if $file_upload_data.file_path}&tmp=y{/if}{/if}[/img]

</td></tr>
</table>
</td></tr>
</table>

{/if}
{/if}

customer/main/products.tpl - only a portion of the file

{if $active_modules.Special_Offers ne "" and $products[product].have_offers}

{include file="modules/Special_Offers/customer/product_offer_thumb.tpl" product=$products[product]}

{else}

{include file="product_thumbnail_products.tpl" productid=$products[product].productid image_x=$config.Appearance.thumbnail_width product=$products[product].product tmbn_url=$products[product].tmbn_url}

{/if}

I also moved the "See details" link and placed it under the short description. That prevents it from being out of place when there isn't a default image.

Like I said, it probably isn't the best way to do it but it works. I haven't tested it in the checkout process but it shouldn't be that difficult to make a small change if something isn't right.