WordPress.org

Make WordPress Core

Opened 7 weeks ago

Last modified 2 weeks ago

#51877 new defect (bug)

Twenty Twenty: Wrong parameter name used in searchform.php

Reported by: poena Owned by:
Milestone: Future Release 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 7 weeks 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 2 weeks 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 (5)

#1 @sabernhardt
7 weeks 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
7 weeks ago

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

#2 @kishanjasani
7 weeks ago

Thanks for Patch @vipulc2. Looks good to me.

@sabernhardt
2 weeks 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
2 weeks 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.

Note: See TracTickets for help on using tickets.