Remove 1 Database Query Per Page

Remove 1 Database Query Per Page

Intro:
The idea is, lets remove 1 query from each page load, if the person is just refreshing the page. At first this may sound not important or just plain stupid, but on "larger" forums, cutting 1 query is always welcomed I know.

Description:
Basically all this does, is check the current Session Location from the DB against the current WOL path; the Session is already provided by vbulletin, so no extra query there.

All we do, is simply check the Location from the Database against the current WOL, if they don't match then the query is allowed to update the database; if they are the same - it is simply ignored.

Affects:
The only things this will do, is prevent the updating of the "lastactivity", and "location" if a person keeps refreshing the page, IMO that's not really a big deal, vbulletin takes care of deleting and remaking new sessions when they expire anyway. As long as your members are browsing around and such, their lastactivity, location will be updated as per usual.

Updates:
Version 1.1 - Scripts whosonline, misc are allowed to always update.
Version 1.0 - Release

Instructions:
1. [ open includes/sessions.php ]
2. [ go to about line 354, or look for $bypass = intval(SESSION_BYPASS); ]
3. [ replace this block of code ]

PHP Code:

    $DB_site->shutdown_query("
        UPDATE " . TABLE_PREFIX . "session
        SET useragent = '" . addslashes(USER_AGENT) . "', lastactivity = " . TIMENOW . $location . ", styleid = $styleid" . iif(VB_AREA !== 'Upgrade', ", bypass = $bypass") . "
        ###REPLACE###
        WHERE sessionhash = '" . addslashes($session['sessionhash']) . "'"
        , 'sessionupdate'
); 


4. [ with this block of code]

PHP Code:

    /**
     * Update Location On Page Change
     *
     * @author Trigunflame
     * @version 1.1
     * @copyright Dusty Burns 2005-2006
     */

    // Check Script
    if ($session['location'] != WOLPATH || THIS_SCRIPT == 'misc' || THIS_SCRIPT == 'online')
    {
        $DB_site->shutdown_query("
            UPDATE " . TABLE_PREFIX . "session
            SET useragent = '" . addslashes(USER_AGENT) . "', lastactivity = " . TIMENOW . $location . ", styleid = $styleid" . iif(VB_AREA !== 'Upgrade', ", bypass = $bypass") . "
             ###REPLACE###
             WHERE sessionhash = '" . addslashes($session['sessionhash']) . "'"
            , 'sessionupdate'
        );
    }

    /**
     * End Update Locations On Page Change
     */ 


5. [ save file ]

[sabret00the]

i'll probably install but i'm happy with my vote for MAF thank you

[Trigunflame]

Originally Posted by sabret00the

i'll probably install but i'm happy with my vote for MAF thank you

lol, poo on you

[Princeton]

anything that improves overall performance is a PLUS ...

[Trigunflame]

Originally Posted by princeton

anything that improves overall performance is a PLUS ...

Couldn't agree more XD

i would like to see a single forum where we can post that kind of Performance hack, because there is some (*i can count at least 25 of these), and they would be more than interesting to add to the next update...

because they are not hacks, they are debugs (when we don't modify a function, it's a debug isn't it?)

[Trigunflame]

Originally Posted by nexialys

i would like to see a single forum where we can post that kind of Performance hack, because there is some (*i can count at least 25 of these), and they would be more than interesting to add to the next update...

because they are not hacks, they are debugs (when we don't modify a function, it's a debug isn't it?)

I imagine, I woulda stuck this in mini-mods, but eh it really belongs here, if anywhere :/

[Yami Shadow]

I like this, but it seems to increase my board's query by 1, rather than knocking one off. Originally had 10, went to 11 after I preformed the mod..

[Trigunflame]

Originally Posted by Yami Shadow

I like this, but it seems to increase my board's query by 1, rather than knocking one off. Originally had 10, went to 11 after I preformed the mod..

Eh, thats not possible. Make sure you copied it exactly as I have it.

All this is doing is an IF condition to check the current page, if its the same page then the query is ignored.

Also should note: These are "Shutdown queries" which are performed after the output of data to the forum, and that would include whatever hack you are using to count queries.

These queries are not "counted" by traditional means, they are executed at the end of the life of the script.

[T3MEDIA]

Hummm how are you guys checking your queries? via putty or something?