Changeset 3927

Timestamp:

05/31/2012 04:30:09 AM (14 years ago)

Author:

johnjamesjacoby

Message:

Theme Compatibility:

• Introduce bbp_is_theme_compat_original_template() and bbp_set_theme_compat_original_template() functions, to allow the template loader to properly use custom post type templates.
• Refactor BBP_Theme_Compat to use magic methods.
• Reorder bbp_template_include_theme_supports() to put _edit() checks before single/archive checks, to prevent them from being stomped.
• Clean up BBP_Theme_Compat usage to work with refactored approach.
• Fixes #1833.

Location:

branches/plugin

Files:

• bbp-includes/bbp-template-loader.php (modified) (5 diffs)
• bbp-includes/bbp-theme-compatibility.php (modified) (4 diffs)
• bbpress.php (modified) (1 diff)

branches/plugin/bbp-includes/bbp-template-loader.php

@@ -14 +14 @@
  * Possibly intercept the template being loaded
  *
- * Listens to the 'template_include' filter and waits for a bbPress post_type
- * to appear. If the current theme does not explicitly support bbPress, it
- * intercepts the page template and uses one served from the bbPress compatable
- * theme, set in the $bbp->theme_compat global. If the current theme does
- * support bbPress, we'll explore the template hierarchy and try to locate one.
+ * Listens to the 'template_include' filter and waits for any bbPress specific
+ * template condition to be met. If one is met and the template file exists,
+ * it will be used; otherwise 
  *
  * @since bbPress (r3032)
@@ -46 +44 @@
 function bbp_template_include_theme_supports( $template = '' ) {
 
+    // Set the original template that WordPress found so we can compare the
+    // one bbPress find's in bbp_template_include_theme_compat().
+    bbp_set_theme_compat_original_template( $template );
+
+    /** bbPress Templates *****************************************************/
+    
+    // Note that the _edit() checks are ahead of their counterparts, to
+    // prevent them from being stomped on accident.
+
+    // Editing a user
+    if     ( bbp_is_single_user_edit() && ( $new_template = bbp_get_single_user_edit_template() ) ) :
+
     // Viewing a user
-    if     ( bbp_is_single_user()      && ( $new_template = bbp_get_single_user_template()      ) ) :
-
-    // Editing a user
-    elseif ( bbp_is_single_user_edit() && ( $new_template = bbp_get_single_user_edit_template() ) ) :
+    elseif ( bbp_is_single_user()      && ( $new_template = bbp_get_single_user_template()      ) ) :
 
     // Single View
     elseif ( bbp_is_single_view()      && ( $new_template = bbp_get_single_view_template()      ) ) :
 
+    // Forum edit
+    elseif ( bbp_is_forum_edit()       && ( $new_template = bbp_get_forum_edit_template()       ) ) :
+
     // Single Forum
     elseif ( bbp_is_single_forum()     && ( $new_template = bbp_get_single_forum_template()     ) ) :
@@ -61 +71 @@
     elseif ( bbp_is_forum_archive()    && ( $new_template = bbp_get_forum_archive_template()    ) ) :
 
-    // Forum edit
-    elseif ( bbp_is_forum_edit()       && ( $new_template = bbp_get_forum_edit_template()       ) ) :
+    // Topic merge
+    elseif ( bbp_is_topic_merge()      && ( $new_template = bbp_get_topic_merge_template()      ) ) :
+
+    // Topic split
+    elseif ( bbp_is_topic_split()      && ( $new_template = bbp_get_topic_split_template()      ) ) :
+
+    // Topic edit
+    elseif ( bbp_is_topic_edit()       && ( $new_template = bbp_get_topic_edit_template()       ) ) :
 
     // Single Topic
@@ -70 +86 @@
     elseif ( bbp_is_topic_archive()    && ( $new_template = bbp_get_topic_archive_template()    ) ) :
 
-    // Topic merge
-    elseif ( bbp_is_topic_merge()      && ( $new_template = bbp_get_topic_merge_template()      ) ) :
-
-    // Topic split
-    elseif ( bbp_is_topic_split()      && ( $new_template = bbp_get_topic_split_template()      ) ) :
-
-    // Topic edit
-    elseif ( bbp_is_topic_edit()       && ( $new_template = bbp_get_topic_edit_template()       ) ) :
+    // Editing a reply
+    elseif ( bbp_is_reply_edit()       && ( $new_template = bbp_get_reply_edit_template()       ) ) :
 
     // Single Reply
     elseif ( bbp_is_single_reply()     && ( $new_template = bbp_get_single_reply_template()     ) ) :
 
-    // Editing a reply
-    elseif ( bbp_is_reply_edit()       && ( $new_template = bbp_get_reply_edit_template()       ) ) :
+    // Editing a topic tag
+    elseif ( bbp_is_topic_tag_edit()   && ( $new_template = bbp_get_topic_tag_edit_template()   ) ) :
 
     // Viewing a topic tag
     elseif ( bbp_is_topic_tag()        && ( $new_template = bbp_get_topic_tag_template()        ) ) :
-
-    // Editing a topic tag
-    elseif ( bbp_is_topic_tag_edit()   && ( $new_template = bbp_get_topic_tag_edit_template()   ) ) :
     endif;
 
-    // Custom template file exists
-    $template = !empty( $new_template ) ? $new_template : $template;
+    // bbPress template file exists
+    if ( !empty( $new_template ) && ! bbp_is_theme_compat_original_template( $new_template ) )
+        $template = $new_template;
 
     return apply_filters( 'bbp_template_include_theme_supports', $template );
@@ -403 +411 @@
     $templates = array(
         'bbpress.php',
+        'forums.php',
         'forum.php',
         'page.php',

branches/plugin/bbp-includes/bbp-theme-compatibility.php

@@ -37 +37 @@
 
     /**
-     * @var string ID of the theme (should be unique)
+     * Should be like:
+     *
+     * array(
+     *     'id'      => ID of the theme (should be unique)
+     *     'name'    => Name of the theme (should match style.css)
+     *     'version' => Theme version for cache busting scripts and styling
+     *     'dir'     => Path to theme
+     *     'url'     => URL to theme
+     * );
+     * @var array 
      */
-    public $id = '';
+    private $_data = array();
 
     /**
-     * @var string Name of the theme (should match style.css)
+     * Pass the $properties to the object on creation.
+     *
+     * @since bbPress (r3926)
+     * @param array $properties
      */
-    public $name = '';
+    public function __construct( Array $properties = array() ) {
+        $this->_data = $properties;
+    }
 
     /**
-     * @var string Theme version for cache busting scripts and styling
+     * Set a theme's property.
+     *
+     * @since bbPress (r3926)
+     * @param string $property
+     * @param mixed $value
+     * @return mixed
      */
-    public $version = '';
+    public function __set( $property, $value ) {
+        return $this->_data[$property] = $value;
+    }
 
     /**
-     * @var string Path to theme
+     * Get a theme's property.
+     *
+     * @since bbPress (r3926)
+     * @param string $property
+     * @param mixed $value
+     * @return mixed
      */
-    public $dir = '';
-
-    /**
-     * @var string URL to theme
-     */
-    public $url = '';
+    public function __get( $property ) {
+        return array_key_exists( $property, $this->_data ) ? $this->_data[$property] : '';
+    }
 }
 
@@ -206 +229 @@
 
     return bbpress()->theme_compat->template;
+}
+
+/**
+ * Set the theme compat original_template global
+ *
+ * Stash the original template file for the current query. Useful for checking
+ * if bbPress was able to find a more appropriate template.
+ *
+ * @since bbPress (r3926)
+ */
+function bbp_set_theme_compat_original_template( $template = '' ) {
+    bbpress()->theme_compat->original_template = $template;
+
+    return bbpress()->theme_compat->original_template;
+}
+
+/**
+ * Set the theme compat original_template global
+ *
+ * Stash the original template file for the current query. Useful for checking
+ * if bbPress was able to find a more appropriate template.
+ *
+ * @since bbPress (r3926)
+ */
+function bbp_is_theme_compat_original_template( $template = '' ) {
+    $bbp = bbpress();
+
+    if ( empty( $bbp->theme_compat->original_template ) )
+        return false;
+
+    return (bool) ( $bbp->theme_compat->original_template == $template );
 }
 
@@ -341 +395 @@
 function bbp_template_include_theme_compat( $template = '' ) {
 
-    // Only filter the main query
-    if ( ! is_main_query() )
+    // Bail if the template doesn't specifically match a bbPress template. This
+    // includes archive-* and single-* WordPress post_type matches, allowing
+    // themes to use the expected format.
+    if ( bbp_is_theme_compat_original_template( $template ) )
         return $template;
 
@@ -739 +795 @@
         if ( !apply_filters( 'bbp_spill_the_beans', false ) ) {
 
-            // Setup the chopping block
-            global $withcomments, $post;
-
-            // Empty out globals that aren't being used in this loop anymore
-            $withcomments = $post = false;
+            // Empty globals that aren't being used in this loop anymore
+            $GLOBALS['withcomments'] = false;
+            $GLOBALS['post']         = false;
 
             // Reset the post data when the next sidebar is fired

branches/plugin/bbpress.php

@@ -533 +533 @@
         /** Default Theme *****************************************************/
 
-        $theme          = new BBP_Theme_Compat();
-        $theme->id      = 'default';
-        $theme->name    = __( 'bbPress Default', 'bbpress' );
-        $theme->version = bbp_get_version();
-        $theme->dir     = trailingslashit( $this->plugin_dir . 'bbp-theme-compat' );
-        $theme->url     = trailingslashit( $this->plugin_url . 'bbp-theme-compat' );
-
-        bbp_register_theme_package( $theme );
-
-        /** Default Theme *****************************************************/
-
-        $theme          = new BBP_Theme_Compat();
-        $theme->id      = 'bbp-twentyten';
-        $theme->name    = __( 'Twenty Ten (bbPress)', 'bbpress' ) ;
-        $theme->version = bbp_get_version();
-        $theme->dir     = trailingslashit( $this->themes_dir . 'bbp-twentyten' );
-        $theme->url     = trailingslashit( $this->themes_url . 'bbp-twentyten' );
-
-        bbp_register_theme_package( $theme );
+        bbp_register_theme_package( new BBP_Theme_Compat( array( 
+            'id'      => 'default',
+            'name'    => __( 'bbPress Default', 'bbpress' ),
+            'version' => bbp_get_version(),
+            'dir'     => trailingslashit( $this->plugin_dir . 'bbp-theme-compat' ),
+            'url'     => trailingslashit( $this->plugin_url . 'bbp-theme-compat' )
+        ) ) );
+
+        /** Twenty Ten ********************************************************/
+
+        bbp_register_theme_package( new BBP_Theme_Compat( array( 
+            'id'      => 'bbp-twentyten',
+            'name'    => __( 'Twenty Ten (bbPress)', 'bbpress' ),
+            'version' => bbp_get_version(),
+            'dir'     => trailingslashit( $this->themes_dir . 'bbp-twentyten' ),
+            'url'     => trailingslashit( $this->themes_url . 'bbp-twentyten' )
+        ) ) );
     }