cbschuld / browser.php Goto Github PK

User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64) SkypeUriPreview Preview/0.5

Similar to FacebookExternalHit

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 :).

Hi
Thanks to add a namespace to your package

     I have implemented browser.php on my project for tracking browsers and platform.When I check my browser  it will display all the browser name except

Google Chrome. It displays following error
Undefined offset: 1 in D:\wamp\www\tracker202\assert\Browser.php on line 647.
please help me how to resolve these error?

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/

The recent commit 012d604 broke detection of IE11. It no longer checks for 'trident' in the user agent string as it was before, and IE11 does not contain MSIE in the user agent string. This wrongly causes getBrowser() to return "Mozilla" instead of "Internet Explorer".

Please create versions in github, so the project can work nicely with composer

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,

ok all work, fine.,
Great job, thanks!

..wouldn't LGPL be more appropriate?

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.

As this class does not contain any HTML, accidential whitespaces in output due to them coming after the ?> can be avoided by simply removing the closing tag. PHP does only requre an opening tag - and if it finds the file's end (EOF), then it will treat the file as complete.

Its just a simple thing I saw while reading the file.

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

When adding Browser as a dependency in Composer, I get the following error:

[RuntimeException]
Failed to clone https://github.com/cbschuld/browser.php.git via git, https
protocols, aborting.

• git://github.com/cbschuld/browser.php.git
  fatal: remote error:
  Repository not found.
• https://github.com/cbschuld/browser.php.git
  fatal: https://github.com/cbschuld/browser.php.git/info/refs?service=git-upload-pack > not found: did you run git update-server-info on the server?

It appears that on packagist (https://packagist.org/packages/cbschuld/browser.php) the canonical url https://github.com/cbschuld/browser.php however it is actually https://github.com/cbschuld/Browser.php
(Not the capital B in Browser.php)

Although viewing either via a web browser doesn't seem to matter, it looks like Composer/Git does.

Is this an issue with Browser or is it a Composer or git issue?

My composer.json has
"require": {
"cbschuld/browser.php": "dev-master"
}

Cheers

Please set up GitHub Service Hook (procedure on https://packagist.org/profile/ , for autoupdate package on packagist). For example now I saw namespace from previous commit (BC break removed by last commit).

Thanks

I just noticed that iPads Pro are not detected as tablets.

Here an example of an iPad Pro user agent string:

Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/16.4 Safari/605.1.15

As far as I could read, as of today iPads Pro are same as intel macs, the only way to potentially detect them would be checking for touchscreen functionality.

Any plans, ideas or workarounds to detect iPad Pro on PHP side?

Thanks in advance.

User agent string: Mozilla/5.0 (iPad; CPU OS 5_1_1 like Mac OS X) AppleWebKit/534.46 (KHTML, like Gecko) Version/5.1 Mobile/9B206 Safari/7534.48.3

Library returns iPad browser instead of Safari.

While http://www.useragentstring.com/index.php is parsing perfect.

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)

This does not work :-/

We screwed up our internal analytics by this issue.

Sample Input UserAgent: "Mozilla/5.0 (iPhone; CPU iPhone OS 6_0 like Mac OS X) AppleWebKit/536.26 (KHTML, like Gecko) Version/6.0 Mobile/10A5376e Safari/8536.25 (compatible; Googlebot/2.1; +http://www.google.com/bot.html)"

Output: iPhone.

Expected Output: GoogleBot

Solution / Fix:
The parsing priority for Googlebot (and other bots) must be high.
Replace the function checkBrowsers with this:

protected function checkBrowsers()
{
    return (
        // well-known, well-used
        // Special Notes:
        // (1) Opera must be checked before FireFox due to the odd
        //     user agents used in some older versions of Opera
        // (2) WebTV is strapped onto Internet Explorer so we must
        //     check for WebTV before IE
        // (3) (deprecated) Galeon is based on Firefox and needs to be
        //     tested before Firefox is tested
        // (4) OmniWeb is based on Safari so OmniWeb check must occur
        //     before Safari
        // (5) Netscape 9+ is based on Firefox so Netscape checks
        //     before FireFox are necessary

        // common bots
        $this->checkBrowserGoogleBot() ||
        $this->checkBrowserMSNBot() ||
        $this->checkBrowserBingBot() ||
        $this->checkBrowserSlurp() ||

        $this->checkBrowserWebTv() ||
        $this->checkBrowserInternetExplorer() ||
        $this->checkBrowserOpera() ||
        $this->checkBrowserGaleon() ||
        $this->checkBrowserNetscapeNavigator9Plus() ||
        $this->checkBrowserFirefox() ||
        $this->checkBrowserChrome() ||
        $this->checkBrowserOmniWeb() ||

        // common mobile
        $this->checkBrowserAndroid() ||
        $this->checkBrowseriPad() ||
        $this->checkBrowseriPod() ||
        $this->checkBrowseriPhone() ||
        $this->checkBrowserBlackBerry() ||
        $this->checkBrowserNokia() ||

        // check for facebook external hit when loading URL
        $this->checkFacebookExternalHit() ||

        // WebKit base check (post mobile and others)
        $this->checkBrowserSafari() ||

        // everyone else
        $this->checkBrowserNetPositive() ||
        $this->checkBrowserFirebird() ||
        $this->checkBrowserKonqueror() ||
        $this->checkBrowserIcab() ||
        $this->checkBrowserPhoenix() ||
        $this->checkBrowserAmaya() ||
        $this->checkBrowserLynx() ||
        $this->checkBrowserShiretoko() ||
        $this->checkBrowserIceCat() ||
        $this->checkBrowserIceweasel() || 
        $this->checkBrowserW3CValidator() ||
        $this->checkBrowserMozilla() /* Mozilla is such an open standard that you must check it last */
    );
}

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.

isMobile() is false for windows phone and Platform is windows instead of windows phone.

I would like to see OS and version detection.

Mozilla/5.0 (iPad; CPU OS 6_1_3 like Mac OS X) AppleWebKit/536.26 (KHTML, like Gecko) CriOS/28.0.1500.12 Mobile/10B329 Safari/8536.25

(This is directly related to #89)

The new chrome-based Microsoft Edge on mobile is still detected as a Chrome Browser.

I tried this on Android 10 with the official Microsoft Edge App (https://play.google.com/store/apps/details?id=com.microsoft.emmx) (App-Version: Edge 45.02.4.4931)

The user agent is:
Mozilla/5.0 (Linux; Android 10; HRY-LX1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/77.0.3865.116 Mobile Safari/537.36 EdgA/45.02.4.4931

I think the A in the EdgA part seems to be the problem since the latest fix only checks for Edg.

This should probably also be tested for the official Apple App (https://apps.apple.com/us/app/microsoft-edge/id1288723196) which I'm currently not able to.

http://msdn.microsoft.com/en-us/library/ie/bg182625(v=vs.85).aspx
Mozilla/5.0 (Windows NT 6.3; Trident/7.0; rv:11.0) like Gecko

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

Hi I can't seem to separate the platform and the browser name. See the browser section has "iPhone" and "iPad".
Any way to get the functionality I'd like? Help is much appreciated.

Hi,

I noticed that when using the class inside a namespace it wold not work because the Browser method was never invoked as constructor. Suggest to change to __construct to work on namespaces

On iOS Safari getBrowser() would return "iPhone" as Browser and (for me) "9.0" as Version, which is actually enough for my purposes, but it could still be a problem for developers that need to go further than me in detecting Browsers. But using Mozilla Firefox on iOS, again the function returns "iPhone", but isn't able to detect the Version, therefore returns "unknown". I didn't test it for other Browsers on iOS but i'm assuming that it's not possible to detect the specific browser as well.
Tested on an iPhone SE (iOS 9.3.5, newest versions of Firefox and (obviously) Safari, atm)

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.

My mac is erroneously being detected as an iPad ($brower->getPlatform()) .

Safari reports: iPad
Chrome reports: Apple
Firefox reports: Apple

device: MacbookPro13,2
Safari Version 10.0.1 (12602.2.14.0.7)

any help would be appreciated

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?

As title, it seems that the http://localhost:3000/projects/browser-php-detecting-a-users-browser-from-php/ demonstration site should be updated.

It should have a global website on the Internet to be visible for everyone.

I can support this domain and demonstration website if @cbschuld wants to do this :).

https://github.com/cbschuld/Browser.php/releases/tag/1.9.1

Thanks for this module, very helpful :)

New version v1.9.1 is not on site
https://packagist.org/packages/cbschuld/browser.php

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 current library doesn't detect the new Edge browser over Windows 10.

Can you create a method to check if Browser is HTML5 Compatible?

Is the "isMobile()" call supposed to return false on the iPad?

That's what I'm getting when trying to use it, but I wasn't sure if this was intended or a bug?

The user agent is:
Mozilla/5.0 (iPad; CPU OS 5_1_1 like Mac OS X) AppleWebKit/534.46 (KHTML, like Gecko) Version/5.1 Mobile/9B206 Safari/7534.48.3

Thanks

Add support for curl & wget

@willemstuursma originally created work to detect curl & wget but the files were out of date; will work in his functions & tests

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.

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")

There's an Open Source project called Pale Moon which primary is based earlier versions of Mozilla Firefox, now independly developed over time. Project home page: https://www.palemoon.org/

The browser mainly available on Windows & Linux platforms, other platforms are still developed.
Linux UA (Pale Moon 28.5.0): Mozilla/5.0 (X11; Linux x86_64; rv:60.9) Gecko/20100101 Goanna/4.2 Firefox/60.9 PaleMoon/28.5.0

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