Legoktm has uploaded a new change for review.
https://gerrit.wikimedia.org/r/94487
Change subject: Make MobileWebEditing logging code support anons
......................................................................
Make MobileWebEditing logging code support anons
Bug: 56833
Change-Id: Ida9de036f5b177ac04d11126a8eff9c4097cd9e8
---
M javascripts/modules/editor/EditorOverlay.js
1 file changed, 5 insertions(+), 2 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MobileFrontend
refs/changes/87/94487/1
diff --git a/javascripts/modules/editor/EditorOverlay.js
b/javascripts/modules/editor/EditorOverlay.js
index 51347a3..a5d6a2c 100644
--- a/javascripts/modules/editor/EditorOverlay.js
+++ b/javascripts/modules/editor/EditorOverlay.js
@@ -43,10 +43,8 @@
action: action,
section: this.sectionId,
namespace: mw.config.get(
'wgNamespaceNumber' ),
- userEditCount: mw.config.get(
'wgUserEditCount' ),
isTestA: M.isTestA,
pageId: mw.config.get( 'wgArticleId' ),
- username: mw.config.get( 'wgUserName' ),
mobileMode: mw.config.get( 'wgMFMode' ),
userAgent: window.navigator.userAgent,
funnel: this.funnel
@@ -54,6 +52,11 @@
if ( errorText ) {
data.errorText = errorText;
}
+ if ( mw.config.get( 'wgUserName' ) ) {
+ // These should only be logged for logged in
users
+ data.username = mw.config.get( 'wgUserName' );
+ data.userEditCount = mw.config.get(
'wgUserEditCount' );
+ }
M.log( 'MobileWebEditing', data );
},
--
To view, visit https://gerrit.wikimedia.org/r/94487
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ida9de036f5b177ac04d11126a8eff9c4097cd9e8
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Legoktm <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
