Make WordPress Core

Opened 4 years ago

Closed 4 years ago

#53182 closed defect (bug) (fixed)

define()s for Network don't conform to WordPress coding standards

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

Description

When you generate a multisite network, WordPress creates some define() statements for you to paste in to wp-config.php.

Those statements lack spacing and don't conform to WordPress coding standards.

Change History (5)

This ticket was mentioned in PR #1248 on WordPress/wordpress-develop by paulschreiber.


4 years ago
#1

  • Keywords has-patch added

Adds spacing so define() statements conform to WordPress coding standards.

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

#2 @johnjamesjacoby
4 years ago

👍 PR looks good to me.

#3 @SergeyBiryukov
4 years ago

  • Milestone changed from Awaiting Review to 5.8

#4 @SergeyBiryukov
4 years ago

  • Focuses coding-standards added

#5 @SergeyBiryukov
4 years ago

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

In 50839:

Coding Standards: Adds spacing so define() statements displayed when creating a network.

This ensures that the code intended to be copied to wp-config.php conforms to WordPress Coding Standards.

Props paulschreiber, johnjamesjacoby.
Fixes #53182.

Note: See TracTickets for help on using tickets.