Changeset 7360 for trunk/src/includes/extend/buddypress/groups.php

Timestamp:

11/16/2025 10:43:01 PM (4 months ago)

Author:

johnjamesjacoby

Message:

Tools - Code Improvement: Updates to the phpcs.xml.dist config file.

This commit implements the WordPress.WP.I18n.MissingTranslatorsComment code sniff, fixes a bunch of whitespace regressions from #3614 and #3613.

Props johnjamesjacoby, sirlouen.

In trunk, for 2.7.

Fixes #3615.

File:

• trunk/src/includes/extend/buddypress/groups.php (modified) (3 diffs)

trunk/src/includes/extend/buddypress/groups.php

@@ -484 +484 @@
                     <label for="bbp_group_forum_id"><?php esc_html_e( 'Group Forum:', 'bbpress' ); ?></label>
                     <?php
-                        bbp_dropdown( array(
-                            'select_id' => 'bbp_group_forum_id',
-                            'show_none' => esc_html__( '&mdash; No forum &mdash;', 'bbpress' ),
-                            'selected'  => $forum_id
-                        ) );
+                        bbp_dropdown(
+                            array(
+                                'select_id' => 'bbp_group_forum_id',
+                                'show_none' => esc_html__( '&mdash; No forum &mdash;', 'bbpress' ),
+                                'selected'  => $forum_id
+                            )
+                        );
                     ?>
                     <p class="description"><?php esc_html_e( 'Network administrators can reconfigure which forum belongs to this group.', 'bbpress' ); ?></p>
@@ -1693 +1695 @@
     }
 
-    /**
-     * Fixes rewrite pagination in BuddyPress Group Forums & Topics.
-     *
-     * Required for compatibility with BuddyPress > 12.0, where the /groups/
+    /**
+     * Fixes rewrite pagination in BuddyPress Group Forums & Topics.
+     *
+     * Required for compatibility with BuddyPress > 12.0, where the /groups/
      * rewrite rule will be caught before bbPress's /page/ rule.
      *
@@ -1704 +1706 @@
      * @param  string $type   Type of variable to check with `is_a()`
      * @return mixed  $object Verified object if valid, Default or null if invalid
-     */
-    public function rewrite_pagination( $object, $type = '' ) {
+     */
+    public function rewrite_pagination( $object, $type = '' ) {
 
         // Bail if wrong global
-        if ( 'wp_query' !== $type ) {
-            return $object;
-        }
+        if ( 'wp_query' !== $type ) {
+            return $object;
+        }
 
         // Bail if not inside a BuddyPress Group
-        if ( ! bp_is_group() ) {
-            return $object;
-        }
+        if ( ! bp_is_group() ) {
+            return $object;
+        }
 
         // Bail if not inside a BuddyPress Group Forum
-        if ( ! bp_is_current_action( 'forum' ) ) {
-            return $object;
-        }
+        if ( ! bp_is_current_action( 'forum' ) ) {
+            return $object;
+        }
 
         // Default "paged" value
-        $page_number = null;
-
-        // Can't use bbp_is_single_topic() because it triggers a loop.
-        $is_single_topic = bp_is_action_variable( 'topic', 0 );
+        $page_number = null;
+
+        // Can't use bbp_is_single_topic() because it triggers a loop.
+        $is_single_topic = bp_is_action_variable( 'topic', 0 );
 
         // Single Topic
-        if ( true === $is_single_topic ) {
+        if ( true === $is_single_topic ) {
 
             // Get the page number from 3rd position
-            if ( bp_is_action_variable( 'page', 2 ) ) {
-                $page_number = bp_action_variable( 3 );
-            }
+            if ( bp_is_action_variable( 'page', 2 ) ) {
+                $page_number = bp_action_variable( 3 );
+            }
 
         // Single Forum
-        } else {
+        } else {
 
             // Get the page number from 1st position
-            if ( bp_is_action_variable( 'page', 0 ) ) {
-                $page_number = bp_action_variable( 1 );
-            }
-        }
+            if ( bp_is_action_variable( 'page', 0 ) ) {
+                $page_number = bp_action_variable( 1 );
+            }
+        }
 
         // Bail if no page number
-        if ( empty( $page_number ) ) {
-            return $object;
-        }
+        if ( empty( $page_number ) ) {
+            return $object;
+        }
 
         // Set the 'paged' WP_Query var to the new action-based value
-        $object->set( 'paged', $page_number );
+        $object->set( 'paged', $page_number );
 
         // Return the filtered/modified object
-        return $object;
-    }
+        return $object;
+    }
 
     /**