jenkins-bot has submitted this change and it was merged.
Change subject: Log no-search-results event
......................................................................
Log no-search-results event
Change-Id: Ia272a6783d56c75ba775d71195104aad21214121
---
M resources/js/ext.uls.eventlogger.js
1 file changed, 19 insertions(+), 4 deletions(-)
Approvals:
Amire80: Looks good to me, approved
jenkins-bot: Verified
diff --git a/resources/js/ext.uls.eventlogger.js
b/resources/js/ext.uls.eventlogger.js
index 8a65935..abb0afb 100644
--- a/resources/js/ext.uls.eventlogger.js
+++ b/resources/js/ext.uls.eventlogger.js
@@ -92,16 +92,19 @@
mw.hook( 'mw.uls.interface.morelanguages' ).add(
$.proxy( this.interfaceMoreLanguages, this ) );
mw.hook( 'mw.uls.interface.language.change' ).add(
$.proxy( this.interfaceLanguageChange, this ) );
mw.hook( 'mw.uls.font.change' ).add( $.proxy(
this.fontChange, this ) );
+ $( 'body' ).on( 'noresults.uls', '.uls-menu
.languagefilter',
+ $.proxy( this.noSearchResults, this )
+ );
},
/**
* Log language settings open
- * @param {Array} args
+ * @param {string} context Where it was opened from
*/
- ulsSettingsOpen: function ( args ) {
+ ulsSettingsOpen: function ( context ) {
this.log( {
action: 'settings-open',
- context: args
+ context: context
} );
},
@@ -179,7 +182,7 @@
/**
* Log font preference changes
*
- * @param {Object} context Either 'interface' or 'content'
+ * @param {string} context Either 'interface' or 'content'
* @param {string} language
* @param {string} font
*/
@@ -200,6 +203,18 @@
}
this.log( logParams );
+ },
+
+ /**
+ * Log search strings which produce no search results.
+ * @param {jQuery.event} event The orignal event
+ * @param {string} context The query string
+ */
+ noSearchResults: function ( event, context ) {
+ this.log( {
+ action: 'no-search-results',
+ context: context
+ } );
}
};
--
To view, visit https://gerrit.wikimedia.org/r/73953
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia272a6783d56c75ba775d71195104aad21214121
Gerrit-PatchSet: 4
Gerrit-Project: mediawiki/extensions/UniversalLanguageSelector
Gerrit-Branch: master
Gerrit-Owner: Nikerabbit <[email protected]>
Gerrit-Reviewer: Amire80 <[email protected]>
Gerrit-Reviewer: Nikerabbit <[email protected]>
Gerrit-Reviewer: Santhosh <[email protected]>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
