Opened 13 years ago
Closed 13 years ago
#26811 closed defect (bug) (fixed)
Bundled Themes: Use `display` context for `get_bloginfo()` in `wp_title` filter
| Reported by: | michelwppi | Owned by: | nacin |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.9 |
| Component: | Bundled Theme | Version: | 3.8 |
| Severity: | normal | Keywords: | has-patch commit fixed-major |
| Cc: | Focuses: |
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
@
13 years ago
- Keywords has-patch commit added
- Milestone Awaiting Review → 3.9
- Summary twentythirteen : unperfect wp_title filter in functions.php → Twenty Thirteen: Use 'display' context for get_bloginfo() in wp_title filter
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Makes sense. Let's do the same for all the default themes that use
wp_titlefilter.