library-mirrors/jquery-ui
1
0
Fork 0
mirror of https://github.com/jquery/jquery-ui.git synced 2024-10-05 01:44:18 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Widget: Don't modify the prototype passed to $.widget(). Fixes #8876 - Calling _super calls wrong inherited widget.

Browse Source
This commit is contained in:
Scott González 2012-12-03 14:18:24 -05:00
parent 7312933c88
commit 8b15aaf496
2 changed files with 71 additions and 33 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

47
tests/unit/widget/widget_core.js
View File

@ -13,18 +13,23 @@ TestHelpers.testJshint( "widget" );
test( "widget creation", function() {
expect( 5 );
var myPrototype = {
_create: function() {},
creationTest: function() {}
var method,
myPrototype = {
_create: function() {
equal( method, "_create", "create function is copied over" );
},
creationTest: function() {
equal( method, "creationTest", "random function is copied over" );
}
};
$.widget( "ui.testWidget", myPrototype );
ok( $.isFunction( $.ui.testWidget ), "constructor was created" );
equal( typeof $.ui.testWidget.prototype, "object", "prototype was created" );
equal( $.ui.testWidget.prototype._create, myPrototype._create,
"create function is copied over" );
equal( $.ui.testWidget.prototype.creationTest, myPrototype.creationTest,
"random function is copied over" );
method = "_create";
$.ui.testWidget.prototype._create();
method = "creationTest";
$.ui.testWidget.prototype.creationTest();
equal( $.ui.testWidget.prototype.option, $.Widget.prototype.option,
"option method copied over from base widget" );
});
@ -1324,6 +1329,34 @@ test( "redefine - widgetEventPrefix", function() {
});
test( "mixins", function() {
expect( 2 );
var mixin = {
method: function() {
return "mixed " + this._super();
}
};
$.widget( "ui.testWidget1", {
method: function() {
return "testWidget1";
}
});
$.widget( "ui.testWidget2", {
method: function() {
return "testWidget2";
}
});
$.widget( "ui.testWidget1", $.ui.testWidget1, mixin );
$.widget( "ui.testWidget2", $.ui.testWidget2, mixin );
equal( $( "<div>" ).testWidget1().testWidget1( "method" ),
"mixed testWidget1", "testWidget1 mixin successful" );
equal( $( "<div>" ).testWidget2().testWidget2( "method" ),
"mixed testWidget2", "testWidget2 mixin successful" );
});
asyncTest( "_delay", function() {
expect( 6 );
var order = 0,

13
ui/jquery.ui.widget.js vendored
View File

@ -25,6 +25,9 @@ $.cleanData = function( elems ) {
$.widget = function( name, base, prototype ) {
var fullName, existingConstructor, constructor, basePrototype,
// proxiedPrototype allows the provided prototype to remain unmodified
// so that it can be used as a mixin for multiple widgets (#8876)
proxiedPrototype = {},
namespace = name.split( "." )[ 0 ];
name = name.split( "." )[ 1 ];
@ -71,8 +74,11 @@ $.widget = function( name, base, prototype ) {
// inheriting from
basePrototype.options = $.widget.extend( {}, basePrototype.options );
$.each( prototype, function( prop, value ) {
if ( $.isFunction( value ) ) {
prototype[ prop ] = (function() {
if ( !$.isFunction( value ) ) {
proxiedPrototype[ prop ] = value;
return;
}
proxiedPrototype[ prop ] = (function() {
var _super = function() {
return base.prototype[ prop ].apply( this, arguments );
},
@ -95,14 +101,13 @@ $.widget = function( name, base, prototype ) {
return returnValue;
};
})();
}
});
constructor.prototype = $.widget.extend( basePrototype, {
// TODO: remove support for widgetEventPrefix
// always use the name + a colon as the prefix, e.g., draggable:start
// don't prefix for widgets that aren't DOM-based
widgetEventPrefix: existingConstructor ? basePrototype.widgetEventPrefix : name
}, prototype, {
}, proxiedPrototype, {
constructor: constructor,
namespace: namespace,
widgetName: name,