Bug Tracker

#1867 closed bug (duplicate)

Opened October 31, 2007 09:48AM UTC

Closed January 10, 2009 10:49PM UTC

Last modified March 15, 2012 12:58AM UTC

Some MS IE 7.0 userAgent strings return a jQuery browser.version of 6.0

Reported by: danielnashnz Owned by:
Priority: major Milestone: 1.2.2
Component: core Version: 1.2.1
Keywords: Cc:
Blocked by: Blocking:
Description

I am not sure how I got it (probably from the numerous .NET patches I have installed?) but my IE 7.0 userAgent string is returned as this:

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)

Which because of the greedy match in the jQuery ver 1.2.1 browser version assignement:

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

^^

returns a version number of "6.0" from the second "MSIE \\d..." string.

I have searched around on the Web and there seem to quite a few userAgents logged with similarly confounding identifiers.

Attachments (0)
Change History (2)

Changed November 22, 2007 03:43AM UTC by achun comment:1

ya

I try fix it;

jQuery.browser = {
	version: (userAgent.match( /.+?(?:rv|it|ra|ie)[\\/: ]([\\d.]+)/ ) || [])[1],//fixed by achun
	safari: /webkit/.test( userAgent ),
	opera: /opera/.test( userAgent ),
	msie: /msie/.test( userAgent ) && !/opera/.test( userAgent ),
	mozilla: /mozilla/.test( userAgent ) && !/(compatible|webkit)/.test( userAgent ),
	firefox: /firefox/.test(userAgent),//fixed by achun
	is:(userAgent.match(/.*(opera|msie|firefox|mozilla)/) || [])[1]//fixed by achun
};

if ('firefox'==jQuery.browser.is) {//fixed by achun
	jQuery.browser.version = (userAgent.match( /.+(?:rv|it|ra|ie)[\\/: ]([\\d.]+)/ ) || [])[1];
}

Changed January 10, 2009 10:49PM UTC by dmethvin comment:2

resolution: → duplicate
status: new → closed

This is a duplicate of #2784 but I left the later ticket open since it had more activity.