Automattic/jetpack

Shortcode Embeds: add embed_oembed_html filter to all shortcodes

Aberta

#1.571 aberto em 30 de jan. de 2015

Ver no GitHubGuia do colaborador
 (8 comentários) (4 reações) (0 responsável)PHP (877 forks)user submission
EnhancementGood For CommunityHacktoberfest[Feature] Shortcodes / Embeds[Pri] Low[Type] Good First Bug

Métricas do repositório

Stars
 (1.808 estrelas)
Métricas de merge de PR
 (Mesclagem média 4d 5h) (645 fundiu PRs em 30d)

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!

Guia do colaborador

Direção de pesquisa
Examine o módulo Shortcode Embeds no Jetpack para encontrar onde a saída oEmbed é gerada e onde o filtro embed oembed html deve ser aplicado. Veja os manipuladores de shortcode para YouTube e outros provedores e garanta que o filtro seja chamado.
Pilha de tecnologia
php
Domain
backend
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
PHPWordPress plugin development
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60