codysnider / tt-rss

plugins/no_iframes/init.php

<?php
class No_Iframes extends Plugin {
    private $host;

    public function about() {
        return array(1.0,
            "Remove embedded iframes (unless whitelisted)",
            "fox");
    }

    public function init($host) {
        $this->host = $host;

        $host->add_hook($host::HOOK_SANITIZE, $this);
    }

    /**
     * @SuppressWarnings(PHPMD.UnusedFormalParameter)
     */
    public function hook_sanitize($doc, $site_url, $allowed_elements, $disallowed_attributes) {

The parameter $site_url is not used and could be removed.

        $xpath = new DOMXpath($doc);
        $entries = $xpath->query('//iframe');

        foreach ($entries as $entry) {
            if (!iframe_whitelisted($entry)) {
                            $entry->parentNode->removeChild($entry);
            }
        }

        return array($doc, $allowed_elements, $disallowed_attributes);
    }

    public function api_version() {
        return 2;
    }

}