#33 incorrect detection on old NS browser

[Anonymous]

From: Donna Robinson < donna at terpsichore.ws >

Hi,

I have the misfortunate to have to deal with old
Netscape browsers (v. 48), and noticed that phpSniff
didn't deal with eg. css1, css2 etc., etc. correctly.

After tracking it through, it turns out that in
function _get_gecko() you re-assign :
$this->_set_browser( 'long_name', 'netscape');
$this->_set_browser( 'browser',
$this->_get_short_name('netscape') );

which means that the final output in your example
index.php _looks_ fine, at a quick glance, but in fact
it isn't.

FIX (or should I say hack)
function _init() {
$this->_get_ip();
$this->_test_cookies();
$this->_get_browser_info();
// --> stick this in *here*
$this->_get_gecko();
$this->_get_languages();
$this->_get_os_info();
$this->_get_javascript();
$this->_get_features();
$this->_get_quirks();
// --> now commented out:
// $this->_get_gecko();
}

A nicer thing to do would be to get it right first time in
_get_browser_info()
but time-shortages being what they are ...

btw, _love_ yr prog. no f_star_see_kay j'script.

Donna