#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
@
12 years ago
- Component changed from Themes to Bundled Theme
- Milestone changed from Awaiting Review to 3.6
#3
follow-up:
↓ 4
@
12 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
@
12 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