Last modified: 2010-03-09 15:41:18 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 T21946, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 19946 - Articles with forward slash in Wikipedia Mobile
Articles with forward slash in Wikipedia Mobile
Status: RESOLVED FIXED
Product: Wikimedia Mobile
Classification: Unclassified
Retired (Ruby) Gateway (Other open bugs)
.5
All All
: Normal normal (vote)
: ---
Assigned To: Hampton Catlin
http://en.m.wikipedia.org/wiki//dev/r...
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-07-26 17:25 UTC by Michael
Modified: 2010-03-09 15:41 UTC (History)
2 users (show)

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


Attachments

Description Michael 2009-07-26 17:25:03 UTC
In the example above, using the mobile site brings up a search page for the article, rather than the article itself. Clicking on the article merely appears to refresh the page.
Comment 1 Hampton Catlin 2009-07-26 18:04:16 UTC
I have done some initial investigation on this bug and it seems
that it is related to the way that URLs in the Merb framework
are interpreted. It just assumes that a double slash is a mistake.
Which, isn't always a bad assumption. However, in this case it bites
us in the ass.

Comment 2 Derk-Jan Hartman 2009-07-26 18:36:38 UTC
It likely changes the // internally to /./
Comment 3 Derk-Jan Hartman 2009-10-19 15:38:04 UTC
#merb

<+merboutpost> thedj: AH :-)
<+merboutpost> thedj: hm...
<+merboutpost> thedj: interesting, that's the router behaviour, feel free to file the Ticket in the lighthouse. It will take me some time to change that. I remember I've seen some similar ticket or conversation.
<+merboutpost> thedj: I remember that the result was: Not easy to fix :-)

No time to figure out how their OpenID works, and don't feel like signing up for Yet Another Bugreporting system.

https://merb.lighthouseapp.com/
Comment 5 Hampton Catlin 2010-03-09 15:41:18 UTC
Fixed this with some custom code in the show action. Only fixed for double slashes at the beginning of articles. Any double slash in the middle will break, but I can't think of any examples of that, so we will consider this closed for the moment.

PS: please don't just arbitrarily create an article with two slashes. ;)

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


Navigation
Links