Changeset 14010 for trunk/src/bp-blogs/classes/class-bp-blogs-theme-compat.php

Timestamp:

08/26/2024 11:10:35 PM (16 months ago)

Author:

espellcaste

Message:

WPCS: Part I: miscellaneous fixes for some of the files of the blogs component.

See #9174
See #9173

File:

• trunk/src/bp-blogs/classes/class-bp-blogs-theme-compat.php (modified) (10 diffs)

trunk/src/bp-blogs/classes/class-bp-blogs-theme-compat.php

@@ -38 +38 @@
 
         // Bail if not looking at a group.
-        if ( ! bp_is_blogs_component() )
+        if ( ! bp_is_blogs_component() ) {
             return;
+        }
 
         // Bail if looking at a users sites.
-        if ( bp_is_user() )
+        if ( bp_is_user() ) {
             return;
+        }
 
         // Blog Directory.
@@ -57 +59 @@
             do_action( 'bp_blogs_screen_index' );
 
-            add_filter( 'bp_get_buddypress_template',                array( $this, 'directory_template_hierarchy' ) );
+            add_filter( 'bp_get_buddypress_template', array( $this, 'directory_template_hierarchy' ) );
             add_action( 'bp_template_include_reset_dummy_post_data', array( $this, 'directory_dummy_post' ) );
-            add_filter( 'bp_replace_the_content',                    array( $this, 'directory_content'    ) );
-
-        // Create blog.
+            add_filter( 'bp_replace_the_content', array( $this, 'directory_content' ) );
+
+            // Create blog.
         } elseif ( is_user_logged_in() && bp_blog_signup_enabled() ) {
-            add_filter( 'bp_get_buddypress_template',                array( $this, 'create_template_hierarchy' ) );
+            add_filter( 'bp_get_buddypress_template', array( $this, 'create_template_hierarchy' ) );
             add_action( 'bp_template_include_reset_dummy_post_data', array( $this, 'create_dummy_post' ) );
-            add_filter( 'bp_replace_the_content',                    array( $this, 'create_content'    ) );
+            add_filter( 'bp_replace_the_content', array( $this, 'create_content' ) );
         }
     }
@@ -79 +81 @@
      * @since 1.8.0
      *
-     * @param string $templates The templates from bp_get_theme_compat_templates().
+     * @param array $templates The templates from bp_get_theme_compat_templates().
      * @return array $templates Array of custom templates to look for.
      */
     public function directory_template_hierarchy( $templates ) {
+
+        if ( empty( $templates ) || ! is_array( $templates ) ) {
+            $templates = array();
+        }
 
         /**
@@ -89 +95 @@
          * @since 1.8.0
          *
-         * @param array $value Array of template paths to add to template list to look for.
+         * @param array $template_paths Array of template paths to add to template list to look for.
          */
-        $new_templates = apply_filters( 'bp_template_hierarchy_blogs_create', array(
-            'blogs/index-directory.php'
-        ) );
+        $new_templates = apply_filters(
+            'bp_template_hierarchy_blogs_create',
+            array(
+                'blogs/index-directory.php',
+            )
+        );
 
         // Merge new templates with existing stack
         // @see bp_get_theme_compat_templates().
-        $templates = array_merge( (array) $new_templates, $templates );
-
-        return $templates;
+        return array_merge( $new_templates, $templates );
     }
 
@@ -108 +115 @@
      */
     public function directory_dummy_post() {
-
-        bp_theme_compat_reset_post( array(
-            'ID'             => 0,
-            'post_title'     => __( 'Sites', 'buddypress' ),
-            'post_author'    => 0,
-            'post_date'      => 0,
-            'post_content'   => '',
-            'post_type'      => 'page',
-            'post_status'    => 'publish',
-            'is_page'        => true,
-            'comment_status' => 'closed'
-        ) );
+        bp_theme_compat_reset_post(
+            array(
+                'ID'             => 0,
+                'post_title'     => __( 'Sites', 'buddypress' ),
+                'post_author'    => 0,
+                'post_date'      => 0,
+                'post_content'   => '',
+                'post_type'      => 'page',
+                'post_status'    => 'publish',
+                'is_page'        => true,
+                'comment_status' => 'closed',
+            )
+        );
     }
 
@@ -126 +134 @@
      *
      * @since 1.7.0
+     *
+     * @return string|null
      */
     public function directory_content() {
@@ -141 +151 @@
      * @since 1.8.0
      *
-     * @param string $templates The templates from bp_get_theme_compat_templates().
+     * @param array $templates The templates from bp_get_theme_compat_templates().
      * @return array $templates Array of custom templates to look for.
      */
     public function create_template_hierarchy( $templates ) {
+
+        if ( empty( $templates ) || ! is_array( $templates ) ) {
+            $templates = array();
+        }
 
         /**
@@ -151 +165 @@
          * @since 1.8.0
          *
-         * @param array $value Array of template paths to add to template list to look for.
+         * @param array $template_paths Array of template paths to add to template list to look for.
          */
-        $new_templates = apply_filters( 'bp_template_hierarchy_blogs_create', array(
-            'blogs/index-create.php'
-        ) );
+        $new_templates = apply_filters(
+            'bp_template_hierarchy_blogs_create',
+            array(
+                'blogs/index-create.php',
+            )
+        );
 
         // Merge new templates with existing stack
         // @see bp_get_theme_compat_templates().
-        $templates = array_merge( (array) $new_templates, $templates );
-
-        return $templates;
+        return array_merge( $new_templates, $templates );
     }
 
@@ -178 +193 @@
         }
 
-        bp_theme_compat_reset_post( array(
-            'ID'             => 0,
-            'post_title'     => $title,
-            'post_author'    => 0,
-            'post_date'      => 0,
-            'post_content'   => '',
-            'post_type'      => 'page',
-            'post_status'    => 'publish',
-            'is_page'        => true,
-            'comment_status' => 'closed'
-        ) );
+        bp_theme_compat_reset_post(
+            array(
+                'ID'             => 0,
+                'post_title'     => $title,
+                'post_author'    => 0,
+                'post_date'      => 0,
+                'post_content'   => '',
+                'post_type'      => 'page',
+                'post_status'    => 'publish',
+                'is_page'        => true,
+                'comment_status' => 'closed',
+            )
+        );
     }
 
@@ -195 +212 @@
      *
      * @since 1.7.0
+     *
+     * @return string|null
      */
     public function create_content() {