Advert Management

This is a script to allow you to manage the adverts on your forums, you can have any sort of advert you can imagine there is no actual limit with this hack to what it can be, javascript, flash, image, text and images.

You add adverts to a category or a forum or the whole board. If the category has child forums they will inherit the adverts as well as their own adverts if they have any.
Adverts can have a set exposure such as 10,000 views and once it has reached that it will stop.

It fills a variable called $advert which you just place in any template, such as the header, you can also use variables and replacements within the adverts, such as $bbuserinfo[userid] for the persons userid in a link within an advert.

Updated June 9th at 19:44 GMT

Added date limits and options for unlimited exposures.
To upgrade from an older version, reapply code in global.php, upload ads.php and run this sql query to alter the ads table you made.

[sql]ALTER TABLE `ads` ADD `time` INT(10) DEFAULT '0' NOT NULL;[/sql]

[sql]ALTER TABLE `ads` ADD `wholeforum` SMALLINT(1) DEFAULT '0' NOT NULL;[/sql]
If you already run that query though are still experiencing problems with the time always being January 1st run

[sql]ALTER TABLE `ads` CHANGE `time` `time` INT(10) DEFAULT '0' NOT NULL[/sql]

New features include showing the advert on the entire forum by using a radio button, some highlighting in the admin panel, italic name indicates the time period has ended and a bold name means the advert has met its quota for exposures.

You must chance RAND() to RAND(NOW()) if you have mysql 3.23.52 or above

Scott

[SpeedStreet]

Would any of my proposed fixes work for this, PPN?

[waherne]

PPN,

I modified your hack and have now got it to randomise the adverts successfully. As I said before, I am not a programmer, so my code is probably inefficient in performing the task - when I post the code here, you and others can hopefully assist in that respect.

The only thing with my modification is that I have disabled the 'show on whole board' functionality. Basically, I didn't need it for my purpose.

I'm at work now and don't have access to my laptop. I will post my code to this thread over the next few days.

Once again, thanks for the hack.

Willie

[thakikka]

I guess I found the error I reported some posts before (banners not shown on directly linked posts).

If the script finds something like "&threadid=19938" then the banner gets displayed.

But if you directly link to a post, it contains "?postid=172199" and the banner does not get displayed.

Can someone fix this so banners get displayed on every page?

Thanks in advance,
Sebastian

[thewolf]

Hi PPN,

any news on the new version of this hack?

Also, will it work with the latest stable version of Vbulletin (2.2.9)?

Keep up the good work!

Thanks,
Marco.

[waherne]

Further to my post above, here are the changes I made to PPN's hack to make it randomise. As I said, let us all work together to make my code more efficient.

Willie

////////////////////START//////////////////////////
In global.php, FIND:

if(isset($advert_forumid)) {
$foruminfo = getforuminfo($advert_forumid);
$advert=$DB_site->query_first("SELECT * FROM ads WHERE LOCATE(CONCAT(',',forumid,','),',$foruminfo[parentlist],')>0 AND

time>=".time()." AND (exposed<=exposures OR exposures=0) ORDER BY RAND()");
} else {
$advert=$DB_site->query_first("SELECT * FROM ads WHERE wholeforum='1' AND time>=".time()." AND (exposed<=exposures OR

exposures=0) ORDER BY RAND()");
}

AND REPLACE IT WITH:

$DB_site;

$sql="SELECT * FROM ads WHERE LOCATE(CONCAT(',',forumid,','),',$foruminfo[parentlist],')>0 AND time>=".time()." AND (exposed<=exposures OR exposures=0) ORDER BY RAND()";

$advert1 = mysql_query($sql);
$advert2 = mysql_query($sql);

$adcounter=0;
while($countrow = mysql_fetch_row($advert1))
{
$adcounter++;
}
$act1=rand(0,$adcounter-1);
$adcounter3=0;
while($countrow3 = mysql_fetch_row($advert2))
{
if($adcounter3==$act1)
{
$DB_site->query("UPDATE ads SET exposed=exposed+1 WHERE adid='$countrow3[0]'");
eval("\$advert = trim(\"".str_replace("\'", "'", addslashes($countrow3[7]))."\");");
}
$adcounter3++;
}

///////////////END///////////////////

I just noticed that there is another block of code that should also be included in the Find It and Replace above, essentially it's the code immediately following that set out above and beginning with $DB_site->query("UPDATE ads SET exposed - one query only.

[SpeedStreet]

Now, let me understand you correctly. This code works, but it eliminates the "Show on all forums" functionality?

[waherne]

Correct.

Adverts on child forums will be pooled together with adverts on their parent forums (and their parents...) as before. One advert will then be picked from the pool. Therefore, insertion of an advert at the ultimate parent level could be one way of displaying it on the entire forum.

Test my code and see if it suits your needs.

Willie

[quickstang22]

Willie

I tried your fix but lost all my ads. Not one is showing up. I made sure they were not selected for show all fourms. Just had them in child forums

Matt

[themonarch]

Originally posted by Visionray
good thing I read the last few pages of this thread and realized this hack isn't working. Maybe the first post should be updated with that info so people don't go instalilng it to only find out it doesn't work.

mmm, good point VisionRay. I installed this hack and then when it didn't work, I came back in here to read the whole thread and found out it's not working right yet.

[ChurchMedia]

When I run this query:

CREATE TABLE `ads` (
`adid` smallint(5) NOT NULL auto_increment,
`name` varchar(25) NOT NULL default '',
`forumid` smallint(5) NOT NULL default '0',
`wholeforum` smallint(1) NOT NULL default '0',
`exposed` smallint(10) NOT NULL default '0',
`exposures` smallint(10) NOT NULL default '0',
`time` int(10) NOT NULL default '0',
`advert` text NOT NULL,
KEY `adid` (`adid`)
);

I get this error:

MySQL said:

You have an error in your SQL syntax near '\'\',
`forumid` smallint(5) NOT NULL default \'0\',
`wholeforum` smallint(' at line 3

Any ideas?