Fixed (bug 27652) [jQuery.client] versionBase is wrong for versions higher with two...
authorKrinkle <krinkle@users.mediawiki.org>
Sun, 6 Mar 2011 14:34:10 +0000 (14:34 +0000)
committerKrinkle <krinkle@users.mediawiki.org>
Sun, 6 Mar 2011 14:34:10 +0000 (14:34 +0000)
* A userAgent like the following:
> "Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10_5_8; nl-nl) AppleWebKit/533.19.4 (KHTML, like Gecko) Version/5.0.3 Safari/533.19.4"
Will now have a versionBase of "10" instead of "1".
* A few JSHint warnings fixed (missing semicolon)

resources/jquery/jquery.client.js

index 95407c9..42e515c 100644 (file)
@@ -125,6 +125,7 @@ $.client = new ( function() {
                        if ( name === 'opera' && version >= 9.8) {
                                version = userAgent.match( /version\/([0-9\.]*)/i )[1] || 10;
                        }
+                       var versionNumber = parseFloat( version, 10 ) || 0.0;
 
                        /* Caching */
 
@@ -134,8 +135,8 @@ $.client = new ( function() {
                                'layoutVersion': layoutversion,
                                'platform': platform,
                                'version': version,
-                               'versionBase': ( version !== x ? new String( version ).substr( 0, 1 ) : x ),
-                               'versionNumber': ( parseFloat( version, 10 ) || 0.0 )
+                               'versionBase': ( version !== x ? Math.floor( versionNumber ).toString() : x ),
+                               'versionNumber': versionNumber
                        };
                }
                return profile;
@@ -191,7 +192,7 @@ $.client = new ( function() {
                        }
                }
                return true;
-       }
+       };
 } )();
 
 $( document ).ready( function() {