Opened 6 years ago
Closed 7 months ago
#4267 closed defect (bug) (fixed)
wp_list_pages using a 'the_title' on all entries
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.0 |
| Component: | Template | Version: | 2.2 |
| Severity: | normal | Keywords: | |
| Cc: | david@…, pembo13 |
Description
I have a plugin that – when a specific ‘page’ is displayed (checked by is_page and ID) a filter changes the page title using a standard add_filter on ‘the_title’.
If wp_list_pages() is being used on the sidebar, ALL page titles are getting altered in the list output.
I have just checked and this does not seem to be the case using 2.1.3
As an aside, optionally passing the post ID into 'the-title' would be useful.
Change History (20)
- Keywords 2nd-opinion added
- Milestone changed from 2.2.1 to 2.4
- Owner ryan deleted
- Milestone changed from 2.4 to 2.2.1
This is a sideeffect of the changes made to wp_list_pages to try and support what is wanted in #4004.
It shows why reusing filters all over the code for similar but not identical things in not a good idea!
filter names should relate to where there are called from not the item of data on which the operate.
comment:5
foolswisdom — 6 years ago
- Milestone changed from 2.2.2 to 2.2.3
- Summary changed from wp_list_pages using a 'the_title' on all entries to wp_list_pages using a 'the_title' on all entries my-2cents
+1 - move the filter out of wp_list_pages. good spoken westi, strong point!
comment:8
foolswisdom — 6 years ago
- Milestone changed from 2.2.3 to 2.3
comment:10
mattyrob — 6 years ago
+1 more for moving the filter out of wp_list_pages or for providing a more elegant solution overall :-)
+1 over here as well -- the_title (or the_content, or...) are for use within the loop.
a widget_title filter would make a lot more sense.
- Resolution set to worksforme
- Status changed from new to closed
seems invalid in wp 2.8
- Milestone changed from 2.9 to 2.8
comment:14
hakre — 4 years ago
- Component changed from Administration to Template
- Keywords reporter-feedback added; wp_list_pages filter 2nd-opinion removed
- Summary changed from wp_list_pages using a 'the_title' on all entries my-2cents to wp_list_pages using a 'the_title' on all entries
have not tested wether this is still the case with 2.8
no direct invoke of the filter 'the_title' in function wp_list_pages() in post-template.php.
it would be good to have a test-plugin. does anybody has an idea which plugin the original reporter has used?
additionally i do not know wether the filter 'the_title' is even correct in there or not. until no more feedback provided i suggest to leave this closed.
corrected the wrong title /meshame.
comment:15
pembo13 — 7 months ago
- Resolution worksforme deleted
- Status changed from closed to reopened
comment:16
pembo13 — 7 months ago
This is still an issue in 3.4.2. The "the_title" filter is used, and the filter has no context of when it's called. A simple change by a filter on "the_title" could have a wide scope.
comment:17
pembo13 — 7 months ago
- Cc pembo13 added
comment:18
SergeyBiryukov — 7 months ago
- Keywords reporter-feedback removed
- Milestone changed from 2.8 to Awaiting Review
comment:19
miqrogroove — 7 months ago
My two cents, as I am chatting with pembo13: Adding an unconditional filter on the_title in version 3.4.2 with the Twenty Eleven theme will cause titles to change in the admin's list of posts, in the front end "Loop", in the menu bar, and in the sidebar list of pages. It is unclear how a plugin author could ever cleanly distinguish between the title that appears in a menu or widget vs. in the standard Loop.
comment:20
SergeyBiryukov — 7 months ago
- Milestone changed from Awaiting Review to 3.0
- Resolution set to fixed
- Status changed from reopened to closed

I'm of the opinion that this is at the fault of the plugin.