[MOD] Popular articles (snews 1.5)

[Mika]

Mandatory requirement: Page view counter mod.

Language variables (snews.php)

$l['posted_in'] = 'Posted in';
$l['views'] = 'Views';

New function (snews.php)

// POPULAR ARTICLES
function popular_articles($size) {
# select all published articles ordered by view column and limiting the output with numeric $size variable
$query = "SELECT * FROM ".db('prefix')."articles WHERE position = 1 AND published = 1 ORDER BY views DESC LIMIT $size";
$result = mysql_query($query);
# "home" title defined outside while-loop
$home = s('home_sef');
#optional formatting (uncomment if needed)
//echo '<ul>';
while ($r = mysql_fetch_array($result)) {
# find category SEF title (used to build links)
$categorySEF = find_cat_sef($r['category']);
# defining category name: if zero, we're at home, else retrieve category name
$categoryName = $r['category'] == 0 ? $home : retrieve('name', 'categories', 'seftitle', $categorySEF);
echo '<li><a href="'.db('website').$categorySEF.'/'.$r['seftitle'].'/" title="'.$r['title'].' - '.l('posted_in').' '.$categoryName.'">'.$r['title'].'</a> '.$r['views'].' '.l('views').'</li>';
}
#optional formatting (uncomment if needed)
//echo '</ul>';
}

Usage (index.php)

<!-- displaying top 5 popular articles -->
<!-- remove unordered list if using optional formatting (see comments above)-->
<ul>
<?php popular_articles(5);?>
</ul>

Live preview at www.ni5ni6.com

[bloop]

Thank you very much , its so useful..

[David]

Mod added on my site without problems! Thank you Mika.

[Sven]

I noticed it doesn't work if you have no reader.
My whole site is a bug !

[codetwist]

What do You mean - have no reader? And whole site a bug ... :lol:

P.S. Site looks great, btw

[mattonik]

nice mod, need to try it

[Sven]

What do You mean - have no reader? And whole site a bug ... :lol:

It was a joke : but now it seems that this mod is not working on "my" sNews.  

P.S. Site looks great, btw

Thanks mate.
Stil under construction but i's coming...  :cool:

[codetwist]

Have You applied that view counter mod that is required for this to work (like have view field in articles table, etc)?

I recall that I had their early versions running without problems.

[Sven]

:|
Yes
but maybe something went wrong with the DB query.
I'll check later in the afternoon.
See you.  

[Sven]

That's it !
Checking it back it seems that I have no table "PREFIX_articles".

EDIT : neither on my localhost

[codetwist]

Good

[Sven]

:mad:
no good !
I can't alter a table which doesn't exist. !  

[philmoz]

That's it !
Checking it back it seems that I have no table "PREFIX_articles".

EDIT : neither on my localhost

Where does it require a prefix to have been set??
If in the sql, ie,

SELECT * FROM ".db('prefix')."articles

, then, if you haven't used a prefix, it can't look for table PREFIX_articles, as db(prefix) will be empty...

It also means that you would use the standard query not the PREFIX_ query here
http://www.solucija.com/forum/viewtopic.php?pid=23166#p23166
to alter your article table

[Sven]

Okay Phil
you got  viagra for the brain ?
I think I need some.
 

[codetwist]

:mad:
no good !
I can't alter a table which doesn't exist. !  

Ughhh ... sorry ... I thought that You solved it ... after all that was offending PREFIX right in Your post