Warning: trim() expects parameter 1 to be string, array given in plugin scriptmerge

yireo-joomla / pkg_scriptmerge

GNU General Public License v3.0

7 stars 10 forks source link

Warning: trim() expects parameter 1 to be string, array given in plugin scriptmerge #1

Closed ghost closed 9 years ago

ghost commented 9 years ago

Above warning

in version 0.9.23 in line 872
in version 0.9.24 in line 896

Reason:

$this->getArrayFromParam('exclude_menuitems');

results in an array. Field exclude_menuitems is a multiple one.

See

private function getArrayFromParam($param)
{
$data = $this->params->get($param);
$data = trim($data);
...

yireo commented 9 years ago

Thanks for posting. Your comments were already so much in detail - I thought you were also going to add a code fix :) I've added a fix for it now.

ghost commented 9 years ago

I thought you were also going to add a code fix :)

I don't use this plugin. Thus I couldn't test an own Pull Request. Just found the error indexed at Google. So, you and I made some anonymous users happy, ;-).