WordPress.org
Get WordPress

Make WordPress Core

Opened 3 months ago

Closed 3 months ago

#62875 closed defect (bug) (fixed)

Document global $wp_local_package in wp-includes/l10n.php file.

Reported by: upadalavipul's profile upadalavipul Owned by: audrasjb's profile audrasjb
Milestone: 6.8 Priority: normal
Severity: normal Version:
Component: I18N Keywords: has-patch commit
Focuses: docs, coding-standards Cc:

Description

I have reviewed the below file code and found that we are using a global $wp_local_package variable. but they do not mention it in the function comment. So I think we need to improve the function comment code.

Files:

trunk/wp-includes/l10n.php

Attachments (2)

62875.patch​ (764 bytes) - added by upadalavipul 3 months ago.
patch Added.
62875-updated.patch​ (501 bytes) - added by dhruvang21 3 months ago.
Updated patch

Download all attachments as: .zip

Change History (5)

@upadalavipul
3 months ago

patch Added.

#1 @mukesh27
3 months ago

  • Component changed from General to I18N
  • Keywords needs-refresh added
  • Milestone changed from Awaiting Review to 6.8
  • Version trunk deleted

Hi there!

Thanks for the ticket and patch.

The patch need updates.

* @global string $pagenow          The filename of the current screen.
* @global string $wp_local_package Locale code of the package.

@dhruvang21
3 months ago

Updated patch

#2 @audrasjb
3 months ago

  • Keywords has-patch commit added; needs-refresh removed
  • Owner set to audrasjb
  • Status changed from new to assigned

#3 @audrasjb
3 months ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 59719:

Coding Standards: Add missing global variable in determine_locale() docblock.

Props upadalavipul, mukesh27, dhruvang21.
Fixes #62875.

Note: See TracTickets for help on using tickets.