[dev] Problem with NAG and Horde::addScriptFile()

Chuck Hagenbuch chuck at horde.org
Wed Sep 3 14:16:43 PDT 2003


Quoting Tyler Colbert <tyler-hordeml at colberts.us>:

> The nag tasklists.php file fails to include the javascript to manage the
> Permissions button.  Seems that addScriptFile decides that the headers havent
> been sent yet, so it queues up the script to be added, but nothing ever calls
> Horde::includeScriptFiles();.
>
> Ive attached a patch, but I dont think its the proper solution.  I just call
> includeScriptFiles() a couple lines after the addScriptFile() call..
>
> It seems like this problem could happen to any part of horde.  Perhaps the
> intention was to call includeScriptFiles() in common-header.inc or something?

Fixed in CVS (in a few other apps, too).

-chuck

--
Charles Hagenbuch, <chuck at horde.org>
They're just looking at a wall of meat.


More information about the dev mailing list