Skip to:
Content

bbPress.org


Ignore:
Timestamp:
07/04/2012 02:17:31 AM (14 years ago)
Author:
johnjamesjacoby
Message:

Admin:

  • Add nonce checks and additional sanity checks to converter, to prevent it being loaded or accessed outside of the converter screen.
  • Use 'load-' actions to conditionally load admin components on their respective screens.
  • Remove post_type checks that are no longer needed as a result of conditionally loading each component.
  • Use get_current_screen() to fill in any extra post_type checks.
  • Sweep through converter, make output more clean, add output method, and store last query in the DB in case of failure.
File:
1 edited

Legend:

Unmodified
Added
Removed

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

    r4007 r4053  
    2626         */
    2727        public function __construct() {
     28
     29                // Bail if request is not correct
     30                switch ( strtoupper( $_SERVER['REQUEST_METHOD'] ) ) {
     31
     32                        // Converter is converting
     33                        case 'POST' :
     34                                if ( ( empty( $_POST['action'] ) || ( 'bbconverter_process' !=  $_POST['action'] ) ) )
     35                                        return;
     36
     37                                break;
     38
     39                        // Some other admin page
     40                        case 'GET'  :
     41                                if ( ( empty( $_GET['page'] ) || ( 'bbp-converter' !=  $_GET['page'] ) ) )
     42                                        return;
     43
     44                                break;
     45                }
     46
     47                // Proceed with the actions
    2848                $this->setup_actions();
    2949        }
    … …  
    6181
    6282                // Add the main section
    63                 add_settings_section( 'bbpress_converter_main',     __( 'Main Settings',    'bbpress' ),  'bbp_converter_setting_callback_main_section', 'bbpress_converter' );
     83                add_settings_section( 'bbpress_converter_main',     __( 'Database Settings', 'bbpress' ),  'bbp_converter_setting_callback_main_section', 'bbpress_converter' );
    6484
    6585                // System Select
    … …  
    140160                                border-color: #E6DB55;
    141161                                font-family: monospace;
     162                                font-weight: bold;
    142163                        }
    143164
    … …  
    145166                                margin: 0.5em 0;
    146167                                padding: 2px;
    147                         }
    148 
    149                         div.bbp-converter-updated p strong.loading {
    150                                 padding: 2px 20px 2px 0;
     168                                float: left;
     169                                clear: left;
     170                        }
     171
     172                        div.bbp-converter-updated p.loading {
     173                                padding: 2px 20px 2px 2px;
    151174                                background-image: url('<?php echo admin_url(); ?>images/wpspin_light.gif');
    152175                                background-repeat: no-repeat;
    … …  
    186209
    187210                                values['action'] = 'bbconverter_process';
     211                                values['_ajax_nonce'] = '<?php echo  wp_create_nonce( 'bbp_converter_process' ); ?>';
     212
    188213                                return values;
    189214                        }
    … …  
    195220                                        jQuery('#bbp-converter-stop').show();
    196221                                        jQuery('#bbp-converter-progress').show();
    197                                         bbconverter_log( "Starting Conversion..." );
     222                                        bbconverter_log( '<p class="loading"><?php _e( 'Starting Conversion', 'bbpress' ); ?></p>' );
    198223                                        bbconverter_run();
    199224                                }
    … …  
    209234
    210235                        function bbconverter_stop() {
     236                                jQuery('#bbp-converter-start').show();
     237                                jQuery('#bbp-converter-stop').hide();
     238                                jQuery('#bbp-converter-progress').hide();
     239                                jQuery('#bbp-converter-message p').removeClass( 'loading' );
    211240                                bbconverter_is_running = false;
    212                                 jQuery('#bbp-converter-message strong').removeClass( 'loading' );
     241                                clearTimeout( bbconverter_run_timer );
    213242                        }
    214243
    … …  
    216245                                bbconverter_log(response);
    217246
    218                                 if ( response == 'Conversion Complete' || response.indexOf('error') > -1 ) {
    219                                         bbconverter_log('<b>Repair any missing information: <a href="<?php echo admin_url(); ?>tools.php?page=bbp-repair">Continue</a></b>');
    220                                         jQuery('#bbp-converter-start').show();
    221                                         jQuery('#bbp-converter-stop').hide();
    222                                         jQuery('#bbp-converter-progress').hide();
     247                                if ( response == '<p class="loading"><?php _e( 'Conversion Complete', 'bbpress' ); ?></p>' || response.indexOf('error') > -1 ) {
     248                                        bbconverter_log('<p>Repair any missing information: <a href="<?php echo admin_url(); ?>tools.php?page=bbp-repair">Continue</a></p>');
    223249                                        bbconverter_stop();
    224                                         clearTimeout( bbconverter_run_timer );
    225250                                } else if( bbconverter_is_running ) { // keep going
    226251                                        jQuery('#bbp-converter-progress').show();
    … …  
    228253                                        bbconverter_run_timer = setTimeout( 'bbconverter_run()', bbconverter_delay_time );
    229254                                } else {
    230                                         jQuery('#bbp-converter-start').show();
    231                                         jQuery('#bbp-converter-stop').hide();
    232                                         jQuery('#bbp-converter-progress').hide();
    233                                         clearTimeout( bbconverter_run_timer );
     255                                        bbconverter_stop();
    234256                                }
    235257                        }
    … …  
    240262                                }
    241263                                if ( text ) {
    242                                         jQuery('#bbp-converter-message strong').removeClass( 'loading' );
    243                                         jQuery('#bbp-converter-message').prepend('<p><strong class="loading">' + text + '</strong></p>');
     264                                        jQuery('#bbp-converter-message p').removeClass( 'loading' );
     265                                        jQuery('#bbp-converter-message').prepend( text );
    244266                                }
    245267                        }
    … …  
    251273
    252274        /**
     275         * Wrap the converter output in paragraph tags, so styling can be applied
     276         *
     277         * @since bbPress (r4052)
     278         *
     279         * @param string $output
     280         */
     281        private static function converter_output( $output = '' ) {
     282
     283                // Get the last query
     284                $before = '<p class="loading">';
     285                $after  = '</p>';
     286                $query  = get_option( '_bbp_converter_query' );
     287
     288                if ( ! empty( $query ) )
     289                        $before = '<p class="loading" title="' . esc_attr( $query ) . '">';
     290
     291                echo $before . $output . $after;
     292        }
     293
     294        /**
    253295         * Callback processor
    254296         *
    … …  
    256298         */
    257299        public function process_callback() {
     300
     301                // Verify intent
     302                check_ajax_referer( 'bbp_converter_process' );
    258303
    259304                if ( ! ini_get( 'safe_mode' ) ) {
    … …  
    294339                                                $this->sync_table();
    295340                                                if ( empty( $start ) ) {
    296                                                         _e( 'No data to clean', 'bbpress' );
     341                                                        $this->converter_output( __( 'No data to clean', 'bbpress' ) );
    297342                                                }
    298343                                        } else {
    299344                                                update_option( '_bbp_converter_start', $max + 1 );
    300                                                 printf( __( 'Deleting previously converted data (%1$s - %2$s)', 'bbpress' ), $min, $max );
     345                                                $this->converter_output( sprintf( __( 'Deleting previously converted data (%1$s - %2$s)', 'bbpress' ), $min, $max ) );
    301346                                        }
    302347                                } else {
    … …  
    314359                                                update_option( '_bbp_converter_start', 0         );
    315360                                                if ( empty( $start ) ) {
    316                                                         _e( 'No users to convert', 'bbpress' );
     361                                                        $this->converter_output( __( 'No users to convert', 'bbpress' ) );
    317362                                                }
    318363                                        } else {
    319364                                                update_option( '_bbp_converter_start', $max + 1 );
    320                                                 printf( __( 'Converting users (%1$s - %2$s)', 'bbpress' ), $min, $max );
     365                                                $this->converter_output( sprintf(  __( 'Converting users (%1$s - %2$s)', 'bbpress' ), $min, $max ) );
    321366                                        }
    322367                                } else {
    … …  
    334379                                                update_option( '_bbp_converter_start', 0         );
    335380                                                if ( empty( $start ) ) {
    336                                                         _e( 'No passwords to clear', 'bbpress' );
     381                                                        $this->converter_output( __( 'No passwords to clear', 'bbpress' ) );
    337382                                                }
    338383                                        } else {
    339384                                                update_option( '_bbp_converter_start', $max + 1 );
    340                                                 printf( __( 'Delete users wordpress default passwords (%1$s - %2$s)', 'bbpress' ), $min, $max );
     385                                                $this->converter_output( sprintf( __( 'Delete users wordpress default passwords (%1$s - %2$s)', 'bbpress' ), $min, $max ) );
    341386                                        }
    342387                                } else {
    … …  
    353398                                        update_option( '_bbp_converter_start', 0         );
    354399                                        if ( empty( $start ) ) {
    355                                                 _e( 'No forums to convert', 'bbpress' );
     400                                                $this->converter_output( __( 'No forums to convert', 'bbpress' ) );
    356401                                        }
    357402                                } else {
    358403                                        update_option( '_bbp_converter_start', $max + 1 );
    359                                         printf( __( 'Converting forums (%1$s - %2$s)', 'bbpress' ), $min, $max );
     404                                        $this->converter_output( sprintf( __( 'Converting forums (%1$s - %2$s)', 'bbpress' ), $min, $max ) );
    360405                                }
    361406
    … …  
    364409                        // STEP 5. Convert forum parents.
    365410                        case 5 :
    366 
    367411                                if ( $converter->convert_forum_parents( $start ) ) {
    368412                                        update_option( '_bbp_converter_step',  $step + 1 );
    369413                                        update_option( '_bbp_converter_start', 0         );
    370414                                        if ( empty( $start ) ) {
    371                                                 _e( 'No forum parents to convert', 'bbpress' );
     415                                                $this->converter_output( __( 'No forum parents to convert', 'bbpress' ) );
    372416                                        }
    373417                                } else {
    374418                                        update_option( '_bbp_converter_start', $max + 1 );
    375                                         printf( __( 'Calculating forum hierarchy (%1$s - %2$s)', 'bbpress' ), $min, $max );
     419                                        $this->converter_output( sprintf( __( 'Calculating forum hierarchy (%1$s - %2$s)', 'bbpress' ), $min, $max ) );
    376420                                }
    377421
    … …  
    380424                        // STEP 6. Convert topics.
    381425                        case 6 :
    382 
    383426                                if ( $converter->convert_topics( $start ) ) {
    384427                                        update_option( '_bbp_converter_step',  $step + 1 );
    385428                                        update_option( '_bbp_converter_start', 0         );
    386429                                        if ( empty( $start ) ) {
    387                                                 _e( 'No topics to convert', 'bbpress' );
     430                                                $this->converter_output( __( 'No topics to convert', 'bbpress' ) );
    388431                                        }
    389432                                } else {
    390433                                        update_option( '_bbp_converter_start', $max + 1 );
    391                                         printf( __( 'Converting topics (%1$s - %2$s)', 'bbpress' ), $min, $max );
     434                                        $this->converter_output( sprintf( __( 'Converting topics (%1$s - %2$s)', 'bbpress' ), $min, $max ) );
    392435                                }
    393436
    … …  
    396439                        // STEP 7. Convert tags.
    397440                        case 7 :
    398 
    399441                                if ( $converter->convert_tags( $start ) ) {
    400442                                        update_option( '_bbp_converter_step',  $step + 1 );
    401443                                        update_option( '_bbp_converter_start', 0         );
    402444                                        if ( empty( $start ) ) {
    403                                                 _e( 'No tags to convert', 'bbpress' );
     445                                                $this->converter_output( __( 'No tags to convert', 'bbpress' ) );
    404446                                        }
    405447                                } else {
    406448                                        update_option( '_bbp_converter_start', $max + 1 );
    407                                         printf( __( 'Converting topic tags (%1$s - %2$s)', 'bbpress' ), $min, $max );
     449                                        $this->converter_output( sprintf( __( 'Converting topic tags (%1$s - %2$s)', 'bbpress' ), $min, $max ) );
    408450                                }
    409451
    … …  
    416458                                        update_option( '_bbp_converter_start', 0         );
    417459                                        if ( empty( $start ) ) {
    418                                                 _e( 'No replies to convert', 'bbpress' );
     460                                                $this->converter_output( __( 'No replies to convert', 'bbpress' ) );
    419461                                        }
    420462                                } else {
    421463                                        update_option( '_bbp_converter_start', $max + 1 );
    422                                         printf( __( 'Converting replies (%1$s - %2$s)', 'bbpress' ), $min, $max );
     464                                        $this->converter_output( sprintf( __( 'Converting replies (%1$s - %2$s)', 'bbpress' ), $min, $max ) );
    423465                                }
    424466
    … …  
    426468
    427469                        default :
    428                                 delete_option( '_bbp_converter_step' );
     470                                delete_option( '_bbp_converter_step'  );
    429471                                delete_option( '_bbp_converter_start' );
    430 
    431                                 _e( 'Conversion Complete', 'bbpress' );
     472                                delete_option( '_bbp_converter_query' );
     473
     474                                $this->converter_output( __( 'Conversion Complete', 'bbpress' ) );
    432475
    433476                                break;
    434 
    435477                }
    436478        }
    … …  
    813855                        $forum_array = $this->opdb->get_results( $forum_query, ARRAY_A );
    814856
    815                         // Output the query, for better debugging
    816                         printf( __( '<span title="%s">View Query</span>%s', 'bbpress' ), esc_attr( $forum_query ), '<br />' );
     857                        // Set this query as the last one ran
     858                        update_option( '_bbp_converter_query', $forum_query );
    817859
    818860                        // Query returned some results
    … …  
    9581000
    9591001        public function convert_forum_parents( $start ) {
     1002
    9601003                $has_update = false;
    9611004
    962                 if ( !empty( $this->sync_table ) ) {
    963                         $forum_array = $this->wpdb->get_results( 'SELECT value_id, meta_value FROM ' . $this->sync_table_name . ' WHERE meta_key = "_bbp_forum_parent_id" AND meta_value > 0 LIMIT ' . $start . ', ' . $this->max_rows );
    964                 } else {
    965                         $forum_array = $this->wpdb->get_results( 'SELECT post_id AS value_id, meta_value FROM ' . $this->wpdb->postmeta . ' WHERE meta_key = "_bbp_forum_parent_id" AND meta_value > 0 LIMIT ' . $start . ', ' . $this->max_rows );
    966                 }
     1005                if ( !empty( $this->sync_table ) )
     1006                        $query = 'SELECT value_id, meta_value FROM ' . $this->sync_table_name . ' WHERE meta_key = "_bbp_forum_parent_id" AND meta_value > 0 LIMIT ' . $start . ', ' . $this->max_rows;
     1007                else
     1008                        $query = 'SELECT post_id AS value_id, meta_value FROM ' . $this->wpdb->postmeta . ' WHERE meta_key = "_bbp_forum_parent_id" AND meta_value > 0 LIMIT ' . $start . ', ' . $this->max_rows;
     1009
     1010                update_option( '_bbp_converter_query', $query );
     1011
     1012                $forum_array = $this->wpdb->get_results( $query );
    9671013
    9681014                foreach ( (array) $forum_array as $row ) {
    … …  
    9791025         */
    9801026        public function clean( $start ) {
     1027
    9811028                $start      = 0;
    9821029                $has_delete = false;
    … …  
    9841031                /** Delete bbconverter topics/forums/posts ****************************/
    9851032
    986                 if ( true === $this->sync_table ) {
    987                         $bbconverter = $this->wpdb->get_results( 'SELECT value_id FROM ' . $this->sync_table_name . ' INNER JOIN ' . $this->wpdb->posts . ' ON(value_id = ID) WHERE meta_key LIKE "_bbp_%" AND value_type = "post" GROUP BY value_id ORDER BY value_id DESC LIMIT ' . $this->max_rows, ARRAY_A );
    988                 } else {
    989                         $bbconverter = $this->wpdb->get_results( 'SELECT post_id AS value_id FROM ' . $this->wpdb->postmeta . ' WHERE meta_key LIKE "_bbp_%" GROUP BY post_id ORDER BY post_id DESC LIMIT ' . $this->max_rows, ARRAY_A );
    990                 }
    991 
    992                 if ( !empty( $bbconverter ) ) {
    993                         foreach ( (array) $bbconverter as $value ) {
     1033                if ( true === $this->sync_table )
     1034                        $query = 'SELECT value_id FROM ' . $this->sync_table_name . ' INNER JOIN ' . $this->wpdb->posts . ' ON(value_id = ID) WHERE meta_key LIKE "_bbp_%" AND value_type = "post" GROUP BY value_id ORDER BY value_id DESC LIMIT ' . $this->max_rows;
     1035                else
     1036                        $query = 'SELECT post_id AS value_id FROM ' . $this->wpdb->postmeta . ' WHERE meta_key LIKE "_bbp_%" GROUP BY post_id ORDER BY post_id DESC LIMIT ' . $this->max_rows;
     1037
     1038                update_option( '_bbp_converter_query', $query );
     1039
     1040                $posts = $this->wpdb->get_results( $query, ARRAY_A );
     1041
     1042                if ( !empty( $posts ) ) {
     1043                        foreach ( (array) $posts as $value ) {
    9941044                                wp_delete_post( $value['value_id'], true );
    9951045                        }
    … …  
    9991049                /** Delete bbconverter users ******************************************/
    10001050
    1001                 if ( true === $this->sync_table ) {
    1002                         $bbconverter = $this->wpdb->get_results( 'SELECT value_id FROM ' . $this->sync_table_name . ' INNER JOIN ' . $this->wpdb->users . ' ON(value_id = ID) WHERE meta_key = "_bbp_user_id" AND value_type = "user" LIMIT ' . $this->max_rows, ARRAY_A );
    1003                 } else {
    1004                         $bbconverter = $this->wpdb->get_results( 'SELECT user_id AS value_id FROM ' . $this->wpdb->usermeta . ' WHERE meta_key = "_bbp_user_id" LIMIT ' . $this->max_rows, ARRAY_A );
    1005                 }
    1006 
    1007                 if ( !empty( $bbconverter ) ) {
    1008                         foreach ( $bbconverter as $value ) {
     1051                if ( true === $this->sync_table )
     1052                        $query = 'SELECT value_id FROM ' . $this->sync_table_name . ' INNER JOIN ' . $this->wpdb->users . ' ON(value_id = ID) WHERE meta_key = "_bbp_user_id" AND value_type = "user" LIMIT ' . $this->max_rows;
     1053                else
     1054                        $query = 'SELECT user_id AS value_id FROM ' . $this->wpdb->usermeta . ' WHERE meta_key = "_bbp_user_id" LIMIT ' . $this->max_rows;
     1055
     1056                update_option( '_bbp_converter_query', $query );
     1057
     1058                $users = $this->wpdb->get_results( $query, ARRAY_A );
     1059
     1060                if ( !empty( $users ) ) {
     1061                        foreach ( $users as $value ) {
    10091062                                wp_delete_user( $value['value_id'] );
    10101063                        }
    … …  
    10121065                }
    10131066
     1067                unset( $posts );
     1068                unset( $users );
     1069
    10141070                return ! $has_delete;
    10151071        }
    … …  
    10211077         */
    10221078        public function clean_passwords( $start ) {
     1079
    10231080                $has_delete = false;
    10241081
    10251082                /** Delete bbconverter passwords **************************************/
    10261083
    1027                 $bbconverter = $this->wpdb->get_results( 'SELECT user_id, meta_value FROM ' . $this->wpdb->usermeta . ' WHERE meta_key = "_bbp_password" LIMIT ' . $start . ', ' . $this->max_rows, ARRAY_A );
     1084                $query       = 'SELECT user_id, meta_value FROM ' . $this->wpdb->usermeta . ' WHERE meta_key = "_bbp_password" LIMIT ' . $start . ', ' . $this->max_rows;
     1085                update_option( '_bbp_converter_query', $query );
     1086
     1087                $bbconverter = $this->wpdb->get_results( $query, ARRAY_A );
     1088
    10281089                if ( !empty( $bbconverter ) ) {
    10291090
    … …  
    10381099                        $has_delete = true;
    10391100                }
     1101
    10401102                return ! $has_delete;
    10411103        }
Note: See TracChangeset for help on using the changeset viewer.