Last modified: 2009-03-21 19:42:01 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 T19799, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 17799 - Database error on [[Special:Random/redirect]]
Database error on [[Special:Random/redirect]]
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Special pages (Other open bugs)
1.15.x
All All
: Normal enhancement with 1 vote (vote)
: ---
Assigned To: Nobody - You can work on this!
:
: 18065 (view as bug list)
Depends on:
Blocks: 16560
  Show dependency treegraph
 
Reported: 2009-03-05 13:23 UTC by Happy-melon
Modified: 2009-03-21 19:42 UTC (History)
2 users (show)

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


Attachments

Description Happy-melon 2009-03-05 13:23:36 UTC
from within function "RandomPage::selectRandomPageFromDB". MySQL returned error "1064: You have an error in your SQL syntax. Check the manual that corresponds to your MySQL server version for the right syntax to use near ') AND page_is_redirect = 0 AND page_random >= 0.987298690 (10.0.6.22)".

This corresponds to lines 94-98 of trunk/phase3/specials/SpecialRandomPage.php; it's probably a result of r45467, applied to fix bug16560.

Interestingly, [[Special:RandomRedirect]] still works.
Comment 1 Chad H. 2009-03-05 13:31:49 UTC
This is a generic issue. Apparently Special:Random dies if you feed it any non-namespace. And afaik, Special:Random/redirect has never been a redirect to Special:RandomRedirect.
Comment 2 Chad H. 2009-03-05 13:39:03 UTC
Fixed in r48059. Do we want to make Special:Random/redirect go to Special:RandomRedirect? Would seem sensible.
Comment 3 Happy-melon 2009-03-05 13:54:08 UTC
As best I can tell, RandomRedirect is actually a bizzarre internal redirect to Random/redirect: it's a derived class that essentially just changes the structure of RandomPage just enough to make it always return redirects, which is wierd, to say the least.  But in the interests of ease of future modification, it would be nice to have it as a pseudo-separate page; at the very least we shouldn't have two similar-but-subtly-different ways of getting (almost) the same outcome. 
Comment 4 Chad H. 2009-03-05 13:55:29 UTC
It's not a redirect, it's a subclass that specifies that the article chosen must be a redirect. Special:Random/redirect does nothing (now).
Comment 5 Happy-melon 2009-03-05 14:55:22 UTC
OH, I see the issue, and the source of the original error.  It was trying to pick a random page from the Redirect: namespace?  Ignore comment #3 then, ravings of a madman :D

In that case, I'd be disinclined to redirect it, it would create a problem for anyone that wanted to *actually have* a Redirect: namespace for whatever reason, and it's unnecessary given that there's a dedicated special page for it.
Comment 6 Chad H. 2009-03-05 14:57:18 UTC
True about the Redirect: namespace, hadn't considered that. Marking as FIXED, then.
Comment 7 Alexandre Emsenhuber [IAlex] 2009-03-21 19:42:01 UTC
*** Bug 18065 has been marked as a duplicate of this bug. ***

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


Navigation
Links