Changeset 5425 for trunk/src/includes/admin/converter.php

Timestamp:

07/07/2014 01:51:38 PM (11 years ago)

Author:

netweb

Message:

Re-factor importing closed topics to import the closed status and update the closed topic post and post meta status using bbp_close_topic(). See #2577

File:

• trunk/src/includes/admin/converter.php (modified) (4 diffs)

trunk/src/includes/admin/converter.php

@@ -468 +468 @@
                 break;
 
-            // STEP 9. Convert tags.
+            // STEP 9. Closed Topics.
             case 9 :
+                if ( $converter->convert_topic_closed_topics( $start ) ) {
+                    update_option( '_bbp_converter_step',  $step + 1 );
+                    update_option( '_bbp_converter_start', 0         );
+                    if ( empty( $start ) ) {
+                        $this->converter_output( __( 'No closed topics to close', 'bbpress' ) );
+                    }
+                } else {
+                    update_option( '_bbp_converter_start', $max + 1 );
+                    $this->converter_output( sprintf( __( 'Calculating closed topics (%1$s - %2$s)', 'bbpress' ), $min, $max ) );
+                }
+
+                break;
+
+            // STEP 10. Convert tags.
+            case 10 :
                 if ( $converter->convert_tags( $start ) ) {
                     update_option( '_bbp_converter_step',  $step + 1 );
@@ -483 +498 @@
                 break;
 
-            // STEP 10. Convert replies.
-            case 10 :
+            // STEP 11. Convert replies.
+            case 11 :
                 if ( $converter->convert_replies( $start ) ) {
                     update_option( '_bbp_converter_step',  $step + 1 );
@@ -498 +513 @@
                 break;
 
-            // STEP 11. Convert reply_to parents.
-            case 11 :
+            // STEP 12. Convert reply_to parents.
+            case 12 :
                 if ( $converter->convert_reply_to_parents( $start ) ) {
                     update_option( '_bbp_converter_step',  $step + 1 );
@@ -1129 +1144 @@
 
     /**
+     * This method converts old closed topics to bbPress closed topics.
+     *
+     * @since bbPress (r5425)
+     *
+     * @uses bbp_close_topic() to close topics properly
+     *
+     */
+    public function convert_topic_closed_topics( $start ) {
+
+        $has_update = false;
+
+        if ( !empty( $this->sync_table ) ) {
+            $query = 'SELECT value_id, meta_value FROM ' . $this->sync_table_name           . ' WHERE meta_key = "_bbp_old_closed_status_id" AND meta_value = "closed" LIMIT ' . $start . ', ' . $this->max_rows;
+        } else {
+            $query = 'SELECT post_id AS value_id, meta_value FROM ' . $this->wpdb->postmeta . ' WHERE meta_key = "_bbp_old_closed_status_id" AND meta_value = "closed" LIMIT ' . $start . ', ' . $this->max_rows;
+        }
+
+        update_option( '_bbp_converter_query', $query );
+
+        $closed_topic = $this->wpdb->get_results( $query );
+
+        foreach ( (array) $closed_topic as $row ) {
+            bbp_close_topic( $row->value_id );
+            $has_update = true;
+        }
+
+        return ! $has_update;
+    }
+
+    /**
      * This method converts old reply_to post id to new bbPress reply_to post id.
      *