NukeCoder

Site Admin Joined: Oct 25, 2004 Posts: 921

I noticed that counter.php is counting yahoo's spider as a user with a netscape browser.
The reason is, is it sees mozilla in the user agent and counts it as netscape and moves on without checking if it is a spider.

Here is the fix...
Open includes/counter.php

Find...

php:

 /* Get the Browser data */



if((ereg("Nav", $_SERVER["HTTP_USER_AGENT"])) || 

(ereg("Gold", $_SERVER["HTTP_USER_AGENT"])) || 

(ereg("X11", $_SERVER["HTTP_USER_AGENT"])) || 

(ereg("Mozilla", $_SERVER["HTTP_USER_AGENT"])) || 

(ereg("Netscape", $_SERVER["HTTP_USER_AGENT"])) AND 

(!ereg("MSIE", $_SERVER["HTTP_USER_AGENT"]) AND 

(!ereg("Konqueror", $_SERVER["HTTP_USER_AGENT"])) $browser = "Netscape";

Change to...

php:

 /* Get the Browser data */



if((ereg("Nav", $_SERVER["HTTP_USER_AGENT"])) || 

(ereg("Gold", $_SERVER["HTTP_USER_AGENT"])) || 

(ereg("X11", $_SERVER["HTTP_USER_AGENT"])) || 

(ereg("Mozilla", $_SERVER["HTTP_USER_AGENT"])) || 

(ereg("Netscape", $_SERVER["HTTP_USER_AGENT"])) AND 

(!ereg("MSIE", $_SERVER["HTTP_USER_AGENT"]) AND 

(!ereg("Konqueror", $_SERVER["HTTP_USER_AGENT"])) AND 

(!eregi("Yahoo", $_SERVER["HTTP_USER_AGENT"])))) $browser = "Netscape";

I added AND (!eregi("Yahoo", $_SERVER["HTTP_USER_AGENT"]))

This will keep the yahoo spider from being counted as netscape and start being counted as a search engine

Joined: Dec 09, 2006 Posts: 335

Wasn't this fixed circa nuke 7.6?
Here's the full code to also include the Firefox browser

Code:

/* Get the Browser data */

if ((ereg("Nav", $_SERVER['HTTP_USER_AGENT'])) || (ereg("Gold", $_SERVER['HTTP_USER_AGENT'])) || (ereg("X11", $_SERVER['HTTP_USER_AGENT'])) || (ereg("Mozilla", $_SERVER['HTTP_USER_AGENT'])) || (ereg("Netscape", $_SERVER['HTTP_USER_AGENT'])) AND (!ereg("MSIE", $_SERVER['HTTP_USER_AGENT'])) AND (!ereg("Konqueror", $_SERVER['HTTP_USER_AGENT'])) AND (!ereg("Yahoo", $_SERVER['HTTP_USER_AGENT'])) AND (!ereg("Firefox", $_SERVER['HTTP_USER_AGENT']))) $browser = "Netscape";
elseif(ereg("Firefox", $_SERVER['HTTP_USER_AGENT'])) $browser = "FireFox";
elseif(ereg("MSIE", $_SERVER['HTTP_USER_AGENT'])) $browser = "MSIE";
elseif(ereg("Lynx", $_SERVER['HTTP_USER_AGENT'])) $browser = "Lynx";
elseif(ereg("Opera", $_SERVER['HTTP_USER_AGENT'])) $browser = "Opera";
elseif(ereg("WebTV", $_SERVER['HTTP_USER_AGENT'])) $browser = "WebTV";
elseif(ereg("Konqueror", $_SERVER['HTTP_USER_AGENT'])) $browser = "Konqueror";
elseif((eregi("bot", $_SERVER['HTTP_USER_AGENT'])) || (ereg("Google", $_SERVER['HTTP_USER_AGENT'])) || (ereg("Slurp", $_SERVER['HTTP_USER_AGENT'])) || (ereg("Scooter", $_SERVER['HTTP_USER_AGENT'])) || (eregi("Spider", $_SERVER['HTTP_USER_AGENT'])) || (eregi("Infoseek", $_SERVER['HTTP_USER_AGENT']))) $browser = "Bot";
else $browser = "Other";

Site Admin Joined: Oct 25, 2004 Posts: 921

this fix was out before 7.6