Class won't properly instantiate on my instance of php5.4+ via public function Browser(). It needs __construct() method instead. This is also backwards compatible.

Its not working for UC Browser. I have made some changes to detect UC Browser too.

/** * Determine if the browser is UCBrowser or not (last updated 1.7) * @return boolean True if the browser is UCBrowser otherwise false */

protected function checkBrowserUCBrowser()
{
    if (stripos($this->_agent, 'UCBrowser') !== false) {
        $resultant = stristr($this->_agent, 'UCBrowser');
        if (preg_match('/\//', $resultant)) {
            $aresult = explode('/', $resultant);
            if (isset($aresult[1])) {
                $aversion = explode(' ', $aresult[1]);
                $this->setVersion($aversion[0]);
            }
        } else {
            $aversion = explode(' ', stristr($resultant, 'UCBrowser'));
            if (isset($aversion[1])) {
                $this->setVersion($aversion[1]);
            }
        }
        if (stripos($this->_agent, 'Mobile') !== false) {
            $this->setMobile(true);
        } else {
            $this->setTablet(true);
        }
        $this->setBrowser(self::BROWSER_UCBROWSER);
        return true;
    }
    return false;
}

Please, review it and add/modify if necessary.

Hello,

There is a problem. IE 11 isn't detected. $browser->getBrowser() => Mozilla $browser->getVersion() => 5.0

There is a new USER-AGENT! My USER-AGENT with IE 11 is: Mozilla/5.0 (Windows NT 6.1; WOW64; Trident/7.0; rv:11.0) like Gecko

Can you update?

Check it: http://msdn.microsoft.com/fr-fr/library/ie/bg182625(v=vs.85).aspx

Thanks

So there is a browser called Brave which is based on the Chromium open-source. Homepage: https://brave.com/

It's available on pretty much same OSes that Chrome is. They are using a slightly changed User-Agent on Windows though (haven't checked linux one yet).

Windows 10 x64: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Brave Chrome/74.0.3729.157 Safari/537.36

Looks like on android they are using same User-Agent as the mobile Chrome.

Android 9: Mozilla/5.0 (Linux; Android 9; *device*) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/74.0.3729.136 Mobile Safari/537.36

I think Browser.php should add support for Brave Chrome keyword in case they change their mobile User-Agent to match the Window's scheme so it's possible to disguise mobile Chrome from Brave.

In newer versions of Edge (based on Chrome) the user agent is using the "Edg" code name instead of "Edge".

Example user agent of Edge Version 81.0.416.41 (Official Build) beta (64 bit) Downloaded from https://www.microsoftedgeinsider.com/

Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/81.0.4044.83 Safari/537.36 Edg/81.0.416.41

Related issues #89

Hi, Edge is erroneously being detected as a Chrome (Browser::BROWSER_EDGE) .

Edge on Mac reports: Chrome Edge on Windows reports: Chrome Edge on Linux reports: Chrome

Edge v.79.0.309.71

Any help would be appreciated to show correct browser name.

Hi At the root of the project it is indicated that the program is licenced under the MIT, but in Browser.php it still mentions the General Public License.

Can that be updated? Mario

The PlayStation 3, 4 and Vita web browser share a pretty simple structure:

Mozilla/5.0 (Playstation $version[;] $firmware) AppleWebkit(......)

$version: 3, 4, Vita $firmware: X.XX (i.e. 3.55)

The PlayStation 3 has a semi-colon, appearently, whilst Vita and 4 do not.

To classify:

    const BROWSER_PLAYSTATION = "PlayStation";
    const PLATFORM_PLAYSTATION = "Sony PlayStation";

PS 3 and 4 are "Desktop" and not mobile. Vita is mobile.

Unfortunately I am not perfectly sure of the results for the PSP.

Could they be implemented?

iPhone unknown Mozilla/5.0 (iPhone; CPU iPhone OS 6_1_4 like Mac OS X) AppleWebKit/536.26 (KHTML, like Gecko) CriOS/28.0.1500.16 Mobile/10B350 Safari/8536.25

Safari unknown Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/536.11 (KHTML, like Gecko) DumpRenderTree/0.0.0.0 Safari/536.11

iPhone 28.0 Mozilla/5.0 (iPhone; CPU iPhone OS 6_1 like Mac OS X) AppleWebKit/536.26 (KHTML, like Gecko) CriOS/28.0.1500.16 Mobile/10B142 Safari/8536.25

Internet Explorer 9.0 Mozilla/5.0 (compatible; MSIE 9.0; Windows Phone OS 7.5; Trident/5.0; IEMobile/9.0; HTC; Radar; Orange) ("Mobile")

Hello,

it would be nice to remove the version property of composer.json file and use git tag to release versions.

This way is the simplest way of providing stable version as long as a developement one.

Hey there! When I work with getters and setters, I really like using magic methods since they make things easyer. Here is what i'd recommend for this very, VERY awesome class!

<?php class Browser {

    // ......all the other code


    /*
     * This function will call a getter function to get the information.
     * Example: $Browser->browser;
     * This will actually call $browser->getBrowser(); - it just looks a little more natural.
     * @return mixed
    */  
    public function __get($what) {
        $method = "get".ucfirst($what);
        if(method_exists($this, $method))
            return call_user_func(array($this, $method));
        else
            return $this->$what;
    }

    /*
     * The same as above, just for functions like setUserAgent.
     * @return void
    */
    public function __set($name, $value) {
        $method = "set".ucfirst($name);
        if(method_exists($this, $method))
            return call_user_func(array($this, $method), $value);
        else
            return $this->$name;
    }

    // ......all the other code

}

Just a thought =)

The getBrowser() method is showing 'Chrome' in browser Microsoft Edge (Version 104.0.1293.70)

The class is checking for Edge/ but Edge changed it to: Edg/

I think that checking the user agent 50+ times is incorrect, you can speed up the script several times by disassembling the user agent, and then checking each parameter according to the conditions.

The getVersion method is documented to return only a single period:

    /**
     * The version of the browser.
     * @return string Version of the browser (will only contain alpha-numeric characters and a period)
     */
    public function getVersion()
    {
        return $this->_version;
    }

However, it may contain multiple. E.g. '15.1.4'.

This was not really an obvious issue, until PHP 8, which changes how automatic casting works...

php -r 'var_dump("15.1.4" == 15.1);' bool(false)

It returns false as it now does a string comparison rather than an automatic cast, if it doesn't fully match the pattern of a float (I think!!). I was relying on this, as I'm sure other users were.

This change to the setVersion method fixes it by making it only have one period section:

    /**
     * Set the version of the browser
     * @param string $version The version of the Browser
     */
    public function setVersion($version)
    {
        $this->_version = preg_replace('#^([^\.]*\.[^\.]*)\..*#', '$1', preg_replace('/[^0-9,.,a-z,A-Z-]/', '', $version));
    }

This provides us a return value that continues to work...

$ php -r 'var_dump("15.1" == 15.1);'
bool(true)

As title, it seems that this class still uses the classmap approach to load the Browser class in composer.json.

I think we can consider using the PSR-4 autoloader and the advantages are as follows:

• The PSR-4 approach will have the namespace and it can avoid the class name collision.

Once this issue is accepted, I can help this issue :).

UserAgent is "Mozilla/5.0 (Linux; Android 6.0.1; SM-T580 Build/MMB29K) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/59.0.3071.125 Safari/537.36 OPR/43.0.2246.121183"

Result for $browser->isTablet() is false