Print Page - list of products per manufacturer, the limit does not work. fix included.

Title: list of products per manufacturer, the limit does not work. fix included.
Post by: AndyGaskell on February 18, 2013, 21:17:43 PM

Hi

I noticed a bug in the list of products per manufacture, it was not using the value in "Configuration" -> "Shopfront" -> "Default items per listing view". It is not using the value there, but is instead defaulting to 10.

This can, I think, be fixed by one line in com_virtuemart/router.php line #335 could be changed from...

Code Select

$vars['limit'] = vmrouterHelper::$limit;

...to...

Code Select

$vars['limit'] = VmConfig::get('list_limit', 20);

...I've tested this here, and it works for me, however there are probably some other use cases that'd need testing.

The route for this is, for example...
index.php?option=com_virtuemart&view=category&virtuemart_manufacturer_id=2

Versions:
Joomla!: 2.5.8
php: same on 5.3.10 and 5.3.2
VirtueMart: 2.0.18a

Hope this is helpful, let me know if any more info is needed.

Thanks

Andy

Title: Re: list of products per manufacturer, the limit does not work. fix included.
Post by: Milbo on February 19, 2013, 01:01:35 AM

thx, added

Title: Re: list of products per manufacturer, the limit does not work. fix included.
Post by: AndyGaskell on February 19, 2013, 09:54:14 AM

Thanks Milbo

Maybe see you at J & Beyond, really enjoyed your seminar at Joomla Day UK 2011

VirtueMart Forum

VirtueMart 2 + 3 + 4 => Virtuemart Development and bug reports => Topic started by: AndyGaskell on February 18, 2013, 21:17:43 PM