[Dr.CustUmz]

Originally Posted by Paul M

I still dont understand the objective, why do you want to remove that record ?

Two things ;

1. Dont use db->query_first() on the delete, thats for table reads. I believe its db->query_write() you need, but Im unable to check that where I am located atm.

2. What you are doing atm isnt going to work properly, because you are running the delete before the insert.

You need to remember that shutdown queries are run when all the page processing has completed - just before vb sends the actual html back to the client (and shuts down).

the recored is being removed, so if thread starter views his own thread the view counter does not go up

this is how i came up with this, i edited showthread.php (search for // update views counter)
and came up with this.

// update views counter
if ($vbulletin->options['threadviewslive']) {
  // doing it as they happen; for optimization purposes, this cannot use a DM!
  if ($vbulletin->userinfo['userid'] == 0) {
    // ignore guest
  } elseif ($thread['postuserid']==$vbulletin->userinfo['userid']) {
    // ignore thread poster
  } else {
    $db->shutdown_query("
      UPDATE " . TABLE_PREFIX . "thread
      SET views = views + 1
      WHERE threadid = " . intval($threadinfo['threadid'])
    );
  }
} else {
  // or doing it once an hour
  if ($vbulletin->userinfo['userid'] == 0) {
    // ignore guest
  } elseif ($thread['postuserid']==$vbulletin->userinfo['userid']) {
    // ignore thread poster
  } else {
    $db->shutdown_query("
      INSERT INTO " . TABLE_PREFIX . "threadviews (threadid)
      VALUES (" . intval($threadinfo['threadid']) . ')'
    );
  }
}

works 100%

im trying to do the same exact thing only without having to edit the showthread.php