ext.uls.eventlogger: Remove unused $.Callbacks() indirection
It is created, unconditionally fired only once (right after creation), and then used only with add(), which becomes a synchronous self-call. I've searched in Wikimedia Git and Codesearch, and found no uses of `logEventQueue` or `mw.uls.eventlogger` outside this file. Change-Id: I3832bcb409fc919c2078e583d50e20413dede163
This commit is contained in:
@@ -27,23 +27,18 @@
|
||||
* @since 2013.08
|
||||
*/
|
||||
function ULSEventLogger() {
|
||||
this.logEventQueue = $.Callbacks( 'memory once' );
|
||||
this.init();
|
||||
this.listen();
|
||||
}
|
||||
|
||||
ULSEventLogger.prototype = {
|
||||
init: function () {
|
||||
var eventLogger = this;
|
||||
|
||||
mw.eventLog.setDefaults( 'UniversalLanguageSelector', {
|
||||
version: 1,
|
||||
token: mw.user.id(),
|
||||
contentLanguage: mw.config.get( 'wgContentLanguage' ),
|
||||
interfaceLanguage: mw.config.get( 'wgUserLanguage' )
|
||||
} );
|
||||
|
||||
eventLogger.logEventQueue.fire();
|
||||
},
|
||||
|
||||
/**
|
||||
@@ -62,11 +57,9 @@
|
||||
|
||||
schema = schema || 'UniversalLanguageSelector';
|
||||
|
||||
this.logEventQueue.add( function () {
|
||||
mw.eventLog.logEvent( schema, event )
|
||||
.done( deferred.resolve )
|
||||
.fail( deferred.reject );
|
||||
} );
|
||||
|
||||
return deferred.promise();
|
||||
},
|
||||
|
||||
Reference in New Issue
Block a user