Changeset 5150

Timestamp:

11/08/2013 03:47:31 AM (8 years ago)

Author:

netweb

Message:

phpBB Importer Improvements. Props netweb. Fixes #2467

• Added forum type callback for if forum is a Forum or Category
• Added reply title callback for clean reply titles
• Improved inline docs

File:

• trunk/includes/admin/converters/phpBB.php (modified) (10 diffs)

trunk/includes/admin/converters/phpBB.php

@@ -24 +24 @@
     public function setup_globals() {
 
-        /** Forum Section ******************************************************/
+        /** Forum Section *****************************************************/
 
         // Forum id (Stored in postmeta)
@@ -66 +66 @@
         );
 
-        // Forum total reply count (Stored in postmeta)
+        // Forum total reply count (Includes unpublished replies, Stored in postmeta)
         $this->field_map[] = array(
             'from_tablename' => 'forums',
@@ -108 +108 @@
         );
 
-        // Forum status (Locked =1 Unlocked =0, Stored in postmeta)
+        // Forum type (Category = 0 or Forum = 1, Stored in postmeta)
+        $this->field_map[] = array(
+            'from_tablename'  => 'forums',
+            'from_fieldname'  => 'forum_type',
+            'to_type'         => 'forum',
+            'to_fieldname'    => '_bbp_forum_type',
+            'callback_method' => 'callback_forum_type'
+        );
+
+        // Forum status (Unlocked = 0 or Locked = 1, Stored in postmeta)
         $this->field_map[] = array(
             'from_tablename'  => 'forums',
@@ -139 +148 @@
         );
 
-        /** Topic Section ******************************************************/
+        /** Topic Section *****************************************************/
 
         // Topic id (Stored in postmeta)
@@ -186 +195 @@
 
         // Topic content.
-        // Note: We join the posts table because topics do not have content.
+        // Note: We join the 'posts' table because 'topics' do not have content.
         $this->field_map[] = array(
             'from_tablename'  => 'posts',
@@ -280 +289 @@
             'to_fieldname'    => '_bbp_author_ip'
         );
-
-        // Sticky Status
 
         /** Tags Section ******************************************************/
@@ -346 +353 @@
         // Reply title.
         $this->field_map[] = array(
-            'from_tablename' => 'posts',
-            'from_fieldname' => 'post_subject',
-            'to_type'        => 'reply',
-            'to_fieldname'   => 'post_title'
+            'from_tablename'  => 'posts',
+            'from_fieldname'  => 'post_subject',
+            'to_type'         => 'reply',
+            'to_fieldname'    => 'post_title',
+            'callback_method' => 'callback_reply_title'
         );
 
@@ -692 +700 @@
 
     /**
+     * Translate the forum type from phpBB v3.x numeric's to WordPress's strings.
+     *
+     * @param int $status phpBB v3.x numeric forum type
+     * @return string WordPress safe
+     */
+    public function callback_forum_type( $status = 1 ) {
+        switch ( $status ) {
+            case 0 :
+                $status = 'category';
+                break;
+
+            case 1  :
+            default :
+                $status = 'forum';
+                break;
+        }
+        return $status;
+    }
+
+    /**
      * Translate the forum status from phpBB v3.x numeric's to WordPress's strings.
      *
@@ -723 +751 @@
                 break;
 
-            case 0  :
+            case 0 :
             default :
                 $status = 'publish';
@@ -740 +768 @@
         $count = absint( (int) $count - 1 );
         return $count;
+    }
+
+    /**
+     * Set the reply title
+     *
+     * @param string $title phpBB v3.x topic title of this reply
+     * @return string Prefixed topic title, or empty string
+     */
+    public function callback_reply_title( $title = '' ) {
+        $title = !empty( $title ) ? __( 'Re: ', 'bbpress' ) . html_entity_decode( $title ) : '';
+        return $title;
     }