WordPress.org

Make WordPress Core

Opened 2 years ago

Closed 2 years ago

#23925 closed defect (bug) (wontfix)

get_post_meta does not check for revision

Reported by: jfarthing84 Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.5.1
Component: General Keywords: has-patch
Focuses: Cc:

Description

Every other *_post_meta function does.

Attachments (1)

23925.patch (486 bytes) - added by jfarthing84 2 years ago.

Download all attachments as: .zip

Change History (5)

@jfarthing842 years ago

comment:1 @kovshenin2 years ago

  • Keywords reporter-feedback added

What is the reasoning behind the proposed change? This will break #20564, #23539 and possibly others.

comment:2 in reply to: ↑ description @jfarthing842 years ago

Replying to kovshenin:

Every other *_post_meta function does.

comment:3 @jfarthing842 years ago

  • Keywords reporter-feedback removed

comment:4 @nacin2 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed

I'm going to wontfix this. The fact that the meta functions check for revisions was a terrible decision at the time, and was done to cover up misuse of the save_post hook. Changing this could break even more things. Leaving it as is. Eventually we'll fork all of these functions and hopefully do away with these entirely.

Note: See TracTickets for help on using tickets.