Changeset 7447

Timestamp:

10/19/2013 07:56:43 PM (11 years ago)

Author:

r-a-y

Message:

Blogs: Modify recorded post permalink to use shortlink.

Previously, when BuddyPress recorded the post permalink in the
activity stream, it would use the current permalink structure of the blog.

The problem with this is if the blog changed its permalink structure,
the post permalink in the activity post would not resolve properly
because of the change.

This commit alters the recorded post permalink to use the shortlink
(domain/?p=POST_ID). Because WP's rewrite rules will pick up the
'p' query parameter and redirect to the proper post permalink, this
will ensure that the post permalink for future activity blog posts
will work as intended.

Fixes #4367.

File:

• trunk/bp-blogs/bp-blogs-functions.php (modified) (1 diff)

trunk/bp-blogs/bp-blogs-functions.php

@@ -289 +289 @@
 
             // Record this in activity streams
-            $post_permalink   = get_permalink( $post_id );
+            $post_permalink = add_query_arg(
+                'p',
+                $post_id,
+                trailingslashit( get_home_url( $blog_id ) )
+            );
 
             if ( is_multisite() )