I suggest developing your code with logging enabled and your error_reporting set to either “E_ALL” or “E_ALL & ~E_NOTICE”. This will help you maintain best practices in your code.

was not correct your will do better.

But I don’t agree on the
if (is_array($depends) && count($depends))
it’s just silly to check for size of array. Even if size is 0, foreach will not trigger.

public $loaded = array();
public $repository = “/common/jloader”;
public $home_path = “/www/mysite”;

/**
*
* @since 1.0.0
*
* @param string $handler Valid external file handler name.
* @param int|bool $ver Optional. Script version number. Default is false.
* @param array|bool $depends Array of valid handler dependencies.
* @return bool
*/
public function load($handler, $ver = false, $src = false, $depends=false)
{
if (!$src)
{
if($ver)
{
$src = $this->repository.”/$handler/$handler.js?ver=$ver”;
} else {
$src = $this->repository.”/$handler/$handler.js”;
}
}

if ($src && !file_exists($src) && !file_exists($this->home_path.$src))
{
trigger_error(”ExtLoader::load($handler, $ver, $src) sad file “.$src.” doesn’t exist in local or home path.”);
return false;
}

if (is_array($depends) && count($depends))
{
foreach($depends as $script)
{
$this->load($script);
}
}
$this->loaded[$handler] = ‘<script type=”text/javascript” src=”%27.$src.%27″></script>’;

return true;
}

/**
* Print loaded Javascript files.
*
* @since 1.0.0
*
* @return unknown
*/
function printjs()
{
foreach($this->loaded as $script)
{
echo $script.”\n”;
}
}
}

Thanks!