Automattic / Automattic/jetpack

Shortcode Embeds: add embed_oembed_html filter to all shortcodes

Open
#1,571 8 comments 4 reactions 0 assignees View on GitHub
[Feature] Shortcodes / Embeds [Pri] Low [Type] Good First Bug Enhancement Good For Community Hacktoberfest
Dominant language
PHP
Stars
1.8k
Forks
898
Avg merge
1d 18h
Merged PRs (30d)
774

Description

Hi,

I've noticed that the filter embed_oembed_html can not be counted on because as soon as Jetpack's Shortcode Embeds is activated, this filter is no longer there even if the new shortcode [youtube](or similar) is not used.

Please either add this filter like it was originally implemented in WP core -OR- please do not override the oembed that occurs when simply pasting a Youtube or other media host URL and limit yourself to the [youtube] and similar added shortcodes.

Hope that explanation was not too confusing ;)

Thanks!

Contributor guide

Open the contributing guide

Research direction

Start by reproducing the behavior with Jetpack's Shortcode Embeds activated, comparing a pasted YouTube or other media URL with the corresponding shortcode. Inspect the Shortcode Embeds handling of embed_oembed_html; done means the filter remains available as in WordPress core or ordinary pasted URLs retain core oEmbed behavior while shortcodes still work.

Written by the indexing model from the issue text.

Assessment

Tech stack
php, wordpress
Domain
content
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.