Obtaining forumid in vB Plugin

[sharkus]

Good morning all,
Following on from this thread, I abused the js code linked to as I didn't want to just work on images, I thought replacing the contents of a div would be better, as some of the banners I would be using would be flash, and thus not a simple img src reference, plus most are img tags within hrefs to allow them to be clickable.

The overall idea is the ability to have per forum rotating banners. I currently use vBSponsors, which allows a single per forum banner to be displayed (small one on the main forum listing, larger one when you view the list of threads in that forum). The disadvantage here is that it only allows a single banner per forum, and I wanted to be able to display multiple banners per forum.

Using the aforementioned, modified js code I do have a solution that is working, but it isn't that elegant, and it means creating a lot of plugins, at least two per forum, which I think is unacceptable, and this is where I come seeking help.

Where to start? I think with my edits to the forumhome_forumbit_level2_post template.

After:

<a href="forumdisplay.php?$session[sessionurl]f=$forum[forumid]"><strong>$forum[title]</strong></a>
			<if condition="$show['browsers']"><span class="smallfont">(<phrase 1="$forum[browsers]">$vbphrase[x_viewing]</phrase>)</span></if>

I have inserted the following:

<!-- START - mrwrb -->
<if condition="$forum[mrwrb_forumhome_banners_enabled] == 1 AND $vboptions[mrwrb_global_enable_banners] == 1">

<div id="mynicebanner$forum[forumid]" style="float:$stylevar[right]; padding-left:10px; height: 80px;">
</div>
	
	<if condition="$forum[forumid] == 39">
	$GLOBALS[mrwrb_forum_39_small]
	<SCRIPT LANGUAGE="JavaScript">
	mycurrentlocation = 'mynicebanner' + $forum[forumid];
	rotateImage_forum39(mycurrentlocation)
	</SCRIPT>
	</if>
	
	<if condition="$forum[forumid] == 48">
	$GLOBALS[mrwrb_forum_48_small]
	<SCRIPT LANGUAGE="JavaScript">
	mycurrentlocation = 'mynicebanner' + $forum[forumid];
	rotateImage_forum48(mycurrentlocation)
	</SCRIPT>
	</if>

</if>

<!-- END - mrwrb -->

The above checks to see if my banner system is globally enabled and if I've also enabled banner display for what I've named forumhome, which actually equates to the forumhome_forumbit_level2_post template. It then checks the forumid and if it matches a specific value, it will do something.

This is part of the reason I'm seeking help. The $GLOBALS[mrwrb_forum_39_small] relates to this plugin:

product: vBulletin
Hook Location: global_start
Title: mrwrb_forum_39_small
Execution Order: 5
Plugin PHP Code:

$mrwrb_forum_39_small = implode('', file('mrwrb/forum39/small.html')); 

This is loading code from a specific html file. This code is the actual javascript code that handles the rotating banners.

Here is where my first question comes into play. As you can see from my template edit, I'm having to create a conditional check for each forum, and at present I have at least 50 forums, thus 50 conditional checks. The key point here is no so much the conditionals it's the plugin code. With my current setup I would need 50 plugins, each reading a different file.
I think is is, every so slightly overkill. If I had a single plugin for things then I could remove the conditional checking for the forumid which would be much better.

Is it possible for me to create a plugin such as this:

$theforumid = $forum[forumid];
$mrw_forum_small =  implode('', file('mrwrb/forum$theforumid/small.html')); 

In other words, I want to have single name for the hook (I think that is the term) that I put into the template, and can I get the forumid within a plugin and then use it as part of the file specification? I know the answer to the former is "yes", but if I cannot do the latter then a single name for the hook isn't much use as the file path would not change depending on the forumid, which is what I need.

Any suggestions or advice is greatly recommended.

[Dave A]

Have you looked at Google Admanager as an option?

I've managed to get it set up for forum and thread targetting with way less strain than trying to hack code.