The vB Geek

Go Back   The vB Geek > vB Geek Products > Geek Auto-Linker

Geek Auto-Linker GAL automatically links keywords that users type in posts to advertising hover pop-ups.

Advertisement
This is an HTML example. Isn't it just wonderful?!?!
  Learn how to remove ads

Reply
 
Thread Tools Display Modes
  #1  
Old 12-28-2005, 04:04 AM
KirbyDE KirbyDE is offline
Member
 
Join Date: Sep 2005
Posts: 31
Geek Article and Review System License Holder Geek Auto-Linker Pro License Holder Geek Advertising Banner License Holder Geek Gazette License Holder 
Default Impression tracking

PHP Code:
function persist_impressions()
{
    
$impressions =& $this->impressions;
    if (
$impressions)
        {
        
$m date('m'TIMENOW);
        
$d date('d'TIMENOW);
        
$y date('y'TIMENOW);
        
$dateline mktime(00$m$d$y);
        
$script $this->db->escape_string(THIS_SCRIPT); // this constant should never-ever be unsafe anyway ...
        
$forumid intval($this->forumid); // not sure if this is deemed necessary either
        
foreach ($impressions AS $key => $value)
        {
            if (
$value)
            {
                
$query[] = array($key$m$d$y$script$forumid$value$dateline);
            }
        }
        if (
is_array($query) AND !emtpy($query))
        {
            
$this->db->query_write("
                INSERT INTO " 
TABLE_PREFIX "geek_autolink_stats
                (galid, m, d, y, script, forum, count, dateline)
                VALUES
                " 
explode(','$query) . "
            "
);
        }
    }

and scrap persist_postponed_impressions()?
Reply With Quote
  #2  
Old 12-28-2005, 10:56 AM
The Geek's Avatar
The Geek The Geek is offline
Senior Member
 
Join Date: Aug 2005
Posts: 6,717
Geek Article and Review System License Holder GeekMart License Holder Geek Auto-Linker Pro License Holder Geek Advertising Banner License Holder Geek Gazette License Holder 
Default

Hey Kirb. I'm confused - what are you suggesting?
Reply With Quote
  #3  
Old 12-28-2005, 03:09 PM
KirbyDE KirbyDE is offline
Member
 
Join Date: Sep 2005
Posts: 31
Geek Article and Review System License Holder Geek Auto-Linker Pro License Holder Geek Advertising Banner License Holder Geek Gazette License Holder 
Default

I am suggesting to replace GAL_base::persist_impressions() with smth. like the code above, remove GAL_admin::persist_postponed_impressions() and the throttle setting :)
Reply With Quote
  #4  
Old 12-28-2005, 10:22 PM
The Geek's Avatar
The Geek The Geek is offline
Senior Member
 
Join Date: Aug 2005
Posts: 6,717
Geek Article and Review System License Holder GeekMart License Holder Geek Auto-Linker Pro License Holder Geek Advertising Banner License Holder Geek Gazette License Holder 
Default

Ah - duh Should of looked a bit closer at the code :P

Fantastic suggestion. Will do.

Thanks
Reply With Quote
Reply

Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump

Similar Threads
Thread Thread Starter Forum Replies Last Post
Is tracking working? KenDude Geek Gazette 6 07-04-2009 01:35 PM
Not tracking clicks? smokenm Geek Advertising Banner System 4 08-29-2006 08:15 AM
Affiliate Program Tracking Code greg watson GeekMart 14 01-05-2006 09:59 AM
First impression of GARS KTBleeding Geek Article and Review System 1 09-03-2005 09:26 AM


All times are GMT. The time now is 01:42 AM.


Powered by vBulletin® Version 3.8.5
Copyright ©2000 - 2017, Jelsoft Enterprises Ltd.