Changeset 5136 for trunk/includes/admin/converters/bbPress1.php

Timestamp:

11/04/2013 04:38:03 AM (12 years ago)

Author:

netweb

Message:

bbPress1.php converter improvements. Props netweb. Fixes #2460

• Add forum category conversion support
• Add _bbp_total_reply_count
• Fix Topic slug field mapping
• Add Reply slug field mapping
• Improved inline docs

File:

• trunk/includes/admin/converters/bbPress1.php (modified) (11 diffs)

trunk/includes/admin/converters/bbPress1.php

@@ -108 +108 @@
         );
 
+        // Forum type (bbPress v1.x Forum > 0 or Category = 0, Stored in postmeta)
+        $this->field_map[] = array(
+            'from_tablename'  => 'meta',
+            'from_fieldname'  => 'meta_value',
+            'join_tablename'  => 'forums',
+            'join_type'       => 'LEFT',
+            'join_expression' => 'ON meta.object_id = forums.forum_id AND meta.meta_key = "forum_is_category"',
+            'to_type'         => 'forum',
+            'to_fieldname'    => '_bbp_forum_type',
+            'callback_method' => 'callback_forum_type'
+        );
+
         // Forum dates.
         $this->field_map[] = array(
@@ -149 +161 @@
         );
 
+        // Topic total reply count (Includes unpublished replies, Stored in postmeta)
+        $this->field_map[] = array(
+            'from_tablename'  => 'topics',
+            'from_fieldname'  => 'topic_posts',
+            'to_type'         => 'topic',
+            'to_fieldname'    => '_bbp_total_reply_count',
+            'callback_method' => 'callback_topic_reply_count'
+        );
+
         // Topic parent forum id (If no parent, then 0. Stored in postmeta)
         $this->field_map[] = array(
@@ -178 +199 @@
         $this->field_map[] = array(
             'from_tablename'  => 'topics',
-            'from_fieldname'  => 'topic_title',
+            'from_fieldname'  => 'topic_slug',
             'to_type'         => 'topic',
             'to_fieldname'    => 'post_name',
@@ -185 +206 @@
 
         // Topic content.
-        // Note: We join the posts table because topics do not have content.
+        // Note: We join the 'posts' table because 'topics' table does not include content.
         $this->field_map[] = array(
             'from_tablename'  => 'posts',
@@ -197 +218 @@
         );
 
-        // Topic status (Spam, Trash or Publish)
+        // Topic status (Spam, Trash or Publish, bbPress v1.x publish = 0, trash = 1 & spam = 2)
         $this->field_map[] = array(
             'from_tablename'  => 'posts',
@@ -210 +231 @@
 
         // Topic author ip (Stored in postmeta)
+        // Note: We join the 'posts' table because 'topics' table does not include author ip.
         $this->field_map[] = array(
             'from_tablename'  => 'posts',
@@ -323 +345 @@
 
         // Reply title.
-        // Note: We join the topics table because post table does not include topic title.
+        // Note: We join the 'topics' table because 'posts' table does not include topic title.
         $this->field_map[] = array(
             'from_tablename'  => 'topics',
@@ -335 +357 @@
         );
 
+        // Reply slug (Clean name to avoid conflicts)
+        // Note: We join the 'topics' table because 'posts' table does not include topic title.
+        $this->field_map[] = array(
+            'from_tablename'  => 'topics',
+            'from_fieldname'  => 'topic_slug',
+            'join_tablename'  => 'posts',
+            'join_type'       => 'INNER',
+            'join_expression' => 'USING (topic_id) WHERE posts.post_position NOT IN (0,1)',
+            'to_type'         => 'reply',
+            'to_fieldname'    => 'post_name',
+            'callback_method' => 'callback_slug'
+        );
+
         // Reply author ip (Stored in postmeta)
         $this->field_map[] = array(
@@ -352 +387 @@
         );
 
-        // Reply status (Spam, Trash or Publish)
+        // Reply status (Spam, Trash or Publish, bbPress v1.x publish = 0, trash = 1 & spam = 2)
         $this->field_map[] = array(
             'from_tablename'  => 'posts',
@@ -500 +535 @@
      * strings.
      *
-     * @param int $status bbPress 1.x numeric status
+     * @param int $status bbPress 1.x numeric post status
      * @return string WordPress safe
      */
@@ -522 +557 @@
 
     /**
+     * Translate the forum type from bbPress 1.x numeric's to WordPress's strings.
+     *
+     * @param int $status bbPress 1.x numeric forum type
+     * @return string WordPress safe
+     */
+    public function callback_forum_type( $status = 0 ) {
+        if ( $status == 1 ) {
+            $status = 'category';
+        } else {
+            $status = 'forum';
+        }
+        return $status;
+    }
+
+    /**
      * Verify the topic reply count.
      *