WordPress.org
Get WordPress

Make WordPress Core

Opened 15 years ago

Closed 14 years ago

#17013 closed enhancement (duplicate)

Make $key argument optional in get_*_meta()

Reported by: scribu's profile scribu Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: General Keywords: has-patch commit 2nd-opinion
Focuses: 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 15 years ago.
17013.2.diff​ (2.7 KB) - added by scribu 15 years ago.
Rename $key to $meta_key

Download all attachments as: .zip

Change History (8)

#1 @scribu
15 years ago

Related: #15483

#2 @scribu
15 years ago

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

@scribu
15 years ago

#3 @hakre
15 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.

@scribu
15 years ago

Rename $key to $meta_key

#4 @markjaquith
15 years ago

  • Keywords 2nd-opinion added

No objection here.

#5 @ryan
15 years ago

  • Milestone changed from 3.2 to Future Release

#6 @scribu
14 years ago

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