Last modified: 2013-02-09 03:46:58 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 T38053, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 36053 - [Regression] Log in "returnto" feature forgets query parameters if no title parameter was specified
[Regression] Log in "returnto" feature forgets query parameters if no title p...
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
General/Unknown (Other open bugs)
1.19
All All
: High normal (vote)
: 1.21.0 release
Assigned To: Marius Hoch
: code-update-regression
: 38251 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-04-18 01:05 UTC by Krinkle
Modified: 2013-02-09 03:46 UTC (History)
6 users (show)

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


Attachments

Description Krinkle 2012-04-18 01:05:52 UTC
I thought I recorded this bug already can't find it. This behavior was okay up until 1.18 and since 1.19wmf1 was deployed, it is breaking.

== Steps to reproduce ==
* When logged out, visit a url to an existing view but without a title query parameter, for example:
- https://nl.wikipedia.org/?curid=2788608&redirect=no
- https://en.wikipedia.org/?diff=123

* Click Login in the top left and continue the login procedure

== Expected ==
The "Back to ..." link should go to where I was

== Actual ==
The link says "Main page" and goes to /wiki/Main_Page, no query at all.

I could understand if it assumes the title 'Main Page' if none was found in the originating url (although even that is somewhat unneeded, since MediaWiki knows the page title by the time the page is rendered, based on oldid and/or pageid). But instead of it becoming ?title=Some_page&originalquery or ?originalquery, it becomes /wiki/Main_Page.
Comment 1 Krinkle 2012-07-08 19:53:27 UTC
*** Bug 38251 has been marked as a duplicate of this bug. ***
Comment 2 Tyler Romeo 2012-08-06 15:25:47 UTC
There are two separate reasons this bug occurs. The first is that if no title is specified, then SkinTemplate::buildPersonalUrls() will not put any returntoquery, even if there is a query to return to. This is easily fixable.

The second issue is that LoginForm::successfulLogin() requires a returnto value in order to build the redirect URL. This is because it calls Title::getFullURL() on the returnto title. To get around this, there needs to be a way to build a full URL without having a valid title object.
Comment 3 Mark A. Hershberger 2012-09-30 18:12:38 UTC
no code yet, so pushing to future.
Comment 4 Marius Hoch 2012-10-23 23:22:31 UTC
Fixed in https://gerrit.wikimedia.org/r/29729

Waiting for review...
Comment 5 Brad Jorsch 2012-11-20 13:46:55 UTC
Change merged.

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


Navigation
Links