WordPress.org
Get WordPress

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#48237 closed defect (bug) (fixed)

Remove spaces around non-variable array items

Reported by: itowhid06's profile itowhid06 Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 5.3 Priority: normal
Severity: trivial Version:
Component: Networks and Sites Keywords: has-patch commit
Focuses: multisite, coding-standards Cc:

Description

From coding standards handbook-
When referring to array items, only include a space around the index if it is a variable

Attachments (1)

48237.diff​ (683 bytes) - added by itowhid06 5 years ago.

Download all attachments as: .zip

Change History (4)

@itowhid06
5 years ago

#1 @jrf
5 years ago

  • Keywords commit added

LGTM, though I'm a bit surprised this wasn't caught in one of the auto-fixer rounds.

#2 @SergeyBiryukov
5 years ago

  • Component changed from General to Networks and Sites
  • Focuses multisite added
  • Milestone changed from Awaiting Review to 5.3

#3 @SergeyBiryukov
5 years ago

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

In 46420:

Coding Standards: Fix WPCS issues in wp-includes/ms-deprecated.php.

Props itowhid06.
Fixes #48237.

Note: See TracTickets for help on using tickets.