EBernhardson has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/247894

Change subject: Further cleanup of ext.wikimediaEvents.search.js
......................................................................

Further cleanup of ext.wikimediaEvents.search.js

These changes were requested in review of I846752cef

Change-Id: Ibb8c8ada6fb86a1832c0d24dbbac7ce2547c1cad
---
M modules/ext.wikimediaEvents.search.js
1 file changed, 8 insertions(+), 11 deletions(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/WikimediaEvents 
refs/changes/94/247894/1

diff --git a/modules/ext.wikimediaEvents.search.js 
b/modules/ext.wikimediaEvents.search.js
index 0edd61b..f2cb2a6 100644
--- a/modules/ext.wikimediaEvents.search.js
+++ b/modules/ext.wikimediaEvents.search.js
@@ -19,22 +19,19 @@
                return;
        }
 
-       depsPromise = mw.loader.using( [
-               'schema.Search',
-               'ext.eventLogging'
-       ] );
-
-       defaults = {
-               platform: 'desktop',
-               userSessionToken: getRandomToken(),
-               searchSessionToken: getRandomToken()
-       };
-
        mw.trackSubscribe( 'mediawiki.searchSuggest', function ( topic, data ) {
                var loggingData = {
                        action: data.action
                };
 
+               depsPromise = depsPromise || mw.loader.using( [ 'schema.Search' 
] );
+
+               defaults = defaults || {
+                       platform: 'desktop',
+                       userSessionToken: getRandomToken(),
+                       searchSessionToken: getRandomToken()
+               };
+
                if ( data.action === 'session-start' ) {
                        // update session token if it's a new search
                        defaults.searchSessionToken = getRandomToken();

-- 
To view, visit https://gerrit.wikimedia.org/r/247894
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibb8c8ada6fb86a1832c0d24dbbac7ce2547c1cad
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WikimediaEvents
Gerrit-Branch: master
Gerrit-Owner: EBernhardson <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to