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 @justinahinon.

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.

Last edited 5 years ago by johnbillion (previous) (diff)

#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.