Event: Restore the `constructor` property on jQuery.Event prototype

The original definition of the jQuery.Event prototype was paving over the
`constructor` property which was causing jQuery.isPlainObject to
improperly report that an instance of jQuery.Event was a plain object.

Fixes #15090
Closes gh-1580

diff --git a/src/event.js b/src/event.js
index 6183f703d4d767d86c630adf1f0f74c06efec730..7cf14c5c9fc6ad18c79fd00394a8b1a9dfebeefb 100644 (file)
--- a/src/event.js
+++ b/src/event.js
@@ -672,6 +672,7 @@ jQuery.Event = function( src, props ) {
 // jQuery.Event is based on DOM3 Events as specified by the ECMAScript Language Binding
 // http://www.w3.org/TR/2003/WD-DOM-Level-3-Events-20030331/ecma-script-binding.html
 jQuery.Event.prototype = {
+       constructor: jQuery.Event,
        isDefaultPrevented: returnFalse,
        isPropagationStopped: returnFalse,
        isImmediatePropagationStopped: returnFalse,

diff --git a/test/unit/event.js b/test/unit/event.js
index 67fe9115b75a8af913421a822ad37911ecf27ec0..fe0f0c2dac621d1f78077f8bdf071bf46082d951 100644 (file)
--- a/test/unit/event.js
+++ b/test/unit/event.js
@@ -1442,7 +1442,7 @@ if ( window.onbeforeunload === null &&
 
 test("jQuery.Event( type, props )", function() {
 
-       expect(5);
+       expect(6);
 
        var event = jQuery.Event( "keydown", { keyCode: 64 }),
                        handler = function( event ) {
@@ -1458,6 +1458,8 @@ test("jQuery.Event( type, props )", function() {
 
        ok( "keyCode" in event, "Special 'keyCode' property exists" );
 
+       strictEqual( jQuery.isPlainObject( event ), false, "Instances of $.Event should not be identified as a plain object." );
+
        jQuery("body").on( "keydown", handler ).trigger( event );
 
        jQuery("body").off( "keydown" );