Last modified: 2013-05-03 02:52:32 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 T49590, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 47590 - isGettingStarted function returns true for account creations regardless of whether they were served GettingStarted
isGettingStarted function returns true for account creations regardless of wh...
Status: RESOLVED WORKSFORME
Product: MediaWiki extensions
Classification: Unclassified
GettingStarted (Other open bugs)
unspecified
All All
: High blocker (vote)
: ---
Assigned To: Matthew Flaschen
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-04-24 04:47 UTC by Steven Walling
Modified: 2013-05-03 02:52 UTC (History)
5 users (show)

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


Attachments

Description Steven Walling 2013-04-24 04:47:57 UTC
For previous split tests, we incorrectly logged views of the control landing page (the MediaWiki default welcome creation) as gettingstarted-impression, giving the false, uh... impression that control users had been served the test condition.
Comment 1 spage 2013-04-25 03:34:43 UTC
GettingStarted.hooks.php sets a flag $isWelcomeCreation if the user has just created an account, regardless of what page the user is shown.  ext.gettingstarted.openTask.js uses it for isNew (good), but also uses it to determine isGettingStarted().

This predates the JS code that switches the browser history state from create account form to Special:GettingStarted.  So I think isGettingStarted() doesn't need to check cfg.wgIsWelcomeCreation any more. (If necessary it could check for something defined in ext.gettingstarted.accountcreation.js, which is only sent to the browser if it's the Special:GS page.)
Comment 2 spage 2013-04-25 04:03:33 UTC
To MediaWiki the page the user is seeing is still Userlogin, so you do have to check for something. Easiest (but fragile) to look for the ext.gettingstarted.accountcreation module, patch forthcoming.
Comment 3 Gerrit Notification Bot 2013-04-25 04:04:26 UTC
Related URL: https://gerrit.wikimedia.org/r/60787 (Gerrit Change Iaff9f7226aa77b0e8b8abb8c32b549f64cc42d8a)
Comment 4 Matthew Flaschen 2013-05-03 02:52:32 UTC
As part of the https://gerrit.wikimedia.org/r/#/c/59575/ restructuring, I removed the isGettingStarted function entirely.

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


Navigation
Links