WordPress.org
Get WordPress

Make WordPress Core

Ticket #11644: 11644.code_cleanup_wpdb.3.patch

File 11644.code_cleanup_wpdb.3.patch, 36.9 KB (added by hakre, 16 years ago)

  • wp-db.php

     
    1212/**
    1313 * @since 0.71
    1414 */
    15 define('EZSQL_VERSION', 'WP1.25');
     15define( 'EZSQL_VERSION', 'WP1.25' );
    1616
    1717/**
    1818 * @since 0.71
    1919 */
    20 define('OBJECT', 'OBJECT', true);
     20define( 'OBJECT', 'OBJECT', true );
    2121
    2222/**
    2323 * @since {@internal Version Unknown}}
    2424 */
    25 define('OBJECT_K', 'OBJECT_K', false);
     25define( 'OBJECT_K', 'OBJECT_K', false );
    2626
    2727/**
    2828 * @since 0.71
    2929 */
    30 define('ARRAY_A', 'ARRAY_A', false);
     30define( 'ARRAY_A', 'ARRAY_A', false );
    3131
    3232/**
    3333 * @since 0.71
    3434 */
    35 define('ARRAY_N', 'ARRAY_N', false);
     35define( 'ARRAY_N', 'ARRAY_N', false );
    3636
    3737/**
    3838 * WordPress Database Access Abstraction Object
    … …  
    6565         * Whether to suppress errors during the DB bootstrapping.
    6666         *
    6767         * @access private
    68          * @since {@internal Version Unknown}}
     68         * @since 2.5
    6969         * @var bool
    7070         */
    7171        var $suppress_errors = false;
    … …  
    7373        /**
    7474         * The last error during query.
    7575         *
    76          * @since {@internal Version Unknown}}
     76         * @see get_last_error()
     77         * @since 2.5
     78         * @access private
    7779         * @var string
    7880         */
    7981        var $last_error = '';
    … …  
    8890        var $num_queries = 0;
    8991
    9092        /**
     93         * Amount of rows returned by last query operation
     94         *
     95         * @since 1.2
     96         * @access private
     97         * @var int
     98         */
     99        var $num_rows = 0;
     100
     101        /**
     102         * Number of affected rows by last query operation
     103         *
     104         * @since 0.71
     105         * @access private
     106         * @var integer
     107         */
     108        var $rows_affected = 0;
     109
     110
     111        /**
    91112         * Saved result of the last query made
    92113         *
    93114         * @since 1.2.0
    … …  
    97118        var $last_query;
    98119
    99120        /**
     121         * Saved resultset of the last query made
     122         *
     123         * @since {@internal Version Unknown}
     124         * @access private
     125         * @var array (null if unintialized)
     126         */
     127        var $last_result;
     128
     129        /**
    100130         * Saved info on the table column
    101131         *
    102132         * @since 1.2.0
    … …  
    135165         * @var bool
    136166         */
    137167        var $ready = false;
     168
     169        /**
     170         * {@internal Missing Description}}
     171         *
     172         * @since 3.0
     173         * @access private
     174         * @var unknown_type
     175         */
    138176        var $blogid = 0;
     177
     178        /**
     179         * {@internal Missing Description}}
     180         *
     181         * @since 3.0
     182         * @access private
     183         * @var unknown_type
     184         */
    139185        var $siteid = 0;
    140186
    141187        /**
    … …  
    157203        var $users;
    158204
    159205        /**
     206         * WordPress Categories table
     207         *
     208         * @since 1.5.0
     209         * @access public
     210         * @var string
     211         */
     212        var $categories;
     213
     214        /**
     215         * WordPress Post to Category table
     216         *
     217         * @since 1.5.0
     218         * @access public
     219         * @var string
     220         */
     221        var $post2cat;
     222
     223        /**
    160224         * WordPress Comments table
    161225         *
    162226         * @since 1.5.0
    … …  
    406470         * @param string $dbname MySQL database name
    407471         * @param string $dbhost MySQL database host
    408472         */
    409         function wpdb($dbuser, $dbpassword, $dbname, $dbhost) {
     473        function wpdb( $dbuser, $dbpassword, $dbname, $dbhost ) {
    410474                if( defined( 'WP_USE_MULTIPLE_DB' ) && WP_USE_MULTIPLE_DB )
    411475                        $this->db_connect();
    412476                return $this->__construct($dbuser, $dbpassword, $dbname, $dbhost);
    … …  
    426490         * @param string $dbname MySQL database name
    427491         * @param string $dbhost MySQL database host
    428492         */
    429         function __construct($dbuser, $dbpassword, $dbname, $dbhost) {
    430                 register_shutdown_function(array(&$this, "__destruct"));
     493        function __construct( $dbuser, $dbpassword, $dbname, $dbhost ) {
     494                register_shutdown_function( array( &$this, '__destruct' ) ); # prevents the object from unloading
    431495
    432496                if ( WP_DEBUG )
    433497                        $this->show_errors();
    … …  
    442506                        $this->collate = DB_COLLATE;
    443507                }
    444508
    445                 if ( defined('DB_CHARSET') )
     509                if ( defined( 'DB_CHARSET' ) )
    446510                        $this->charset = DB_CHARSET;
    447511
    448512                $this->dbuser = $dbuser;
    449513
    450                 $this->dbh = @mysql_connect($dbhost, $dbuser, $dbpassword, true);
    451                 if (!$this->dbh) {
    452                         $this->bail(sprintf(/*WP_I18N_DB_CONN_ERROR*/"
     514                $this->dbh = @mysql_connect( $dbhost, $dbuser, $dbpassword, true );
     515                if ( !$this->dbh ) {
     516                        $this->bail( sprintf( /*WP_I18N_DB_CONN_ERROR*/"
    453517<h1>Error establishing a database connection</h1>
    454518<p>This either means that the username and password information in your <code>wp-config.php</code> file is incorrect or we can't contact the database server at <code>%s</code>. This could mean your host's database server is down.</p>
    455519<ul>
    … …  
    458522        <li>Are you sure that the database server is running?</li>
    459523</ul>
    460524<p>If you're unsure what these terms mean you should probably contact your host. If you still need help you can always visit the <a href='http://wordpress.org/support/'>WordPress Support Forums</a>.</p>
    461 "/*/WP_I18N_DB_CONN_ERROR*/, $dbhost), 'db_connect_fail');
     525"/*/WP_I18N_DB_CONN_ERROR*/, $dbhost ), 'db_connect_fail' );
    462526                        return;
    463527                }
    464528
    465529                $this->ready = true;
    466530
    467                 if ( $this->has_cap( 'collation' ) && !empty($this->charset) ) {
    468                         if ( function_exists('mysql_set_charset') ) {
    469                                 mysql_set_charset($this->charset, $this->dbh);
     531                if ( $this->has_cap( 'collation' ) && !empty( $this->charset ) ) {
     532                        if ( function_exists( 'mysql_set_charset' ) ) {
     533                                mysql_set_charset( $this->charset, $this->dbh );
    470534                                $this->real_escape = true;
    471535                        } else {
    472                                 $collation_query = "SET NAMES '{$this->charset}'";
    473                                 if ( !empty($this->collate) )
    474                                         $collation_query .= " COLLATE '{$this->collate}'";
    475                                 $this->query($collation_query);
     536                                $query = $this->prepare( 'SET NAMES %s', $this->charset );
     537                                if ( !empty( $this->collate ) )
     538                                        $query .= $this->prepare( ' COLLATE %s', $this->collate );
     539                                $this->query( $query );
    476540                        }
    477541                }
    478542
    479                 $this->select($dbname, $this->dbh);
     543                $this->select( $dbname, $this->dbh );
    480544        }
    481545
    482546        /**
    483547         * PHP5 style destructor and will run when database object is destroyed.
    484548         *
     549         *
     550         *   Please see {@link __construct() class constructor} and the
     551         *   {@link register_shutdown_function() register_shutdown_function()}. for
     552         *   more details.
     553         *
     554         * @link http://www.php.net/__destruct
     555         * @link http://www.php.net/register_shutdown_function
     556         *
    485557         * @since 2.0.8
    486558         *
    487          * @return bool Always true
     559         * @return bool true
    488560         */
    489561        function __destruct() {
    490562                return true;
    … …  
    503575         */
    504576        function set_prefix( $prefix ) {
    505577
    506                 if ( preg_match('|[^a-z0-9_]|i', $prefix) )
     578                if ( preg_match( '|[^a-z0-9_]|i', $prefix ) )
    507579                        return new WP_Error('invalid_db_prefix', /*WP_I18N_DB_BAD_PREFIX*/'Invalid database prefix'/*/WP_I18N_DB_BAD_PREFIX*/);
    508580
    509581                if ( is_multisite() )
    … …  
    517589                foreach ( $this->tables( 'global' ) as $table )
    518590                        $this->$table = $prefix . $table;
    519591
    520                 if ( defined('VHOST') && empty( $this->blogid ) )
     592                if ( defined( 'VHOST' ) && empty( $this->blogid ) )
    521593                        return $old_prefix;
    522594
    523595                $this->prefix = $this->get_blog_prefix( $this->blogid );
    524596
     597                // @todo (hakre) merge those two operations into one
    525598                foreach ( (array) $this->tables( 'blog' ) as $table )
    526599                        $this->$table = $this->prefix . $table;
    527 
     600                       
    528601                foreach ( (array) $this->tables( 'old' ) as $table )
    529602                        $this->$table = $this->prefix . $table;
    530603
    531                 if ( defined('CUSTOM_USER_TABLE') )
     604                if ( defined( 'CUSTOM_USER_TABLE' ) )
    532605                        $this->users = CUSTOM_USER_TABLE;
    533606
    534                 if ( defined('CUSTOM_USER_META_TABLE') )
     607                if ( defined( 'CUSTOM_USER_META_TABLE' ) )
    535608                        $this->usermeta = CUSTOM_USER_META_TABLE;
    536609
    537610                return $old_prefix;
    538611        }
    539612
    540         function set_blog_id($blog_id, $site_id = '') {
     613        /**
     614         * blog id setter
     615         *
     616         * {@internal Missing Description}}
     617         *
     618         * @since 3.0
     619         * @access public
     620         * @param string $blog_id
     621         * @param string $site_id (optional)
     622         * @return string previous blog id
     623         */     
     624        function set_blog_id( $blog_id, $site_id = '' ) {
    541625                if ( !empty($site_id) )
    542626                        $this->siteid = $site_id;
    543627
    544                 $old_blog_id = $this->blogid;
     628                $old_blog_id  = $this->blogid;
    545629                $this->blogid = $blog_id;
    546630
    547631                $this->prefix = $this->get_blog_prefix( $this->blogid );
    548632
    549                 foreach ( $this->tables( 'blog' ) as $table )
     633                foreach ( $this->tables as $table )
    550634                        $this->$table = $this->prefix . $table;
    551635
    552                 foreach ( $this->tables( 'old' ) as $table )
    553                         $this->$table = $this->prefix . $table;
    554 
    555636                return $old_blog_id;
    556637        }
    557638
     639        /**
     640         * blog prefix getter
     641         *
     642         * {@internal Missing Description}}
     643         *
     644         * @param string $blog_id (optional)
     645         * @return unknown_type
     646         */
    558647        function get_blog_prefix( $blog_id = '' ) {
    559648                if ( is_multisite() && $blog_id ) {
    560649                        if ( defined('MULTISITE') && ( $blog_id == 0 || $blog_id == 1 ) )
    … …  
    570659         * Returns an array of WordPress tables.
    571660         *
    572661         * @since 3.0.0
    573          * @uses wpdb::tables
    574          * @uses wpdb::old_tables
    575          * @uses wpdb::global_tables
     662         * @uses wpdb::$tables
     663         * @uses wpdb::$old_tables
     664         * @uses wpdb::$global_tables
     665         * @uses wpdb::$ms_tables
    576666         * @uses is_multisite()
    577667         *
    578668         * @param string $scope Can be all, global, blog, or old tables. Default all.
    … …  
    620710         * @param string $db MySQL database name
    621711         * @return null Always null.
    622712         */
    623         function select($db, &$dbh) {
    624                 if (!@mysql_select_db($db, $dbh)) {
     713        function select( $db, &$dbh ) {
     714                if ( !@mysql_select_db( $db, $dbh ) ) {
    625715                        $this->ready = false;
    626                         $this->bail(sprintf(/*WP_I18N_DB_SELECT_DB*/'
     716                        $this->bail( sprintf( /*WP_I18N_DB_SELECT_DB*/'
    627717<h1>Can&#8217;t select database</h1>
    628718<p>We were able to connect to the database server (which means your username and password is okay) but not able to select the <code>%1$s</code> database.</p>
    629719<ul>
    … …  
    631721<li>Does the user <code>%2$s</code> have permission to use the <code>%1$s</code> database?</li>
    632722<li>On some systems the name of your database is prefixed with your username, so it would be like <code>username_%1$s</code>. Could that be the problem?</li>
    633723</ul>
    634 <p>If you don\'t know how to setup a database you should <strong>contact your host</strong>. If all else fails you may find help at the <a href="http://wordpress.org/support/">WordPress Support Forums</a>.</p>'/*/WP_I18N_DB_SELECT_DB*/, $db, $this->dbuser), 'db_select_fail');
     724<p>If you don\'t know how to setup a database you should <strong>contact your host</strong>. If all else fails you may find help at the <a href="http://wordpress.org/support/">WordPress Support Forums</a>.</p>'/*/WP_I18N_DB_SELECT_DB*/, $db, $this->dbuser ), 'db_select_fail' );
    635725                        return;
    636726                }
    637727        }
    638728
    639         function _weak_escape($string) {
    640                 return addslashes($string);
     729        /**
     730         * weak escape
     731         *
     732         * an alias to addslashes()
     733         *
     734         * @see addslashes()
     735         *
     736         * @since {@internal Version Unknown}}
     737         * @access private
     738         * @param string $string
     739         * @return string
     740         */
     741        function _weak_escape( $string ) {
     742                return addslashes( $string );
    641743        }
    642744
    643         function _real_escape($string) {
     745        /**
     746         * real escape
     747         *
     748         * escape via mysql_real_escape_string() or addslashes() 
     749         *
     750         * @since  2.8
     751         * @access private
     752         *
     753         * @param  string $string to escape
     754         * @return string escaped
     755         */
     756        function _real_escape( $string ) {
    644757                if ( $this->dbh && $this->real_escape )
    645758                        return mysql_real_escape_string( $string, $this->dbh );
    646759                else
    647760                        return addslashes( $string );
    648761        }
    649762
    650         function _escape($data) {
    651                 if ( is_array($data) ) {
     763        /**
     764         * escape
     765         *
     766         * escape data, uses {@see _real_escape()} and works
     767         * on arrays as well.
     768         *
     769         * @since  2.8
     770         * @access private
     771         *
     772         * @param  string|array $data to escape
     773         * @return string|array escaped
     774         */
     775        function _escape( $data ) {
     776                if ( is_array( $data ) ) {
    652777                        foreach ( (array) $data as $k => $v ) {
    653778                                if ( is_array($v) )
    654779                                        $data[$k] = $this->_escape( $v );
    … …  
    667792         *
    668793         * @since 0.71
    669794         *
    670          * @param string|array $data
    671          * @return string query safe string
     795         * @param  string|array $data to escape
     796         * @return string|array escaped as query safe string
    672797         */
    673         function escape($data) {
    674                 if ( is_array($data) ) {
     798        function escape( $data ) {
     799                if ( is_array( $data ) ) {
    675800                        foreach ( (array) $data as $k => $v ) {
    676                                 if ( is_array($v) )
     801                                if ( is_array( $v ) )
    677802                                        $data[$k] = $this->escape( $v );
    678803                                else
    679804                                        $data[$k] = $this->_weak_escape( $v );
    … …  
    690815         *
    691816         * @since 2.3.0
    692817         *
    693          * @param string $s
     818         * @param  string $string to escape
     819         * @return void
    694820         */
    695         function escape_by_ref(&$string) {
     821        function escape_by_ref( &$string ) {
    696822                $string = $this->_real_escape( $string );
    697823        }
    698824
    699825        /**
    700826         * Prepares a SQL query for safe execution.  Uses sprintf()-like syntax.
    701827         *
     828         * The following directives can be used in the query format string:
     829         *
     830         *   %d (decimal number) 
     831         *   %s (string)
     832         *   %% (literal percentage sign - no argument needed)
     833         *
     834         * Both %d and %s are to be left unquoted in the query string and   
     835         * they need an argument passed for them.
     836         * Literals (%) as parts of the query must be properly written
     837         * as %%.
     838         * 
    702839         * This function only supports a small subset of the sprintf syntax; it only supports %d (decimal number), %s (string).
    703840         * Does not support sign, padding, alignment, width or precision specifiers.
    704841         * Does not support argument numbering/swapping.
    … …  
    709846         *
    710847         * <code>
    711848         * wpdb::prepare( "SELECT * FROM `table` WHERE `column` = %s AND `field` = %d", "foo", 1337 )
     849         * wpdb::prepare( "SELECT DATE_FORMAT(`field`, '%%c') FROM `table` WHERE `column` = %s", 'foo' );
    712850         * </code>
    713851         *
    714852         * @link http://php.net/sprintf Description of syntax.
    715853         * @since 2.3.0
    716854         *
    717855         * @param string $query Query statement with sprintf()-like placeholders
    718          * @param array|mixed $args The array of variables to substitute into the query's placeholders if being called like {@link http://php.net/vsprintf vsprintf()}, or the first variable to substitute into the query's placeholders if being called like {@link http://php.net/sprintf sprintf()}.
     856         * @param mixed $args The array of variables to substitute into the query's placeholders if being called like {@link http://php.net/vsprintf vsprintf()}, or the first variable to substitute into the query's placeholders if being called like {@link http://php.net/sprintf sprintf()}.
    719857         * @param mixed $args,... further variables to substitute into the query's placeholders if being called like {@link http://php.net/sprintf sprintf()}.
    720          * @return null|string Sanitized query string
     858         * @return null|false|string Sanitized query string, null if there is no query, false if there is an error and string if there was something to prepare
    721859         */
    722         function prepare($query = null) { // ( $query, *$args )
     860        function prepare( $query = null ) { // ( $query, *$args )
    723861                if ( is_null( $query ) )
    724862                        return;
     863
    725864                $args = func_get_args();
    726                 array_shift($args);
    727                 // If args were passed as an array (as in vsprintf), move them up
    728                 if ( isset($args[0]) && is_array($args[0]) )
    729                         $args = $args[0];
    730                 $query = str_replace("'%s'", '%s', $query); // in case someone mistakenly already singlequoted it
    731                 $query = str_replace('"%s"', '%s', $query); // doublequote unquoting
    732                 $query = str_replace('%s', "'%s'", $query); // quote the strings
    733                 array_walk($args, array(&$this, 'escape_by_ref'));
    734                 return @vsprintf($query, $args);
     865                array_shift( $args );
     866                if ( isset( $args[0] ) && is_array( $args[0] ) )
     867                        $args = $args[0]; # re-assign args passed as array like in vsprintf
     868
     869                $query = str_replace( "'%s'", '%s', $query ); // in case someone mistakenly already singlequoted it
     870                $query = str_replace( '"%s"', '%s', $query ); // doublequote unquoting
     871                $query = str_replace( '%s', "'%s'", $query ); // quote the strings
     872                array_walk( $args, array( &$this, 'escape_by_ref' ) );
     873                return @vsprintf( $query, $args );
    735874        }
    736875
    737876        /**
    … …  
    743882         * @param string $str The error to display
    744883         * @return bool False if the showing of errors is disabled.
    745884         */
    746         function print_error($str = '') {
     885        function print_error( $str = '' ) {
    747886                global $EZSQL_ERROR;
    748887
    749                 if (!$str) $str = mysql_error($this->dbh);
    750                 $EZSQL_ERROR[] = array ('query' => $this->last_query, 'error_str' => $str);
     888                if ( !$str ) $str = mysql_error( $this->dbh );
     889                $EZSQL_ERROR[] = array( 'query' => $this->last_query, 'error_str' => $str );
    751890
    752891                if ( $this->suppress_errors )
    753892                        return false;
    … …  
    757896                else
    758897                        $error_str = sprintf(/*WP_I18N_DB_QUERY_ERROR*/'WordPress database error %1$s for query %2$s'/*/WP_I18N_DB_QUERY_ERROR*/, $str, $this->last_query);
    759898
    760                 $log_error = true;
    761                 if ( ! function_exists('error_log') )
    762                         $log_error = false;
     899                if ( function_exists( 'error_log' )
     900                     && !( $log_file = @ini_get( 'error_log' ) )
     901                     && ( 'syslog' != $log_file )
     902                     && @is_writable( $log_file ) )
     903                    @error_log( $error_str );
    763904
    764                 $log_file = @ini_get('error_log');
    765                 if ( !empty($log_file) && ('syslog' != $log_file) && !@is_writable($log_file) )
    766                         $log_error = false;
    767 
    768                 if ( $log_error )
    769                         @error_log($error_str, 0);
    770 
    771905                // Is error output turned on or not..
    772906                if ( !$this->show_errors )
    773907                        return false;
    … …  
    778912                        if ( defined( 'ERRORLOGFILE' ) )
    779913                                error_log( $msg, 3, ERRORLOGFILE );
    780914                        if ( defined( 'DIEONDBERROR' ) )
    781                                 die( $msg );
     915                                wp_die( $msg );
    782916                } else {
    783                         $str = htmlspecialchars($str, ENT_QUOTES);
    784                         $query = htmlspecialchars($this->last_query, ENT_QUOTES);
     917                        $str   = htmlspecialchars( $str, ENT_QUOTES );
     918                        $query = htmlspecialchars( $this->last_query, ENT_QUOTES );
    785919
    786920                        print "<div id='error'>
    787921                        <p class='wpdberror'><strong>WordPress database error:</strong> [$str]<br />
    … …  
    799933         * errors.
    800934         *
    801935         * @since 0.71
     936         * @see hide_errors()
    802937         *
    803938         * @param bool $show Whether to show or hide errors
    804939         * @return bool Old value for showing errors.
    … …  
    812947        /**
    813948         * Disables showing of database errors.
    814949         *
     950         * By default database errors are not shown.
     951         *
    815952         * @since 0.71
     953         * @see show_errors()
    816954         *
    817955         * @return bool Whether showing of errors was active or not
    818956         */
    … …  
    825963        /**
    826964         * Whether to suppress database errors.
    827965         *
    828          * @param unknown_type $suppress
    829          * @return unknown
     966         * By default database errors are suppressed, with a simple
     967         * call to this function they can be enabled.
     968         *
     969         * @since 2.5
     970         * @see hide_errors()
     971         * @param bool $suppress (optional) new value, defaults to true
     972         * @return bool old suppress errors value
    830973         */
    831974        function suppress_errors( $suppress = true ) {
    832975                $errors = $this->suppress_errors;
    833                 $this->suppress_errors = $suppress;
     976                $this->suppress_errors = (bool) $suppress;
    834977                return $errors;
    835978        }
    836979
    … …  
    838981         * Kill cached query results.
    839982         *
    840983         * @since 0.71
     984         *
     985         * @return void
    841986         */
    842987        function flush() {
    843988                $this->last_result = array();
    844                 $this->col_info = null;
    845                 $this->last_query = null;
     989                $this->col_info    = null;
     990                $this->last_query  = null;
    846991        }
    847992
    848993        function db_connect( $query = "SELECT" ) {
    … …  
    8651010                }
    8661011
    8671012                $dbhname = "dbh" . $action;
    868                 $this->$dbhname = @mysql_connect( $details[ 'db_host' ], $details[ 'db_user' ], $details[ 'db_password' ] );
     1013                $this->$dbhname = @mysql_connect( $details[ 'db_host' ], $details[ 'db_user' ], $details[ 'db_password' ] );           
    8691014                if (!$this->$dbhname ) {
    870                         $this->bail("
     1015                        $this->bail( sprintf( /*WP_I18N_DB_CONNECT_DB*/'
    8711016<h1>Error establishing a database connection</h1>
    872 <p>This either means that the username and password information in your <code>wp-config.php</code> file is incorrect or we can't contact the database server at <code>{$details['db_host']}</code>. This could mean your host's database server is down.</p>
     1017<p>This either means that the username and password information in your <code>wp-config.php</code> file is incorrect or we can\'t contact the database server at <code>%s</code>. This could mean your host\'s database server is down.</p>
    8731018<ul>
    8741019        <li>Are you sure you have the correct username and password?</li>
    8751020        <li>Are you sure that you have typed the correct hostname?</li>
    8761021        <li>Are you sure that the database server is running?</li>
    8771022</ul>
    878 <p>If you're unsure what these terms mean you should probably contact your host. If you still need help you can always visit the <a href='http://wordpress.org/support/'>WordPress Support Forums</a>.</p>
    879 ");
     1023<p>If you\'re unsure what these terms mean you should probably contact your host. If you still need help you can always visit the <a href="http://wordpress.org/support/">WordPress Support Forums</a>.</p>'/*/WP_I18N_DB_CONNECT_DB*/, $dbhost) );
    8801024                }
    8811025                $this->select( $details[ 'db_name' ], $this->$dbhname );
    8821026        }
    … …  
    8881032         *
    8891033         * @since 0.71
    8901034         *
    891          * @param string $query
     1035         * @param string $query database query
    8921036         * @return int|false Number of rows affected/selected or false on error
    8931037         */
    894         function query($query) {
     1038        function query( $query ) {
    8951039                if ( ! $this->ready )
    8961040                        return false;
    8971041
    898                 // filter the query, if filters are available
    899                 // NOTE: some queries are made before the plugins have been loaded, and thus cannot be filtered with this method
    900                 if ( function_exists('apply_filters') )
    901                         $query = apply_filters('query', $query);
     1042                if ( function_exists( 'apply_filters' ) )
     1043                        $query = apply_filters( 'query', $query );
    9021044
    903                 // initialise return
     1045                // init return values and object's state
    9041046                $return_val = 0;
    9051047                $this->flush();
    9061048
    … …  
    9101052                // Keep track of the last query for debug..
    9111053                $this->last_query = $query;
    9121054
    913                 // Perform the query via std mysql_query function..
    914                 if ( defined('SAVEQUERIES') && SAVEQUERIES )
     1055                if ( defined( 'SAVEQUERIES' ) && SAVEQUERIES )
    9151056                        $this->timer_start();
    9161057
    9171058                // use $this->dbh for read ops, and $this->dbhwrite for write ops
    … …  
    9391080                        $this->last_db_used = "other/read";
    9401081                }
    9411082
    942                 $this->result = @mysql_query($query, $dbh);
    943                 ++$this->num_queries;
     1083                // Perform the query via std mysql_query function..
     1084                $this->result = @mysql_query( $query, $dbh );
     1085                $this->num_queries++;
    9441086
    9451087                if ( defined('SAVEQUERIES') && SAVEQUERIES )
    9461088                        $this->queries[] = array( $query, $this->timer_stop(), $this->get_caller() );
    9471089
    9481090                // If there is an error then take note of it..
    949                 if ( $this->last_error = mysql_error($dbh) ) {
     1091                if ( $this->last_error = mysql_error( $dbh ) ) {
    9501092                        $this->print_error();
    9511093                        return false;
    9521094                }
    9531095
    954                 if ( preg_match("/^\\s*(insert|delete|update|replace|alter) /i",$query) ) {
    955                         $this->rows_affected = mysql_affected_rows($dbh);
     1096                if ( preg_match( "/^\\s*(insert|delete|update|replace|alter) /i", $query ) ) {
     1097                        $this->rows_affected = mysql_affected_rows( $dbh );
    9561098                        // Take note of the insert_id
    957                         if ( preg_match("/^\\s*(insert|replace) /i",$query) ) {
     1099                        if ( preg_match( "/^\\s*(insert|replace) /i", $query ) ) {
    9581100                                $this->insert_id = mysql_insert_id($dbh);
    9591101                        }
    9601102                        // Return number of rows affected
    9611103                        $return_val = $this->rows_affected;
    9621104                } else {
    9631105                        $i = 0;
    964                         while ($i < @mysql_num_fields($this->result)) {
    965                                 $this->col_info[$i] = @mysql_fetch_field($this->result);
     1106                        while ( $i < @mysql_num_fields( $this->result ) ) {
     1107                                $this->col_info[$i] = @mysql_fetch_field( $this->result );
    9661108                                $i++;
    9671109                        }
    9681110                        $num_rows = 0;
    969                         while ( $row = @mysql_fetch_object($this->result) ) {
     1111                        while ( $row = @mysql_fetch_object( $this->result ) ) {
    9701112                                $this->last_result[$num_rows] = $row;
    9711113                                $num_rows++;
    9721114                        }
    9731115
    974                         @mysql_free_result($this->result);
     1116                        @mysql_free_result( $this->result );
    9751117
    9761118                        // Log number of rows the query returned
     1119                        // and Return number of rows selected
    9771120                        $this->num_rows = $num_rows;
    978 
    979                         // Return number of rows selected
    980                         $return_val = $this->num_rows;
     1121                        $return_val     = $num_rows;
    9811122                }
    9821123
    9831124                return $return_val;
    … …  
    9871128         * Insert a row into a table.
    9881129         *
    9891130         * <code>
     1131         * wpdb::insert( 'table', array( 'column' => 'foo', 'field' => 1337 ) )
    9901132         * wpdb::insert( 'table', array( 'column' => 'foo', 'field' => 1337 ), array( '%s', '%d' ) )
    9911133         * </code>
    9921134         *
    … …  
    9981140         * @param array|string $format (optional) An array of formats to be mapped to each of the value in $data.  If string, that format will be used for all of the values in $data.  A format is one of '%d', '%s' (decimal number, string).  If omitted, all values in $data will be treated as strings.
    9991141         * @return int|false The number of rows inserted, or false on error.
    10001142         */
    1001         function insert($table, $data, $format = null) {
     1143        function insert( $table, $data, $format = null ) {
    10021144                $formats = $format = (array) $format;
    1003                 $fields = array_keys($data);
     1145                $fields = array_keys( $data );
    10041146                $formatted_fields = array();
    10051147                foreach ( $fields as $field ) {
    1006                         if ( !empty($format) )
    1007                                 $form = ( $form = array_shift($formats) ) ? $form : $format[0];
    1008                         elseif ( isset($this->field_types[$field]) )
     1148                        if ( !empty( $format ) )
     1149                                $form = ( $form = array_shift( $formats ) ) ? $form : $format[0];
     1150                        elseif ( isset( $this->field_types[$field] ) )
    10091151                                $form = $this->field_types[$field];
    10101152                        else
    10111153                                $form = '%s';
    10121154                        $formatted_fields[] = $form;
    10131155                }
    10141156                $sql = "INSERT INTO `$table` (`" . implode( '`,`', $fields ) . "`) VALUES ('" . implode( "','", $formatted_fields ) . "')";
    1015                 return $this->query( $this->prepare( $sql, $data) );
     1157                return $this->query( $this->prepare( $sql, $data ) );
    10161158        }
    10171159
    10181160
    … …  
    10201162         * Update a row in the table
    10211163         *
    10221164         * <code>
     1165         * wpdb::update( 'table', array( 'column' => 'foo', 'field' => 1337 ), array( 'ID' => 1 ) )
    10231166         * wpdb::update( 'table', array( 'column' => 'foo', 'field' => 1337 ), array( 'ID' => 1 ), array( '%s', '%d' ), array( '%d' ) )
    10241167         * </code>
    10251168         *
    … …  
    10331176         * @param array|string $format_where (optional) An array of formats to be mapped to each of the values in $where.  If string, that format will be used for all of  the items in $where.  A format is one of '%d', '%s' (decimal number, string).  If omitted, all values in $where will be treated as strings.
    10341177         * @return int|false The number of rows updated, or false on error.
    10351178         */
    1036         function update($table, $data, $where, $format = null, $where_format = null) {
    1037                 if ( !is_array( $where ) )
     1179        function update( $table, $data, $where, $format = null, $where_format = null ) {
     1180                if ( !is_array( $data ) || !is_array( $where ) )
    10381181                        return false;
    10391182
    10401183                $formats = $format = (array) $format;
    10411184                $bits = $wheres = array();
    1042                 foreach ( (array) array_keys($data) as $field ) {
    1043                         if ( !empty($format) )
    1044                                 $form = ( $form = array_shift($formats) ) ? $form : $format[0];
     1185                foreach ( (array) array_keys( $data ) as $field ) {
     1186                        if ( !empty( $format ) )
     1187                                $form = ( $form = array_shift( $formats ) ) ? $form : $format[0];
    10451188                        elseif ( isset($this->field_types[$field]) )
    10461189                                $form = $this->field_types[$field];
    10471190                        else
    … …  
    10501193                }
    10511194
    10521195                $where_formats = $where_format = (array) $where_format;
    1053                 foreach ( (array) array_keys($where) as $field ) {
    1054                         if ( !empty($where_format) )
    1055                                 $form = ( $form = array_shift($where_formats) ) ? $form : $where_format[0];
    1056                         elseif ( isset($this->field_types[$field]) )
     1196                foreach ( (array) array_keys( $where ) as $field ) {
     1197                        if ( !empty( $where_format ) )
     1198                                $form = ( $form = array_shift( $where_formats ) ) ? $form : $where_format[0];
     1199                        elseif ( isset( $this->field_types[$field] ) )
    10571200                                $form = $this->field_types[$field];
    10581201                        else
    10591202                                $form = '%s';
    … …  
    10611204                }
    10621205
    10631206                $sql = "UPDATE `$table` SET " . implode( ', ', $bits ) . ' WHERE ' . implode( ' AND ', $wheres );
    1064                 return $this->query( $this->prepare( $sql, array_merge(array_values($data), array_values($where))) );
     1207                return $this->query( $this->prepare( $sql, array_merge( array_values( $data ), array_values( $where ) ) ) );
    10651208        }
    10661209
    10671210        /**
    … …  
    10731216         *
    10741217         * @since 0.71
    10751218         *
    1076          * @param string|null $query SQL query.  If null, use the result from the previous query.
     1219         * @param string|null $query (optional) SQL query. Defaults to NULL, use the result from the previous query.
    10771220         * @param int $x (optional) Column of value to return.  Indexed from 0.
    10781221         * @param int $y (optional) Row of value to return.  Indexed from 0.
    1079          * @return string Database query result
     1222         * @return string|null Database query result on success (as string), NULL if a value on failure
    10801223         */
    1081         function get_var($query=null, $x = 0, $y = 0) {
    1082                 $this->func_call = "\$db->get_var(\"$query\",$x,$y)";
     1224        function get_var( $query = null, $x = 0, $y = 0 ) {
     1225                $this->func_call = "\$db->get_var(\"$query\", $x, $y)";
    10831226                if ( $query )
    1084                         $this->query($query);
     1227                        $this->query( $query );
    10851228
    10861229                // Extract var out of cached results based x,y vals
    10871230                if ( !empty( $this->last_result[$y] ) ) {
    1088                         $values = array_values(get_object_vars($this->last_result[$y]));
     1231                        $values = array_values( get_object_vars( $this->last_result[$y] ) );
    10891232                }
    10901233
    10911234                // If there is a value return it else return null
    1092                 return (isset($values[$x]) && $values[$x]!=='') ? $values[$x] : null;
     1235                return ( isset( $values[$x] ) && $values[$x] !== '' ) ? $values[$x] : null;
    10931236        }
    10941237
    10951238        /**
    … …  
    11021245         * @param string|null $query SQL query.
    11031246         * @param string $output (optional) one of ARRAY_A | ARRAY_N | OBJECT constants.  Return an associative array (column => value, ...), a numerically indexed array (0 => value, ...) or an object ( ->column = value ), respectively.
    11041247         * @param int $y (optional) Row to return.  Indexed from 0.
    1105          * @return mixed Database query result in format specifed by $output
     1248         * @return mixed Database query result in format specifed by $output or NULL
     1249         * @return void
    11061250         */
    1107         function get_row($query = null, $output = OBJECT, $y = 0) {
     1251        function get_row( $query = null, $output = OBJECT, $y = 0 ) {
    11081252                $this->func_call = "\$db->get_row(\"$query\",$output,$y)";
    11091253                if ( $query )
    1110                         $this->query($query);
     1254                        $this->query( $query );
    11111255                else
    11121256                        return null;
    11131257
    1114                 if ( !isset($this->last_result[$y]) )
     1258                if ( !isset( $this->last_result[$y] ) )
    11151259                        return null;
    11161260
    11171261                if ( $output == OBJECT ) {
    11181262                        return $this->last_result[$y] ? $this->last_result[$y] : null;
    11191263                } elseif ( $output == ARRAY_A ) {
    1120                         return $this->last_result[$y] ? get_object_vars($this->last_result[$y]) : null;
     1264                        return $this->last_result[$y] ? get_object_vars( $this->last_result[$y] ) : null;
    11211265                } elseif ( $output == ARRAY_N ) {
    1122                         return $this->last_result[$y] ? array_values(get_object_vars($this->last_result[$y])) : null;
     1266                        return $this->last_result[$y] ? array_values( get_object_vars( $this->last_result[$y] ) ) : null;
    11231267                } else {
    11241268                        $this->print_error(/*WP_I18N_DB_GETROW_ERROR*/" \$db->get_row(string query, output type, int offset) -- Output type must be one of: OBJECT, ARRAY_A, ARRAY_N"/*/WP_I18N_DB_GETROW_ERROR*/);
    11251269                }
    … …  
    11341278         *
    11351279         * @since 0.71
    11361280         *
    1137          * @param string|null $query SQL query. If null, use the result from the previous query.
    1138          * @param int $x Column to return.  Indexed from 0.
     1281         * @param string|null $query (optional) SQL query. If null, use the result from the previous query.
     1282         * @param int $x (optional) Column to return.  Indexed from 0.
    11391283         * @return array Database query result.  Array indexed from 0 by SQL result row number.
    11401284         */
    1141         function get_col($query = null , $x = 0) {
     1285        function get_col( $query = null , $x = 0 ) {
    11421286                if ( $query )
    1143                         $this->query($query);
     1287                        $this->query( $query );
    11441288
    11451289                $new_array = array();
    11461290                // Extract the column values
    1147                 for ( $i=0; $i < count($this->last_result); $i++ ) {
    1148                         $new_array[$i] = $this->get_var(null, $x, $i);
     1291                for ( $i=0; $i < count( $this->last_result ); $i++ ) {
     1292                        $new_array[$i] = $this->get_var( null, $x, $i );
    11491293                }
    11501294                return $new_array;
    11511295        }
    … …  
    11611305         * @param string $output (optional) ane of ARRAY_A | ARRAY_N | OBJECT | OBJECT_K constants.  With one of the first three, return an array of rows indexed from 0 by SQL result row number.  Each row is an associative array (column => value, ...), a numerically indexed array (0 => value, ...), or an object. ( ->column = value ), respectively.  With OBJECT_K, return an associative array of row objects keyed by the value of each row's first column's value.  Duplicate keys are discarded.
    11621306         * @return mixed Database query results
    11631307         */
    1164         function get_results($query = null, $output = OBJECT) {
     1308        function get_results( $query = null, $output = OBJECT ) {
    11651309                $this->func_call = "\$db->get_results(\"$query\", $output)";
    11661310
    11671311                if ( $query )
    1168                         $this->query($query);
     1312                        $this->query( $query );
    11691313                else
    11701314                        return null;
    11711315
    … …  
    12061350         *
    12071351         * @since 0.71
    12081352         *
    1209          * @param string $info_type one of name, table, def, max_length, not_null, primary_key, multiple_key, unique_key, numeric, blob, type, unsigned, zerofill
    1210          * @param int $col_offset 0: col name. 1: which table the col's in. 2: col's max length. 3: if the col is numeric. 4: col's type
     1353         * @param string $info_type(optional) type one of name, table, def, max_length, not_null, primary_key, multiple_key, unique_key, numeric, blob, type, unsigned, zerofill
     1354         * @param int $col_offset (optional) 0: col name. 1: which table the col's in. 2: col's max length. 3: if the col is numeric. 4: col's type
    12111355         * @return mixed Column Results
    12121356         */
    1213         function get_col_info($info_type = 'name', $col_offset = -1) {
     1357        function get_col_info( $info_type = 'name', $col_offset = -1 ) {
    12141358                if ( $this->col_info ) {
    12151359                        if ( $col_offset == -1 ) {
    1216                                 $i = 0;
     1360                                $i         = 0;
    12171361                                $new_array = array();
    12181362                                foreach( (array) $this->col_info as $col ) {
    12191363                                        $new_array[$i] = $col->{$info_type};
    … …  
    12341378         * @return true
    12351379         */
    12361380        function timer_start() {
    1237                 $mtime = microtime();
    1238                 $mtime = explode(' ', $mtime);
     1381                $mtime            = explode( ' ', microtime() );
    12391382                $this->time_start = $mtime[1] + $mtime[0];
    12401383                return true;
    12411384        }
    … …  
    12481391         * @return int Total time spent on the query, in milliseconds
    12491392         */
    12501393        function timer_stop() {
    1251                 $mtime = microtime();
    1252                 $mtime = explode(' ', $mtime);
    1253                 $time_end = $mtime[1] + $mtime[0];
     1394                $mtime      = explode( ' ', microtime() );
     1395                $time_end   = $mtime[1] + $mtime[0];
    12541396                $time_total = $time_end - $this->time_start;
    12551397                return $time_total;
    12561398        }
    … …  
    12661408         * @param string $error_code (optional) A Computer readable string to identify the error.
    12671409         * @return false|void
    12681410         */
    1269         function bail($message, $error_code = '500') {
     1411        function bail( $message, $error_code = '500' ) {
    12701412                if ( !$this->show_errors ) {
    1271                         if ( class_exists('WP_Error') )
     1413                        if ( class_exists( 'WP_Error' ) )
    12721414                                $this->error = new WP_Error($error_code, $message);
    12731415                        else
    12741416                                $this->error = $message;
    … …  
    12821424         *
    12831425         * @since 2.5.0
    12841426         * @uses $wp_version
     1427         * @uses $required_mysql_version
    12851428         *
    12861429         * @return WP_Error
    12871430         */
    … …  
    12891432                global $wp_version, $required_mysql_version;
    12901433                // Make sure the server has the required MySQL version
    12911434                if ( version_compare($this->db_version(), $required_mysql_version, '<') )
    1292                         return new WP_Error('database_version',sprintf(__('<strong>ERROR</strong>: WordPress %1$s requires MySQL %2$s or higher'), $wp_version, $required_mysql_version));
     1435                        return new WP_Error('database_version', sprintf( __( '<strong>ERROR</strong>: WordPress %1$s requires MySQL %2$s or higher' ), $wp_version, $required_mysql_version ));
    12931436        }
    12941437
    12951438        /**
    … …  
    13061449        }
    13071450
    13081451        /**
    1309          * Generic function to determine if a database supports a particular feature
     1452         * has capability
     1453         *
     1454         * Determine if a database supports a particular feature
     1455         *
     1456         * @since  2.7
     1457         * @see    db_version()
     1458         *
    13101459         * @param string $db_cap the feature
    13111460         * @param false|string|resource $dbh_or_table (not implemented) Which database to test.  False = the currently selected database, string = the database containing the specified table, resource = the database corresponding to the specified mysql resource.
    13121461         * @return bool
    … …  
    13181467                case 'collation' :    // @since 2.5.0
    13191468                case 'group_concat' : // @since 2.7
    13201469                case 'subqueries' :   // @since 2.7
    1321                         return version_compare($version, '4.1', '>=');
     1470                        return version_compare( $version, '4.1', '>=' );
    13221471                        break;
    13231472                endswitch;
    13241473
    … …  
    13361485         * @return string The name of the calling function
    13371486         */
    13381487        function get_caller() {
    1339                 $bt = debug_backtrace();
     1488                $trace  = array_reverse( debug_backtrace() );
    13401489                $caller = array();
    13411490
    1342                 $bt = array_reverse( $bt );
    1343                 foreach ( (array) $bt as $call ) {
     1491                foreach ( $trace as $call ) {
    13441492                        if ( isset( $call['class'] ) && __CLASS__ == $call['class'] )
    1345                                 continue;
     1493                                continue; # filter out function calls of this object's class
     1494
    13461495                        $function = $call['function'];
     1496
    13471497                        if ( isset( $call['class'] ) )
    13481498                                $function = $call['class'] . "->$function";
     1499
    13491500                        $caller[] = $function;
    13501501                }
    1351                 $caller = join( ', ', $caller );
    13521502
    1353                 return $caller;
     1503                return join( ', ', $caller );
    13541504        }
    13551505
    13561506        /**
    1357          * The database version number
    1358          * @param false|string|resource $dbh_or_table (not implemented) Which database to test.  False = the currently selected database, string = the database containing the specified table, resource = the database corresponding to the specified mysql resource.
     1507         * get database version number
     1508         *
     1509         * ADDITIONAL PARAMETER NOTICE
     1510         *
     1511         * there once was a proposed second parameter which has never been
     1512         * implemented. It was describben as "Which database to test" ($dbh_or_table)
     1513         *
     1514         * It would have had three different types:
     1515         *
     1516         *      false    : currently selected database
     1517         *  string   : database containing this table
     1518         *  resource : database by mysql resource
     1519         *
     1520         * regarding that third parameter please see {@see db_version()} as well
     1521         *
     1522         * @since  2.7
     1523         * @see    has_cap()
     1524         *
    13591525         * @return false|string false on failure, version number on success
    13601526         */
    13611527        function db_version() {
    1362                 return preg_replace('/[^0-9.].*/', '', mysql_get_server_info( $this->dbh ));
     1528                return preg_replace( '/[^0-9.].*/', '', mysql_get_server_info( $this->dbh ) );
    13631529        }
    13641530}
    13651531
    … …  
    13711537         */
    13721538        $wpdb = new wpdb(DB_USER, DB_PASSWORD, DB_NAME, DB_HOST);
    13731539}
    1374 ?>
     1540?>
     1541 No newline at end of file