Skip to:
Content

bbPress.org


Ignore:
Timestamp:
06/23/2012 06:49:39 AM (14 years ago)
Author:
johnjamesjacoby
Message:

Converter:

  • Pinking shears.
  • Run $field_list through array_unique() to remove duplicate fields.
  • Properly internationalize converter output strings.
File:
1 edited

Legend:

Unmodified
Added
Removed

  • branches/plugin/bbp-admin/bbp-converter.php

    r4000 r4003  
    146146                padding: 2px;
    147147            }
    148            
     148
    149149            div.bbp-converter-updated p strong.loading {
    150150                padding: 2px 20px 2px 0;
    … …  
    176176                    values[field.name] = field.value;
    177177                });
    178                
     178
    179179                if( values['_bbp_converter_restart'] ) {
    180180                    jQuery('#_bbp_converter_restart').removeAttr("checked");
    … …  
    219219            function bbconverter_success(response) {
    220220                bbconverter_log(response);
    221                
     221
    222222                if ( response == 'Conversion Complete' || response.indexOf('error') > -1 ) {
    223223                    bbconverter_log('<b>Repair any missing information: <a href="<?php echo admin_url(); ?>tools.php?page=bbp-repair">Continue</a></b>');
    … …  
    276276        $step  = (int) get_option( '_bbp_converter_step',  1 );
    277277        $min   = (int) get_option( '_bbp_converter_start', 0 );
    278         $count = (int) empty( $_POST['_bbp_converter_rows'] ) ? $_POST['_bbp_converter_rows'] : 100;
     278        $count = (int) ! empty( $_POST['_bbp_converter_rows'] ) ? $_POST['_bbp_converter_rows'] : 100;
    279279        $max   = ( $min + $count ) - 1;
    280280        $start = $min;
    … …  
    297297                        update_option( '_bbp_converter_start', 0         );
    298298                        $this->sync_table();
    299 
    300299                        if ( empty( $start ) ) {
    301300                            _e( 'No data to clean', 'bbpress' );
    … …  
    303302                    } else {
    304303                        update_option( '_bbp_converter_start', $max + 1 );
    305 
    306                         _e( 'Deleting previously converted data (' . $min . ' - ' . $max . ')', 'bbpress' );
     304                        printf( __( 'Deleting previously converted data (%1$s - %2$s)', 'bbpress' ), $min, $max );
    307305                    }
    308306                } else {
    … …  
    310308                    update_option( '_bbp_converter_start', 0         );
    311309                }
    312                
     310
    313311                break;
    314312
    … …  
    319317                        update_option( '_bbp_converter_step',  $step + 1 );
    320318                        update_option( '_bbp_converter_start', 0         );
    321 
    322319                        if ( empty( $start ) ) {
    323320                            _e( 'No users to convert', 'bbpress' );
    … …  
    325322                    } else {
    326323                        update_option( '_bbp_converter_start', $max + 1 );
    327 
    328                         _e( 'Converting users (' . $min . ' - ' . $max . ')', 'bbpress' );
     324                        printf( __( 'Converting users (%1$s - %2$s)', 'bbpress' ), $min, $max );
    329325                    }
    330326                } else {
    … …  
    341337                        update_option( '_bbp_converter_step',  $step + 1 );
    342338                        update_option( '_bbp_converter_start', 0         );
    343 
    344339                        if ( empty( $start ) ) {
    345340                            _e( 'No passwords to clear', 'bbpress' );
    … …  
    347342                    } else {
    348343                        update_option( '_bbp_converter_start', $max + 1 );
    349 
    350                         _e( 'Delete users wordpress default passwords (' . $min . ' - ' . $max . ')', 'bbpress' );
     344                        printf( __( 'Delete users wordpress default passwords (%1$s - %2$s)', 'bbpress' ), $min, $max );
    351345                    }
    352346                } else {
    … …  
    362356                    update_option( '_bbp_converter_step',  $step + 1 );
    363357                    update_option( '_bbp_converter_start', 0         );
    364 
    365358                    if ( empty( $start ) ) {
    366359                        _e( 'No forums to convert', 'bbpress' );
    … …  
    368361                } else {
    369362                    update_option( '_bbp_converter_start', $max + 1 );
    370 
    371                     _e( 'Converting forums (' . $min . ' - ' . $max . ')', 'bbpress' );
     363                    printf( __( 'Converting forums (%1$s - %2$s)', 'bbpress' ), $min, $max );
    372364                }
    373365
    … …  
    376368            // STEP 5. Convert forum parents.
    377369            case 5 :
    378                
     370
    379371                if ( $converter->convert_forum_parents( $start ) ) {
    380372                    update_option( '_bbp_converter_step',  $step + 1 );
    381373                    update_option( '_bbp_converter_start', 0         );
    382 
    383374                    if ( empty( $start ) ) {
    384375                        _e( 'No forum parents to convert', 'bbpress' );
    … …  
    386377                } else {
    387378                    update_option( '_bbp_converter_start', $max + 1 );
    388 
    389                     _e( 'Converting forum parents (' . $min . ' - ' . $max . ')', 'bbpress' );
     379                    printf( __( 'Calculating forum hierarchy (%1$s - %2$s)', 'bbpress' ), $min, $max );
    390380                }
    391381
    … …  
    398388                    update_option( '_bbp_converter_step',  $step + 1 );
    399389                    update_option( '_bbp_converter_start', 0         );
    400 
    401                     if ( !$start ) {
     390                    if ( empty( $start ) ) {
    402391                        _e( 'No topics to convert', 'bbpress' );
    403392                    }
    404393                } else {
    405394                    update_option( '_bbp_converter_start', $max + 1 );
    406 
    407                     _e( 'Converting topics (' . $min . ' - ' . $max . ')', 'bbpress' );
     395                    printf( __( 'Converting topics (%1$s - %2$s)', 'bbpress' ), $min, $max );
    408396                }
    409397
    … …  
    412400            // STEP 7. Convert tags.
    413401            case 7 :
    414                
     402
    415403                if ( $converter->convert_tags( $start ) ) {
    416404                    update_option( '_bbp_converter_step',  $step + 1 );
    417405                    update_option( '_bbp_converter_start', 0         );
    418 
    419406                    if ( empty( $start ) ) {
    420407                        _e( 'No tags to convert', 'bbpress' );
    … …  
    422409                } else {
    423410                    update_option( '_bbp_converter_start', $max + 1 );
    424 
    425                     _e( 'Converting tags (' . $min . ' - ' . $max . ')', 'bbpress' );
     411                    printf( __( 'Converting topic tags (%1$s - %2$s)', 'bbpress' ), $min, $max );
    426412                }
    427413
    … …  
    438424                } else {
    439425                    update_option( '_bbp_converter_start', $max + 1 );
    440 
    441                     _e( 'Converting replies (' . $min . ' - ' . $max . ')', 'bbpress' );
     426                    printf( __( 'Converting replies (%1$s - %2$s)', 'bbpress' ), $min, $max );
    442427                }
    443428
    444429                break;
    445            
     430
    446431            default :
    447432                delete_option( '_bbp_converter_step' );
    … …  
    451436
    452437                break;
    453            
     438
    454439        }
    455440    }
    … …  
    457442    /**
    458443     * Convert passwords from previous forum to wordpress.
    459      * 
     444     *
    460445     * @since bbPress (r3813)
    461446     * @global WPDB $wpdb
    … …  
    480465    /**
    481466     * Create Tables for fast syncing
    482      * 
     467     *
    483468     * @since bbPress (r3813)
    484469     */
    … …  
    828813
    829814            // Get some data from the old forums
    830             $forum_array = $this->opdb->get_results( 'SELECT ' . implode( ',', $field_list ) . ' FROM ' . $this->opdb->prefix . $from_tablename . ' LIMIT ' . $start . ', ' . $this->max_rows, ARRAY_A );
     815            $field_list  = array_unique( $field_list );
     816            $forum_query = 'SELECT ' . implode( ',', $field_list ) . ' FROM ' . $this->opdb->prefix . $from_tablename . ' LIMIT ' . $start . ', ' . $this->max_rows;
     817            $forum_array = $this->opdb->get_results( $forum_query, ARRAY_A );
     818
     819            // Output the query, for better debugging
     820            printf( __( '<span title="%s">View Query</span>%s', 'bbpress' ), esc_attr( $forum_query ), '<br />' );
    831821
    832822            // Query returned some results
    … …  
    849839                            // columns in this table.
    850840                            if ( in_array( $row['to_fieldname'], $tablefield_array ) ) {
    851                                
     841
    852842                                // Allows us to set default fields.
    853843                                if ( isset( $row['default'] ) ) {
    … …  
    861851                                        $insert_post[$row['to_fieldname']] = call_user_func_array( array( $this, $row['callback_method'] ), array( $forum[$row['from_fieldname']], $forum ) );
    862852                                    }
    863                                    
     853
    864854                                // Maps the field from the old forum.
    865855                                } else {
    … …  
    870860                            // to do some extra work or set a default.
    871861                            } elseif ( !empty( $row['to_fieldname'] ) ) {
    872                                
     862
    873863                                // Allows us to set default fields.
    874864                                if ( isset( $row['default'] ) ) {
    … …  
    897887
    898888                        switch ( $to_type ) {
    899                            
     889
    900890                            /** New user **************************************/
    901891
    … …  
    10881078        return $rval;
    10891079    }
    1090    
     1080
    10911081    /** Callbacks *************************************************************/
    10921082
    … …  
    10951085     *
    10961086     * @param string $username
    1097      * @param string $password 
     1087     * @param string $password
    10981088     */
    10991089    public function callback_pass( $username, $password ) {
    … …  
    11151105     *
    11161106     * @param string $field
    1117      * @return string 
     1107     * @return string
    11181108     */
    11191109    private function callback_forumid( $field ) {
    … …  
    11381128     *
    11391129     * @param string $field
    1140      * @return string 
     1130     * @return string
    11411131     */
    11421132    private function callback_topicid( $field ) {
    … …  
    11611151     *
    11621152     * @param string $field
    1163      * @return string 
     1153     * @return string
    11641154     */
    11651155    private function callback_userid( $field ) {
    … …  
    11881178     *
    11891179     * @param string $field
    1190      * @return string 
     1180     * @return string
    11911181     */
    11921182    private function callback_topicid_to_forumid( $field ) {
Note: See TracChangeset for help on using the changeset viewer.