lat9 / cross_sell_advanced

"Cross Sell Advanced II" for Zen Cart v1.5.7.
Other
0 stars 1 forks source link

Pagination Dropdown does not change the page #8

Closed webchills closed 7 months ago

webchills commented 10 months ago

Zen Cart 2.0.0 (maybe same with 1.5.8) Have more than 10 cross sells Set maximum value for admin search results per page to 10 Use the drop down on the top to select page 2 1 No effect, you stay on index.php?cmd=xsell&page=1 Using the dropdown on the bottom is working though

lat9 commented 10 months ago

Thanks for the report, @webchills. I've got this in my queue but have no ETA for correction at this time.

lat9 commented 10 months ago

@webchills, would you grab of copy of the generated HTML and send it to my direct email address? Looking at the code (I don't have a current installation), it's exactly the same code to generate to top and bottom pagination elements so I'm very confused.

webchills commented 10 months ago

The issue is caused by not correctly getting the current page and a </form> at the wrong place. I solved the issue by changes in admin/xsell.php and admin/includes/modules/xsell/category_product_selection.php (using the same logic as in other 1.5.8 pages as specials I have replaced the$xsell_page functionality with the normal $currentPage one). Files attached lat9fixed.zip

lat9 commented 10 months ago

Thanks, @webchills, I'll review those changes later today.