WordPress.org

Make WordPress Core

Opened 5 months ago

Last modified 6 weeks ago

#51877 accepted defect (bug)

Twenty Twenty: Wrong parameter name used in searchform.php

Reported by: poena Owned by: sabernhardt
Milestone: 5.8 Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords: has-patch
Focuses: Cc:

Description

In searchform.php, the parameter/argument that is used is named
$args['label'], the correct name for use with get_search_form() is $args['aria_label'].

See https://developer.wordpress.org/reference/functions/get_search_form/#parameters

Attachments (2)

51877.patch (680 bytes) - added by vipulc2 5 months ago.
Wrong parameter name used in searchform.php in Twenty Twenty changed $argslabel? to $argsaria_label?
51877.2.patch (2.5 KB) - added by sabernhardt 4 months ago.
editing argument to aria-label in get_search_form() functions for 3 files, and updating searchform.php to give a fallback for child theme templates that might use label

Download all attachments as: .zip

Change History (6)

#1 @sabernhardt
5 months ago

  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to Future Release

Would this be a good first bug?

In addition to replacing 'label' with 'aria_label', Twenty Twenty's searchform template still needs to accept 'label' as a valid parameter (if used in a child theme).

@vipulc2
5 months ago

Wrong parameter name used in searchform.php in Twenty Twenty changed $argslabel? to $argsaria_label?

#2 @kishanjasani
5 months ago

Thanks for Patch @vipulc2. Looks good to me.

@sabernhardt
4 months ago

editing argument to aria-label in get_search_form() functions for 3 files, and updating searchform.php to give a fallback for child theme templates that might use label

#3 @sabernhardt
4 months ago

  • Keywords has-patch added; needs-patch removed

This was a little more complicated than I had thought it might be. In addition to updating the argument in 4 files, 51877.2.patch adds a fallback for child theme backward compatibility.

#4 @sabernhardt
6 weeks ago

  • Milestone changed from Future Release to 5.8
  • Owner set to sabernhardt
  • Status changed from new to accepted
Note: See TracTickets for help on using tickets.