Top 5 posters

[Bellardia]

Hopefully this will be my final revision :x

Fixed alignment, and updates every 20minutes.

<?xml version="1.0" encoding="ISO-8859-1"?>

<product productid="top5posters" active="1">
	<title>Top 5 Posters</title>
	<description>Gets a string of the top five posters, and parses where $Top5 is included in a template</description>
	<version>0.1</version>
	<url />
	<versioncheckurl />
	<dependencies>
	</dependencies>
	<codes>
	</codes>
	<templates>
	</templates>
	<plugins>
		<plugin active="1" executionorder="10">
			<title>Parse Names</title>
			<hookname>global_start</hookname>
			<phpcode><![CDATA[global $vbulletin;
	$Top5Data = unserialize($vbulletin->TopPosters);
        if (count($Top5Data) != 2) $Top5Data[0] = 0;
	$Top5 = $Top5Data[1];
	if (($Top5Data[0] + 1200) < TIMENOW)
	{
		$GrabString = $vbulletin->db->query_read("select `username`, `posts` from `".TABLE_PREFIX."user` ORDER BY `posts` DESC limit 0, 4;");
		while ($GrabTop = $vbulletin->db->fetch_array($GrabString))
		{
			$Top5Store[] = '<tr><td width="100%" style="color:#848484;">'.$GrabTop['username'].'</td><td style="white-space:nowrap; color:#FFFF00" align="right">'.$GrabTop['posts'].'</td></tr>';	
		}
		$Top5Store = '<table width="100%" border="0" style="color:#FF0">'.implode('', $Top5Store).'</table>';
		htmlspecialchars_uni($Top5Store);
		$Top5Storage = array(TIMENOW, $Top5Store);
		build_datastore('TopPosters', serialize($Top5Storage));
		$Top5 = $Top5Store;
	}]]></phpcode>
		</plugin>
		<plugin active="1" executionorder="5">
			<title>DataStore</title>
			<hookname>init_startup</hookname>
			<phpcode><![CDATA[$datastore_fetch[] = "'TopPosters'";
$Top5 = $vbulletin->TopPosters;]]></phpcode>
		</plugin>
	</plugins>
	<phrases>
	</phrases>
	<options>
	</options>
	<helptopics>
	</helptopics>
	<cronentries>
	</cronentries>
	<faqentries>
	</faqentries>
</product>

[Ghost Shadow]

Bellardia, you are a "Coding Saint". Now it is PERFECT!!
Today is my birthday; thank you for all the time you spent helping me, fantastic present.
Peace my friend.


:up:

[Dismounted]

Originally Posted by Bellardia

Hopefully this will be my final revision :x

Maybe not.

First line of the plugin at init_startup, you have too many sets of quotes. Second line is not needed at all. When using build_datastore(), you can set a third parameter, which can automatically unserialize the data for you. Also, you want to run htmlspecialchars_uni on the username, not the whole thing. Finally, you might want to look at the vBulletin Coding Standards document.

[F0xy]

it would be amazing if this could be edited to show the top 10 latest posts for the sidebar.

When I added the latest it only displayed the top 4 posters?

[slinky]

I'm using an old hack and need to work on this one. If I had the time I'd probably take a better look. I'm shocked that nobody has done a good one for this very popular function.