X-Cart forums

[tecdady]

unfortunately, its clean install. not sure why this isnt working right out the box, but christ its frustrating to have to hold up delivery over this.. i wish they didnt ad it to begin with.

i cant even find the smarty template that holds the logic for that crap so i can just rip it out. any ideas?

[tecdady]

found it:

{if %product.index% is not odd}
<TR>
{/if}


thats in the products.tpl... that logic doesnt apear to be working what so ever. no to mention "is not odd"? how could that possibly work? how can that statement test 3 conditions, two of which are odd and determine the difference between 1 and 3, let alone 2 if it even worked.... another x-cart flop?

anyways, if you DONT want multicolmn format like me, comment the "if" statments out on the top of the main table and the bottom of the main table in products.tpl.

[balinor]

Strange, worked fine in 4.0.12, must have botched it up on 4.0.13....

[mattzarb]

I found this some problem, and it seems like the products.tpl template lost some functionality somewhere.

I looked back at a copy of the file on a 3.4.x version of a store I have running, and products.tpl was set up to check the appropriate configuration settings and include products_t.tpl if needed, otherwise it would display a single column product list.

I'm working on a 4.0.14 cart right now and see that the products.tpl template has no code at all to even check the $featured variable or any config settings. It will always display everything in a 2 column format.

Trying to find where products_t.tpl went, I see that this is now only called from cart_contents.tpl and cart_details.tpl.

So....

If you want to "fix" products.tpl to actually pay attention to the configuration settings the way it used to, add this to the top of the products.tpl:

{if $usertype eq "C" and $config.Appearance.products_per_row ne "" and $config.Appearance.products_per_row gt 0 and $config.Appearance.products_per_row lt 4 and ($featured eq "Y" or $config.Appearance.featured_only_multicolumn eq "N")}
{include file="customer/main/products_t.tpl" products=$products}
{else}

[ .. original products.tpl content ]

{/if}

There are a couple of things to consider on this fix:

1. The original products.tpl inside the {else}{/if} will still be configured to always display in 2 columns. If you want it to default to 1 column (the way it used to be) then you'll need to edit that part of the template.

2. products_t.tpl by default will not display product descriptions. So if you want your products list to appear the way they do in products.tpl, only with variable columns, you'll need to either modify products_t.tpl (which will have side effects in the cart view) or hack products.tpl to include much the same code as in products_t.tpl

I think I'll be hacking a version of products.tpl to abide by the settings instead of calling products_t.tpl so if you're interested in this let me know. I'm also making this a pure css (w/o table) implementation, so I'd need to create another version of the template for the more standard table implementation if anyone wants it.

Matt

[airntam]

Thanks for this! That was driving me crazy!