Browser detection IE (and possibly other browsers)

[cybernetix]

If MEGAUPLOAD or similar plugin is installed jQuery.browser returns wrong IE version.

For example userAgent string for IE7 with installed MEGAUPLOAD plugin is:

Mozilla/4.0 (compatible; MSIE 7.0; Windows NT 5.1; Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; SV1) ; .NET CLR 2.0.50727; .NET CLR 1.1.4322; .NET CLR 3.0.04506.30; MEGAUPLOAD 2.0)

jQuery.browser.version returns 6 instead of 7

First plus (+) should be removed from regExp and than it will work fine. So, this line:

(userAgent.match( /.+(?:rv|it|ra|ie)[\/: ]([\d.]+)/ )

[])[1]

should read:

(userAgent.match( /.(?:rv|it|ra|ie)[\/: ]([\d.]+)/ )

[])[1]

[flesler]

Same as #2784

[eventualbudd]

The solution you posted doesn't actually work with Opera. Here's one that passes all the jQuery tests (thanks Brad Greenlee!):

/.+?(?:rv|it|ra|ie)[\/: ]([\d.]+)(?!.+opera)/

[dmethvin]

See also #3169 (dup but essentially agrees on the regexp)

[dmethvin]

Tickets #3696, #3402, #2661, and #2413 are all related, but for different user-agents.

[mesoconcepts]

Has anyone looked into this? It's clearly related, and his patch seems to work...

​http://jamazon.co.uk/web/2008/03/14/jquerybrowserversion-doesnt-recognise-ie7/

​http://jamazon.co.uk/web/2008/03/17/thickbox-31-ie7-positioning-bug/

[mesoconcepts]

cross-referencing in case someone on the WP end submits a patch:

​http://trac.wordpress.org/ticket/8933#comment:5

[brandon]

I've closed the following bugs as duplicates of this one to consolidate the conversations/fixes. When fixing this bug, please be sure that the fix addresses these other issues as well. #4338, #3696, #3402, #2413, #3169, #2748, and #1537

[dmethvin]

Adding dup #4674 to the pile.

[dmethvin]

Also dup #4722.

[dmethvin]

Boop boop be dup #4754.

[dmethvin]

See also #5279.

[dmethvin]

See also #5127.

[thorn]

see also ​http://jamazon.co.uk/web/2008/07/23/an-ie7-bug-that-returns-msie-60-user-agent-string/

[hm2k]

​http://www.hm2k.com/posts/jquery-sucks-at-browser-detection

[john]

Landed. ​http://github.com/jquery/jquery/commit/d3dc2d1234af10cb856519a01a97358eb4103008