From 9e6095a646f7e198a5ba29efd3efc36f010c23aa Mon Sep 17 00:00:00 2001 From: =?utf8?q?Scott=20Gonz=C3=A1lez?= Date: Wed, 23 Oct 2013 08:17:21 -0400 Subject: [PATCH] Core: Move variables for .uniqueId()/.removeUniqueId() into their implementations. --- ui/jquery.ui.core.js | 23 ++++++++++++----------- 1 file changed, 12 insertions(+), 11 deletions(-) diff --git a/ui/jquery.ui.core.js b/ui/jquery.ui.core.js index 9145d5d45..7faae0760 100644 --- a/ui/jquery.ui.core.js +++ b/ui/jquery.ui.core.js @@ -10,9 +10,6 @@ */ (function( $, undefined ) { -var uuid = 0, - runiqueId = /^ui-id-\d+$/; - // $.ui might exist from components with no dependencies, e.g., $.ui.position $.ui = $.ui || {}; @@ -72,17 +69,21 @@ $.fn.extend({ return ( /fixed/ ).test( this.css( "position") ) || !scrollParent.length ? $( this[ 0 ].ownerDocument || document ) : scrollParent; }, - uniqueId: function() { - return this.each(function() { - if ( !this.id ) { - this.id = "ui-id-" + (++uuid); - } - }); - }, + uniqueId: (function() { + var uuid = 0; + + return function() { + return this.each(function() { + if ( !this.id ) { + this.id = "ui-id-" + ( ++uuid ); + } + }); + }; + })(), removeUniqueId: function() { return this.each(function() { - if ( runiqueId.test( this.id ) ) { + if ( /^ui-id-\d+$/.test( this.id ) ) { $( this ).removeAttr( "id" ); } }); -- 2.39.5