Changeset 9701 for trunk/src/bp-core/bp-core-buddybar.php

Timestamp:

04/06/2015 03:19:56 PM (10 years ago)

Author:

boonebgorges

Message:

Don't trailingslash the href of subnav items if the 'link' was provided explicitly in bp_core_new_subnav_item().

trailingslashit() is not smart enough to detect links with query params, so
that links can be improperly trailingslashed in some cases. More generally,
it's not the case that all links should be trailingslashed (in particular,
those that do not lead to WP/BP pages), so plugin authors should be responsible
for trailingslashing on their own.

Props pareshradadiya.
See #6353.

File:

• trunk/src/bp-core/bp-core-buddybar.php (modified) (2 diffs)

trunk/src/bp-core/bp-core-buddybar.php

@@ -297 +297 @@
     // Link was not forced, so create one
     if ( empty( $link ) ) {
-        $link = $parent_url . $slug;
+        $link = trailingslashit( $parent_url . $slug );
 
         // If this sub item is the default for its parent, skip the slug
         if ( ! empty( $bp->bp_nav[$parent_slug]['default_subnav_slug'] ) && $slug == $bp->bp_nav[$parent_slug]['default_subnav_slug'] ) {
-            $link = $parent_url;
+            $link = trailingslashit( $parent_url );
         }
     }
@@ -314 +314 @@
     $subnav_item = array(
         'name'              => $name,
-        'link'              => trailingslashit( $link ),
+        'link'              => $link,
         'slug'              => $slug,
         'css_id'            => $item_css_id,