Changeset 14022

Timestamp:

09/10/2024 05:10:17 PM (2 months ago)

Author:

imath

Message:

WPCS: improve BP Nouveau's main functions file code formatting.

Props narenin

Fixes #9230
See #9174
Closes ​https://github.com/buddypress/buddypress/pull/372

File:

• trunk/src/bp-templates/bp-nouveau/buddypress-functions.php (modified) (11 diffs)

trunk/src/bp-templates/bp-nouveau/buddypress-functions.php

@@ -5 +5 @@
  * @since 3.0.0
  * @package BuddyPress
- * @version 12.0.0
+ * @version 15.0.0
  *
  * @buddypress-template-pack {
@@ -51 +51 @@
     public static function get_instance() {
         if ( null === self::$instance ) {
-            self::$instance = new self;
+            self::$instance = new self();
         }
 
@@ -101 +101 @@
             require $this->includes_dir . 'ajax.php';
 
-        // Load AJAX code only on AJAX requests.
+            // Load AJAX code only on AJAX requests.
         } else {
-            add_action( 'admin_init', function () {
-                if ( defined( 'DOING_AJAX' ) && true === DOING_AJAX ) {
-                    require bp_nouveau()->includes_dir . 'ajax.php';
-                }
-            }, 0 );
+            add_action(
+                'admin_init',
+                function () {
+                    if ( wp_doing_ajax() ) {
+                        require bp_nouveau()->includes_dir . 'ajax.php';
+                    }
+                },
+                0
+            );
         }
 
@@ -122 +126 @@
             );
 
-            // When BP Classic is activated, regular themes need this filter. 
+            // When BP Classic is activated, regular themes need this filter.
             if ( function_exists( 'bp_classic' ) ) {
                 // Set the BP Uri for the Ajax customizer preview.
@@ -138 +142 @@
             }
 
-            require( $component_loader );
+            require $component_loader;
         }
 
@@ -242 +246 @@
         add_action( 'bp_nouveau_feedback_messages', array( $this, 'filter_registration_messages' ), 99 );
 
-        /** Override **********************************************************/
-
         /**
          * Fires after all of the BuddyPress theme compat actions have been added.
@@ -287 +289 @@
          * @param array $value Array of styles to enqueue.
          */
-        $styles = apply_filters( 'bp_nouveau_enqueue_styles', array(
-            'bp-nouveau' => array(
-                'file' => 'css/buddypress%1$s%2$s.css', 'dependencies' => $css_dependencies, 'version' => $this->version,
-            ),
-            'bp-nouveau-priority-nav' => array(
-                'file' => 'css/priority-nav%1$s%2$s.css', 'dependencies' => array( 'dashicons' ), 'version' => $this->version,
-            ),
-        ) );
+        $styles = apply_filters(
+            'bp_nouveau_enqueue_styles',
+            array(
+                'bp-nouveau'              => array(
+                    'file'         => 'css/buddypress%1$s%2$s.css',
+                    'dependencies' => $css_dependencies,
+                    'version'      => $this->version,
+                ),
+                'bp-nouveau-priority-nav' => array(
+                    'file'         => 'css/priority-nav%1$s%2$s.css',
+                    'dependencies' => array( 'dashicons' ),
+                    'version'      => $this->version,
+                ),
+            )
+        );
 
         if ( $styles ) {
@@ -388 +397 @@
             'bp_nouveau_register_scripts',
             array(
-                'bp-nouveau' => array(
+                'bp-nouveau'               => array(
                     'file'         => 'js/buddypress-nouveau%s.js',
                     'dependencies' => $dependencies,
@@ -399 +408 @@
                     'version'      => $this->version,
                     'footer'       => true,
-                )
+                ),
             )
         );
@@ -493 +502 @@
     public function localize_scripts() {
         $params = array(
-            'ajaxurl'             => bp_core_ajax_url(),
-            'confirm'             => __( 'Are you sure?', 'buddypress' ),
+            'ajaxurl'           => bp_core_ajax_url(),
+            'confirm'           => __( 'Are you sure?', 'buddypress' ),
 
             /* translators: %s: number of activity comments */
-            'show_x_comments'     => __( 'Show all %d comments', 'buddypress' ),
-            'unsaved_changes'     => __( 'Your profile has unsaved changes. If you leave the page, the changes will be lost.', 'buddypress' ),
-            'object_nav_parent'   => '#buddypress',
+            'show_x_comments'   => __( 'Show all %d comments', 'buddypress' ),
+            'unsaved_changes'   => __( 'Your profile has unsaved changes. If you leave the page, the changes will be lost.', 'buddypress' ),
+            'object_nav_parent' => '#buddypress',
         );
 
@@ -678 +687 @@
 
         foreach ( $nav_items as $nav_item ) {
-            if ( empty( $nav_item['component'] ) || $nav_item['component'] !== bp_current_component() ) {
+            if ( empty( $nav_item['component'] ) || bp_current_component() !== $nav_item['component'] ) {
                 continue;
             }