Changeset 5183

Timestamp:

11/25/2013 02:20:06 AM (11 years ago)

Author:

johnjamesjacoby

Message:

Update default theme compatibility bbpress-functions.php to use new bbp_enqueue_style() and bbp_enqueue_script() functions. Enhances style and script loading behavior to be compatible with bbPress's template stack. Fixes #2478.

File:

• trunk/templates/default/bbpress-functions.php (modified) (6 diffs)

trunk/templates/default/bbpress-functions.php

@@ -45 +45 @@
      * @uses BBP_Default::setup_actions()
      */
-    public function __construct() {
-        $this->setup_globals();
+    public function __construct( $properties = array() ) {
+
+        parent::__construct( bbp_parse_args( $properties, array(
+            'id'      => 'default',
+            'name'    => __( 'bbPress Default', 'bbpress' ),
+            'version' => bbp_get_version(),
+            'dir'     => trailingslashit( bbpress()->themes_dir . 'default' ),
+            'url'     => trailingslashit( bbpress()->themes_url . 'default' ),
+        ), 'default_theme' ) );
+
         $this->setup_actions();
-    }
-
-    /**
-     * Component global variables
-     *
-     * Note that this function is currently commented out in the constructor.
-     * It will only be used if you copy this file into your current theme and
-     * uncomment the line above.
-     *
-     * You'll want to customize the values in here, so they match whatever your
-     * needs are.
-     *
-     * @since bbPress (r3732)
-     * @access private
-     */
-    private function setup_globals() {
-        $bbp           = bbpress();
-        $this->id      = 'default';
-        $this->name    = __( 'bbPress Default', 'bbpress' );
-        $this->version = bbp_get_version();
-        $this->dir     = trailingslashit( $bbp->themes_dir . 'default' );
-        $this->url     = trailingslashit( $bbp->themes_url . 'default' );
     }
 
@@ -141 +127 @@
     public function enqueue_styles() {
 
-        $styles = array( 'bbp-default' => 'css/bbpress.css' );
-
-        if ( is_rtl() )
-            $styles['bbp-default-rtl'] = 'css/bbpress-rtl.css';
-
-        foreach ( $styles as $handle => $file ) {
-
-            // Check child theme
-            if ( file_exists( trailingslashit( get_stylesheet_directory() ) . $file ) ) {
-                $location = trailingslashit( get_stylesheet_directory_uri() );
-
-            // Check parent theme
-            } elseif ( file_exists( trailingslashit( get_template_directory() ) . $file ) ) {
-                $location = trailingslashit( get_template_directory_uri() );
-
-            // bbPress Theme Compatibility
-            } else {
-                $location = trailingslashit( $this->url );
-            }
-
-            // Enqueue the bbPress styling
-            wp_enqueue_style( $handle, $location . $file, array(), $this->version, 'screen' );
+        // Setup styles array
+        $styles = array();
+
+        // LTR
+        $styles['bbp-default'] = array(
+            'file'         => 'css/bbpress.css',
+            'dependencies' => array()
+        );
+
+        // RTL helpers
+        if ( is_rtl() ) {
+            $styles['bbp-default-rtl'] = array(
+                'file'         => 'css/bbpress-rtl.css',
+                'dependencies' => array( 'bbp-default' )
+            );
+        }
+
+        // Filter the scripts
+        $styles = apply_filters( 'bbp_default_styles', $styles );
+
+        // Enqueue the styles
+        foreach ( $styles as $handle => $attributes ) {
+            bbp_enqueue_style( $handle, $attributes['file'], $attributes['dependencies'], $this->version, 'screen' );
         }
     }
@@ -179 +166 @@
     public function enqueue_scripts() {
 
+        // Setup scripts array
+        $scripts = array();
+
         // Always pull in jQuery for TinyMCE shortcode usage
         if ( bbp_use_wp_editor() ) {
-            wp_enqueue_script( 'jquery' );
-            wp_enqueue_script( 'bbpress-editor', $this->url . 'js/editor.js', array( 'jquery' ), $this->version );
+            $scripts['bbpress-editor'] = array(
+                'file'         => 'js/editor.js',
+                'dependencies' => array( 'jquery' )
+            );
         }
 
         // Forum-specific scripts
         if ( bbp_is_single_forum() ) {
-
-            // Forum subscribe/unsubscribe
-            wp_enqueue_script( 'bbpress-forum', $this->url . 'js/forum.js', array( 'jquery' ), $this->version );
+            $scripts['bbpress-forum'] = array(
+                'file'         => 'js/forum.js',
+                'dependencies' => array( 'jquery' )
+            );
         }
 
@@ -196 +189 @@
 
             // Topic favorite/unsubscribe
-            wp_enqueue_script( 'bbpress-topic', $this->url . 'js/topic.js', array( 'jquery' ), $this->version );
+            $scripts['bbpress-topic'] = array(
+                'file'         => 'js/topic.js',
+                'dependencies' => array( 'jquery' )
+            );
 
             // Hierarchical replies
             if ( bbp_thread_replies() ) {
-                wp_enqueue_script( 'bbpress-reply', $this->url . 'js/reply.js', array(), $this->version );
+                $scripts['bbpress-reply'] = array(
+                    'file'         => 'js/reply.js',
+                    'dependencies' => array( 'jquery' )
+                );
             }
         }
@@ -206 +205 @@
         // User Profile edit
         if ( bbp_is_single_user_edit() ) {
-            wp_enqueue_script( 'user-profile' );
-            wp_enqueue_script( 'bbpress-user', $this->url . 'js/user.js', array(), $this->version );
+            $scripts['bbpress-user'] = array(
+                'file'         => 'js/user.js',
+                'dependencies' => array( 'user-query' )
+            );
+        }
+
+        // Filter the scripts
+        $scripts = apply_filters( 'bbp_default_scripts', $scripts );
+
+        // Enqueue the scripts
+        foreach ( $scripts as $handle => $attributes ) {
+            bbp_enqueue_script( $handle, $attributes['file'], $attributes['dependencies'], $this->version, 'screen' );
         }
     }
@@ -241 +250 @@
                 'subs_nonce'         => wp_create_nonce( 'toggle-subscription_' . get_the_ID() )
             ) );
-            
+
         // Single topic
         } elseif ( bbp_is_single_topic() ) {