Make WordPress Core

Opened 4 months ago

Closed 3 months ago

#60146 closed defect (bug) (fixed)

$global Should add after @since according to PHP Documentation Standards in wp-settings.php

Reported by: shailu25's profile shailu25 Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 6.5 Priority: normal
Severity: trivial Version:
Component: General Keywords: has-patch commit
Focuses: docs, coding-standards Cc:

Description

In wp-settings.php file

$global Should add after @since according to PHP Documentation Standards

Attachments (2)

60146.patch (2.4 KB) - added by shailu25 4 months ago.
Patch Added
60146.2.patch (2.7 KB) - added by shailu25 4 months ago.
Patch Updated

Download all attachments as: .zip

Change History (9)

@shailu25
4 months ago

Patch Added

#1 @shailu25
4 months ago

  • Keywords has-patch added

#2 @shailu25
4 months ago

  • Version set to trunk

#3 @mukesh27
4 months ago

  • Focuses coding-standards added
  • Keywords needs-refresh added
  • Milestone changed from Awaiting Review to 6.5
  • Version trunk deleted

Thanks for ticket and patch!

Could you please update https://github.com/WordPress/wordpress-develop/blob/trunk/src/wp-settings.php#L56-L57 as it missing in patch.

@shailu25
4 months ago

Patch Updated

#4 @shailu25
4 months ago

I have Updated Patch as per Suggested.

#5 @shailu25
3 months ago

  • Keywords needs-refresh removed

#6 @mukesh27
3 months ago

  • Keywords commit added

Thanks @shailu25 for the update.

@audrasjb could you please review when you get some moment.

#7 @SergeyBiryukov
3 months ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 57320:

Docs: Correct the placement of @global tags in wp-settings.php.

Props shailu25, mukesh27.
Fixes #60146.

Note: See TracTickets for help on using tickets.