Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#51426 closed defect (bug) (fixed)

@param boolean To @param bool

Reported by: ravipatel's profile ravipatel Owned by: johnbillion's profile johnbillion
Milestone: 5.6 Priority: normal
Severity: normal Version:
Component: General Keywords: has-patch
Focuses: docs Cc:

Description

@param boolean to bool

Per core's PHP documentation standards:
For boolean and integer types, use bool and int, respectively.

Attachments (4)

51426.patch (2.8 KB) - added by ravipatel 5 years ago.
Per core's PHP documentation standards: For boolean and integer types, use bool and int, respectively.
51426-php-code-standards.patch (14.0 KB) - added by ravipatel 5 years ago.
Per core's PHP documentation standards: use bool and int
51426-php-code-standards-1.patch (4.0 KB) - added by ravipatel 5 years ago.
include this patch with 2 replaced @return integer to int
51426.diff (33.0 KB) - added by justinahinon 5 years ago.

Download all attachments as: .zip

Change History (10)

@ravipatel
5 years ago

Per core's PHP documentation standards: For boolean and integer types, use bool and int, respectively.

This ticket was mentioned in Slack in #core by ravi. View the logs.


5 years ago

@ravipatel
5 years ago

Per core's PHP documentation standards: use bool and int

@ravipatel
5 years ago

include this patch with 2 replaced @return integer to int

#3 @justinahinon
5 years ago

It may be good to explore other parts in Core when this change can be made also.

@justinahinon
5 years ago

#4 @justinahinon
5 years ago

I've merged the changes in the three patches proposed by @ravipatel as well as refreshed against trunk.

#5 @johnbillion
5 years ago

  • Focuses coding-standards removed
  • Milestone changed from Awaiting Review to 5.6
  • Owner set to johnbillion
  • Status changed from new to accepted
  • Version trunk deleted

Thanks for the patches @ravipatel @ravipatel .

Note that some of these files are third party libraries that we usually don't change (eg. Requests and SimplePie), but the changes in WordPress core files look good.

Version 0, edited 5 years ago by johnbillion (next)

#6 @johnbillion
5 years ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 49120:

Docs: Standardise the type name for booleans and integers.

This brings these docs inline with the documentation standards.

Props ravipatel, justinahinon

Fixes #51426

Note: See TracTickets for help on using tickets.