JavaScript "strict mode" fixes

jdomnitz · scottgonzalez · commit 57fe6ed01d07 · 2012-02-12T19:58:56.000-05:00
diff --git a/ui/jquery.ui.position.js b/ui/jquery.ui.position.js
@@ -414,7 +414,7 @@ $.ui.position = {
 
 // fraction support test
 (function () {
-	var testElement, testElementParent, testElementStyle, offsetLeft, i
+	var testElement, testElementParent, testElementStyle, offsetLeft, i,
 		body = document.getElementsByTagName( "body" )[ 0 ],
 		div = document.createElement( "div" );
 
diff --git a/ui/jquery.ui.tabs.js b/ui/jquery.ui.tabs.js
@@ -947,9 +947,7 @@ if ( $.uiBackCompat !== false ) {
 
 	// cookie option
 	var listId = 0;
-	function getNextListId() {
-		return ++listId;
-	}
+
 	$.widget( "ui.tabs", $.ui.tabs, {
 		options: {
 			cookie: null // e.g. { expires: 7, path: '/', domain: 'jquery.com', secure: true }
@@ -968,7 +966,7 @@ if ( $.uiBackCompat !== false ) {
 		},
 		_cookie: function( active ) {
 			var cookie = [ this.cookie ||
-				( this.cookie = this.options.cookie.name || "ui-tabs-" + getNextListId() ) ];
+				( this.cookie = this.options.cookie.name || "ui-tabs-" + ++listId ) ];
 			if ( arguments.length ) {
 				cookie.push( active === false ? -1 : active );
 				cookie.push( this.options.cookie );
diff --git a/ui/jquery.ui.widget.js b/ui/jquery.ui.widget.js
@@ -182,7 +182,7 @@ $.widget.bridge = function( name, object ) {
 				if ( instance ) {
 					instance.option( options || {} )._init();
 				} else {
-					object( options, this );
+					new object( options, this );
 				}
 			});
 		}

Original file line number	Diff line number	Diff line change
`@@ -182,7 +182,7 @@ $.widget.bridge = function( name, object ) {`
`182`	`182`	`if ( instance ) {`
`183`	`183`	`instance.option( options \|\| {} )._init();`
`184`	`184`	`} else {`
`185`		`- object( options, this );`
	`185`	`+ new object( options, this );`
`186`	`186`	`}`
`187`	`187`	`});`
`188`	`188`	`}`