#23511 closed defect (bug) (fixed)
Twenty Thirteen: twentythirteen_aside_date() breaks oEmbed in some cases
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 3.6 | Priority: | normal |
| Severity: | minor | Version: | 3.6 |
| Component: | Bundled Theme | Keywords: | has-patch commit |
| Focuses: | Cc: |
Description
Trying out Twenty Thirteen, I found some oEmbed elements on 'aside' posts were displayed as plain text (specifically those where the URL was at the end)
Digging around, I found twentythirteen_aside_date adds the date just right to the content (without a new line); and the function was added before oEmbed processing. The quickest solution I found was add the new line before executing twentythirteen_entry_date
Attachments (2)
Change History (9)
#1
@
13 years ago
- Component changed from Themes to Bundled Theme
- Milestone changed from Awaiting Review to 3.6
#3
follow-up:
↓ 4
@
13 years ago
PotterSys, thanks for the ticket and patch.
Testing note: we should try to change the priority of the filter instead of adding the \n. The newline breaks the display of the datestamp for asides, and isn't really needed.
#4
in reply to:
↑ 3
@
13 years ago
Testing note: we should try to change the priority of the filter instead of adding the
\n. The newline breaks the display of the datestamp for asides, and isn't really needed.
As the default value for priority in add_filter() is 10, I thought changing that value could break something else. However, the patch submitted by obenland fixes the bug by changing the priority
Quick fix to oEmbed not being shown on asides