The actual error is always in English, so wrap it in ltr element
[lhc/web/wiklou.git] / includes / db / Database.php
index 3ff4145..4d75379 100644 (file)
@@ -28,7 +28,7 @@ interface DatabaseType {
         *
         * @return string
         */
-       public function getType();
+       function getType();
 
        /**
         * Open a connection to the database. Usually aborts on failure
@@ -40,7 +40,7 @@ interface DatabaseType {
         * @return bool
         * @throws DBConnectionError
         */
-       public function open( $server, $user, $password, $dbName );
+       function open( $server, $user, $password, $dbName );
 
        /**
         * The DBMS-dependent part of query()
@@ -50,7 +50,7 @@ interface DatabaseType {
         * @return Result object to feed to fetchObject, fetchRow, ...; or false on failure
         * @private
         */
-       /*private*/ function doQuery( $sql );
+       function doQuery( $sql );
 
        /**
         * Fetch the next row from the given result object, in object form.
@@ -61,7 +61,7 @@ interface DatabaseType {
         * @return Row object
         * @throws DBUnexpectedError Thrown if the database returns an error
         */
-       public function fetchObject( $res );
+       function fetchObject( $res );
 
        /**
         * Fetch the next row from the given result object, in associative array
@@ -71,7 +71,7 @@ interface DatabaseType {
         * @return Row object
         * @throws DBUnexpectedError Thrown if the database returns an error
         */
-       public function fetchRow( $res );
+       function fetchRow( $res );
 
        /**
         * Get the number of rows in a result object
@@ -79,7 +79,7 @@ interface DatabaseType {
         * @param $res Mixed: A SQL result
         * @return int
         */
-       public function numRows( $res );
+       function numRows( $res );
 
        /**
         * Get the number of fields in a result object
@@ -88,7 +88,7 @@ interface DatabaseType {
         * @param $res Mixed: A SQL result
         * @return int
         */
-       public function numFields( $res );
+       function numFields( $res );
 
        /**
         * Get a field name in a result object
@@ -98,7 +98,7 @@ interface DatabaseType {
         * @param $n Integer
         * @return string
         */
-       public function fieldName( $res, $n );
+       function fieldName( $res, $n );
 
        /**
         * Get the inserted value of an auto-increment row
@@ -112,7 +112,7 @@ interface DatabaseType {
         *
         * @return int
         */
-       public function insertId();
+       function insertId();
 
        /**
         * Change the position of the cursor in a result object
@@ -121,7 +121,7 @@ interface DatabaseType {
         * @param $res Mixed: A SQL result
         * @param $row Mixed: Either MySQL row or ResultWrapper
         */
-       public function dataSeek( $res, $row );
+       function dataSeek( $res, $row );
 
        /**
         * Get the last error number
@@ -129,7 +129,7 @@ interface DatabaseType {
         *
         * @return int
         */
-       public function lastErrno();
+       function lastErrno();
 
        /**
         * Get a description of the last error
@@ -137,7 +137,7 @@ interface DatabaseType {
         *
         * @return string
         */
-       public function lastError();
+       function lastError();
 
        /**
         * mysql_fetch_field() wrapper
@@ -146,7 +146,7 @@ interface DatabaseType {
         * @param $table string: table name
         * @param $field string: field name
         */
-       public function fieldInfo( $table, $field );
+       function fieldInfo( $table, $field );
 
        /**
         * Get information about an index into an object
@@ -163,7 +163,7 @@ interface DatabaseType {
         *
         * @return int
         */
-       public function affectedRows();
+       function affectedRows();
 
        /**
         * Wrapper for addslashes()
@@ -171,7 +171,7 @@ interface DatabaseType {
         * @param $s string: to be slashed.
         * @return string: slashed string.
         */
-       public function strencode( $s );
+       function strencode( $s );
 
        /**
         * Returns a wikitext link to the DB's website, e.g.,
@@ -181,7 +181,7 @@ interface DatabaseType {
         *
         * @return string: wikitext of a link to the server software's web site
         */
-       public static function getSoftwareLink();
+       static function getSoftwareLink();
 
        /**
         * A string describing the current software version, like from
@@ -189,7 +189,7 @@ interface DatabaseType {
         *
         * @return string: Version information from the database server.
         */
-       public function getServerVersion();
+       function getServerVersion();
 
        /**
         * A string describing the current software version, and possibly
@@ -198,7 +198,7 @@ interface DatabaseType {
         *
         * @return string: Version information from the database server
         */
-       public function getServerInfo();
+       function getServerInfo();
 }
 
 /**
@@ -288,7 +288,7 @@ abstract class DatabaseBase implements DatabaseType {
        }
 
        function tablePrefix( $prefix = null ) {
-               return wfSetVar( $this->mTablePrefix, $prefix );
+               return wfSetVar( $this->mTablePrefix, $prefix, true );
        }
 
        /**
@@ -508,13 +508,6 @@ abstract class DatabaseBase implements DatabaseType {
                        }
                }
 
-               /*
-               // Faster read-only access
-               if ( wfReadOnly() ) {
-                       $this->mFlags |= DBO_PERSISTENT;
-                       $this->mFlags &= ~DBO_TRX;
-               }*/
-
                /** Get the default table prefix*/
                if ( $tablePrefix == 'get from global' ) {
                        $this->mTablePrefix = $wgDBprefix;
@@ -540,6 +533,27 @@ abstract class DatabaseBase implements DatabaseType {
                return new DatabaseMysql( $server, $user, $password, $dbName, $flags );
        }
 
+       /**
+        * Given a DB type, construct the name of the appropriate child class of
+        * DatabaseBase. This is designed to replace all of the manual stuff like:
+        *      $class = 'Database' . ucfirst( strtolower( $type ) );
+        * as well as validate against the canonical list of DB types we have
+        *
+        * @param $dbType String A possible DB type
+        * @return DatabaseBase subclass or null
+        */
+       public final static function classFromType( $dbType ) {
+               $canonicalDBTypes = array(
+                       'mysql', 'postgres', 'sqlite', 'oracle', 'mssql', 'ibm_db2'
+               );
+               $dbType = strtolower( $dbType );
+               if( in_array( $dbType, $canonicalDBTypes ) ) {
+                       return 'Database' . ucfirst( $dbType );
+               } else {
+                       return null;
+               }
+       }
+
        protected function installErrorHandler() {
                $this->mPHPError = false;
                $this->htmlErrors = ini_set( 'html_errors', '0' );
@@ -604,7 +618,7 @@ abstract class DatabaseBase implements DatabaseType {
         *     comment (you can use __METHOD__ or add some extra info)
         * @param  $tempIgnore Boolean:   Whether to avoid throwing an exception on errors...
         *     maybe best to catch the exception instead?
-        * @return boolean|ResultWrapper true for a successful write query, ResultWrapper object for a successful read query,
+        * @return boolean or ResultWrapper. true for a successful write query, ResultWrapper object for a successful read query,
         *     or false on failure if $tempIgnore set
         * @throws DBQueryError Thrown when the database returns an error of any kind
         */
@@ -1012,7 +1026,7 @@ abstract class DatabaseBase implements DatabaseType {
        /**
         * SELECT wrapper
         *
-        * @param $table   Mixed:  Array or string, table name(s) (prefix auto-added)
+        * @param $table   Mixed:  Array or string, table name(s) (prefix auto-added). Array keys are table aliases (optional)
         * @param $vars    Mixed:  Array or string, field name(s) to be retrieved
         * @param $conds   Mixed:  Array or string, condition(s) for WHERE
         * @param $fname   String: Calling function name (use __METHOD__) for logs/profiling
@@ -1035,7 +1049,7 @@ abstract class DatabaseBase implements DatabaseType {
                        if ( !empty( $join_conds ) || ( isset( $options['USE INDEX'] ) && is_array( @$options['USE INDEX'] ) ) ) {
                                $from = ' FROM ' . $this->tableNamesWithUseIndexOrJOIN( $table, @$options['USE INDEX'], $join_conds );
                        } else {
-                               $from = ' FROM ' . implode( ',', array_map( array( &$this, 'tableName' ), $table ) );
+                               $from = ' FROM ' . implode( ',', $this->tableNamesWithAlias( $table ) );
                        }
                } elseif ( $table != '' ) {
                        if ( $table { 0 } == ' ' ) {
@@ -1575,6 +1589,39 @@ abstract class DatabaseBase implements DatabaseType {
                return $retVal;
        }
 
+       /**
+        * Get an aliased table name
+        * e.g. tableName AS newTableName
+        *
+        * @param $name string Table name, see tableName()
+        * @param $alias string Alias (optional)
+        * @return string SQL name for aliased table. Will not alias a table to its own name
+        */
+       public function tableNameWithAlias( $name, $alias = false ) {
+               if ( !$alias || $alias == $name ) {
+                       return $this->tableName( $name );
+               } else {
+                       return $this->tableName( $name ) . ' `' . $alias . '`';
+               }
+       }
+
+       /**
+        * Gets an array of aliased table names
+        *
+        * @param $tables array( [alias] => table )
+        * @return array of strings, see tableNameWithAlias()
+        */
+       public function tableNamesWithAlias( $tables ) {
+               $retval = array();
+               foreach ( $tables as $alias => $table ) {
+                       if ( is_numeric( $alias ) ) {
+                               $alias = $table;
+                       }
+                       $retval[] = $this->tableNameWithAlias( $table, $alias );
+               }
+               return $retval;
+       }
+
        /**
         * @private
         */
@@ -1584,35 +1631,37 @@ abstract class DatabaseBase implements DatabaseType {
                $use_index_safe = is_array( $use_index ) ? $use_index : array();
                $join_conds_safe = is_array( $join_conds ) ? $join_conds : array();
 
-               foreach ( $tables as $table ) {
+               foreach ( $tables as $alias => $table ) {
+                       if ( !is_string( $alias ) ) {
+                               // No alias? Set it equal to the table name
+                               $alias = $table;
+                       }
                        // Is there a JOIN and INDEX clause for this table?
-                       if ( isset( $join_conds_safe[$table] ) && isset( $use_index_safe[$table] ) ) {
-                               $tableClause = $join_conds_safe[$table][0] . ' ' . $this->tableName( $table );
-                               $tableClause .= ' ' . $this->useIndexClause( implode( ',', (array)$use_index_safe[$table] ) );
-                               $on = $this->makeList( (array)$join_conds_safe[$table][1], LIST_AND );
-
+                       if ( isset( $join_conds_safe[$alias] ) && isset( $use_index_safe[$alias] ) ) {
+                               $tableClause = $join_conds_safe[$alias][0] . ' ' . $this->tableNameWithAlias( $table, $alias );
+                               $tableClause .= ' ' . $this->useIndexClause( implode( ',', (array)$use_index_safe[$alias] ) );
+                               $on = $this->makeList( (array)$join_conds_safe[$alias][1], LIST_AND );
                                if ( $on != '' ) {
                                        $tableClause .= ' ON (' . $on . ')';
                                }
 
                                $retJOIN[] = $tableClause;
                        // Is there an INDEX clause?
-                       } else if ( isset( $use_index_safe[$table] ) ) {
-                               $tableClause = $this->tableName( $table );
-                               $tableClause .= ' ' . $this->useIndexClause( implode( ',', (array)$use_index_safe[$table] ) );
+                       } else if ( isset( $use_index_safe[$alias] ) ) {
+                               $tableClause = $this->tableNameWithAlias( $table, $alias );
+                               $tableClause .= ' ' . $this->useIndexClause( implode( ',', (array)$use_index_safe[$alias] ) );
                                $ret[] = $tableClause;
                        // Is there a JOIN clause?
-                       } else if ( isset( $join_conds_safe[$table] ) ) {
-                               $tableClause = $join_conds_safe[$table][0] . ' ' . $this->tableName( $table );
-                               $on = $this->makeList( (array)$join_conds_safe[$table][1], LIST_AND );
-
+                       } else if ( isset( $join_conds_safe[$alias] ) ) {
+                               $tableClause = $join_conds_safe[$alias][0] . ' ' . $this->tableNameWithAlias( $table, $alias );
+                               $on = $this->makeList( (array)$join_conds_safe[$alias][1], LIST_AND );
                                if ( $on != '' ) {
                                        $tableClause .= ' ON (' . $on . ')';
                                }
 
                                $retJOIN[] = $tableClause;
                        } else {
-                               $tableClause = $this->tableName( $table );
+                               $tableClause = $this->tableNameWithAlias( $table, $alias );
                                $ret[] = $tableClause;
                        }
                }
@@ -1659,6 +1708,27 @@ abstract class DatabaseBase implements DatabaseType {
                }
        }
 
+       /**
+        * Quotes an identifier using `backticks` or "double quotes" depending on the database type.
+        * MySQL uses `backticks` while basically everything else uses double quotes.
+        * Since MySQL is the odd one out here the double quotes are our generic
+        * and we implement backticks in DatabaseMysql.
+        */      
+       public function addIdentifierQuotes( $s ) {
+               return '"' . str_replace( '"', '""', $s ) . '"';
+       }
+
+       /**
+        * Backwards compatibility, identifier quoting originated in DatabasePostgres
+        * which used quote_ident which does not follow our naming conventions
+        * was renamed to addIdentifierQuotes.
+        * @deprecated use addIdentifierQuotes
+        */
+       function quote_ident( $s ) {
+               wfDeprecated( __METHOD__ );
+               return $this->addIdentifierQuotes( $s );
+       }
+
        /**
         * Escape string for safe LIKE usage.
         * WARNING: you should almost never use this function directly,
@@ -1988,16 +2058,6 @@ abstract class DatabaseBase implements DatabaseType {
                return "REPLACE({$orig}, {$old}, {$new})";
        }
 
-       /**
-        * Convert a field to an unix timestamp
-        *
-        * @param $field String: field name
-        * @return String: SQL statement
-        */
-       public function unixTimestamp( $field ) {
-               return "EXTRACT(epoch FROM $field)";
-       }
-
        /**
         * Determines if the last failure was due to a deadlock
         * STUB
@@ -2232,6 +2292,16 @@ abstract class DatabaseBase implements DatabaseType {
        function duplicateTableStructure( $oldName, $newName, $temporary = false, $fname = 'DatabaseBase::duplicateTableStructure' ) {
                throw new MWException( 'DatabaseBase::duplicateTableStructure is not implemented in descendant class' );
        }
+       
+       /**
+        * List all tables on the database
+        *
+        * @param $prefix Only show tables with this prefix, e.g. mw_
+        * @param $fname String: calling function name
+        */
+       function listTables( $prefix = null, $fname = 'DatabaseBase::listTables' ) {
+               throw new MWException( 'DatabaseBase::listTables is not implemented in descendant class' );
+       }
 
        /**
         * Return MW-style timestamp used for MySQL schema
@@ -2342,13 +2412,12 @@ abstract class DatabaseBase implements DatabaseType {
         *              using $filename
         */
        function sourceFile( $filename, $lineCallback = false, $resultCallback = false, $fname = false ) {
+               wfSuppressWarnings();
                $fp = fopen( $filename, 'r' );
+               wfRestoreWarnings();
 
                if ( false === $fp ) {
-                       if ( !defined( "MEDIAWIKI_INSTALL" ) )
-                               throw new MWException( "Could not open \"{$filename}\".\n" );
-                       else
-                               return "Could not open \"{$filename}\".\n";
+                       throw new MWException( "Could not open \"{$filename}\".\n" );
                }
 
                if ( !$fname ) {
@@ -2359,12 +2428,8 @@ abstract class DatabaseBase implements DatabaseType {
                        $error = $this->sourceStream( $fp, $lineCallback, $resultCallback, $fname );
                }
                catch ( MWException $e ) {
-                       if ( defined( "MEDIAWIKI_INSTALL" ) ) {
-                               $error = $e->getMessage();
-                       } else {
-                               fclose( $fp );
-                               throw $e;
-                       }
+                       fclose( $fp );
+                       throw $e;
                }
 
                fclose( $fp );
@@ -2465,6 +2530,32 @@ abstract class DatabaseBase implements DatabaseType {
                return true;
        }
 
+       /**
+        * Database independent variable replacement, replaces a set of named variables
+        * in a sql statement with the contents of their global variables.
+        * Supports '{$var}' `{$var}` and / *$var* / (without the spaces) style variables
+        * 
+        * '{$var}' should be used for text and is passed through the database's addQuotes method
+        * `{$var}` should be used for identifiers (eg: table and database names), it is passed through
+        *          the database's addIdentifierQuotes method which can be overridden if the database
+        *          uses something other than backticks.
+        * / *$var* / is just encoded, besides traditional dbprefix and tableoptions it's use should be avoided
+        * 
+        * @param $ins String: SQL statement to replace variables in
+        * @param $varnames Array: Array of global variable names to replace
+        * @return String The new SQL statement with variables replaced
+        */
+       protected function replaceGlobalVars( $ins, $varnames ) {
+               foreach ( $varnames as $var ) {
+                       if ( isset( $GLOBALS[$var] ) ) {
+                               $ins = str_replace( '\'{$' . $var . '}\'', $this->addQuotes( $GLOBALS[$var] ), $ins ); // replace '{$var}'
+                               $ins = str_replace( '`{$' . $var . '}`', $this->addIdentifierQuotes( $GLOBALS[$var] ), $ins ); // replace `{$var}`
+                               $ins = str_replace( '/*$' . $var . '*/', $this->strencode( $GLOBALS[$var] ) , $ins ); // replace /*$var*/
+                       }
+               }
+               return $ins;
+       }
+
        /**
         * Replace variables in sourced SQL
         */
@@ -2475,15 +2566,7 @@ abstract class DatabaseBase implements DatabaseType {
                        'wgDBadminuser', 'wgDBadminpassword', 'wgDBTableOptions',
                );
 
-               // Ordinary variables
-               foreach ( $varnames as $var ) {
-                       if ( isset( $GLOBALS[$var] ) ) {
-                               $val = $this->addQuotes( $GLOBALS[$var] ); // FIXME: safety check?
-                               $ins = str_replace( '{$' . $var . '}', $val, $ins );
-                               $ins = str_replace( '/*$' . $var . '*/`', '`' . $val, $ins );
-                               $ins = str_replace( '/*$' . $var . '*/', $val, $ins );
-                       }
-               }
+               $ins = $this->replaceGlobalVars( $ins, $varnames );
 
                // Table prefixes
                $ins = preg_replace_callback( '!/\*(?:\$wgDBprefix|_)\*/([a-zA-Z_0-9]*)!',
@@ -2570,6 +2653,20 @@ abstract class DatabaseBase implements DatabaseType {
                return true;
        }
 
+       /**
+        * Delete a table
+        */
+       public function dropTable( $tableName, $fName = 'DatabaseBase::dropTable' ) {
+               if( !$this->tableExists( $tableName ) ) {
+                       return false;
+               }
+               $sql = "DROP TABLE " . $this->tableName( $tableName );
+               if( $this->cascadingDeletes() ) {
+                       $sql .= " CASCADE";
+               }
+               return $this->query( $sql, $fName );
+       }
+
        /**
         * Get search engine class. All subclasses of this need to implement this
         * if they wish to use searching.
@@ -2580,6 +2677,17 @@ abstract class DatabaseBase implements DatabaseType {
                return 'SearchEngineDummy';
        }
 
+       /**
+        * Find out when 'infinity' is. Most DBMSes support this. This is a special
+        * keyword for timestamps in PostgreSQL, and works with CHAR(14) as well
+        * because "i" sorts after all numbers.
+        *
+        * @return String
+        */
+       public function getInfinity() {
+               return 'infinity';
+       }
+
        /**
         * Allow or deny "big selects" for this session only. This is done by setting
         * the sql_big_selects session variable.
@@ -2766,6 +2874,8 @@ class DBConnectionError extends DBError {
                        $this->error = $this->db->getProperty( 'mServer' );
                }
 
+               $this->error = Html::element( 'span', array( 'dir' => 'ltr' ), $this->error );
+
                $noconnect = "<p><strong>$sorry</strong><br />$again</p><p><small>$info</small></p>";
                $text = str_replace( '$1', $this->error, $noconnect );
 
@@ -2844,8 +2954,8 @@ EOT;
                return $trygoogle;
        }
 
-       function fileCachedPage() {
-               global $wgTitle, $title, $wgLang, $wgOut;
+       private function fileCachedPage() {
+               global $wgTitle, $wgLang, $wgOut;
 
                if ( $wgOut->isDisabled() ) {
                        return; // Done already?
@@ -2854,13 +2964,11 @@ EOT;
                $mainpage = 'Main Page';
 
                if ( $wgLang instanceof Language ) {
-                       $mainpage    = htmlspecialchars( $wgLang->getMessage( 'mainpage' ) );
+                       $mainpage = htmlspecialchars( $wgLang->getMessage( 'mainpage' ) );
                }
 
                if ( $wgTitle ) {
                        $t =& $wgTitle;
-               } elseif ( $title ) {
-                       $t = Title::newFromURL( $title );
                } else {
                        $t = Title::newFromText( $mainpage );
                }