Opened 11 years ago
Closed 11 years ago
#26811 closed defect (bug) (fixed)
Bundled Themes: Use `display` context for `get_bloginfo()` in `wp_title` filter
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 3.9 | Priority: | normal |
Severity: | normal | Version: | 3.8 |
Component: | Bundled Theme | Keywords: | has-patch commit fixed-major |
Focuses: | Cc: |
Description
as developer of xili-language multilingual plugin, after a support enquiry, I discover that wp_filter (twentythirteen_wp_title function) inside functions.php of 2013 show a issue:
to display blogname you forgot to add second parameter 'display' in
get_bloginfo ('name');
as done in wp_title for description ( see #25808 )...
get_bloginfo ('name', 'display');
By using this parameter, get_bloginfo is filterable (apply_filters at end of function get_bloginfo ) as wp_title used in header...
I think it will be coherent to be done in parent 2013 and not with a remove filter in child theme and add a new fixed filter...
Happy new year
M.
Attachments (1)
Change History (6)
#1
@
11 years ago
- Keywords has-patch commit added
- Milestone changed from Awaiting Review to 3.9
- Summary changed from twentythirteen : unperfect wp_title filter in functions.php to Twenty Thirteen: Use 'display' context for get_bloginfo() in wp_title filter
#2
@
11 years ago
- Milestone changed from 3.9 to 3.8.2
- Summary changed from Twenty Thirteen: Use 'display' context for get_bloginfo() in wp_title filter to Bundled Themes: Use `display` context for `get_bloginfo()` in `wp_title` filter
#3
@
11 years ago
- Owner set to nacin
- Resolution set to fixed
- Status changed from new to closed
In 27091:
Note: See
TracTickets for help on using
tickets.
Makes sense. Let's do the same for all the default themes that use
wp_title
filter.