Context Navigation

← Previous Ticket
Next Ticket →

#38371 closed defect (bug) (fixed)

PHP documentation for wp_list_comments parameter reverse_top_level is incorrect

Reported by:	bcole808	Owned by:	boonebgorges
Priority:	normal	Milestone:	4.7
Component:	Comments	Version:	4.7
Severity:	normal	Keywords:
Cc:		Focuses:	docs

Description

The PHP docs for the parameter currently reads:

@type string $reverse_top_level Ordering of the listed comments. Default null. Accepts 'desc', 'asc'.

however in reading how this parameter is used, it must be set to a boolean value. Using the string "desc" or "asc" will cause the function to always show the oldest comments first, as it gets converted to a boolean value.

Attached is a patch to update the docs to reflect actual behavior.

Attachments (1)

38371.patch (1.0 KB ) - added by bcole808 10 years ago.

Download all attachments as: .zip

Change History (3)

@bcole808
10 years ago

Attachment 38371.patch added

#1 @boonebgorges
10 years ago

Milestone Awaiting Review → 4.7

Hi @bcole808 - Thanks for the ticket and the patch. Looks good to me.

Introduced in [25567].

#2 @boonebgorges
10 years ago

Owner set to boonebgorges
Resolution → fixed
Status new → closed

In 38845:

Comments: Fix description for reverse_top_level argument in wp_list_comments().

It's a boolean, not a string. Introduced in [25567].

Props bcole808.
Fixes #38371.

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core