Changeset 5313 for trunk/bp-core/bp-core-buddybar.php

Timestamp:

11/08/2011 09:37:42 PM (15 years ago)

Author:

boonebgorges

Message:

Force ambiguous item URIs to resolve to their most specific versions, to avoid content duplication. Fixes #1741

File:

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

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

@@ -70 +70 @@
                         add_action( 'bp_screens', array( &$screen_function[0], $screen_function[1] ), 3 );
 
-                if ( !empty( $default_subnav_slug ) )
-                        $bp->current_action = apply_filters( 'bp_default_component_subnav', $default_subnav_slug, $r );
+                if ( !empty( $default_subnav_slug ) ) {
+                        /**
+                         * BuddyPress will attempt to resolve to the most specific URL possible,
+                         * to avoid search-engine-unfriendly content reduplication. Filter
+                         * bp_guarantee_unique_uris (and return false) to avoid this behavior
+                         */
+                        if ( apply_filters( 'bp_guarantee_unique_uris', true ) ) {
+                                bp_core_redirect( bp_displayed_user_domain() . $slug . '/' . apply_filters( 'bp_default_component_subnav', $default_subnav_slug, $r ) );
+                        } else {
+                                 $bp->current_action = apply_filters( 'bp_default_component_subnav', $default_subnav_slug, $r );
+                        }
+                }
 
         // Look for current item
@@ -80 +90 @@
                         add_action( 'bp_screens', array( &$screen_function[0], $screen_function[1] ), 3 );
 
-                if ( !empty( $default_subnav_slug ) )
-                        $bp->current_action = apply_filters( 'bp_default_component_subnav', $default_subnav_slug, $r );
+                if ( !empty( $default_subnav_slug ) ) {
+                        /**
+                         * BuddyPress will attempt to resolve to the most specific URL possible,
+                         * to avoid search-engine-unfriendly content reduplication. Filter
+                         * bp_guarantee_unique_uris (and return false) to avoid this behavior
+                         */
+                        if ( apply_filters( 'bp_guarantee_unique_uris', true ) ) {
+                                bp_core_redirect( bp_get_root_domain() . '/' . bp_get_root_slug( bp_current_component() ) . '/' . $slug . '/' . apply_filters( 'bp_default_component_subnav', $default_subnav_slug, $r ) );
+                        } else {
+                                 $bp->current_action = apply_filters( 'bp_default_component_subnav', $default_subnav_slug, $r );
+                        }
+                }
         }