Opened 2 years ago

Closed 14 months ago

#17013 closed enhancement (duplicate)

Make $key argument optional in get_*_meta()

Reported by: scribu Owned by:
Priority: normal Milestone:
Component: General Version:
Severity: normal Keywords: has-patch commit 2nd-opinion
Cc:

Description

Altough the $key argument is optional in get_metadata(), it's not optional in the wrapper functions for some reason.

This should be fixed.

Attachments (2)

17013.diff (2.7 KB) - added by scribu 2 years ago.
17013.2.diff (2.7 KB) - added by scribu 2 years ago.
Rename $key to $meta_key

Download all attachments as: .zip

Change History (8)

Related: #15483

  • Keywords has-patch commit added; needs-patch removed

scribu2 years ago

Take care with renaming @param string $meta_key while not renaming $key in the functions' signatures. Looks like a copy&paste glitch.

$meta_key says more btw, so I wouldn't mind if you even rename the parameter names in the signature.

scribu2 years ago

Rename $key to $meta_key

  • Keywords 2nd-opinion added

No objection here.

comment:5   ryan2 years ago

  • Milestone changed from 3.2 to Future Release
  • Milestone Future Release deleted
  • Resolution set to duplicate
  • Status changed from new to closed

Those who can't search through trac tickets are bound to re-create them: #19882

Note: See TracTickets for help on using tickets.