Most viewed plugin

[kayfi]

Hi everybofy,

I have some troubles using the most viewed plugin.
I inserted the code below in my default/layout.tpl:
{most_viewed number=5 cols=1 table_attr='width="100%" cellpadding="3" cellspacing="3"' td_attr='style="border-top: 1px dashed #999;"'}

But if I change the number of listings displayed like 'number=3", i still have 5 listings displayed. Moreover it displayed all the most viewed ads.
How can i fix it ?

Best regards !

[Mike-N-Tosh +]

I moved this thread to v4.1 from v4 since your signature indicates that you are running v4.1.

Originally Posted by kayfi

Hi everybofy,

I have some troubles using the most viewed plugin.
I inserted the code below in my default/layout.tpl:
{most_viewed number=5 cols=1 table_attr='width="100%" cellpadding="3" cellspacing="3"' td_attr='style="border-top: 1px dashed #999;"'}

But if I change the number of listings displayed like 'number=3", i still have 5 listings displayed. Moreover it displayed all the most viewed ads.
How can i fix it ?

Best regards !

I don't know if it was a "typo" in your post, but if it wasn't then that is the issue: 'number=3"

This should be number=3 (No single or double quotes)

[kayfi]

Yes it was a mistake in my typo, but when i change the number of displayed ads, nothing changes...

[kayfi]

I still don't understand why i have all the new listings displayed in my Most viewed section.
I put the following code in my layout.tpl:
{most_viewed number=5 cols=1 table_attr='width="100%" cellpadding="3" cellspacing="3"' td_attr='style="border-top: 1px dashed #999;"'}

I cleaned all the IE cache, i tested it with firefox, and it's still the same... i don't have 5 ads displayed.
Please can you help me ?

[Lhotch +]

The plugin has a bug in it and there is no limit clause on the end of the query.

open the file function.most_viewed.php and find this line...

$sSQL="SELECT p.id, p.owner, p.title, p.featured, p.section, p.description, p.price, p.dateadded, p.expiration, p.pHighlighted, p.pBold, u.state, u.city, u.country FROM ".PREFIX."listings AS p LEFT JOIN ".PREFIX."users AS u ON p.owner = u.id WHERE p.expiration > NOW() AND p.display = 'Y' ORDER BY hitcount DESC";


and add "LIMIT $number" to the end so it looks like this....

$sSQL="SELECT p.id, p.owner, p.title, p.featured, p.section, p.description, p.price, p.dateadded, p.expiration, p.pHighlighted, p.pBold, u.state, u.city, u.country FROM ".PREFIX."listings AS p LEFT JOIN ".PREFIX."users AS u ON p.owner = u.id WHERE p.expiration > NOW() AND p.display = 'Y' ORDER BY hitcount DESC LIMIT $number";

[kayfi]

Thank you so much Larry ! It fixed out the problem !!!!