Make WordPress Core


Ignore:
Timestamp:
09/10/2015 07:13:02 AM (9 years ago)
Author:
pento
Message:

WPDB: get_table_from_query() didn't find table names with hyphens in them.

Merge of [33718] to the 3.8 branch.

Props dustinbolton, pento.

See #33470.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • branches/3.8/tests/phpunit/tests/db.php

    r32415 r33996  
    291291    function data_get_table_from_query() {
    292292        $table = 'a_test_table_name';
    293         $db_table = '`a_test_db`.`another_test_table`';
     293        $more_tables = array(
     294            // table_name => expected_value
     295            '`a_test_db`.`another_test_table`' => 'a_test_db.another_test_table',
     296            'a-test-with-dashes'               => 'a-test-with-dashes',
     297        );
    294298
    295299        $queries = array(
     
    396400        $querycount = count( $queries );
    397401        for ( $ii = 0; $ii < $querycount; $ii++ ) {
    398             $db_query = str_replace( $table, $db_table, $queries[ $ii ] );
    399             $expected_db_table = str_replace( '`', '', $db_table );
     402            foreach ( $more_tables as $name => $expected_name ) {
     403                $new_query = str_replace( $table, $name, $queries[ $ii ] );
     404                $queries[] = array( $new_query, $expected_name );
     405            }
    400406
    401407            $queries[ $ii ] = array( $queries[ $ii ], $table );
    402             $queries[] = array( $db_query, $expected_db_table );
    403408        }
    404409        return $queries;
Note: See TracChangeset for help on using the changeset viewer.