Skip to content

Pouvoir interdire explicitement certains providers

Glop a demandé de fusionner gh-2fba0a0d/7/unknown/refs/pull/7/head vers master

Bonjour,

Il est parfois nécessaire de pouvoir interdire certains providers oEmbed, tout en gardant l'option detecter_lien activée. (Par exemple lorsqu'on a un alias de nom de domaine www.monsite.tld → monsite.tld : oEmbed ne détecte pas www.monsite.tld comme un lien vers lui-même.)

Le premier commit de cette PR permet de considérer les providers dont l'endpoint a été mis à la chaîne vide '' comme étant explicitement interdits : même si detecter_lien est activée, oEmbed n'ira pas voir s'il s'agit d'un provider oEmbed.

Le principe est que oembed_verifier_provider() ne renvoie false que lorsque le provider n'est pas dans la liste des providers connus, mais renvoie array('endpoint' => '') lorsque le provider est interdit (y compris dans le cas d'un lien vers soi-même).

Cette modification rend superflu un test oembed_embarquer_lien(), supprimé par le commit suivant.

Enfin, le dernier commit uniformise le comportement du plugin en rajoutant le support de l'option detecter_lien au filtre |oembed (qui n'en tenait pas compte).

Merci !

++ Glop

Rapports de requête de fusion