Partial revert of r50478 "Preparations for Oracle database abstraction update."
authorAryeh Gregor <simetrical@users.mediawiki.org>
Mon, 11 May 2009 14:05:21 +0000 (14:05 +0000)
committerAryeh Gregor <simetrical@users.mediawiki.org>
Mon, 11 May 2009 14:05:21 +0000 (14:05 +0000)
Syntax error in RecentChangesLinked with MySQL 5.0: "1248: Every derived
table must have its own alias (localhost)".  Also likely to fail in
MySQL 4.0 even with an alias added, since subqueries don't work.

includes/specials/SpecialRecentchangeslinked.php

index e1b5ae4..82aa5b9 100644 (file)
@@ -155,9 +155,7 @@ class SpecialRecentchangeslinked extends SpecialRecentchanges {
                        $sql = $subsql[0];
                else {
                        // need to resort and relimit after union
-                       // unwrapped UNION block will not work in Oracle. Wrapper aded.
-                       $sql = "SELECT * FROM ((" . implode( ") UNION (", $subsql ) . ")) ORDER BY rc_timestamp DESC";
-                       $sql = $dbr->limitResult($sql, $limit, false);
+                       $sql = "(" . implode( ") UNION (", $subsql ) . ") ORDER BY rc_timestamp DESC LIMIT {$limit}";
                }
 
                $res = $dbr->query( $sql, __METHOD__ );