Changeset 10663 for sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-profiles-wp-activity-notifier/wporg-profiles-wp-activity-notifier.php

Timestamp:

02/12/2021 12:16:38 AM (4 years ago)

Author:

dd32

Message:

Support: Don't include moderator query vars in the permalink sent to the Profiles feed.

Fixes #5607.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-profiles-wp-activity-notifier/wporg-profiles-wp-activity-notifier.php (modified) (5 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-profiles-wp-activity-notifier/wporg-profiles-wp-activity-notifier.php

@@ -256 +256 @@
         }
 
+        $url = bbp_get_topic_permalink( $topic_id );
+        // Remove moderator flags
+        $url = remove_query_arg( [ 'view' ], $url );
+
         $args = array(
             'body' => array(
@@ -266 +270 @@
                 'forum_id'  => bbp_get_topic_forum_id( $topic_id ),
                 'title'     => strip_tags( bbp_get_topic_title( $topic_id ) ),
-                'url'       => bbp_get_topic_permalink( $topic_id ),
+                'url'       => $url,
                 'message'   => bbp_get_topic_excerpt( $topic_id, 55 ),
                 'site'      => get_bloginfo( 'name' ),
@@ -273 +277 @@
         );
 
-        $x = wp_remote_post( $this->activity_handler_url, $args );
+        wp_remote_post( $this->activity_handler_url, $args );
     }
 
@@ -323 +327 @@
             return;
         }
+
+        $url = bbp_get_reply_url( $reply_id );
+        // Remove moderator flags
+        $url = remove_query_arg( [ 'view' ], $url );
 
         $args = array(
@@ -334 +342 @@
                 'forum_id'  => bbp_get_reply_forum_id( $reply_id ),
                 'title'     => strip_tags( bbp_get_reply_topic_title( $reply_id ) ),
-                'url'       => bbp_get_reply_url( $reply_id ),
+                'url'       => $url,
                 'message'   => $this->get_reply_excerpt( $reply_id, 15 ),
                 'site'      => get_bloginfo( 'name' ),