Make WordPress Core

Opened 14 years ago

Closed 12 years ago

#23724 closed enhancement (fixed)

Pass $post into 'protected_title_format' and 'private_title_format' filters

Reported by: johnjamesjacoby Owned by: SergeyBiryukov
Priority: low Milestone: 4.0
Component: Posts, Post Types Version: 3.5
Severity: minor Keywords: has-patch
Cc: Focuses: template

Description

Currently it's only possible to change these prefixes globally.

Being able to change these prefixes based on specific $post data would be helpful for narrowing down the scope, say to specific custom post types.

I noticed also that we don't pass $post into the 'the_title' filter, which could cut down on an additional get_post() call later if we pass it here. Obviously not essential, but wouldn't hurt.

Attachments (3)

23724.patch (909 bytes ) - added by johnjamesjacoby 14 years ago.
23724.2.patch (786 bytes ) - added by johnjamesjacoby 14 years ago.
23724.3.patch (1.5 KB ) - added by DrewAPicture 12 years ago.
refresh + hook docs

Download all attachments as: .zip

Change History (10)

#1 follow-up: @SergeyBiryukov
14 years ago

  • Version trunk3.5

I noticed also that we don't pass $post into the 'the_title' filter, which could cut down on an additional get_post() call later if we pass it here. Obviously not essential, but wouldn't hurt.

I've found 14 instances of applying the_title filter in current trunk. Changing it in one place would be inconsistent, so this should be done a separate ticket. Previous inconsistencies (passing only one parameter) were fixed in #13558 and #16688.

Passing $post to protected_title_format and private_title_format makes sense to me.

#2 in reply to: ↑ 1 @johnjamesjacoby
14 years ago

Replying to SergeyBiryukov:

I've found 14 instances of applying the_title filter in current trunk. Changing it in one place would be inconsistent, so this should be done a separate ticket. Previous inconsistencies (passing only one parameter) were fixed in #13558 and #16688.

I hadn't considered these. Agree adding anything to 'the_title' should be consistent. Will repatch with that bit. Thanks for the heads up there.

Last edited 14 years ago by johnjamesjacoby (previous) (diff)

#3 @nacin
13 years ago

  • Component TemplatePosts, Post Types
  • Focuses template added

#4 @obenland
12 years ago

  • Keywords needs-refresh added

#5 @obenland
12 years ago

Needs refresh with filter documentation update.

@DrewAPicture
12 years ago

refresh + hook docs

#6 @DrewAPicture
12 years ago

  • Keywords needs-refresh removed
  • Milestone Awaiting Review4.0

23724.3.patch adds the hook doc updates.

#7 @SergeyBiryukov
12 years ago

  • Owner set to SergeyBiryukov
  • Resolutionfixed
  • Status newclosed

In 28571:

Pass $post to 'protected_title_format' and 'private_title_format' filters.

props johnjamesjacoby, DrewAPicture.
fixes #23724.

Note: See TracTickets for help on using tickets.