Opened 12 years ago
Closed 10 years ago
#23724 closed enhancement (fixed)
Pass $post into 'protected_title_format' and 'private_title_format' filters
Reported by: | johnjamesjacoby | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 4.0 | Priority: | low |
Severity: | minor | Version: | 3.5 |
Component: | Posts, Post Types | Keywords: | has-patch |
Focuses: | template | Cc: |
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)
Change History (10)
#2
in reply to:
↑ 1
@
12 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. Very much agree this bit needs a rethink. Will repatch without that bit.
#6
@
10 years ago
- Keywords needs-refresh removed
- Milestone changed from Awaiting Review to 4.0
23724.3.patch adds the hook doc updates.
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
toprotected_title_format
andprivate_title_format
makes sense to me.