Set number of products displayed per page (support thread)

[rbarbour]

any inputs ?

This is happening to me as well, pretty sure it has to do with my local comp security tho.

On another note:

I have the newest version of this plugin 1.3 complete and will post a download link by days end as well as submit it.

1.) It now utilizes the init_system for:

- less merging into the product_listing.php file
- to ease ZC upgrades
- to carry $_SESSION['product_listing_max_display'] from 1 category to the next remembering # of products to display

2.) I added an "active" class for selected links (post 98 & 104)
3.) Re-coded for HTML Validation
4.) Includes the product_listing.php file for:

- Flexible Product Listing
- Column Layout Grid for Product Listing
- SNAF

Only the $max_results code needs merging now! (post 108)

5.) Includes the stock tpl_modules_product_listing.php file to ease merging process.

6.) Added admin configuration to display link/drop-down at below product listings

A DEMO can be seen here!

Demo defaults to show links but I coded category 22 (Big Linked) to display drop-down.

[rbarbour]

Version 1.3 has been submitted!

You can download directly from here.

[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.

[swguy]

Another thing I noticed was that when the "Items per page" selection was made into a dropdown, it seemed to clobber the next/previous links. I think this might be reproduceable on your test site if you set PRODUCT_LISTING_DISPLAY_OPTION to true and look at this page:

http://zcadditions.com/24/index.php?...&keyword=price