Set number of products displayed per page (support thread)

[rbarbour]

Version: 1.3 is now available!

[swguy]

There is a bug in includes/templates/YOUR_TEMPLATE_FOLDER/templates/tpl_modules_product_listing.php~

$ php -l tpl_modules_product_listing.php~
PHP Parse error: syntax error, unexpected $end in tpl_modules_product_listing.php~ on line 104

I fixed this issue but probably incorrectly.

But even so the screen looks wonky. I set the products per page to 6 - first row has 2, second row 1, third row 3. I am using Column Layout Grid.

@PMrbarbour, would you please double check your work?

[rbarbour]

There is a bug in includes/templates/YOUR_TEMPLATE_FOLDER/templates/tpl_modules_product_listing.php~

$ php -l tpl_modules_product_listing.php~
PHP Parse error: syntax error, unexpected $end in tpl_modules_product_listing.php~ on line 104

I suspect the intention was to add a closing brace on line 50 and on line 85.

But even so the screen looks wonky. I set the products per page to 6 - first row has 2, second row 1, third row 3. I am using Column Layout Grid.

@PMrbarbour, would you please double check your work?

- /includes/modules/YOUR_TEMPLATE_FOLDER/product_listing.php~Column Layout Grid for Product Listing
- /includes/modules/YOUR_TEMPLATE_FOLDER/product_listing.php~Flexible Product Listing
- /includes/modules/YOUR_TEMPLATE_FOLDER/product_listing.php~SNAF

Are example files extracted directly from the original downloaded from the plugins section and unaltered other than to replace the $max_results (1) line of code.

Even if one was to upload these files they wouldn't have any effect.

I will double check my uploaded against the original downloaded from the plugins section just to make sure though.

[swguy]

I will revert includes/modules/YOUR_TEMPLATE_FOLDER/product_listing.php (based on the ~ Column Layout Grid for Product Listing copy) to see if the problem goes away.

But you missed what I said about includes/templates/YOUR_TEMPLATE_FOLDER/templates/tpl_modules_product_listing.php~. This is broken.

[rbarbour]

I will revert includes/modules/YOUR_TEMPLATE_FOLDER/product_listing.php (based on the ~ Column Layout Grid for Product Listing copy) to see if the problem goes away.

But you missed what I said about includes/templates/YOUR_TEMPLATE_FOLDER/templates/tpl_modules_product_listing.php~. This is broken.

You are correct sir,

includes/templates/YOUR_TEMPLATE_FOLDER/templates/tpl_modules_product_listing.php~ is broken.

Copy/Paste Error from demo site

find both instances of:

<!-- (BOF - 1.3) Set number of products displayed per page (24) -->
<?php } ?>
<!-- (EOF - 1.3) Set number of products displayed per page (24) --><br class="clearBoth" />

and change to:

<!-- (EOF - 1.3) Set number of products displayed per page (24) -->
<?php }
  }
 ?>
<!-- (EOF - 1.3) Set number of products displayed per page (24) --><br class="clearBoth" />

I will update the example file.

Thanks for pointing this out Scott

[swguy]

And sure enough, that fixed my display issue. So that was the root cause of the problem.

Thanks for the quick diagnosis! I will edit out my suggested fix which was wrong.

[rbarbour]

And sure enough, that fixed my display issue. So that was the root cause of the problem.

Thanks for the quick diagnosis! I will edit out my suggested fix which was wrong.

I suspect the intention was to add a closing brace on line 50 and on line 85.

Only wrong depending on the text editor and line count, I think you had it figured out way before I did.