Changeset 13436 for trunk/src/bp-blogs/classes/class-bp-blogs-component.php

Timestamp:

03/15/2023 08:16:46 AM (22 months ago)

Author:

imath

Message:

Make more BuddyPress generated links ready for BP Rewrites

• Improve the Members component adding permastructs and custom rewrite

rules for registration and activation pages.

• Improve the Blogs component adding custom rewrite rule to handle the

Blogs create page.

• Make a huge progress about replacing all occurrences of

bp_get_root_domain() (45 replacements were performed).

• Deprecate bp_groups_directory_permalink(),

bp_get_groups_directory_permalink(), bp_blogs_directory_permalink() &
bp_get_blogs_directory_permalink() and replace them with these new
functions: bp_groups_directory_url(), bp_get_groups_directory_url(),
bp_blogs_directory_url() & bp_get_blogs_directory_url().

• Although bp_loggedin_user_domain() & bp_displayed_user_domain()

should also be deprecated, we're leaving them as aliases of the right
functions to use. Plugin authors shouldn't use them to build other links
than member's profile home url.

NB: these deprecations are required because these functions were used
to build BuddyPress URLs concatenating URL chunks. Once the BP Classic
plugin will be built we will adapt the code to remove these deprecation
notices.

Props r-a-y, johnjamesjacoby, boonebgorges

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

File:

• trunk/src/bp-blogs/classes/class-bp-blogs-component.php (modified) (3 diffs)

trunk/src/bp-blogs/classes/class-bp-blogs-component.php

@@ -82 +82 @@
             'has_directory'         => is_multisite(), // Non-multisite installs don't need a top-level Sites directory, since there's only one site.
             'rewrite_ids'           => array(
-                'directory'                    => 'blogs',
-                'single_item_action'           => 'blogs_action',
-                'single_item_action_variables' => 'blogs_action_variables',
+                'directory'          => 'blogs',
+                'create_single_item' => 'blog_create',
             ),
             'directory_title'       => isset( $bp->pages->blogs->title ) ? $bp->pages->blogs->title : $default_directory_title,
@@ -312 +311 @@
             // Create a Site.
             if ( bp_blog_signup_enabled() ) {
+                $url = bp_get_blogs_directory_url(
+                    array(
+                        'create_single_item' => 1,
+                    )
+                );
+
                 $wp_admin_nav[] = array(
                     'parent'   => 'my-account-' . $this->id,
                     'id'       => 'my-account-' . $this->id . '-create',
                     'title'    => __( 'Create a Site', 'buddypress' ),
-                    'href'     => trailingslashit( bp_get_blogs_directory_permalink() . 'create' ),
+                    'href'     => $url,
                     'position' => 99
                 );
@@ -371 +376 @@
 
         parent::setup_cache_groups();
+    }
+
+    /**
+     * Add the Blog Create rewrite tags.
+     *
+     * @since 12.0.0
+     *
+     * @param array $rewrite_tags Optional. See BP_Component::add_rewrite_tags() for
+     *                            description.
+     */
+    public function add_rewrite_tags( $rewrite_tags = array() ) {
+        $rewrite_tags = array(
+            'create_single_item' => '([1]{1,})',
+        );
+
+        parent::add_rewrite_tags( $rewrite_tags );
+    }
+
+    /**
+     * Add the Registration and Activation rewrite rules.
+     *
+     * @since 12.0.0
+     *
+     * @param array $rewrite_rules Optional. See BP_Component::add_rewrite_rules() for
+     *                             description.
+     */
+    public function add_rewrite_rules( $rewrite_rules = array() ) {
+        $create_slug = bp_rewrites_get_slug( 'blogs', 'blog_create', 'create' );
+
+        $rewrite_rules = array(
+            'create_single_item' => array(
+                'regex' => $this->root_slug . '/' . $create_slug . '/?$',
+                'order' => 50,
+                'query' => 'index.php?' . $this->rewrite_ids['directory'] . '=1&' . $this->rewrite_ids['create_single_item'] . '=1',
+            ),
+        );
+
+        parent::add_rewrite_rules( $rewrite_rules );
     }