Changeset 62000

Timestamp:

03/13/2026 01:36:00 AM (2 months ago)

Author:

westonruter

Message:

Twenty Seventeen: Restore $post_id parameter to twentyseventeen_edit_link() which is being passed in some templates.

This addresses arguments.count errors identified by PHPStan at rule level 1.

Developed in ​https://github.com/WordPress/wordpress-develop/pull/11242

Follow-up to WordPress/twentyseventeen commits: ​1afb73c, ​7dd3025, ​6948288.

Props sabernhardt, westonruter.
See #64680.
Fixes #64825.

File:

• trunk/src/wp-content/themes/twentyseventeen/inc/template-tags.php (modified) (1 diff)

trunk/src/wp-content/themes/twentyseventeen/inc/template-tags.php

@@ -111 +111 @@
 if ( ! function_exists( 'twentyseventeen_edit_link' ) ) :
     /**
-     * Returns an accessibility-friendly link to edit a post or page.
-     *
-     * This also gives a little context about what exactly we're editing
-     * (post or page?) so that users understand a bit more where they are in terms
-     * of the template hierarchy and their content. Helpful when/if the single-page
-     * layout with multiple posts/pages shown gets confusing.
-     */
-    function twentyseventeen_edit_link() {
+     * Displays an accessibility-friendly link to edit a post or page.
+     *
+     * @since Twenty Seventeen 1.0
+     * @since Twenty Seventeen 4.1 Added `$post_id` parameter.
+     *
+     * @param int $post_id Post ID. Default 0.
+     */
+    function twentyseventeen_edit_link( $post_id = 0 ) {
         edit_post_link(
             sprintf(
                 /* translators: %s: Post title. Only visible to screen readers. */
                 __( 'Edit<span class="screen-reader-text"> "%s"</span>', 'twentyseventeen' ),
-                get_the_title()
+                get_the_title( $post_id )
             ),
             '<span class="edit-link">',
-            '</span>'
+            '</span>',
+            $post_id
         );
     }