Make WordPress Core

Opened 3 months ago

Last modified 3 months ago

#59347 new enhancement

Typo on the Gettext_Translations class

Reported by: tohincoderex's profile tohincoderex Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: 6.3.1
Component: General Keywords: has-patch close
Focuses: coding-standards Cc:

Description

There appears to be a typo in the function name and PHPDoc comments within the file located at src/wp-includes/pomo/translations.php

Line - https://github.com/WordPress/wordpress-develop/blob/943e2b95511794417d545ca06fa22c0167391267/src/wp-includes/pomo/translations.php#L255

Change History (2)

This ticket was mentioned in PR #5213 on WordPress/wordpress-develop by @tohincoderex.


3 months ago
#1

Trac ticket: https://core.trac.wordpress.org/ticket/59347

Description: Typo on the Gettext_Translations class

  • changeparenthesize_plural_exression function name toparenthesize_plural_expression
  • change oerators word tooperators

#2 @jrf
3 months ago

  • Focuses php-compatibility removed
  • Keywords close added

@tohincoderex Thanks for opening this ticket and bringing this to our attention.

This file is regarded as a file from an external dependency and should for that reason not be changed without due cause.

Renaming the function would break every single function call to that function and would cause fatal errors. Breaking changes like that are not acceptable.

The typo has been in the original external dependency since (at least) 2009.
Of course, the function could be deprecated, a new function without the typo introduced and calls to the function could be updated, but as this is an external dependency, that is the kind of change we normally do not do for those kind of files (at least, not unless the change has first been made in the externally maintained code first).

Note: See TracTickets for help on using tickets.