Skip to content
Permalink
Browse files

Remove block notice tracking

This is a clean up after collecting the necessary data related to
blocks and how often users see the block notices

See: https://phabricator.wikimedia.org/T189724

Bug: T214214
Change-Id: I047587c064e63e8bd4b933351edfec298f7c7956
  • Loading branch information...
dayllanmaza committed Apr 23, 2019
1 parent bfb9ce9 commit eebdabe65c067dbcfd9ddf59ab283ddcdba61b40
Showing with 0 additions and 24 deletions.
  1. +0 −8 README.md
  2. +0 −2 includes/MobileFrontendHooks.php
  3. +0 −14 src/mobile.editor.overlay/BlockMessage.js
@@ -295,14 +295,6 @@ risk!
* Type: `Boolean`
* Default: `false`

#### $wgMFTrackBlockNotices

If set to true Block Notices will be tracked on display and a count will be stored
for later analysis. No personal data will be collected.

* Type: `Boolean`
* Default: `false`

#### $wgMFMobileHeader

Requests containing header with this name will be considered as coming from
@@ -424,8 +424,6 @@ public static function getResourceLoaderMFConfigVars() {
// and signal to API requests that the origin parameter should be used.
// A boolean would also suffice here but let's keep things simple and pass verbatim
'wgMFContentProviderScriptPath' => $config->get( 'MFContentProviderScriptPath' ),
// BlockMessage.js
'wgEnableBlockNoticeStats' => $config->get( 'EnableBlockNoticeStats' ),
];
return $vars;
}
@@ -59,20 +59,6 @@ mfExtend( BlockMessage, Drawer, {
button: Button.prototype.template,
icon: Icon.prototype.template
} ),
/**
* @inheritdoc
* @memberof BlockMessage
* @instance
*/
onShowDrawer: function () {
var wiki = mw.config.get( 'wgDBname' );

Drawer.prototype.onShowDrawer.apply( this );

if ( mw.config.get( 'wgEnableBlockNoticeStats' ) ) {
mw.track( 'counter.MediaWiki.BlockNotices.' + wiki + '.MobileFrontend.shown', 1 );
}
},
/**
* @memberof BlockMessage
* @instance

0 comments on commit eebdabe

Please sign in to comment.
You can’t perform that action at this time.