WordPress.org

Make WordPress Core

Opened 14 months ago

Last modified 14 months ago

#46842 new defect (bug)

Doc: Missing @param in inline document

Reported by: SaeedFard Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Administration Keywords: has-patch needs-refresh
Focuses: docs, administration Cc:

Description

Missing @param in inline document in wp-admin\includes\class-wp-list-table-compat.php

Attachments (3)

46842.patch (537 bytes) - added by SaeedFard 14 months ago.
46842-2.patch (540 bytes) - added by SaeedFard 14 months ago.
46842-3.patch (595 bytes) - added by SaeedFard 14 months ago.

Download all attachments as: .zip

Change History (8)

@SaeedFard
14 months ago

@SaeedFard
14 months ago

#1 @swissspidy
14 months ago

  • Keywords needs-refresh added

@SaeedFard In your patch you added $args and $screen, but the actual parameters are $screen and $columns.

Also, patches should follow the PHP documentation standards outlined in https://make.wordpress.org/core/handbook/best-practices/inline-documentation-standards/php/.

#2 @SaeedFard
14 months ago

Thanks @swissspidy
I'll update it.

@SaeedFard
14 months ago

#3 @SaeedFard
14 months ago

@swissspidy would you please check the new patch?
Thanks

#4 @desrosj
14 months ago

@SaeedFard The parameters should line up at the $. Also, please add a description for each parameter.

#5 @desrosj
14 months ago

  • Milestone changed from Awaiting Review to Future Release
Note: See TracTickets for help on using tickets.