Changeset 13446 for trunk/src/bp-groups/bp-groups-functions.php

Timestamp:

04/04/2023 01:06:03 AM (2 years ago)

Author:

imath

Message:

Use the BP Rewrites API to generate single Groups item links

• Introduce the bp_groups_get_path_chunks() function to build BP Rewrites arguments using a regular array.
• Improve bp_members_get_path_chunks() so that it avoids looking for a custom slug when the chunk is a numeric value.
• Introduce the bp_get_group_manage_url() function to build Group's front-end Admin URLs using BP Rewrites.
• Deprecate bp_group_admin_permalink() & bp_get_group_admin_permalink() in favor of bp_group_manage_url() & bp_get_group_manage_url().
• Replace all remaining usage of bp_get_group_permalink() in favor of bp_get_group_url() or bp_get_group_manage_url().

Props r-a-y, johnjamesjacoby, boonebgorges

Closes ​https://github.com/buddypress/buddypress/pull/81
See #4954

File:

• trunk/src/bp-groups/bp-groups-functions.php (modified) (2 diffs)

trunk/src/bp-groups/bp-groups-functions.php

@@ -3836 +3836 @@
                 'position'        => 10,
                 'item_css_id'     => 'home',
+            ),
+            'activity'            => array(
+                'rewrite_id'      => 'bp_group_read_activity',
+                'slug'            => 'activity',
+                'name'            => _x( 'Activity', 'Group read screen', 'buddypress' ),
+                'screen_function' => 'groups_screen_group_activity',
+                'position'        => 11,
+                'user_has_access' => false,
+                'no_access_url'   => '',
+                'item_css_id'     => 'activity',
             ),
             'request-membership' => array(
@@ -3980 +3990 @@
     return $context_screens;
 }
+
+/**
+ * Get single group's path chunks using an array of URL slugs.
+ *
+ * @since 12.0.0
+ *
+ * @param array $chunks An array of URL slugs.
+ * @return array An array of BP Rewrites URL arguments.
+ */
+function bp_groups_get_path_chunks( $chunks = array(), $context = 'read' ) {
+    $path_chunks   = array();
+    $group_screens = bp_get_group_screens( $context );
+
+    if ( 'read' === $context ) {
+        $single_item_action = array_shift( $chunks );
+        if ( $single_item_action ) {
+            if ( isset( $group_screens[ $single_item_action ]['rewrite_id'] ) ) {
+                $item_action_rewrite_id            = $group_screens[ $single_item_action ]['rewrite_id'];
+                $path_chunks['single_item_action'] = bp_rewrites_get_slug( 'groups', $item_action_rewrite_id, $single_item_action );
+            } else {
+                $path_chunks['single_item_action'] = $single_item_action;
+            }
+        }
+    }
+
+    if ( $chunks ) {
+        foreach ( $chunks as $chunk ) {
+            if ( is_numeric( $chunk ) ) {
+                $path_chunks['single_item_action_variables'][] = $chunk;
+            } else {
+                if ( isset( $group_screens[ $chunk ]['rewrite_id'] ) ) {
+                    $item_action_variable_rewrite_id               = $group_screens[ $chunk ]['rewrite_id'];
+                    $path_chunks['single_item_action_variables'][] = bp_rewrites_get_slug( 'groups', $item_action_variable_rewrite_id, $chunk );
+                } else {
+                    $path_chunks['single_item_action_variables'][] = $chunk;
+                }
+            }
+        }
+    }
+
+    return $path_chunks;
+}