From 1993c51a49cde5e39c684fa6419fb633a886f3e8 Mon Sep 17 00:00:00 2001 From: "James D. Forrester" Date: Mon, 22 Aug 2016 11:28:22 -0700 Subject: [PATCH] Follow-up 51dd790 & c40070b: Drop T135171 debug tracking Apparently this is now fixed, so no need to keep this. Bug: T135171 Change-Id: Ie5c0f9140c6de4bb73770276011adce23fa69f9e --- ApiVisualEditor.php | 3 --- modules/ve-mw/init/ve.init.mw.ArticleTargetLoader.js | 3 --- 2 files changed, 6 deletions(-) diff --git a/ApiVisualEditor.php b/ApiVisualEditor.php index 799db4958b..7c774b5c80 100644 --- a/ApiVisualEditor.php +++ b/ApiVisualEditor.php @@ -164,9 +164,6 @@ class ApiVisualEditor extends ApiBase { if ( $parserParams['oldid'] ) { $path .= '/' . $parserParams['oldid']; } - if ( !is_string( $etag ) || $etag === '' ) { - wfDebugLog( 'AdHocDebug', 'VisualEditor T135171 - bad etag: ' . var_export( $etag, true ) ); - } return $this->requestRestbase( 'POST', $path, diff --git a/modules/ve-mw/init/ve.init.mw.ArticleTargetLoader.js b/modules/ve-mw/init/ve.init.mw.ArticleTargetLoader.js index c1ded971e2..c3b9bc0055 100644 --- a/modules/ve-mw/init/ve.init.mw.ArticleTargetLoader.js +++ b/modules/ve-mw/init/ve.init.mw.ArticleTargetLoader.js @@ -196,9 +196,6 @@ duration: ve.now() - start, targetName: targetName } ); - mw.track( 'event.VET135171', { - msg: 'getResponseHeader: ' + jqxhr.getResponseHeader( 'etag' ) - } ); return [ data, jqxhr.getResponseHeader( 'etag' ) ]; }, function ( response ) {