Fixed a E_DEPRECATED
[lhc/web/wiklou.git] / includes / DatabaseFunctions.php
index f87ac9a..2df5611 100644 (file)
@@ -1,34 +1,40 @@
 <?php
-# $Id$
-
 /**
- * Backwards compatibility wrapper for Database.php
- * 
- * Note: $wgDatabase has ceased to exist. Destroy all references.
+ * Legacy database functions, for compatibility with pre-1.3 code
+ * NOTE: this file is no longer loaded by default.
+ * @file
+ * @ingroup Database
  */
 
 /**
  * Usually aborts on failure
  * If errors are explicitly ignored, returns success
+ * @param $sql String: SQL query
+ * @param $db Mixed: database handler
+ * @param $fname String: name of the php function calling
  */
-function wfQuery( $sql, $db, $fname = '' )
-{
-       global $wgOut;
+function wfQuery( $sql, $db, $fname = '' ) {
        if ( !is_numeric( $db ) ) {
                # Someone has tried to call this the old way
-               $wgOut->fatalError( wfMsgNoDB( 'wrong_wfQuery_params', $db, $sql ) );
+               throw new FatalError( wfMsgNoDB( 'wrong_wfQuery_params', $db, $sql ) );
        }
-       $c =& wfGetDB( $db );
+       $c = wfGetDB( $db );
        if ( $c !== false ) {
                return $c->query( $sql, $fname );
-       } else {        
+       } else {
                return false;
        }
 }
 
-function wfSingleQuery( $sql, $dbi, $fname = '' )
-{
-       $db =& wfGetDB( $dbi );
+/**
+ *
+ * @param $sql String: SQL query
+ * @param $dbi
+ * @param $fname String: name of the php function calling
+ * @return Array: first row from the database
+ */
+function wfSingleQuery( $sql, $dbi, $fname = '' ) {
+       $db = wfGetDB( $dbi );
        $res = $db->query($sql, $fname );
        $row = $db->fetchRow( $res );
        $ret = $row[0];
@@ -36,237 +42,329 @@ function wfSingleQuery( $sql, $dbi, $fname = '' )
        return $ret;
 }
 
-function &wfGetDB( $db = DB_LAST )
-{
-       global $wgLoadBalancer;
-       return $wgLoadBalancer->getConnection( $db );
-}
-       
-# Turns buffering of SQL result sets on (true) or off (false). Default is
-# "on" and it should not be changed without good reasons. 
-# Returns the previous state.
-
-function wfBufferSQLResults( $newstate, $dbi = DB_LAST )
-{
-       $db =& wfGetDB( $dbi );
+/**
+ * Turns on (false) or off (true) the automatic generation and sending
+ * of a "we're sorry, but there has been a database error" page on
+ * database errors. Default is on (false). When turned off, the
+ * code should use wfLastErrno() and wfLastError() to handle the
+ * situation as appropriate.
+ *
+ * @param $newstate
+ * @param $dbi
+ * @return Returns the previous state.
+ */
+function wfIgnoreSQLErrors( $newstate, $dbi = DB_LAST ) {
+       $db = wfGetDB( $dbi );
        if ( $db !== false ) {
-               return $db->setBufferResults( $newstate );
+               return $db->ignoreErrors( $newstate );
        } else {
-               return NULL;
+               return null;
        }
 }
 
-# Turns on (false) or off (true) the automatic generation and sending
-# of a "we're sorry, but there has been a database error" page on
-# database errors. Default is on (false). When turned off, the
-# code should use wfLastErrno() and wfLastError() to handle the
-# situation as appropriate.
-# Returns the previous state.
+/**#@+
+ * @param $res Database result handler
+ * @param $dbi
+*/
 
-function wfIgnoreSQLErrors( $newstate, $dbi = DB_LAST )
+/**
+ * Free a database result
+ * @return Bool: whether result is sucessful or not.
+ */
+function wfFreeResult( $res, $dbi = DB_LAST )
 {
-       $db =& wfGetDB( $dbi );
-       if ( $db !== false ) {
-               return $db->ignoreErrors( $newstate );
-       } else {
-               return NULL;
-       }
-}
-
-function wfFreeResult( $res, $dbi = DB_LAST ) 
-{ 
-       $db =& wfGetDB( $dbi );
+       $db = wfGetDB( $dbi );
        if ( $db !== false ) {
-               $db->freeResult( $res ); 
+               $db->freeResult( $res );
                return true;
-       } else {        
+       } else {
                return false;
        }
 }
 
-function wfFetchObject( $res, $dbi = DB_LAST ) 
-{ 
-       $db =& wfGetDB( $dbi );
+/**
+ * Get an object from a database result
+ * @return object|false object we requested
+ */
+function wfFetchObject( $res, $dbi = DB_LAST ) {
+       $db = wfGetDB( $dbi );
        if ( $db !== false ) {
-               return $db->fetchObject( $res, $dbi = DB_LAST ); 
-       } else {        
+               return $db->fetchObject( $res, $dbi = DB_LAST );
+       } else {
                return false;
        }
 }
 
-function wfFetchRow( $res, $dbi = DB_LAST )
-{
-       $db =& wfGetDB( $dbi );
+/**
+ * Get a row from a database result
+ * @return object|false row we requested
+ */
+function wfFetchRow( $res, $dbi = DB_LAST ) {
+       $db = wfGetDB( $dbi );
        if ( $db !== false ) {
                return $db->fetchRow ( $res, $dbi = DB_LAST );
-       } else {        
+       } else {
                return false;
        }
 }
 
-function wfNumRows( $res, $dbi = DB_LAST ) 
-{ 
-       $db =& wfGetDB( $dbi );
+/**
+ * Get a number of rows from a database result
+ * @return integer|false number of rows
+ */
+function wfNumRows( $res, $dbi = DB_LAST ) {
+       $db = wfGetDB( $dbi );
        if ( $db !== false ) {
-               return $db->numRows( $res, $dbi = DB_LAST ); 
-       } else {        
+               return $db->numRows( $res, $dbi = DB_LAST );
+       } else {
                return false;
        }
 }
 
-function wfNumFields( $res, $dbi = DB_LAST ) 
-{ 
-       $db =& wfGetDB( $dbi );
+/**
+ * Get the number of fields from a database result
+ * @return integer|false number of fields
+ */
+function wfNumFields( $res, $dbi = DB_LAST ) {
+       $db = wfGetDB( $dbi );
        if ( $db !== false ) {
-               return $db->numFields( $res ); 
-       } else {        
+               return $db->numFields( $res );
+       } else {
                return false;
        }
 }
 
-function wfFieldName( $res, $n, $dbi = DB_LAST ) 
-{ 
-       $db =& wfGetDB( $dbi );
+/**
+ * Return name of a field in a result
+ * @param $res Mixed: Ressource link see Database::fieldName()
+ * @param $n Integer: id of the field
+ * @param $dbi Default DB_LAST
+ * @return string|false name of field
+ */
+function wfFieldName( $res, $n, $dbi = DB_LAST )
+{
+       $db = wfGetDB( $dbi );
        if ( $db !== false ) {
-               return $db->fieldName( $res, $n, $dbi = DB_LAST ); 
-       } else {        
+               return $db->fieldName( $res, $n, $dbi = DB_LAST );
+       } else {
                return false;
        }
 }
+/**#@-*/
 
-function wfInsertId( $dbi = DB_LAST ) 
-{ 
-       $db =& wfGetDB( $dbi );
+/**
+ * @todo document function
+ * @see Database::insertId()
+ */
+function wfInsertId( $dbi = DB_LAST ) {
+       $db = wfGetDB( $dbi );
        if ( $db !== false ) {
-               return $db->insertId(); 
-       } else {        
+               return $db->insertId();
+       } else {
                return false;
        }
 }
 
-function wfDataSeek( $res, $row, $dbi = DB_LAST ) 
-{ 
-       $db =& wfGetDB( $dbi );
+/**
+ * @todo document function
+ * @see Database::dataSeek()
+ */
+function wfDataSeek( $res, $row, $dbi = DB_LAST ) {
+       $db = wfGetDB( $dbi );
        if ( $db !== false ) {
-               return $db->dataSeek( $res, $row ); 
-       } else {        
+               return $db->dataSeek( $res, $row );
+       } else {
                return false;
        }
 }
 
-function wfLastErrno( $dbi = DB_LAST )  
-{ 
-       $db =& wfGetDB( $dbi );
+/**
+ * Get the last error number
+ * @see Database::lastErrno()
+ */
+function wfLastErrno( $dbi = DB_LAST ) {
+       $db = wfGetDB( $dbi );
        if ( $db !== false ) {
-               return $db->lastErrno(); 
-       } else {        
+               return $db->lastErrno();
+       } else {
                return false;
        }
 }
 
-function wfLastError( $dbi = DB_LAST )  
-{ 
-       $db =& wfGetDB( $dbi );
+/**
+ * Get the last error
+ * @see Database::lastError()
+ */
+function wfLastError( $dbi = DB_LAST ) {
+       $db = wfGetDB( $dbi );
        if ( $db !== false ) {
-               return $db->lastError(); 
-       } else {        
+               return $db->lastError();
+       } else {
                return false;
        }
 }
 
-function wfAffectedRows( $dbi = DB_LAST )
-{ 
-       $db =& wfGetDB( $dbi );
+/**
+ * Get the number of affected rows
+ * @see Database::affectedRows()
+ */
+function wfAffectedRows( $dbi = DB_LAST ) {
+       $db = wfGetDB( $dbi );
        if ( $db !== false ) {
-               return $db->affectedRows(); 
-       } else {        
+               return $db->affectedRows();
+       } else {
                return false;
        }
 }
 
-function wfLastDBquery( $dbi = DB_LAST )
-{
-       $db =& wfGetDB( $dbi );
+/**
+ * Get the last query ran
+ * @see Database::lastQuery
+ */
+function wfLastDBquery( $dbi = DB_LAST ) {
+       $db = wfGetDB( $dbi );
        if ( $db !== false ) {
                return $db->lastQuery();
-       } else {        
+       } else {
                return false;
        }
 }
 
+/**
+ * @see Database::Set()
+ * @todo document function
+ * @param $table
+ * @param $var
+ * @param $value
+ * @param $cond
+ * @param $dbi Default DB_MASTER
+ */
 function wfSetSQL( $table, $var, $value, $cond, $dbi = DB_MASTER )
 {
-       $db =& wfGetDB( $dbi );
+       $db = wfGetDB( $dbi );
        if ( $db !== false ) {
                return $db->set( $table, $var, $value, $cond );
-       } else {        
+       } else {
                return false;
        }
 }
 
+
+/**
+ * Simple select wrapper, return one field
+ * @see Database::selectField()
+ * @param $table
+ * @param $var
+ * @param $cond Default ''
+ * @param $dbi Default DB_LAST
+ */
 function wfGetSQL( $table, $var, $cond='', $dbi = DB_LAST )
 {
-       $db =& wfGetDB( $dbi );
+       $db = wfGetDB( $dbi );
        if ( $db !== false ) {
-               return $db->getField( $table, $var, $cond );
-       } else {        
+               return $db->selectField( $table, $var, $cond );
+       } else {
                return false;
        }
 }
 
-function wfFieldExists( $table, $field, $dbi = DB_LAST )
-{
-       $db =& wfGetDB( $dbi );
+/**
+ * Does a given field exist on the specified table?
+ * @see Database::fieldExists()
+ * @param $table
+ * @param $field
+ * @param $dbi Default DB_LAST
+ * @return Result of Database::fieldExists() or false.
+ */
+function wfFieldExists( $table, $field, $dbi = DB_LAST ) {
+       $db = wfGetDB( $dbi );
        if ( $db !== false ) {
                return $db->fieldExists( $table, $field );
-       } else {        
+       } else {
                return false;
        }
 }
 
-function wfIndexExists( $table, $index, $dbi = DB_LAST ) 
-{
-       $db =& wfGetDB( $dbi );
+/**
+ * Does the requested index exist on the specified table?
+ * @see Database::indexExists()
+ * @param $table String
+ * @param $index
+ * @param $dbi Default DB_LAST
+ * @return Result of Database::indexExists() or false.
+ */
+function wfIndexExists( $table, $index, $dbi = DB_LAST ) {
+       $db = wfGetDB( $dbi );
        if ( $db !== false ) {
                return $db->indexExists( $table, $index );
-       } else {        
+       } else {
                return false;
        }
 }
 
-function wfInsertArray( $table, $array, $fname = 'wfInsertArray', $dbi = DB_MASTER ) 
-{
-       $db =& wfGetDB( $dbi );
+/**
+ * @see Database::insert()
+ * @todo document function
+ * @param $table String
+ * @param $array Array
+ * @param $fname String, default 'wfInsertArray'.
+ * @param $dbi Default DB_MASTER
+ * @return result of Database::insert() or false.
+ */
+function wfInsertArray( $table, $array, $fname = 'wfInsertArray', $dbi = DB_MASTER ) {
+       $db = wfGetDB( $dbi );
        if ( $db !== false ) {
-               return $db->insertArray( $table, $array, $fname );
-       } else {        
+               return $db->insert( $table, $array, $fname );
+       } else {
                return false;
        }
 }
 
-function wfGetArray( $table, $vars, $conds, $fname = 'wfGetArray', $dbi = DB_LAST )
-{
-       $db =& wfGetDB( $dbi );
+/**
+ * @see Database::getArray()
+ * @todo document function
+ * @param $table String
+ * @param $vars
+ * @param $conds
+ * @param $fname String, default 'wfGetArray'.
+ * @param $dbi Default DB_LAST
+ * @return result of Database::getArray() or false.
+ */
+function wfGetArray( $table, $vars, $conds, $fname = 'wfGetArray', $dbi = DB_LAST ) {
+       $db = wfGetDB( $dbi );
        if ( $db !== false ) {
                return $db->getArray( $table, $vars, $conds, $fname );
-       } else {        
+       } else {
                return false;
        }
 }
 
-function wfUpdateArray( $table, $values, $conds, $fname = 'wfUpdateArray', $dbi = DB_MASTER )
-{
-       $db =& wfGetDB( $dbi );
+/**
+ * @see Database::update()
+ * @param $table String
+ * @param $values
+ * @param $conds
+ * @param $fname String, default 'wfUpdateArray'
+ * @param $dbi Default DB_MASTER
+ * @return Result of Database::update()) or false;
+ * @todo document function
+ */
+function wfUpdateArray( $table, $values, $conds, $fname = 'wfUpdateArray', $dbi = DB_MASTER ) {
+       $db = wfGetDB( $dbi );
        if ( $db !== false ) {
-               $db->updateArray( $table, $values, $conds, $fname );
+               $db->update( $table, $values, $conds, $fname );
                return true;
        } else {
                return false;
        }
 }
 
+/**
+ * Get fully usable table name
+ * @see Database::tableName()
+ */
 function wfTableName( $name, $dbi = DB_LAST ) {
-       $db =& wfGetDB( $dbi );
+       $db = wfGetDB( $dbi );
        if ( $db !== false ) {
                return $db->tableName( $name );
        } else {
@@ -274,9 +372,12 @@ function wfTableName( $name, $dbi = DB_LAST ) {
        }
 }
 
-function wfStrencode( $s, $dbi = DB_LAST )
-{
-       $db =& wfGetDB( $dbi );
+/**
+ * @todo document function
+ * @see Database::strencode()
+ */
+function wfStrencode( $s, $dbi = DB_LAST ) {
+       $db = wfGetDB( $dbi );
        if ( $db !== false ) {
                return $db->strencode( $s );
        } else {
@@ -284,8 +385,12 @@ function wfStrencode( $s, $dbi = DB_LAST )
        }
 }
 
+/**
+ * @todo document function
+ * @see Database::nextSequenceValue()
+ */
 function wfNextSequenceValue( $seqName, $dbi = DB_MASTER ) {
-       $db =& wfGetDB( $dbi );
+       $db = wfGetDB( $dbi );
        if ( $db !== false ) {
                return $db->nextSequenceValue( $seqName );
        } else {
@@ -293,12 +398,15 @@ function wfNextSequenceValue( $seqName, $dbi = DB_MASTER ) {
        }
 }
 
+/**
+ * @todo document function
+ * @see Database::useIndexClause()
+ */
 function wfUseIndexClause( $index, $dbi = DB_SLAVE ) {
-       $db =& wfGetDB( $dbi );
+       $db = wfGetDB( $dbi );
        if ( $db !== false ) {
                return $db->useIndexClause( $index );
        } else {
                return false;
        }
 }
-?>