From: Timo Tijhof Date: Fri, 18 Aug 2017 01:20:40 +0000 (-0700) Subject: jquery.migrate: Instrument with mw.track() X-Git-Tag: 1.31.0-rc.0~2343^2 X-Git-Url: http://git.cyclocoop.org/%27.parametre_url%28%20%20%20generer_action_auteur%28%27charger_plugin%27%2C%20%27update_flux%27%29%2C%27update_flux%27%2C%20%27oui%27%29.%27?a=commitdiff_plain;h=4107d1d4df7874bb1a73211582754ab4f17b7b7a;p=lhc%2Fweb%2Fwiklou.git jquery.migrate: Instrument with mw.track() Inspired by cb20b5aa5a68d, which added similar instrumentation during the jQuery 1.11 migration for MediaWiki in 2014. This way we can plot overall usage in a Grafana dashboard and help us track the progres of the on-going migration. Bug: T124742 Change-Id: Ifab13d52d5681189cfb2faae342373558369b8c0 --- diff --git a/resources/lib/jquery/jquery.migrate.js b/resources/lib/jquery/jquery.migrate.js index f094aaa2cc..12212ffcfb 100644 --- a/resources/lib/jquery/jquery.migrate.js +++ b/resources/lib/jquery/jquery.migrate.js @@ -6,6 +6,7 @@ * - Preserve handler of uncaught exceptions in promise chains * https://gerrit.wikimedia.org/r/#/c/360999/ * https://github.com/jquery/jquery-migrate/pull/262 + * - Add mw.track instrumentation for statistics. */ ;( function( factory ) { if ( typeof define === "function" && define.amd ) { @@ -81,6 +82,8 @@ function migrateWarn( msg ) { if ( !warnedAbout[ msg ] ) { warnedAbout[ msg ] = true; jQuery.migrateWarnings.push( msg ); + // PATCH: Add instrumentation for statistics --Krinkle + mw.track( "mw.deprecate", "jquery-migrate" ); if ( console && console.warn && !jQuery.migrateMute ) { console.warn( "JQMIGRATE: " + msg ); if ( jQuery.migrateTrace && console.trace ) {