Fix HTML sanitizer to skip <!-- node type X --> in output (#1490583)

Aleksander Machniak

2015-11-05 e7d1a80a800f6f08c0a683d2be04b0db2a1f6523

 program/lib/Roundcube/rcube_plugin_api.php

@@ -182,7 +182,7 @@
        }

        // plugin already loaded
        if ($this->plugins[$plugin_name] || class_exists($plugin_name, false)) {
        if ($this->plugins[$plugin_name]) {
            return true;
        }

@@ -190,7 +190,9 @@
            . DIRECTORY_SEPARATOR . $plugin_name . '.php';

        if (file_exists($fn)) {
            include $fn;
            if (!class_exists($plugin_name, false)) {
                include $fn;
            }

            // instantiate class if exists
            if (class_exists($plugin_name, false)) {
@@ -231,7 +233,7 @@

    /**
     * Get information about a specific plugin.
     * This is either provided my a plugin's info() method or extracted from a package.xml or a composer.json file
     * This is either provided by a plugin's info() method or extracted from a package.xml or a composer.json file
     *
     * @param string Plugin name
     * @return array Meta information about a plugin or False if plugin was not found
@@ -277,13 +279,14 @@
        include($fn);

      if (class_exists($plugin_name))
        $info = $plugin_name::info();
        $info = call_user_func(array($plugin_name, 'info'));

      // fall back to composer.json file
      if (!$info) {
        $composer = INSTALL_PATH . "/plugins/$plugin_name/composer.json";
        if (file_exists($composer) && ($json = @json_decode(file_get_contents($composer), true))) {
          list($info['vendor'], $info['name']) = explode('/', $json['name']);
          $info['version'] = $json['version'];
          $info['license'] = $json['license'];
          if ($license_uri = $license_uris[$info['license']])
            $info['license_uri'] = $license_uri;