Make WordPress Core

Context Navigation

← Previous Ticket
Next Ticket →

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#57209 closed defect (bug) (fixed)

Inconsistent function definition brace

Reported by:	Alberuni Azad.	Owned by:	audrasjb
Priority:	normal	Milestone:	6.2
Component:	General	Version:	4.4
Severity:	normal	Keywords:	has-patch commit
Cc:		Focuses:	coding-standards

Description

On line 69 of the wp-admin/includes/noop.php file. The right brace of the function is not in the new line which is not consistent. The brace should be in the new line.

Change History (7)

#1 @audrasjb
4 years ago

Owner set to audrasjb
Status new → reviewing

Hello @alberuni-azad,

Thanks for reporting this. Would you like to propose a patch?

This ticket was mentioned in PR #3687 on WordPress/wordpress-develop by @Alberuni Azad..

4 years ago #2

Keywords has-patch added

Trac ticket: https://core.trac.wordpress.org/ticket/57209#ticket

#3 @audrasjb
4 years ago

Milestone Awaiting Review → 6.2
Version → 4.4

Introduced in [34037].

#4 @Alberuni Azad.
4 years ago

Hi @audrasjb I've submitted the patch.

#5 @audrasjb
4 years ago

Keywords commit added
Status reviewing → accepted

Ah thanks for the patch. Self-assigning for commit.

#6 @audrasjb
4 years ago

Resolution → fixed
Status accepted → closed

Coding Standards: Fix brace indentation in wp-align/includes/noop.php.

Props alberuni-azad.
Fixes #57209.
See #56792.

@audrasjb commented on PR #3687:

4 years ago #7

Committed in https://core.trac.wordpress.org/changeset/54876

Note: See TracTickets for help on using tickets.

Download in other formats: