Remove jQuery Migrate and $wgIncludejQueryMigrate
authorJames D. Forrester <jforrester@wikimedia.org>
Tue, 3 Jun 2014 20:51:39 +0000 (13:51 -0700)
committerTimo Tijhof <krinklemail@gmail.com>
Fri, 5 Dec 2014 01:20:57 +0000 (01:20 +0000)
As indicated in MediaWiki 1.24, removing the jQuery Migrate bridging patch
that allows legacy Javascript code that depends ancient features of
jQuery removed in jQuery 1.9 to work.

Change-Id: I0934247182e2737716c9d40f4bf93415adcdadf3

RELEASE-NOTES-1.25
includes/DefaultSettings.php
resources/Resources.php
resources/lib/jquery/jquery.migrate.js [deleted file]

index ce825e5..36d698f 100644 (file)
@@ -29,6 +29,9 @@ production.
 * $wgOpenSearchTemplate is deprecated in favor of $wgOpenSearchTemplates.
 * Edits are now prepared via AJAX as users type edit summaries. This behavior
   can be disabled via $wgAjaxEditStash.
+* (bug 44740) The temporary option $wgIncludejQueryMigrate was removed, along
+  with the jQuery Migrate library, as indicated when this option was provided in
+  MediaWiki 1.24.
 
 === New features in 1.25 ===
 * (T64861) Updated plural rules to CLDR 26. Includes incompatible changes
index e644e09..64d9892 100644 (file)
@@ -3413,15 +3413,6 @@ $wgResourceLoaderMinifierMaxLineLength = 1000;
  */
 $wgIncludeLegacyJavaScript = true;
 
-/**
- * Whether to include the jQuery Migrate library, which lets legacy JS that
- * requires jQuery 1.8.x to work and breaks with 1.9.x+.
- *
- * @since 1.24
- * @deprecated since 1.24, to be removed in 1.25
- */
-$wgIncludejQueryMigrate = false;
-
 /**
  * Whether to preload the mediawiki.util module as blocking module in the top
  * queue.
index 72cc2ef..d6a3181 100644 (file)
@@ -121,14 +121,9 @@ return array(
        /* jQuery */
 
        'jquery' => array(
-               'scripts' => ( $GLOBALS['wgIncludejQueryMigrate'] ?
-                       array(
-                               'resources/lib/jquery/jquery.js',
-                               'resources/lib/jquery/jquery.migrate.js'
-                       ) :
-                       array(
-                               'resources/lib/jquery/jquery.js'
-                       ) ),
+               'scripts' => array(
+                       'resources/lib/jquery/jquery.js',
+               ),
                'raw' => true,
                'targets' => array( 'desktop', 'mobile' ),
        ),
diff --git a/resources/lib/jquery/jquery.migrate.js b/resources/lib/jquery/jquery.migrate.js
deleted file mode 100644 (file)
index 5b18236..0000000
+++ /dev/null
@@ -1,551 +0,0 @@
-/*!
- * jQuery Migrate - v1.2.1 - 2013-05-08
- * https://github.com/jquery/jquery-migrate
- * Copyright 2005, 2013 jQuery Foundation, Inc. and other contributors; Licensed MIT
- *
- * Patched for MediaWiki to add mw.track calls. --Krinkle 2014-04-14
- */
-(function( jQuery, window, undefined ) {
-// See http://bugs.jquery.com/ticket/13335
-// "use strict";
-
-
-var warnedAbout = {};
-
-// List of warnings already given; public read only
-jQuery.migrateWarnings = [];
-
-// Set to true to prevent console output; migrateWarnings still maintained
-// jQuery.migrateMute = false;
-
-// Show a message on the console so devs know we're active
-if ( !jQuery.migrateMute && window.console && window.console.log ) {
-       window.console.log("JQMIGRATE: Logging is active");
-}
-
-// Set to false to disable traces that appear with warnings
-if ( jQuery.migrateTrace === undefined ) {
-       jQuery.migrateTrace = true;
-}
-
-// Forget any warnings we've already given; public
-jQuery.migrateReset = function() {
-       warnedAbout = {};
-       jQuery.migrateWarnings.length = 0;
-};
-
-function migrateWarn( msg, key ) {
-       var console = window.console;
-       /*
-               MediaWiki patch for tracking usage.
-
-               Custom keys:
-               - andSelf
-               - attr-pass
-               - attr-prop
-               - bind-error
-               - clean
-               - create-html
-               - data-events
-               - die
-               - event-ajax
-               - event-global
-               - event-hover
-               - event-handle
-               - input-type
-               - json-invalid
-               - live
-               - sub
-               - toggle-handle
-
-               Prop keys:
-               - attrFn
-               - browser
-       */
-       mw.track( "jquery.migrate", key || "unknown" );
-
-       if ( !warnedAbout[ msg ] ) {
-               warnedAbout[ msg ] = true;
-               jQuery.migrateWarnings.push( msg );
-               if ( console && console.warn && !jQuery.migrateMute ) {
-                       console.warn( "JQMIGRATE: " + msg );
-                       if ( jQuery.migrateTrace && console.trace ) {
-                               console.trace();
-                       }
-               }
-       }
-}
-
-function migrateWarnProp( obj, prop, value, msg, key ) {
-       if ( Object.defineProperty ) {
-               // On ES5 browsers (non-oldIE), warn if the code tries to get prop;
-               // allow property to be overwritten in case some other plugin wants it
-               try {
-                       Object.defineProperty( obj, prop, {
-                               configurable: true,
-                               enumerable: true,
-                               get: function() {
-                                       migrateWarn( msg, key || prop );
-                                       return value;
-                               },
-                               set: function( newValue ) {
-                                       migrateWarn( msg, key || prop );
-                                       value = newValue;
-                               }
-                       });
-                       return;
-               } catch( err ) {
-                       // IE8 is a dope about Object.defineProperty, can't warn there
-               }
-       }
-
-       // Non-ES5 (or broken) browser; just set the property
-       jQuery._definePropertyBroken = true;
-       obj[ prop ] = value;
-}
-
-if ( document.compatMode === "BackCompat" ) {
-       // jQuery has never supported or tested Quirks Mode
-       migrateWarn( "jQuery is not compatible with Quirks Mode" );
-}
-
-
-var attrFn = jQuery( "<input/>", { size: 1 } ).attr("size") && jQuery.attrFn,
-       oldAttr = jQuery.attr,
-       valueAttrGet = jQuery.attrHooks.value && jQuery.attrHooks.value.get ||
-               function() { return null; },
-       valueAttrSet = jQuery.attrHooks.value && jQuery.attrHooks.value.set ||
-               function() { return undefined; },
-       rnoType = /^(?:input|button)$/i,
-       rnoAttrNodeType = /^[238]$/,
-       rboolean = /^(?:autofocus|autoplay|async|checked|controls|defer|disabled|hidden|loop|multiple|open|readonly|required|scoped|selected)$/i,
-       ruseDefault = /^(?:checked|selected)$/i;
-
-// jQuery.attrFn
-migrateWarnProp( jQuery, "attrFn", attrFn || {}, "jQuery.attrFn is deprecated" );
-
-jQuery.attr = function( elem, name, value, pass ) {
-       var lowerName = name.toLowerCase(),
-               nType = elem && elem.nodeType;
-
-       if ( pass ) {
-               // Since pass is used internally, we only warn for new jQuery
-               // versions where there isn't a pass arg in the formal params
-               if ( oldAttr.length < 4 ) {
-                       migrateWarn("jQuery.fn.attr( props, pass ) is deprecated", "attr-pass" );
-               }
-               if ( elem && !rnoAttrNodeType.test( nType ) &&
-                       (attrFn ? name in attrFn : jQuery.isFunction(jQuery.fn[name])) ) {
-                       return jQuery( elem )[ name ]( value );
-               }
-       }
-
-       // Warn if user tries to set `type`, since it breaks on IE 6/7/8; by checking
-       // for disconnected elements we don't warn on $( "<button>", { type: "button" } ).
-       if ( name === "type" && value !== undefined && rnoType.test( elem.nodeName ) && elem.parentNode ) {
-               migrateWarn("Can't change the 'type' of an input or button in IE 6/7/8", "input-type");
-       }
-
-       // Restore boolHook for boolean property/attribute synchronization
-       if ( !jQuery.attrHooks[ lowerName ] && rboolean.test( lowerName ) ) {
-               jQuery.attrHooks[ lowerName ] = {
-                       get: function( elem, name ) {
-                               // Align boolean attributes with corresponding properties
-                               // Fall back to attribute presence where some booleans are not supported
-                               var attrNode,
-                                       property = jQuery.prop( elem, name );
-                               return property === true || typeof property !== "boolean" &&
-                                       ( attrNode = elem.getAttributeNode(name) ) && attrNode.nodeValue !== false ?
-
-                                       name.toLowerCase() :
-                                       undefined;
-                       },
-                       set: function( elem, value, name ) {
-                               var propName;
-                               if ( value === false ) {
-                                       // Remove boolean attributes when set to false
-                                       jQuery.removeAttr( elem, name );
-                               } else {
-                                       // value is true since we know at this point it's type boolean and not false
-                                       // Set boolean attributes to the same name and set the DOM property
-                                       propName = jQuery.propFix[ name ] || name;
-                                       if ( propName in elem ) {
-                                               // Only set the IDL specifically if it already exists on the element
-                                               elem[ propName ] = true;
-                                       }
-
-                                       elem.setAttribute( name, name.toLowerCase() );
-                               }
-                               return name;
-                       }
-               };
-
-               // Warn only for attributes that can remain distinct from their properties post-1.9
-               if ( ruseDefault.test( lowerName ) ) {
-                       migrateWarn( "jQuery.fn.attr('" + lowerName + "') may use property instead of attribute", "attr-prop" );
-               }
-       }
-
-       return oldAttr.call( jQuery, elem, name, value );
-};
-
-// attrHooks: value
-jQuery.attrHooks.value = {
-       get: function( elem, name ) {
-               var nodeName = ( elem.nodeName || "" ).toLowerCase();
-               if ( nodeName === "button" ) {
-                       return valueAttrGet.apply( this, arguments );
-               }
-               if ( nodeName !== "input" && nodeName !== "option" ) {
-                       migrateWarn("jQuery.fn.attr('value') no longer gets properties", "attr-prop");
-               }
-               return name in elem ?
-                       elem.value :
-                       null;
-       },
-       set: function( elem, value ) {
-               var nodeName = ( elem.nodeName || "" ).toLowerCase();
-               if ( nodeName === "button" ) {
-                       return valueAttrSet.apply( this, arguments );
-               }
-               if ( nodeName !== "input" && nodeName !== "option" ) {
-                       migrateWarn("jQuery.fn.attr('value', val) no longer sets properties", "attr-prop");
-               }
-               // Does not return so that setAttribute is also used
-               elem.value = value;
-       }
-};
-
-
-var matched, browser,
-       oldInit = jQuery.fn.init,
-       oldParseJSON = jQuery.parseJSON,
-       // Note: XSS check is done below after string is trimmed
-       rquickExpr = /^([^<]*)(<[\w\W]+>)([^>]*)$/;
-
-// $(html) "looks like html" rule change
-jQuery.fn.init = function( selector, context, rootjQuery ) {
-       var match;
-
-       if ( selector && typeof selector === "string" && !jQuery.isPlainObject( context ) &&
-                       (match = rquickExpr.exec( jQuery.trim( selector ) )) && match[ 0 ] ) {
-               // This is an HTML string according to the "old" rules; is it still?
-               if ( selector.charAt( 0 ) !== "<" ) {
-                       migrateWarn("$(html) HTML strings must start with '<' character", "create-html");
-               }
-               if ( match[ 3 ] ) {
-                       migrateWarn("$(html) HTML text after last tag is ignored", "create-html");
-               }
-               // Consistently reject any HTML-like string starting with a hash (#9521)
-               // Note that this may break jQuery 1.6.x code that otherwise would work.
-               if ( match[ 0 ].charAt( 0 ) === "#" ) {
-                       migrateWarn("HTML string cannot start with a '#' character", "create-html");
-                       jQuery.error("JQMIGRATE: Invalid selector string (XSS)");
-               }
-               // Now process using loose rules; let pre-1.8 play too
-               if ( context && context.context ) {
-                       // jQuery object as context; parseHTML expects a DOM object
-                       context = context.context;
-               }
-               if ( jQuery.parseHTML ) {
-                       return oldInit.call( this, jQuery.parseHTML( match[ 2 ], context, true ),
-                                       context, rootjQuery );
-               }
-       }
-       return oldInit.apply( this, arguments );
-};
-jQuery.fn.init.prototype = jQuery.fn;
-
-// Let $.parseJSON(falsy_value) return null
-jQuery.parseJSON = function( json ) {
-       if ( !json && json !== null ) {
-               migrateWarn("jQuery.parseJSON requires a valid JSON string", "json-invalid");
-               return null;
-       }
-       return oldParseJSON.apply( this, arguments );
-};
-
-jQuery.uaMatch = function( ua ) {
-       ua = ua.toLowerCase();
-
-       var match = /(chrome)[ \/]([\w.]+)/.exec( ua ) ||
-               /(webkit)[ \/]([\w.]+)/.exec( ua ) ||
-               /(opera)(?:.*version|)[ \/]([\w.]+)/.exec( ua ) ||
-               /(msie) ([\w.]+)/.exec( ua ) ||
-               ua.indexOf("compatible") < 0 && /(mozilla)(?:.*? rv:([\w.]+)|)/.exec( ua ) ||
-               [];
-
-       return {
-               browser: match[ 1 ] || "",
-               version: match[ 2 ] || "0"
-       };
-};
-
-// Don't clobber any existing jQuery.browser in case it's different
-if ( !jQuery.browser ) {
-       matched = jQuery.uaMatch( navigator.userAgent );
-       browser = {};
-
-       if ( matched.browser ) {
-               browser[ matched.browser ] = true;
-               browser.version = matched.version;
-       }
-
-       // Chrome is Webkit, but Webkit is also Safari.
-       if ( browser.chrome ) {
-               browser.webkit = true;
-       } else if ( browser.webkit ) {
-               browser.safari = true;
-       }
-
-       jQuery.browser = browser;
-}
-
-// Warn if the code tries to get jQuery.browser
-migrateWarnProp( jQuery, "browser", jQuery.browser, "jQuery.browser is deprecated" );
-
-jQuery.sub = function() {
-       function jQuerySub( selector, context ) {
-               return new jQuerySub.fn.init( selector, context );
-       }
-       jQuery.extend( true, jQuerySub, this );
-       jQuerySub.superclass = this;
-       jQuerySub.fn = jQuerySub.prototype = this();
-       jQuerySub.fn.constructor = jQuerySub;
-       jQuerySub.sub = this.sub;
-       jQuerySub.fn.init = function init( selector, context ) {
-               if ( context && context instanceof jQuery && !(context instanceof jQuerySub) ) {
-                       context = jQuerySub( context );
-               }
-
-               return jQuery.fn.init.call( this, selector, context, rootjQuerySub );
-       };
-       jQuerySub.fn.init.prototype = jQuerySub.fn;
-       var rootjQuerySub = jQuerySub(document);
-       migrateWarn( "jQuery.sub() is deprecated", "sub" );
-       return jQuerySub;
-};
-
-
-// Ensure that $.ajax gets the new parseJSON defined in core.js
-jQuery.ajaxSetup({
-       converters: {
-               "text json": jQuery.parseJSON
-       }
-});
-
-
-var oldFnData = jQuery.fn.data;
-
-jQuery.fn.data = function( name ) {
-       var ret, evt,
-               elem = this[0];
-
-       // Handles 1.7 which has this behavior and 1.8 which doesn't
-       if ( elem && name === "events" && arguments.length === 1 ) {
-               ret = jQuery.data( elem, name );
-               evt = jQuery._data( elem, name );
-               if ( ( ret === undefined || ret === evt ) && evt !== undefined ) {
-                       migrateWarn("Use of jQuery.fn.data('events') is deprecated", "data-events");
-                       return evt;
-               }
-       }
-       return oldFnData.apply( this, arguments );
-};
-
-
-var rscriptType = /\/(java|ecma)script/i,
-       oldSelf = jQuery.fn.andSelf || jQuery.fn.addBack;
-
-jQuery.fn.andSelf = function() {
-       migrateWarn("jQuery.fn.andSelf() replaced by jQuery.fn.addBack()", "andSelf");
-       return oldSelf.apply( this, arguments );
-};
-
-// Since jQuery.clean is used internally on older versions, we only shim if it's missing
-if ( !jQuery.clean ) {
-       jQuery.clean = function( elems, context, fragment, scripts ) {
-               // Set context per 1.8 logic
-               context = context || document;
-               context = !context.nodeType && context[0] || context;
-               context = context.ownerDocument || context;
-
-               migrateWarn("jQuery.clean() is deprecated", "clean");
-
-               var i, elem, handleScript, jsTags,
-                       ret = [];
-
-               jQuery.merge( ret, jQuery.buildFragment( elems, context ).childNodes );
-
-               // Complex logic lifted directly from jQuery 1.8
-               if ( fragment ) {
-                       // Special handling of each script element
-                       handleScript = function( elem ) {
-                               // Check if we consider it executable
-                               if ( !elem.type || rscriptType.test( elem.type ) ) {
-                                       // Detach the script and store it in the scripts array (if provided) or the fragment
-                                       // Return truthy to indicate that it has been handled
-                                       return scripts ?
-                                               scripts.push( elem.parentNode ? elem.parentNode.removeChild( elem ) : elem ) :
-                                               fragment.appendChild( elem );
-                               }
-                       };
-
-                       for ( i = 0; (elem = ret[i]) != null; i++ ) {
-                               // Check if we're done after handling an executable script
-                               if ( !( jQuery.nodeName( elem, "script" ) && handleScript( elem ) ) ) {
-                                       // Append to fragment and handle embedded scripts
-                                       fragment.appendChild( elem );
-                                       if ( typeof elem.getElementsByTagName !== "undefined" ) {
-                                               // handleScript alters the DOM, so use jQuery.merge to ensure snapshot iteration
-                                               jsTags = jQuery.grep( jQuery.merge( [], elem.getElementsByTagName("script") ), handleScript );
-
-                                               // Splice the scripts into ret after their former ancestor and advance our index beyond them
-                                               ret.splice.apply( ret, [i + 1, 0].concat( jsTags ) );
-                                               i += jsTags.length;
-                                       }
-                               }
-                       }
-               }
-
-               return ret;
-       };
-}
-
-var eventAdd = jQuery.event.add,
-       eventRemove = jQuery.event.remove,
-       eventTrigger = jQuery.event.trigger,
-       oldToggle = jQuery.fn.toggle,
-       oldLive = jQuery.fn.live,
-       oldDie = jQuery.fn.die,
-       ajaxEvents = "ajaxStart|ajaxStop|ajaxSend|ajaxComplete|ajaxError|ajaxSuccess",
-       rajaxEvent = new RegExp( "\\b(?:" + ajaxEvents + ")\\b" ),
-       rhoverHack = /(?:^|\s)hover(\.\S+|)\b/,
-       hoverHack = function( events ) {
-               if ( typeof( events ) !== "string" || jQuery.event.special.hover ) {
-                       return events;
-               }
-               if ( rhoverHack.test( events ) ) {
-                       migrateWarn("'hover' pseudo-event is deprecated, use 'mouseenter mouseleave'", "event-hover");
-               }
-               return events && events.replace( rhoverHack, "mouseenter$1 mouseleave$1" );
-       };
-
-// Event props removed in 1.9, put them back if needed; no practical way to warn them
-if ( jQuery.event.props && jQuery.event.props[ 0 ] !== "attrChange" ) {
-       jQuery.event.props.unshift( "attrChange", "attrName", "relatedNode", "srcElement" );
-}
-
-// Undocumented jQuery.event.handle was "deprecated" in jQuery 1.7
-if ( jQuery.event.dispatch ) {
-       migrateWarnProp( jQuery.event, "handle", jQuery.event.dispatch, "jQuery.event.handle is undocumented and deprecated", "event-handle" );
-}
-
-// Support for 'hover' pseudo-event and ajax event warnings
-jQuery.event.add = function( elem, types, handler, data, selector ){
-       if ( elem !== document && rajaxEvent.test( types ) ) {
-               migrateWarn( "AJAX events should be attached to document: " + types, "event-ajax" );
-       }
-       eventAdd.call( this, elem, hoverHack( types || "" ), handler, data, selector );
-};
-jQuery.event.remove = function( elem, types, handler, selector, mappedTypes ){
-       eventRemove.call( this, elem, hoverHack( types ) || "", handler, selector, mappedTypes );
-};
-
-jQuery.fn.error = function() {
-       var args = Array.prototype.slice.call( arguments, 0);
-       migrateWarn("jQuery.fn.error() is deprecated", "bind-error");
-       args.splice( 0, 0, "error" );
-       if ( arguments.length ) {
-               return this.bind.apply( this, args );
-       }
-       // error event should not bubble to window, although it does pre-1.7
-       this.triggerHandler.apply( this, args );
-       return this;
-};
-
-jQuery.fn.toggle = function( fn, fn2 ) {
-
-       // Don't mess with animation or css toggles
-       if ( !jQuery.isFunction( fn ) || !jQuery.isFunction( fn2 ) ) {
-               return oldToggle.apply( this, arguments );
-       }
-       migrateWarn("jQuery.fn.toggle(handler, handler...) is deprecated", "toggle-handle");
-
-       // Save reference to arguments for access in closure
-       var args = arguments,
-               guid = fn.guid || jQuery.guid++,
-               i = 0,
-               toggler = function( event ) {
-                       // Figure out which function to execute
-                       var lastToggle = ( jQuery._data( this, "lastToggle" + fn.guid ) || 0 ) % i;
-                       jQuery._data( this, "lastToggle" + fn.guid, lastToggle + 1 );
-
-                       // Make sure that clicks stop
-                       event.preventDefault();
-
-                       // and execute the function
-                       return args[ lastToggle ].apply( this, arguments ) || false;
-               };
-
-       // link all the functions, so any of them can unbind this click handler
-       toggler.guid = guid;
-       while ( i < args.length ) {
-               args[ i++ ].guid = guid;
-       }
-
-       return this.click( toggler );
-};
-
-jQuery.fn.live = function( types, data, fn ) {
-       migrateWarn("jQuery.fn.live() is deprecated", "live");
-       if ( oldLive ) {
-               return oldLive.apply( this, arguments );
-       }
-       jQuery( this.context ).on( types, this.selector, data, fn );
-       return this;
-};
-
-jQuery.fn.die = function( types, fn ) {
-       migrateWarn("jQuery.fn.die() is deprecated", "die");
-       if ( oldDie ) {
-               return oldDie.apply( this, arguments );
-       }
-       jQuery( this.context ).off( types, this.selector || "**", fn );
-       return this;
-};
-
-// Turn global events into document-triggered events
-jQuery.event.trigger = function( event, data, elem, onlyHandlers  ){
-       if ( !elem && !rajaxEvent.test( event ) ) {
-               migrateWarn( "Global events are undocumented and deprecated", "event-global" );
-       }
-       return eventTrigger.call( this,  event, data, elem || document, onlyHandlers  );
-};
-jQuery.each( ajaxEvents.split("|"),
-       function( _, name ) {
-               jQuery.event.special[ name ] = {
-                       setup: function() {
-                               var elem = this;
-
-                               // The document needs no shimming; must be !== for oldIE
-                               if ( elem !== document ) {
-                                       jQuery.event.add( document, name + "." + jQuery.guid, function() {
-                                               jQuery.event.trigger( name, null, elem, true );
-                                       });
-                                       jQuery._data( this, name, jQuery.guid++ );
-                               }
-                               return false;
-                       },
-                       teardown: function() {
-                               if ( this !== document ) {
-                                       jQuery.event.remove( document, name + "." + jQuery._data( this, name ) );
-                               }
-                               return false;
-                       }
-               };
-       }
-);
-
-
-})( jQuery, window );