WordPress.org
Get WordPress

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#57210 closed defect (bug) (fixed)

Inconsistent function definition brace

Reported by: mukesh27 Owned by: audrasjb
Priority: normal Milestone: 6.2
Component: General Version:
Severity: normal Keywords: has-patch good-first-bug
Cc: Focuses: coding-standards

Description

Related to #57209.

This is inconsistent because the function's right bracket is not present in the new line. The new line should contain the brace.

Change History (7)

This ticket was mentioned in ​PR #3688 on ​WordPress/wordpress-develop by ​@mukesh27.
4 years ago #1

  • Keywords has-patch added; needs-patch removed

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

#2 @audrasjb
4 years ago

  • Owner set to audrasjb
  • Status new → reviewing

#3 @davidbaumwald
4 years ago

  • Keywords good-first-bug added

#4 @audrasjb
4 years ago

  • Resolution → fixed
  • Status reviewing → closed

In 54881:

Coding Standards: Various brace indentation corrections.

Props mukesh27.
Fixes #57210.
See #56791.

#5 @audrasjb
4 years ago

Please note that I separated this patch in two commits. One for Core fixes, and one for Twentyten.

#7 @audrasjb
4 years ago

In 54882:

Twenty Ten: Fixes brace indentation in loop-attachment template.

Props mukesh27.
See #57210, #56791.

Note: See TracTickets for help on using tickets.