Last modified: 2014-02-12 23:53:43 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T47392, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 45392 - EL doesn't log user IDs on non-enwiki projects
EL doesn't log user IDs on non-enwiki projects
Status: RESOLVED FIXED
Product: MobileFrontend
Classification: Unclassified
stable (Other open bugs)
unspecified
All All
: Unprioritized normal
: ---
Assigned To: Jon
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-02-25 22:51 UTC by Maryana Pinchuk
Modified: 2014-02-12 23:53 UTC (History)
15 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Maryana Pinchuk 2013-02-25 22:51:53 UTC
Looking at the data coming in from https://meta.wikimedia.org/wiki/Schema:MobileBetaWatchlist, it appears that non-enwiki users are not being assigned event_userId. To quote Matt Flaschen: "It's probably time to move wgUserId into the core JavaScript interface." :)
Comment 1 Ori Livneh 2013-02-25 22:53:26 UTC
This is properly a MobileFrontend bug. EventLogging doesn't know about user IDs.
Comment 2 Matthew Flaschen 2013-02-25 22:56:56 UTC
Also, as a general note, we should consider using the extension name in extension-defined keys, so these sorts of dependencies are obvious.  We ran into this with PostEdit too.

Consider it a vendor prefix.
Comment 3 Maryana Pinchuk 2013-02-27 02:05:51 UTC
An easy solution from the data analyst perspective: for MobileFrontend schemas, log username rather than user ID, since it does appear to work globally (tested on tawiki and plwiki). We should replace the event_userId field in our upload error schema (https://meta.wikimedia.org/wiki/Schema:MobileWebUploads) with event_username, so we can weed out staff uploads and reliably measure unique users' uploads globally.
Comment 4 Maryana Pinchuk 2013-02-27 02:07:19 UTC
An easy solution from the data analyst perspective: for MobileFrontend schemas, log username rather than user ID, since it does appear to work globally (tested on tawiki and plwiki). We should replace the event_userId field in our upload error schema (https://meta.wikimedia.org/wiki/Schema:MobileWebUploads) with event_username, so we can weed out staff uploads and reliably measure unique users' uploads globally.
Comment 5 Maryana Pinchuk 2013-02-27 02:13:35 UTC
Durr, we're not currently logging userId in Schema:MobileWebUploads. I meant we should change it to username on Schema:MobileBetaWatchlist and add it to Schema:MobileWebUploads. Sorry, long day :)
Comment 6 Ori Livneh 2013-02-27 05:03:23 UTC
Gerrit change Id0adb795567d4ba932104938f2793cdb4084883c adds the variable to core. Until that is deployed, Gerrit change Ie9daf89c0acadbe24f95d7d6f3dded5d82af109e adds the variable in MobileFrontend. Whether or not you need the latter is up to you.
Comment 7 Jon 2013-02-27 23:19:10 UTC
https://gerrit.wikimedia.org/r/51292
Comment 8 Matthew Flaschen 2013-02-27 23:27:05 UTC
Id0adb795567d4ba932104938f2793cdb4084883c (same one Ori mentioned above), which adds wgUserId to core (among other things) is merged.

Given that, up to MobileFrontend whether to use user ID or username.
Comment 9 Jon 2013-03-01 21:49:54 UTC
Changes got merged.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links