Skip to:
Content

bbPress.org


Ignore:
Timestamp:
02/19/2018 07:00:55 AM (7 years ago)
Author:
johnjamesjacoby
Message:

Converter: remove WP_SETUP_CONFIG constant setting from converter process.

This introduces a helper database class to avoid directly connecting to the external database. Instead, we'll attempt to control that connection, and provide feedback to the user in the event a connection cannot be made. The WP_SETUP_CONFIG was causing calls to the options API to fail, resulting in broken calculations and invalid offsets.

This commit includes some general load order clean-up, which also fixes a regression causing fatal errors when attempting to upgrade converted user passwords from the old platform. It also fixes the condition where boundaries being converted would display beyond their maximums.

Fixes #3191.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/includes/admin/classes/class-bbp-converter-base.php

    r6780 r6785  
    139139        // Maybe add port to server
    140140        if ( ! empty( $db_port ) && ! empty( $db_host ) && ! strstr( $db_host, ':' ) ) {
    141             $db_host = $db_host . ':' . $db_port;
     141            $db_host = "{$db_host}:{$db_port}";
    142142        }
    143143
    … …  
    147147        $this->wpdb = bbp_db();
    148148
    149         // Control WPDB db_connect() bailing
    150         define( 'WP_SETUP_CONFIG', true );
    151 
    152149        // Setup old forum Database
    153         $this->opdb = new wpdb( $db_user, $db_pass, $db_name, $db_host );
     150        $this->opdb = new BBP_Converter_DB( $db_user, $db_pass, $db_name, $db_host );
    154151
    155152        // Connection failed
    … …  
    499496                            /** New user **************************************/
    500497
    501                             case 'user':
     498                            case 'user' :
    502499                                if ( username_exists( $insert_post['user_login'] ) ) {
    503                                     $insert_post['user_login'] = 'imported_' . $insert_post['user_login'];
     500                                    $insert_post['user_login'] = "imported_{$insert_post['user_login']}";
    504501                                }
    505502
    506503                                if ( email_exists( $insert_post['user_email'] ) ) {
    507                                     $insert_post['user_email'] = 'imported_' . $insert_post['user_email'];
     504                                    $insert_post['user_email'] = "imported_{$insert_post['user_email']}";
    508505                                }
    509506
    … …  
    523520
    524521                                        if ( '_id' == substr( $key, -3 ) && ( true === $this->sync_table ) ) {
    525                                             $this->wpdb->insert( $this->sync_table_name, array( 'value_type' => 'user', 'value_id' => $post_id, 'meta_key' => $key, 'meta_value' => $value ) );
     522                                            $this->wpdb->insert( $this->sync_table_name, array(
     523                                                'value_type' => 'user',
     524                                                'value_id'   => $post_id,
     525                                                'meta_key'   => $key,
     526                                                'meta_value' => $value
     527                                            ) );
    526528                                        }
    527529                                    }
    … …  
    531533                            /** New Topic-Tag *********************************/
    532534
    533                             case 'tags':
     535                            case 'tags' :
    534536                                $post_id = wp_set_object_terms( $insert_postmeta['objectid'], $insert_postmeta['name'], 'topic-tag', true );
    535537                                $term = get_term_by( 'name', $insert_postmeta['name'], 'topic-tag');
    … …  
    544546                            /** Forum Subscriptions ***************************/
    545547
    546                             case 'forum_subscriptions':
     548                            case 'forum_subscriptions' :
    547549                                $user_id = $insert_post['user_id'];
    548550                                $items   = wp_list_pluck( $insert_postmeta, '_bbp_forum_subscriptions' );
    … …  
    565567                            /** Subscriptions *********************************/
    566568
    567                             case 'topic_subscriptions':
     569                            case 'topic_subscriptions' :
    568570                                $user_id = $insert_post['user_id'];
    569571                                $items   = wp_list_pluck( $insert_postmeta, '_bbp_subscriptions' );
    … …  
    586588                            /** Favorites *************************************/
    587589
    588                             case 'favorites':
     590                            case 'favorites' :
    589591                                $user_id = $insert_post['user_id'];
    590592                                $items   = wp_list_pluck( $insert_postmeta, '_bbp_favorites' );
    … …  
    607609                            /** Forum, Topic, Reply ***************************/
    608610
    609                             default:
     611                            default :
    610612                                $post_id = wp_insert_post( $insert_post, true );
    611613
    … …  
    634636                                         */
    635637                                        if ( '_id' === substr( $key, -3 ) && ( true === $this->sync_table ) ) {
    636                                             $this->wpdb->insert( $this->sync_table_name, array( 'value_type' => 'post', 'value_id' => $post_id, 'meta_key' => $key, 'meta_value' => $value ) );
     638                                            $this->wpdb->insert( $this->sync_table_name, array(
     639                                                'value_type' => 'post',
     640                                                'value_id'   => $post_id,
     641                                                'meta_key'   => $key,
     642                                                'meta_value' => $value
     643                                            ) );
    637644                                        }
    638645
    … …  
    665672        $has_update = false;
    666673        $query      = ! empty( $this->sync_table )
    667             ? $this->wpdb->prepare( "SELECT value_id, meta_value FROM {$this->sync_table_name} WHERE meta_key = %s AND meta_value > 0 LIMIT {$start}, {$this->max_rows}", '_bbp_old_forum_parent_id' )
     674            ? $this->wpdb->prepare( "SELECT value_id, meta_value FROM {$this->sync_table_name} WHERE meta_key = %s AND meta_value > 0 LIMIT {$start}, {$this->max_rows}",           '_bbp_old_forum_parent_id' )
    668675            : $this->wpdb->prepare( "SELECT post_id AS value_id, meta_value FROM {$this->wpdb->postmeta} WHERE meta_key = %s AND meta_value > 0 LIMIT {$start}, {$this->max_rows}", '_bbp_old_forum_parent_id' );
    669676
    … …  
    685692        $has_update = false;
    686693        $query      = ! empty( $this->sync_table )
    687             ? $this->wpdb->prepare( "SELECT value_id, meta_value FROM {$this->sync_table_name} WHERE meta_key = %s AND meta_value = %s LIMIT {$start}, {$this->max_rows}", '_bbp_old_sticky_status_id', 'sticky' )
     694            ? $this->wpdb->prepare( "SELECT value_id, meta_value FROM {$this->sync_table_name} WHERE meta_key = %s AND meta_value = %s LIMIT {$start}, {$this->max_rows}",           '_bbp_old_sticky_status_id', 'sticky' )
    688695            : $this->wpdb->prepare( "SELECT post_id AS value_id, meta_value FROM {$this->wpdb->postmeta} WHERE meta_key = %s AND meta_value = %s LIMIT {$start}, {$this->max_rows}", '_bbp_old_sticky_status_id', 'sticky' );
    689696
    … …  
    704711        $has_update = false;
    705712        $query      = ! empty( $this->sync_table )
    706             ? $this->wpdb->prepare( "SELECT value_id, meta_value FROM {$this->sync_table_name} WHERE meta_key = %s AND meta_value = %s LIMIT {$start}, {$this->max_rows}", '_bbp_old_sticky_status_id', 'super-sticky' )
     713            ? $this->wpdb->prepare( "SELECT value_id, meta_value FROM {$this->sync_table_name} WHERE meta_key = %s AND meta_value = %s LIMIT {$start}, {$this->max_rows}",           '_bbp_old_sticky_status_id', 'super-sticky' )
    707714            : $this->wpdb->prepare( "SELECT post_id AS value_id, meta_value FROM {$this->wpdb->postmeta} WHERE meta_key = %s AND meta_value = %s LIMIT {$start}, {$this->max_rows}", '_bbp_old_sticky_status_id', 'super-sticky' );
    708715
    … …  
    724731        $has_update = false;
    725732        $query      = ! empty( $this->sync_table )
    726             ? $this->wpdb->prepare( "SELECT value_id, meta_value FROM {$this->sync_table_name} WHERE meta_key = %s AND meta_value = %s LIMIT {$start}, {$this->max_rows}", '_bbp_old_closed_status_id', 'closed' )
     733            ? $this->wpdb->prepare( "SELECT value_id, meta_value FROM {$this->sync_table_name} WHERE meta_key = %s AND meta_value = %s LIMIT {$start}, {$this->max_rows}",           '_bbp_old_closed_status_id', 'closed' )
    727734            : $this->wpdb->prepare( "SELECT post_id AS value_id, meta_value FROM {$this->wpdb->postmeta} WHERE meta_key = %s AND meta_value = %s LIMIT {$start}, {$this->max_rows}", '_bbp_old_closed_status_id', 'closed' );
    728735
    … …  
    743750        $has_update = false;
    744751        $query      = ! empty( $this->sync_table )
    745             ? $this->wpdb->prepare( "SELECT value_id, meta_value FROM {$this->sync_table_name} WHERE meta_key = %s AND meta_value > 0 LIMIT {$start}, {$this->max_rows}", '_bbp_old_reply_to_id' )
     752            ? $this->wpdb->prepare( "SELECT value_id, meta_value FROM {$this->sync_table_name} WHERE meta_key = %s AND meta_value > 0 LIMIT {$start}, {$this->max_rows}",           '_bbp_old_reply_to_id' )
    746753            : $this->wpdb->prepare( "SELECT post_id AS value_id, meta_value FROM {$this->wpdb->postmeta} WHERE meta_key = %s AND meta_value > 0 LIMIT {$start}, {$this->max_rows}", '_bbp_old_reply_to_id' );
    747754
    … …  
    766773        if ( ! empty( $this->sync_table ) ) {
    767774            $query = $this->wpdb->prepare( "SELECT sync_table1.value_id AS topic_id, sync_table1.meta_value AS topic_is_anonymous, sync_table2.meta_value AS topic_author
    768                             FROM {$this->sync_table_name} AS sync_table1
    769                             INNER JOIN {$this->sync_table_name} AS sync_table2
    770                             ON ( sync_table1.value_id = sync_table2.value_id )
    771                             WHERE sync_table1.meta_value = %s
    772                             AND sync_table2.meta_key = %s
    773                             LIMIT {$start}, {$this->max_rows}", 'true', '_bbp_old_topic_author_name_id' );
     775                FROM {$this->sync_table_name} AS sync_table1
     776                INNER JOIN {$this->sync_table_name} AS sync_table2
     777                ON ( sync_table1.value_id = sync_table2.value_id )
     778                WHERE sync_table1.meta_value = %s
     779                AND sync_table2.meta_key = %s
     780                LIMIT {$start}, {$this->max_rows}", 'true', '_bbp_old_topic_author_name_id' );
    774781        } else {
    775782            $query = $this->wpdb->prepare( "SELECT wp_postmeta1.post_id AS topic_id, wp_postmeta1.meta_value AS topic_is_anonymous, wp_postmeta2.meta_value AS topic_author
    776                             FROM {$this->wpdb->postmeta} AS wp_postmeta1
    777                             INNER JOIN {$this->wpdb->postmeta} AS wp_postmeta2
    778                             ON ( wp_postmeta1.post_id = wp_postmeta2.post_id )
    779                             WHERE wp_postmeta1.meta_value = %s
    780                             AND wp_postmeta2.meta_key = %s
    781                             LIMIT {$start}, {$this->max_rows}", 'true', '_bbp_old_topic_author_name_id' );
     783                FROM {$this->wpdb->postmeta} AS wp_postmeta1
     784                INNER JOIN {$this->wpdb->postmeta} AS wp_postmeta2
     785                ON ( wp_postmeta1.post_id = wp_postmeta2.post_id )
     786                WHERE wp_postmeta1.meta_value = %s
     787                AND wp_postmeta2.meta_key = %s
     788                LIMIT {$start}, {$this->max_rows}", 'true', '_bbp_old_topic_author_name_id' );
    782789        }
    783790
    … …  
    805812        if ( ! empty( $this->sync_table ) ) {
    806813            $query = $this->wpdb->prepare( "SELECT sync_table1.value_id AS reply_id, sync_table1.meta_value AS reply_is_anonymous, sync_table2.meta_value AS reply_author
    807                             FROM {$this->sync_table_name} AS sync_table1
    808                             INNER JOIN {$this->sync_table_name} AS sync_table2
    809                             ON ( sync_table1.value_id = sync_table2.value_id )
    810                             WHERE sync_table1.meta_value = %s
    811                             AND sync_table2.meta_key = %s
    812                             LIMIT {$start}, {$this->max_rows}", 'true', '_bbp_old_reply_author_name_id' );
     814                FROM {$this->sync_table_name} AS sync_table1
     815                INNER JOIN {$this->sync_table_name} AS sync_table2
     816                ON ( sync_table1.value_id = sync_table2.value_id )
     817                WHERE sync_table1.meta_value = %s
     818                AND sync_table2.meta_key = %s
     819                LIMIT {$start}, {$this->max_rows}", 'true', '_bbp_old_reply_author_name_id' );
    813820        } else {
    814821            $query = $this->wpdb->prepare( "SELECT wp_postmeta1.post_id AS reply_id, wp_postmeta1.meta_value AS reply_is_anonymous, wp_postmeta2.meta_value AS reply_author
    815                             FROM {$this->wpdb->postmeta} AS wp_postmeta1
    816                             INNER JOIN {$this->wpdb->postmeta} AS wp_postmeta2
    817                             ON ( wp_postmeta1.post_id = wp_postmeta2.post_id )
    818                             WHERE wp_postmeta1.meta_value = %s
    819                             AND wp_postmeta2.meta_key = %s
    820                             LIMIT {$start}, {$this->max_rows}", 'true', '_bbp_old_reply_author_name_id' );
     822                FROM {$this->wpdb->postmeta} AS wp_postmeta1
     823                INNER JOIN {$this->wpdb->postmeta} AS wp_postmeta2
     824                ON ( wp_postmeta1.post_id = wp_postmeta2.post_id )
     825                WHERE wp_postmeta1.meta_value = %s
     826                AND wp_postmeta2.meta_key = %s
     827                LIMIT {$start}, {$this->max_rows}", 'true', '_bbp_old_reply_author_name_id' );
    821828        }
    822829
    … …  
    928935     * @param string The table name to grab fields from
    929936     */
    930     private function get_fields( $tablename ) {
    931         $rval        = array();
    932         $field_array = $this->get_results( 'DESCRIBE ' . $tablename, ARRAY_A );
    933 
     937    private function get_fields( $tablename = '' ) {
     938        $retval      = array();
     939        $field_array = $this->get_results( "DESCRIBE {$tablename}", ARRAY_A );
     940
     941        // Bail if no fields
     942        if ( empty( $field_array ) ) {
     943            return $retval;
     944        }
     945
     946        // Add fields to array
    934947        foreach ( $field_array as $field ) {
    935             $rval[] = $field['Field'];
    936         }
    937 
     948            if ( ! empty( $field['Field'] ) ) {
     949                $retval[] = $field['Field'];
     950            }
     951        }
     952
     953        // Add social fields for users table
    938954        if ( $tablename === $this->wpdb->users ) {
    939             $rval[] = 'role';
    940             $rval[] = 'yim';
    941             $rval[] = 'aim';
    942             $rval[] = 'jabber';
    943         }
    944 
    945         return $rval;
     955            $retval[] = 'role';
     956            $retval[] = 'yim';
     957            $retval[] = 'aim';
     958            $retval[] = 'jabber';
     959        }
     960
     961        return $retval;
    946962    }
    947963
Note: See TracChangeset for help on using the changeset viewer.