Opened 9 months ago
Last modified 9 months ago
#59992 new defect (bug)
Document global $locale in wp-admin/options.php
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | Awaiting Review | Priority: | normal |
Severity: | normal | Version: | |
Component: | General | Keywords: | 2nd-opinion |
Focuses: | docs, coding-standards | Cc: |
Description
I have review the below file code and found the we are used a global locale variable. but they do not mention in function comment. So I think we need improve function comment code.
File Names: wp-admin/options.php
Attachments (1)
Change History (2)
Note: See
TracTickets for help on using
tickets.
The
$locale
global is on line 347, not inside a named function with its own docblock. 59992.patch adds to the docblock for theallowed_options
filter, which does not include the global variable. The comment immediately above line 347 could be good enough.[39122] added the
$locale
variable (and the comment).