Ticket #2440: 2440-converter.php.diff

bbPress/includes/admin/converter.php

@@ -467 +467 @@
 
                                 break;
 
-                        // STEP 9. Convert reply_to parents.
+                        // STEP 9. Convert comments.
                         case 9 :
+                                if ( $converter->convert_comments( $start ) ) {
+                                        update_option( '_bbp_converter_step',  $step + 1 );
+                                        update_option( '_bbp_converter_start', 0         );
+                                        if ( empty( $start ) ) {
+                                                $this->converter_output( __( 'No comments to convert', 'bbpress' ) );
+                                        }
+                                } else {
+                                        update_option( '_bbp_converter_start', $max + 1 );
+                                        $this->converter_output( sprintf( __( 'Converting comments (%1$s - %2$s)', 'bbpress' ), $min, $max ) );
+                                }
+
+                                break;
+
+                        // STEP 10. Convert reply_to parents.
+                        case 10 :
                                 if ( $converter->convert_reply_to_parents( $start ) ) {
                                         update_option( '_bbp_converter_step',  $step + 1 );
                                         update_option( '_bbp_converter_start', 0         );
@@ -692 +707 @@
                         'default'      => 'topic'
                 );
 
-                /** Post Section ******************************************************/
+                /** Reply Section *****************************************************/
 
                 $this->field_map[] = array(
                         'to_type'      => 'reply',
@@ -715 +730 @@
                         'default'      => 'reply'
                 );
 
+                /** Comment Section ***************************************************/
+
+                $this->field_map[] = array(
+                        'to_type'      => 'comment',
+                        'to_fieldname' => 'post_status',
+                        'default'      => 'publish'
+                );
+                $this->field_map[] = array(
+                        'to_type'      => 'comment',
+                        'to_fieldname' => 'comment_status',
+                        'default'      => 'closed'
+                );
+                $this->field_map[] = array(
+                        'to_type'      => 'comment',
+                        'to_fieldname' => 'ping_status',
+                        'default'      => 'closed'
+                );
+                $this->field_map[] = array(
+                        'to_type'      => 'comment',
+                        'to_fieldname' => 'post_type',
+                        'default'      => 'reply'
+                );
+
                 /** User Section ******************************************************/
 
                 $this->field_map[] = array(
@@ -739 +777 @@
         }
 
         /**
-         * Convert Posts
+         * Convert Reples / Posts
          */
         public function convert_replies( $start = 1 ) {
                 return $this->convert_table( 'reply', $start );
         }
 
         /**
+         * Convert Comments
+         */
+        public function convert_comments( $start = 1 ) {
+                return $this->convert_table( 'comment', $start );
+        }
+
+        /**
          * Convert Users
          */
         public function convert_users( $start = 1 ) {
@@ -863 +908 @@
                                         // Loop through field map, again...
                                         foreach ( $this->field_map as $row ) {
 
-                                                // Types matchand to_fieldname is present. This means
+                                                // Types match and to_fieldname is present. This means
                                                 // we have some work to do here.
                                                 if ( ( $row['to_type'] == $to_type ) && ! is_null( $row['to_fieldname'] ) ) {
 
@@ -951 +996 @@
                                                                 $post_id = wp_set_object_terms( $insert_postmeta['objectid'], $insert_postmeta['name'], 'topic-tag', true );
                                                                 break;
 
-                                                        /** Forum, Topic, Reply ***************************/
+                                                        /** Forum, Topic, Reply, Comment ******************/
 
                                                         default:
                                                                 $post_id = wp_insert_post( $insert_post );
@@ -1358 +1403 @@
         $found = false;
 
         if ( $curdir = opendir( bbpress()->admin->admin_dir . 'converters/' ) ) {
-                while ( $file = readdir( $curdir ) ) {
+                while ( false !== ( $file = readdir( $curdir ) ) ) {
                         if ( stristr( $file, '.php' ) && stristr( $file, 'index' ) === FALSE ) {
                                 $file = preg_replace( '/.php/', '', $file );
                                 if ( $platform == $file ) {