Changeset 14957

Timestamp:

06/11/2026 08:35:32 AM (2 months ago)

Author:

yani.iliev

Message:

Profiles: Link the handbook page from handbook-update activity items.

The activity notifier already sends the page title and permalink on handbook updates, but the handler's digest path discarded them. While a digest holds a single update, link the page itself: "Updated the handbook page [title] on [site]". Multiple same-day updates still collapse to the existing "Made N updates" summary. The singular string is percent-escaped because digest_bump() passes it back through sprintf().

Props dd32
Fixes #8228

File:

• sites/trunk/profiles.wordpress.org/public_html/wp-content/plugins/wporg-profiles-activity-handler/wporg-profiles-activity-handler.php (modified) (1 diff)

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

@@ -846 +846 @@
                                 $primary_link = sanitize_url( $data['blog_url'] ); // To group digest entries by site.
 
-                                $singular = sprintf(
-                                        'Updated a handbook page on <a href="%s">%s</a>.',
-                                        sanitize_url( $data['blog_url'] ),
-                                        sanitize_text_field( $data['blog'] )
-                                );
+                                if ( ! empty( $data['url'] ) && ! empty( $data['title'] ) ) {
+                                        // Link the page itself while the digest holds a single update (#8228).
+                                        $singular = sprintf(
+                                                'Updated the handbook page <i><a href="%s">%s</a></i> on <a href="%s">%s</a>.',
+                                                esc_url( $data['url'] ),
+                                                sanitize_text_field( $data['title'] ),
+                                                sanitize_url( $data['blog_url'] ),
+                                                sanitize_text_field( $data['blog'] )
+                                        );
+                                        // `digest_bump()` runs this through sprintf(); escape literal
+                                        // percent signs from encoded URLs or titles.
+                                        $singular = str_replace( '%', '%%', $singular );
+                                } else {
+                                        $singular = sprintf(
+                                                'Updated a handbook page on <a href="%s">%s</a>.',
+                                                sanitize_url( $data['blog_url'] ),
+                                                sanitize_text_field( $data['blog'] )
+                                        );
+                                }
 
                                 $plural = sprintf(