WordPress.org

Make WordPress Core

#47481 closed enhancement (fixed)

Wrong arguments order in the_meta()

Reported by: loyaltymanufaktur Owned by: SergeyBiryukov
Milestone: 5.3 Priority: normal
Severity: minor Version: 1.0
Component: Posts, Post Types Keywords: has-patch commit
Focuses: template Cc:

Description

./wp-includes/post-template.php:1107

$value  = implode( $values, ', ' );

The glue argument should be the first one:

$value  = implode( ', ', $values );

Attachments (1)

100644.patch (465 bytes) - added by loyaltymanufaktur 18 months ago.

Download all attachments as: .zip

Change History (3)

#1 @ocean90
18 months ago

  • Keywords commit added
  • Milestone changed from Awaiting Review to 5.3
  • Type changed from defect (bug) to enhancement
  • Version changed from trunk to 1.0

Hello @loyaltymanufaktur, welcome to Trac!

Thanks for the ticket and the patch. As noted in the docs, "implode() can, for historical reasons, accept its parameters in either order". I'm fine with changing it, though.

Related: [946], [948]

#2 @SergeyBiryukov
18 months ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 45491:

Posts, Post Types: Switch the order of arguments for implode() in the_meta() to match the function notation.

Props loyaltymanufaktur.
Fixes #47481.

Note: See TracTickets for help on using tickets.