The Arcive of vBulletin Modifications Site.

Number of views per announcement · **Released**: 04-08-2002

I wasn't able to find this hack anywhere. So I wrote this.

This hack will show the number of views per announcement.

Tables affected: announcement
Templates affected: forumdisplay_announcement
Files affected: forumdisplay.php and announcement.php

First we need to add a new field to announcement:

alter table announcement add visits smallint(5) unsigned DEFAULT 0 NOT NULL;

Then we need to modify forumdisplay_announcement template:

Look for the following:

Code:

<smallfont><br>($announcement[usertitle])</smallfont></td>
	<td bgcolor="#F1F1F1"><normalfont>-</normalfont></td>
	<td bgcolor="#DFDFDF"><normalfont>-</normalfont></td>

And replace it with the following:

Code:

<smallfont><br>($announcement[usertitle])</smallfont></td>
	<td bgcolor="#F1F1F1"><normalfont>-</normalfont></td>
	<td bgcolor="#DFDFDF"><normalfont>$announcement[visits]</normalfont></td>

In forumdisplay.php:

Look for the following:

PHP Code:


			if ($announcement=$DB_site->query_first("SELECT announcementid,startdate,title,user.username,user.userid,user.usertitle,user.customtitle
                                         FROM announcement
                                         LEFT JOIN user
                                           ON user.userid=announcement.userid
                                         WHERE startdate<=$datenow AND enddate>=$datenow
                                           AND $forumlist
                                         ORDER BY startdate DESC
                                         LIMIT 1")) {
    if ($foruminfo[allowratings]) {
      $thread[rating]='clear.gif';
      eval("\$threadrating = \"".gettemplate('forumdisplaybit_threadrate')."\";");
      $backcolor = '#DFDFDF';
          $bgclass = "alt2";
    } else {
                        unset($threadrating);
      $backcolor = '#F1F1F1';
          $bgclass = "alt1";
    }

     if ($announcement[customtitle]==2)
       $announcement[usertitle] = htmlspecialchars($announcement[usertitle]);
           $announcement[postdate]=vbdate($dateformat,$announcement[startdate]);

  if ($announcement[startdate]>$bbuserinfo[lastvisit]) {
        $announcement[icon]='newannounce.gif';
  } else {
        $announcement[icon]='announce.gif';
  }
  eval("\$announcement = \"".gettemplate('forumdisplay_announcement')."\";");
  $doneannouncements = 1;
}

And replace it with the following:

PHP Code:


			if ($announcement=$DB_site->query_first("SELECT announcementid,startdate,title,user.username,user.userid,user.usertitle,user.customtitle,visits
                                         FROM announcement
                                         LEFT JOIN user
                                           ON user.userid=announcement.userid
                                         WHERE startdate<=$datenow AND enddate>=$datenow
                                           AND $forumlist
                                         ORDER BY startdate DESC
                                         LIMIT 1")) {
    if ($foruminfo[allowratings]) {
      $thread[rating]='clear.gif';
      eval("\$threadrating = \"".gettemplate('forumdisplaybit_threadrate')."\";");
      $backcolor = '#DFDFDF';
          $bgclass = "alt2";
    } else {
                        unset($threadrating);
      $backcolor = '#F1F1F1';
          $bgclass = "alt1";
    }

     if ($announcement[customtitle]==2) {
       $announcement[usertitle] = htmlspecialchars($announcement[usertitle]);
     }
     $announcement[postdate]=vbdate($dateformat,$announcement[startdate]);

  if ($announcement[startdate]>$bbuserinfo[lastvisit]) {
        $announcement[icon]='newannounce.gif';
  } else {
        $announcement[icon]='announce.gif';
  }
  eval("\$announcement = \"".gettemplate('forumdisplay_announcement')."\";");
  $doneannouncements = 1;
}

In announcement.php

Look for the following:

PHP Code:


			$announcements=$DB_site->query("
SELECT
announcementid,startdate,enddate,announcement.title,pagetext,user.*,userfield.*
".iif($avatarenabled,",avatar.avatarpath,NOT ISNULL(customavatar.avatardata) AS hascustomavatar,customavatar.dateline AS avatardateline","")."
FROM announcement
LEFT JOIN user ON user.userid=announcement.userid
LEFT JOIN userfield ON userfield.userid=announcement.userid
".iif ($avatarenabled,"LEFT JOIN avatar ON avatar.avatarid=user.avatarid LEFT JOIN customavatar ON customavatar.userid=announcement.userid","")\
."
WHERE startdate<='$datenow' AND enddate>='$datenow' AND $forumlist ORDER BY startdate DESC");

And replace it with the following:

PHP Code:


			$announcements=$DB_site->query("
SELECT
announcementid,startdate,enddate,announcement.title,pagetext,visits,user.*,userfield.*
".iif($avatarenabled,",avatar.avatarpath,NOT ISNULL(customavatar.avatardata) AS hascustomavatar,customavatar.dateline AS avatardateline","")."
FROM announcement
LEFT JOIN user ON user.userid=announcement.userid
LEFT JOIN userfield ON userfield.userid=announcement.userid
".iif ($avatarenabled,"LEFT JOIN avatar ON avatar.avatarid=user.avatarid LEFT JOIN customavatar ON customavatar.userid=announcement.userid","")\
."
WHERE startdate<='$datenow' AND enddate>='$datenow' AND $forumlist ORDER BY startdate DESC");

In announcement.php

Look for the following:

PHP Code:


			  $counter++;
  //$allowhtml = 1;
  //$announcebits .= getpostbit($post);

And annd the following directly below it:

PHP Code:


			  $DB_site->query("UPDATE announcement set visits=visits+1 where announcementid=$post[announcementid];");

That's it.

MarkB · #2 04-09-2002, 08:18 PM

YES!! Been wanting something like this for ages

Thanks! Works great

Admin · #3 04-10-2002, 09:09 AM

Great, now we can prove that no one, but no one, reads the announcements.

Two things:
a) Instead of updating the counter for every announcement, you could just run a query that goes more or less like this:

Code:

UPDATE announcement SET visits = visits+1 WHERE forumid IN ($forumlist);

(generate $forumlist if needed). That way you only add one query instead of one per announcement.

b) Use the shutdown queries of vBulletin.

An example can be found in showthread.php:

Code:

if ($noshutdownfunc) {
  $DB_site->query("UPDATE thread SET views=views+1 WHERE threadid='$threadid'");
} else {
  $shutdownqueries[]="UPDATE LOW_PRIORITY thread SET views=views+1 WHERE threadid='$threadid'";
}

Zzed · #4 04-10-2002, 09:49 AM

As always, I am most grateful for the wonderful advice.

Thank you.

Jawelin · #5 04-10-2002, 06:42 PM

Very nice....
Should we wait for a upgrade, so ?

Thnx

Zzed · #6 04-10-2002, 07:01 PM

Sure.

I'll post an update later tonite.

Xelation · #7 04-10-2002, 08:00 PM

also for the forumdisplay.php file, when your trying to find the first part it should be....

PHP Code:


			if ($announcement=$DB_site->query_first("SELECT announcementid,startdate,title,user.username,user.userid,user.usertitle,user.customtitle
                                         FROM announcement
                                         LEFT JOIN user
                                           ON user.userid=announcement.userid
                                         WHERE startdate<=$datenow AND enddate>=$datenow
                                           AND $forumlist
                                         ORDER BY startdate DESC
                                         LIMIT 1")) {
    if ($foruminfo[allowratings]) {
      $thread[rating]='clear.gif';
      eval("\$threadrating = \"".gettemplate('forumdisplaybit_threadrate')."\";");
      $backcolor = '#1C5780';
          $bgclass = "alt2";
    } else {
            unset($threadrating);
      $backcolor = '#13486D';
          $bgclass = "alt1";
    }

     if ($announcement[customtitle]==2) {
       $announcement[usertitle] = htmlspecialchars($announcement[usertitle]);
         }
        $announcement[postdate]=vbdate($dateformat,$announcement[startdate]);

  if ($announcement[startdate]>$bbuserinfo[lastvisit]) {
      $announcement[icon]='newannounce.gif';
  } else {
      $announcement[icon]='announce.gif';
  }
  eval("\$announcement = \"".gettemplate('forumdisplay_announcement')."\";");
  $doneannouncements = 1;
}

and not

PHP Code:


			if ($announcement=$DB_site->query_first("SELECT announcementid,startdate,title,user.username,user.userid,user.usertitle,user.customtitle,visits
                                         FROM announcement
                                         LEFT JOIN user
                                           ON user.userid=announcement.userid
                                         WHERE startdate<=$datenow AND enddate>=$datenow
                                           AND $forumlist
                                         ORDER BY startdate DESC
                                         LIMIT 1")) {
    if ($foruminfo[allowratings]) {
      $thread[rating]='clear.gif';
      eval("\$threadrating = \"".gettemplate('forumdisplaybit_threadrate')."\";");
      $backcolor = '#1C5780';
          $bgclass = "alt2";
    } else {
                        unset($threadrating);
      $backcolor = '#13486D';
          $bgclass = "alt1";
    }

     if ($announcement[customtitle]==2)
       $announcement[usertitle] = htmlspecialchars($announcement[usertitle]);
           $announcement[postdate]=vbdate($dateformat,$announcement[startdate]);

  if ($announcement[startdate]>$bbuserinfo[lastvisit]) {
        $announcement[icon]='newannounce.gif';
  } else {
        $announcement[icon]='announce.gif';
  }
  eval("\$announcement = \"".gettemplate('forumdisplay_announcement')."\";");
  $doneannouncements = 1;
}

you forgot to add 2 brackets, just thought I would let you know... it could be confusing for newbies.

Crazy Mofo · #8 04-16-2002, 12:53 AM

so any updates for this?

coz i get this error

Warning: Unexpected character in input: '\' (ASCII=92) state=1 in /home/domain/domain119/web/forum/announcement.php on line 34

inetd · #9 04-16-2002, 05:10 PM

ZZed, pleaser release update this wonderful hack!

Zzed · #10 04-16-2002, 06:57 PM

Ok, I updated it.

It is all in the original post. And I updated the attachment.