WordPress.org
Get WordPress

Make WordPress Core

Opened 16 years ago

Closed 16 years ago

#13257 closed defect (bug) (duplicate)

Pass $post to post_type_link filter, instead of $id

Reported by: bigdawggi Owned by: nacin
Priority: normal Milestone:
Component: General Version: 3.0
Severity: normal Keywords: needs-patch
Cc: Focuses:

Description

To keep with the standard set by the filter 'post_link' inside the get_permalink() function, should the second parameter of the 'post_type_link' be the $post, instead of the $id?

attached is the proposed diff

Attachments (1)

post-type-link-filter.diff​ (516 bytes ) - added by bigdawggi 16 years ago.

Download all attachments as: .zip

Change History (4)

@bigdawggi
16 years ago

#1 @hakre
16 years ago

  • Keywords needs-patch added; filter post-type removed

I like the idea (makes sense) but patch is stale.

Related: #12526 (for mixed/optional post argument)

#2 @nacin
16 years ago

  • Owner set to nacin
  • Status new → reviewing

#3 @nacin
16 years ago

  • Milestone Awaiting Review
  • Resolution → duplicate
  • Status reviewing → closed

This was actually fixed via a different ticket, #14299, and was backported to 3.0.

Note: See TracTickets for help on using tickets.