pierre-lannoy/wp-oembed-manager

A WordPress plugin to manage oEmbed capabilities of your website and to take a new step in the GDPR compliance of your embedded content.

wp-oembed-manager's Issues

GDPR integration: Enabling exception rules with emtpy 'item to check' causes 500

Describe the bug
If using together with the 'GDPR' plugin, I can enable an 'Exception Rule' for 'Consent Management' or 'Cookie Management', but I can leave the 'Item to check' empty... which in turn causes an issue, because it would make GDPR call a function with null argument.

To Reproduce
Steps to reproduce the behavior:

1. Create a page with an embedded YouTube video using the OOTB embeding feature in WP.
2. Install 'oEmbed Manager'
3. Install 'GDPR'
4. Go to the admin, and enable 'Cookie managemen' under 'Exception Rules', but leave the 'consent ID' field blank.
5. Try to open a page, and check, whether the YouTube embed is blocked.
6. Observer the page is broken; check the server logs, and see a 500 error, complaining about the is_allowed_cookie() being called without arguments instead of at least 1.

Expected behavior
The page opens, and fails gracefully. But actually, and more preferably the admin interface not letting me save the config, that doesn't make sense.

Environment (please complete the following information):

• PHP version: 7.3.27
• WordPress version and flavour 5.7.2
• Plugin versions: GDPR (by Trew Knowledge) 2.1.2 and oEmbed Manager 2.4.0
• Other info: Running on Debian 10.10 (buster)