Is it a bug for the method of addJqueryPlugin method in /class/controller/Controller.php?

[ealio]

Is it a bug for the method of addJqueryPlugin method in /class/controller/Controller.php?

 

public function addJqueryPlugin($name, $folder = null)
    {
        $plugin_path = array();
        if (is_array($name))
        {
            foreach ($name as $plugin)
            {
                $plugin_path = Media::getJqueryPluginPath($plugin, $folder);
                if(!empty($plugin_path['js']))
                    $this->addJS($plugin_path['js']);
                if(!empty($plugin_path['css']))        
                    $this->addCSS($plugin_path['css']);
            }
        }
        else
            $plugin_path = Media::getJqueryPluginPath($name, $folder);

        if(!empty($plugin_path['css']))
            $this->addCSS($plugin_path['css']);
        if(!empty($plugin_path['js']))
            $this->addJS($plugin_path['js']);
    }

 

If the parameter of $name is an array, the last

        if(!empty($plugin_path['css']))
            $this->addCSS($plugin_path['css']);
        if(!empty($plugin_path['js']))
            $this->addJS($plugin_path['js']);

should not be executed.

 

I think it should be changed to

public function addJqueryPlugin($name, $folder = null)
    {
        $plugin_path = array();
        if (is_array($name))
        {
            foreach ($name as $plugin)
            {
                $plugin_path = Media::getJqueryPluginPath($plugin, $folder);
                if(!empty($plugin_path['js']))
                    $this->addJS($plugin_path['js']);
                if(!empty($plugin_path['css']))        
                    $this->addCSS($plugin_path['css']);
            }
        }
        else{
            $plugin_path = Media::getJqueryPluginPath($name, $folder);

            if(!empty($plugin_path['css']))
                $this->addCSS($plugin_path['css']);
            if(!empty($plugin_path['js']))
                $this->addJS($plugin_path['js']);

        }
    }

 

What do you think?

[PascalVG]

ealio,

think you have a point there. it twice adds the css and js of the last $plugin, one inside the foreach, one at the end. your expansion of the else-reach eliminates it :-)

 

pascal