Ticket #4098: 4098.2.diff

bp-themes/bp-default/functions.php

@@ -83 +83 @@
         // This theme uses wp_nav_menu() in one location.
         register_nav_menus( array(
                 'primary' => __( 'Primary Navigation', 'buddypress' ),
-        ) );
-
+        ) );    
+        
+        $custom_background_args = array(
+                'wp-head-callback' => 'bp_dtheme_custom_background_style'
+        );
+        
         // This theme allows users to set a custom background
         // Backward-compatibility with WP < 3.4 will be removed in a future release
-        if ( bp_get_major_wp_version() >= 3.4 ) {
-                $custom_background_args = array(
-                        'wp-head-callback' => 'bp_dtheme_custom_background_style'
-                );
+        if ( function_exists( 'get_custom_header' ) ) {
                 add_theme_support( 'custom-background', $custom_background_args );
         } else {
-                add_custom_background( 'bp_dtheme_custom_background_style' );
+                add_custom_background( $custom_background_args['wp-head-callback'] );
         }
 
         // Add custom header support if allowed
         if ( !defined( 'BP_DTHEME_DISABLE_CUSTOM_HEADER' ) ) {
-                define( 'HEADER_TEXTCOLOR', 'FFFFFF' );
+                $custom_header_args = array(
+                        'default-text-color' => 'fff',
+                        'width' => apply_filters( 'bp_dtheme_header_image_width',  1250 ),
+                        'height' => apply_filters( 'bp_dtheme_header_image_height', 133  ),
+                        'wp-head-callback' => 'bp_dtheme_header_style',
+                        'admin-head-callback' => 'bp_dtheme_admin_header_style'
+                );
 
-                // The height and width of your custom header. You can hook into the theme's own filters to change these values.
-                // Add a filter to bp_dtheme_header_image_width and bp_dtheme_header_image_height to change these values.
-                define( 'HEADER_IMAGE_WIDTH',  apply_filters( 'bp_dtheme_header_image_width',  1250 ) );
-                define( 'HEADER_IMAGE_HEIGHT', apply_filters( 'bp_dtheme_header_image_height', 133  ) );
-
                 // We'll be using post thumbnails for custom header images on posts and pages. We want them to be 1250 pixels wide by 133 pixels tall.
                 // Larger images will be auto-cropped to fit, smaller ones will be ignored.
-                set_post_thumbnail_size( HEADER_IMAGE_WIDTH, HEADER_IMAGE_HEIGHT, true );
+                set_post_thumbnail_size( $custom_header_args['width'], $custom_header_args['height'], true );
 
                 // Add a way for the custom header to be styled in the admin panel that controls custom headers.
                 // Backward-compatibility with WP < 3.4 will be removed in a future release
-                if ( bp_get_major_wp_version() >= 3.4 ) {
-                        $custom_header_args = array(
-                                'wp-head-callback' => 'bp_dtheme_header_style',
-                                'admin-head-callback' => 'bp_dtheme_admin_header_style'
-                        );
+                if ( function_exists( 'get_custom_header' ) ) {                 
                         add_theme_support( 'custom-header', $custom_header_args );
                 } else {
-                        add_custom_image_header( 'bp_dtheme_header_style', 'bp_dtheme_admin_header_style' );
+                        define( 'HEADER_TEXTCOLOR', $custom_header_args['default-text-color'] );
+                        define( 'HEADER_IMAGE_WIDTH', $custom_header_args['width'] );
+                        define( 'HEADER_IMAGE_HEIGHT', $custom_header_args['height'] );
+                        add_custom_image_header( $custom_header_args['wp-head-callback'], $custom_header_args['admin-head-callback'] );
                 }
         }