TinyMCE - Code Metrics - Inspection of "eliminate notice "Only variables should be assigne..." - XOOPS/XoopsCore25 - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#133)

by Goffy

created 2016-06-23 14:17 UTC

TinyMCE C

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	358
Duplicated Lines	12.57 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
wmc	56
c	1
b	0
f	0
lcom	1
cbo	1
dl	45
loc	358
rs	6.5957

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
F	init()	45	133	29
A	loadPlugins()	0	18	4
B	get_xoopsPlugins()	0	14	5
D	render()	0	99	9
A	instance()	0	11	2
A	setConfig()	0	6	2
B	loadCss()	0	22	4

How to fix Duplicated Code Complexity

Duplicated Code

Duplicate code is one of the most pungent code smells. A rule that is often used is to re-structure code once it is duplicated in three or more places.

Common duplication problems, and corresponding solutions are:

If you have the same expression in different places: Extract expression to a method
If you have the same method in different sub-classes: Extract method, and pull up field to the parent class
If you have the same code in unrelated classes: Consider extracting the code to a new class, and injecting that class

Complex Class

Tip: Before tackling complexity, make sure that you eliminate any duplication first. This often can reduce the size of classes significantly.

Complex classes like TinyMCE often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes. You can also have a look at the cohesion graph to spot any un-connected, or weakly-connected components.

Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.

While breaking up the class, it is a good idea to analyze how other classes use TinyMCE, and based on these observations, apply Extract Interface, too.

<?php
/*
 You may not change or alter any portion of this comment or credits
 of supporting developers from this source code or any supporting source code
 which is considered copyrighted (c) material of the original comment or credit authors.

 This program is distributed in the hope that it will be useful,
 but WITHOUT ANY WARRANTY; without even the implied warranty of
 MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
*/

/**
 *  TinyMCE adapter for XOOPS
 *
 * @copyright       (c) 2000-2016 XOOPS Project (www.xoops.org)
 * @license             GNU GPL 2 or later (http://www.gnu.org/licenses/gpl-2.0.html)
 * @package             class
 * @subpackage          editor
 * @since               2.3.0
 * @author              Taiwen Jiang <[email protected]>
 * @author              Lucio Rota <[email protected]>
 * @author              Laurent JEN <[email protected]>
 */
class TinyMCE
{
    public        $rootpath;
    public        $config                = array();
    public        $setting               = array();
    public static $LastOfElementsTinymce = '';
    public static $ListOfElementsTinymce = array();

    // PHP 5 Constructor
    /**
     * @param $config
     */
    public function __construct($config)
    {
        $this->setConfig($config);
        $this->rootpath                = $this->config['rootpath'] . '/tinymce/jscripts/tiny_mce';
        $this->xoopsPlugins            = $this->get_xoopsPlugins();
class MyClass { }

$x = new MyClass();
$x->foo = true;
        self::$LastOfElementsTinymce   = $this->config['elements'];
        self::$ListOfElementsTinymce[] = self::$LastOfElementsTinymce;
    }

    /**
     * @param $config
     *
     * @return TinyMCE
     */
    public function &instance($config)
    {
        static $instance;
        if (!isset($instance)) {
            $instance = new TinyMCE($config);
        } else {
            $instance->setConfig($config);
        }

        return $instance;
    }

    /**
     * @param $config
     */
    public function setConfig($config)
    {
        foreach ($config as $key => $val) {
            $this->config[$key] = $val;
        }
    }

    /**
     * @return bool
     */
    public function init()
    {
        // list of configured options
        $configured = array();

        // Load default settings
        if (!($this->setting = @include($GLOBALS['xoops']->path('var/configs/tinymce.php')))) {
            $this->setting = include __DIR__ . '/settings.php';
        }

        // get editor language (from ...)
        if (isset($this->config['language']) && is_readable(XOOPS_ROOT_PATH . $this->rootpath . '/langs/' . $this->config['language'] . '.js')) {
            $this->setting['language'] = $this->config['language'];
            $configured[]              = 'language';
        }

        $this->setting['content_css'] = implode(',', $this->loadCss());
        $configured[]                 = 'content_css';

        if (!empty($this->config['theme']) && is_dir(XOOPS_ROOT_PATH . $this->rootpath . '/themes/' . $this->config['theme'])) {
            $this->setting['theme'] = $this->config['theme'];
            $configured[]           = 'theme';
        }

        if (!empty($this->config['mode'])) {
            $this->setting['mode'] = $this->config['mode'];
            $configured[]          = 'mode';
        }

        // load all plugins except the plugins in setting["exclude_plugins"]
        $this->setting['plugins'] = implode(',', $this->loadPlugins());
        $configured[]             = 'plugins';

        if ($this->setting['theme'] !== 'simple') {
            if (empty($this->config['buttons'])) {
                $this->config['buttons'][] = array(
                    'before' => '',
                    'add' => '');
                $this->config['buttons'][] = array(
                    'before' => '',
                    'add' => '');
                $this->config['buttons'][] = array(
                    'before' => '',
                    'add' => '');
            }
            $i = 0;
            foreach ($this->config['buttons'] as $button) {
                ++$i;
                if (isset($button['before'])) {
                    $this->setting['theme_' . $this->setting['theme'] . "_buttons{$i}_add_before"] = $button['before'];
                }
                if (isset($button['add'])) {
                    $this->setting['theme_' . $this->setting['theme'] . "_buttons{$i}_add"] = $button['add'];
                }
                if (isset($button[''])) {
                    $this->setting['theme_' . $this->setting['theme'] . "_buttons{$i}"] = $button[''];
                }
            }
            $configured[] = 'buttons';

            if (isset($this->config['toolbar_location'])) {
                $this->setting['theme_' . $this->setting['theme'] . '_toolbar_location'] = $this->config['toolbar_location'];
                $configured[]                                                            = 'toolbar_location';
            } else {
                $this->setting['theme_' . $this->setting['theme'] . '_toolbar_location'] = 'top';
            }

            if (isset($this->config['toolbar_align'])) {
                $this->setting['theme_' . $this->setting['theme'] . '_toolbar_align'] = $this->config['toolbar_align'];
                $configured[]                                                         = 'toolbar_align';
            } else {
                $this->setting['theme_' . $this->setting['theme'] . '_toolbar_align'] = 'left';
            }

            if (isset($this->config['statusbar_location'])) {
                $this->setting['theme_' . $this->setting['theme'] . '_statusbar_location'] = $this->config['statusbar_location'];
                $configured[]                                                              = 'statusbar_location';
            }

            if (isset($this->config['path_location'])) {
                $this->setting['theme_' . $this->setting['theme'] . '_path_location'] = $this->config['path_location'];
                $configured[]                                                         = 'path_location';
            }

            if (isset($this->config['resize_horizontal'])) {
                $this->setting['theme_' . $this->setting['theme'] . '_resize_horizontal'] = $this->config['resize_horizontal'];
                $configured[]                                                             = 'resize_horizontal';
            }

            if (isset($this->config['resizing'])) {
                $this->setting['theme_' . $this->setting['theme'] . '_resizing'] = $this->config['resizing'];
                $configured[]                                                    = 'resizing';
            }

            if (!empty($this->config['fonts'])) {
                $this->setting['theme_' . $this->setting['theme'] . '_fonts'] = $this->config['fonts'];
                $configured[]                                                 = 'fonts';
            }

            for ($i = 1; $i <= 4; ++$i) {
                $buttons = array();
                if (isset($this->setting['theme_' . $this->setting['theme'] . "_buttons{$i}"])) {
                    $checklist = explode(',', $this->setting['theme_' . $this->setting['theme'] . "_buttons{$i}"]);
                    foreach ($checklist as $plugin) {
                        if (strpos(strtolower($plugin), 'xoops') != false) {

                            if (in_array($plugin, $this->xoopsPlugins)) {
                                $buttons[] = $plugin;
                            }
                        } else {
                            $buttons[] = $plugin;
                        }
                    }
                    $this->setting['theme_' . $this->setting['theme'] . "_buttons{$i}"] = implode(',', $buttons);
                }
            }
        }

        $configured = array_unique($configured);
        foreach ($this->config as $key => $val) {
            if (isset($this->setting[$key]) || in_array($key, $configured)) {
                continue;
            }
            $this->setting[$key] = $val;
        }

        if (!is_dir(XOOPS_ROOT_PATH . $this->rootpath . '/themes/' . $this->setting['theme'] . '/docs/' . $this->setting['language'] . '/')) {
            $this->setting['docs_language'] = 'en';
        }

        unset($this->config, $configured);

        return true;
    }

    // load all plugins execpt the plugins in setting["exclude_plugins"]
    /**
     * @return array
     */
    public function loadPlugins()
    {
        $plugins      = array();
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        $plugins_list = XoopsLists::getDirListAsArray(XOOPS_ROOT_PATH . $this->rootpath . '/plugins');
        if (empty($this->setting['plugins'])) {
            $plugins = $plugins_list;
        } else {
            $plugins = array_intersect(explode(',', $this->setting['plugins']), $plugins_list);
        }
        if (!empty($this->setting['exclude_plugins'])) {
            $plugins = array_diff($plugins, explode(',', $this->setting['exclude_plugins']));
        }
        if (!empty($this->config['plugins'])) {
            $plugins = array_merge($plugins, $this->config['plugins']);
        }

        return $plugins;
    }

    // return all xoops plugins
    /**
     * @return array
     */
    public function get_xoopsPlugins()
    {
        $xoopsPlugins = array();
        $allplugins   = XoopsLists::getDirListAsArray(XOOPS_ROOT_PATH . $this->rootpath . '/plugins');
        foreach ($allplugins as $plugin) {
            if (strpos(strtolower($plugin), 'xoops') != false && file_exists(XOOPS_ROOT_PATH . $this->config['rootpath'] . "/include/$plugin.php")) {

                if ($right = @include XOOPS_ROOT_PATH . $this->config['rootpath'] . "/include/$plugin.php") {
                    $xoopsPlugins[$plugin] = $plugin;
                }
            }
        }

        return $xoopsPlugins;
    }

    /**
     * @param string $css_file
     *
     * @return array
     */
    public function loadCss($css_file = 'style.css')
    {
        static $css_url, $css_path;

        if (!isset($css_url)) {
            $css_url  = dirname(xoops_getcss($GLOBALS['xoopsConfig']['theme_set']));
            $css_path = str_replace(XOOPS_THEME_URL, XOOPS_THEME_PATH, $css_url);
        }

        $css         = array();
        $css[]       = $css_url . '/' . $css_file;
        $css_content = file_get_contents($css_path . '/' . $css_file);

        // get all import css files
        if (preg_match_all("~\@import url\((.*\.css)\);~sUi", $css_content, $matches, PREG_PATTERN_ORDER)) {
            foreach ($matches[1] as $key => $css_import) {
                $css = array_merge($css, $this->loadCss($css_import));
            }
        }

        return $css;
    }

    /**
     * @return string
     */
    public function render()
    {
        static $isTinyMceJsLoaded = false;

        $this->init();
        if (isset($this->setting['elements']) && self::$LastOfElementsTinymce != $this->setting['elements']) {
            $ret = "\n<!-- 'tiny_mce.js' SCRIPT NOT YET " . $this->setting['elements'] . " -->\n"; //debug

            return $ret;
        } else {
            $this->setting['elements'] = implode(',', self::$ListOfElementsTinymce);
        }
        if (!empty($this->setting['callback'])) {
            $callback = $this->setting['callback'];
            unset($this->setting['callback']);
        } else {
            $callback = '';
        }
        if (!empty($this->setting['file_browser_callback'])) {
            $fbc_name = XOOPS_ROOT_PATH . '/class/xoopseditor/tinymce/include/' . $this->setting['file_browser_callback'] . '.js';
            //suis passé la /lesrevespossibles/x244/class/xoopseditor/tinymce/tinymce/jscripts/include/openFinder.js
            $file_browser_callback = "MyXoopsUrl ='" . XOOPS_URL . "';\n";
            $file_browser_callback .= file_get_contents($fbc_name);
            $file_browser_callback .= "\n//suis passé la " . $fbc_name;
            //unset($this->setting["file_browser_callback"]);
        } else {
            $file_browser_callback = '//suis absent';
        }

        // create returned string - start
        $ret = "\n";

        /*
        // IE BUG - start
        // more info here:
        // http://www.456bereastreet.com/archive/200802/beware_of_id_and_name_attribute_mixups_when_using_getelementbyid_in_internet_explorer/
        // possible solution here:
        // http://www.sixteensmallstones.org/ie-javascript-bugs-overriding-internet-explorers-documentgetelementbyid-to-be-w3c-compliant-exposes-an-additional-bug-in-getattributes
                $ret =<<<EOF
        <script language='javascript' type='text/javascript'>
            if (/msie/i.test (navigator.userAgent)) { //only override IE
                document.nativeGetElementById = document.getElementById;
                document.getElementById = function(id) {
                    var elem = document.nativeGetElementById(id);
                    if (elem) {
                        //make sure that it is a valid match on id
                        if (elem.attributes['id'].value == id) {
                            return elem;
                        } else {
                            //otherwise find the correct element
                            for (var i=1;i<document.all[id].length;i++) {
                                if (document.all[id][i].attributes['id'].value == id) {
                                    return document.all[id][i];
                                }
                            }
                        }
                    }

                    return null;
                };
            }
        </script>
        \n
        EOF;
        // IE BUG - end
        */

        $ret .= "<!-- Start TinyMce Rendering -->\n"; //debug
        if ($isTinyMceJsLoaded) {
            $ret .= "<!-- 'tiny_mce.js' SCRIPT IS ALREADY LOADED -->\n"; //debug
        } else {
            $ret .= "<script type='text/javascript' src='" . XOOPS_URL . $this->rootpath . "/tiny_mce.js'></script>\n";
            $isTinyMceJsLoaded = true;
        }
        $ret .= "<script type='text/javascript'>\n";
        $ret .= "tinyMCE.init({\n";
        // set options - start
        foreach ($this->setting as $key => $val) {
            $ret .= $key . ':';
            if ($val === true) {
                $ret .= 'true,';
            } elseif ($val === false) {
                $ret .= 'false,';
            } else {
                $ret .= "'{$val}',";
            }
            $ret .= "\n";
        }
        // set options - end
        $ret .= "tinymceload: true\n";
        $ret .= "});\n";
        $ret .= $callback . "\n";
        $ret .= $file_browser_callback . "\n";
        //$ret .= "function toggleEditor(id) {tinyMCE.execCommand('mceToggleEditor',false, id);}\n";
        $ret .= "</script>\n";
        $ret .= "<!-- End TinyMce Rendering -->\n";//debug
        // create returned string - end
        return $ret;
    }
}


1		<?php
2		/*
3		You may not change or alter any portion of this comment or credits
4		of supporting developers from this source code or any supporting source code
5		which is considered copyrighted (c) material of the original comment or credit authors.
6
7		This program is distributed in the hope that it will be useful,
8		but WITHOUT ANY WARRANTY; without even the implied warranty of
9		MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
10		*/
11
12		/**
13		* TinyMCE adapter for XOOPS
14		*
15		* @copyright (c) 2000-2016 XOOPS Project (www.xoops.org)
16		* @license GNU GPL 2 or later (http://www.gnu.org/licenses/gpl-2.0.html)
17		* @package class
18		* @subpackage editor
19		* @since 2.3.0
20		* @author Taiwen Jiang <[email protected]>
21		* @author Lucio Rota <[email protected]>
22		* @author Laurent JEN <[email protected]>
23		*/
24		class TinyMCE
25		{
26		public $rootpath;
27		public $config = array();
28		public $setting = array();
29		public static $LastOfElementsTinymce = '';
30		public static $ListOfElementsTinymce = array();
31
32		// PHP 5 Constructor
33		/**
34		* @param $config
35		*/
36		public function __construct($config)
37		{
38		$this->setConfig($config);
39		$this->rootpath = $this->config['rootpath'] . '/tinymce/jscripts/tiny_mce';
40		$this->xoopsPlugins = $this->get_xoopsPlugins();
		0 ignored issues – show Bug introduced 2016-03-07 05:16 UTC by Report Bug Copy Issue Report The property `xoopsPlugins` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
41		self::$LastOfElementsTinymce = $this->config['elements'];
42		self::$ListOfElementsTinymce[] = self::$LastOfElementsTinymce;
43		}
44
45		/**
46		* @param $config
47		*
48		* @return TinyMCE
49		*/
50		public function &instance($config)
51		{
52		static $instance;
53		if (!isset($instance)) {
54		$instance = new TinyMCE($config);
55		} else {
56		$instance->setConfig($config);
57		}
58
59		return $instance;
60		}
61
62		/**
63		* @param $config
64		*/
65		public function setConfig($config)
66		{
67		foreach ($config as $key => $val) {
68		$this->config[$key] = $val;
69		}
70		}
71
72		/**
73		* @return bool
74		*/
75		public function init()
76		{
77		// list of configured options
78		$configured = array();
79
80		// Load default settings
81		if (!($this->setting = @include($GLOBALS['xoops']->path('var/configs/tinymce.php')))) {
82		$this->setting = include __DIR__ . '/settings.php';
83		}
84
85		// get editor language (from ...)
86	View Code Duplication	if (isset($this->config['language']) && is_readable(XOOPS_ROOT_PATH . $this->rootpath . '/langs/' . $this->config['language'] . '.js')) {
87		$this->setting['language'] = $this->config['language'];
88		$configured[] = 'language';
89		}
90
91		$this->setting['content_css'] = implode(',', $this->loadCss());
92		$configured[] = 'content_css';
93
94	View Code Duplication	if (!empty($this->config['theme']) && is_dir(XOOPS_ROOT_PATH . $this->rootpath . '/themes/' . $this->config['theme'])) {
95		$this->setting['theme'] = $this->config['theme'];
96		$configured[] = 'theme';
97		}
98
99		if (!empty($this->config['mode'])) {
100		$this->setting['mode'] = $this->config['mode'];
101		$configured[] = 'mode';
102		}
103
104		// load all plugins except the plugins in setting["exclude_plugins"]
105		$this->setting['plugins'] = implode(',', $this->loadPlugins());
106		$configured[] = 'plugins';
107
108		if ($this->setting['theme'] !== 'simple') {
109		if (empty($this->config['buttons'])) {
110		$this->config['buttons'][] = array(
111		'before' => '',
112		'add' => '');
113		$this->config['buttons'][] = array(
114		'before' => '',
115		'add' => '');
116		$this->config['buttons'][] = array(
117		'before' => '',
118		'add' => '');
119		}
120		$i = 0;
121		foreach ($this->config['buttons'] as $button) {
122		++$i;
123	View Code Duplication	if (isset($button['before'])) {
124		$this->setting['theme_' . $this->setting['theme'] . "_buttons{$i}_add_before"] = $button['before'];
125		}
126	View Code Duplication	if (isset($button['add'])) {
127		$this->setting['theme_' . $this->setting['theme'] . "_buttons{$i}_add"] = $button['add'];
128		}
129	View Code Duplication	if (isset($button[''])) {
130		$this->setting['theme_' . $this->setting['theme'] . "_buttons{$i}"] = $button[''];
131		}
132		}
133		$configured[] = 'buttons';
134
135	View Code Duplication	if (isset($this->config['toolbar_location'])) {
136		$this->setting['theme_' . $this->setting['theme'] . '_toolbar_location'] = $this->config['toolbar_location'];
137		$configured[] = 'toolbar_location';
138		} else {
139		$this->setting['theme_' . $this->setting['theme'] . '_toolbar_location'] = 'top';
140		}
141
142	View Code Duplication	if (isset($this->config['toolbar_align'])) {
143		$this->setting['theme_' . $this->setting['theme'] . '_toolbar_align'] = $this->config['toolbar_align'];
144		$configured[] = 'toolbar_align';
145		} else {
146		$this->setting['theme_' . $this->setting['theme'] . '_toolbar_align'] = 'left';
147		}
148
149	View Code Duplication	if (isset($this->config['statusbar_location'])) {
150		$this->setting['theme_' . $this->setting['theme'] . '_statusbar_location'] = $this->config['statusbar_location'];
151		$configured[] = 'statusbar_location';
152		}
153
154	View Code Duplication	if (isset($this->config['path_location'])) {
155		$this->setting['theme_' . $this->setting['theme'] . '_path_location'] = $this->config['path_location'];
156		$configured[] = 'path_location';
157		}
158
159	View Code Duplication	if (isset($this->config['resize_horizontal'])) {
160		$this->setting['theme_' . $this->setting['theme'] . '_resize_horizontal'] = $this->config['resize_horizontal'];
161		$configured[] = 'resize_horizontal';
162		}
163
164	View Code Duplication	if (isset($this->config['resizing'])) {
165		$this->setting['theme_' . $this->setting['theme'] . '_resizing'] = $this->config['resizing'];
166		$configured[] = 'resizing';
167		}
168
169		if (!empty($this->config['fonts'])) {
170		$this->setting['theme_' . $this->setting['theme'] . '_fonts'] = $this->config['fonts'];
171		$configured[] = 'fonts';
172		}
173
174		for ($i = 1; $i <= 4; ++$i) {
175		$buttons = array();
176		if (isset($this->setting['theme_' . $this->setting['theme'] . "_buttons{$i}"])) {
177		$checklist = explode(',', $this->setting['theme_' . $this->setting['theme'] . "_buttons{$i}"]);
178		foreach ($checklist as $plugin) {
179		if (strpos(strtolower($plugin), 'xoops') != false) {
		0 ignored issues – show Bug Best Practice introduced 2016-03-07 05:16 UTC by Report Bug Copy Issue Report It seems like you are loosely comparing `strpos(strtolower($plugin), 'xoops')` of type `integer` to the boolean `false`. If you are specifically checking for non-zero, consider using something more explicit like `> 0` or `!== 0` instead. Loading history...
180		if (in_array($plugin, $this->xoopsPlugins)) {
181		$buttons[] = $plugin;
182		}
183		} else {
184		$buttons[] = $plugin;
185		}
186		}
187		$this->setting['theme_' . $this->setting['theme'] . "_buttons{$i}"] = implode(',', $buttons);
188		}
189		}
190		}
191
192		$configured = array_unique($configured);
193		foreach ($this->config as $key => $val) {
194		if (isset($this->setting[$key]) \|\| in_array($key, $configured)) {
195		continue;
196		}
197		$this->setting[$key] = $val;
198		}
199
200		if (!is_dir(XOOPS_ROOT_PATH . $this->rootpath . '/themes/' . $this->setting['theme'] . '/docs/' . $this->setting['language'] . '/')) {
201		$this->setting['docs_language'] = 'en';
202		}
203
204		unset($this->config, $configured);
205
206		return true;
207		}
208
209		// load all plugins execpt the plugins in setting["exclude_plugins"]
210		/**
211		* @return array
212		*/
213		public function loadPlugins()
214		{
215		$plugins = array();
		0 ignored issues – show Unused Code introduced 2016-03-07 05:16 UTC by Report Bug Copy Issue Report `$plugins` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
216		$plugins_list = XoopsLists::getDirListAsArray(XOOPS_ROOT_PATH . $this->rootpath . '/plugins');
217		if (empty($this->setting['plugins'])) {
218		$plugins = $plugins_list;
219		} else {
220		$plugins = array_intersect(explode(',', $this->setting['plugins']), $plugins_list);
221		}
222		if (!empty($this->setting['exclude_plugins'])) {
223		$plugins = array_diff($plugins, explode(',', $this->setting['exclude_plugins']));
224		}
225		if (!empty($this->config['plugins'])) {
226		$plugins = array_merge($plugins, $this->config['plugins']);
227		}
228
229		return $plugins;
230		}
231
232		// return all xoops plugins
233		/**
234		* @return array
235		*/
236		public function get_xoopsPlugins()
237		{
238		$xoopsPlugins = array();
239		$allplugins = XoopsLists::getDirListAsArray(XOOPS_ROOT_PATH . $this->rootpath . '/plugins');
240		foreach ($allplugins as $plugin) {
241		if (strpos(strtolower($plugin), 'xoops') != false && file_exists(XOOPS_ROOT_PATH . $this->config['rootpath'] . "/include/$plugin.php")) {
		0 ignored issues – show Bug Best Practice introduced 2016-03-07 05:16 UTC by Report Bug Copy Issue Report It seems like you are loosely comparing `strpos(strtolower($plugin), 'xoops')` of type `integer` to the boolean `false`. If you are specifically checking for non-zero, consider using something more explicit like `> 0` or `!== 0` instead. Loading history...
242		if ($right = @include XOOPS_ROOT_PATH . $this->config['rootpath'] . "/include/$plugin.php") {
243		$xoopsPlugins[$plugin] = $plugin;
244		}
245		}
246		}
247
248		return $xoopsPlugins;
249		}
250
251		/**
252		* @param string $css_file
253		*
254		* @return array
255		*/
256		public function loadCss($css_file = 'style.css')
257		{
258		static $css_url, $css_path;
259
260		if (!isset($css_url)) {
261		$css_url = dirname(xoops_getcss($GLOBALS['xoopsConfig']['theme_set']));
262		$css_path = str_replace(XOOPS_THEME_URL, XOOPS_THEME_PATH, $css_url);
263		}
264
265		$css = array();
266		$css[] = $css_url . '/' . $css_file;
267		$css_content = file_get_contents($css_path . '/' . $css_file);
268
269		// get all import css files
270		if (preg_match_all("~\@import url\((.*\.css)\);~sUi", $css_content, $matches, PREG_PATTERN_ORDER)) {
271		foreach ($matches[1] as $key => $css_import) {
272		$css = array_merge($css, $this->loadCss($css_import));
273		}
274		}
275
276		return $css;
277		}
278
279		/**
280		* @return string
281		*/
282		public function render()
283		{
284		static $isTinyMceJsLoaded = false;
285
286		$this->init();
287		if (isset($this->setting['elements']) && self::$LastOfElementsTinymce != $this->setting['elements']) {
288		$ret = "\n<!-- 'tiny_mce.js' SCRIPT NOT YET " . $this->setting['elements'] . " -->\n"; //debug
289
290		return $ret;
291		} else {
292		$this->setting['elements'] = implode(',', self::$ListOfElementsTinymce);
293		}
294		if (!empty($this->setting['callback'])) {
295		$callback = $this->setting['callback'];
296		unset($this->setting['callback']);
297		} else {
298		$callback = '';
299		}
300		if (!empty($this->setting['file_browser_callback'])) {
301		$fbc_name = XOOPS_ROOT_PATH . '/class/xoopseditor/tinymce/include/' . $this->setting['file_browser_callback'] . '.js';
302		//suis passé la /lesrevespossibles/x244/class/xoopseditor/tinymce/tinymce/jscripts/include/openFinder.js
303		$file_browser_callback = "MyXoopsUrl ='" . XOOPS_URL . "';\n";
304		$file_browser_callback .= file_get_contents($fbc_name);
305		$file_browser_callback .= "\n//suis passé la " . $fbc_name;
306		//unset($this->setting["file_browser_callback"]);
307		} else {
308		$file_browser_callback = '//suis absent';
309		}
310
311		// create returned string - start
312		$ret = "\n";
313
314		/*
315		// IE BUG - start
316		// more info here:
317		// http://www.456bereastreet.com/archive/200802/beware_of_id_and_name_attribute_mixups_when_using_getelementbyid_in_internet_explorer/
318		// possible solution here:
319		// http://www.sixteensmallstones.org/ie-javascript-bugs-overriding-internet-explorers-documentgetelementbyid-to-be-w3c-compliant-exposes-an-additional-bug-in-getattributes
320		$ret =<<<EOF
321		<script language='javascript' type='text/javascript'>
322		if (/msie/i.test (navigator.userAgent)) { //only override IE
323		document.nativeGetElementById = document.getElementById;
324		document.getElementById = function(id) {
325		var elem = document.nativeGetElementById(id);
326		if (elem) {
327		//make sure that it is a valid match on id
328		if (elem.attributes['id'].value == id) {
329		return elem;
330		} else {
331		//otherwise find the correct element
332		for (var i=1;i<document.all[id].length;i++) {
333		if (document.all[id][i].attributes['id'].value == id) {
334		return document.all[id][i];
335		}
336		}
337		}
338		}
339
340		return null;
341		};
342		}
343		</script>
344		\n
345		EOF;
346		// IE BUG - end
347		*/
348
349		$ret .= "<!-- Start TinyMce Rendering -->\n"; //debug
350		if ($isTinyMceJsLoaded) {
351		$ret .= "<!-- 'tiny_mce.js' SCRIPT IS ALREADY LOADED -->\n"; //debug
352		} else {
353		$ret .= "<script type='text/javascript' src='" . XOOPS_URL . $this->rootpath . "/tiny_mce.js'></script>\n";
354		$isTinyMceJsLoaded = true;
355		}
356		$ret .= "<script type='text/javascript'>\n";
357		$ret .= "tinyMCE.init({\n";
358		// set options - start
359		foreach ($this->setting as $key => $val) {
360		$ret .= $key . ':';
361		if ($val === true) {
362		$ret .= 'true,';
363		} elseif ($val === false) {
364		$ret .= 'false,';
365		} else {
366		$ret .= "'{$val}',";
367		}
368		$ret .= "\n";
369		}
370		// set options - end
371		$ret .= "tinymceload: true\n";
372		$ret .= "});\n";
373		$ret .= $callback . "\n";
374		$ret .= $file_browser_callback . "\n";
375		//$ret .= "function toggleEditor(id) {tinyMCE.execCommand('mceToggleEditor',false, id);}\n";
376		$ret .= "</script>\n";
377		$ret .= "<!-- End TinyMce Rendering -->\n";//debug
378		// create returned string - end
379		return $ret;
380		}
381		}
382

XOOPS / XoopsCore25