[dev] [patch] logout is not application fix
Chuck Hagenbuch
chuck at horde.org
Fri Sep 24 17:45:09 PDT 2004
Quoting Vilius Sumskas <vilius at lnk.lt>:
> patch fixes bug #624 (http://bugs.horde.org/ticket/?id=624).
This is okay, but what about following the same path we did with the problem
reporting link and move it out of registry.php entirely, so we can remove the
special cases? Just handle it in Horde::getServiceLink()?
> By the way, this code:
>
> /* Skip app if no conf.xml file. */
> $path = $registry->getParam('fileroot', $app) . '/config';
> if (!file_exists($path . '/conf.xml')) {
> continue;
> }
>
> in /horde/setup/index.php on line 88 will not do, because getParam returns
> Horde's path, which conf.xml is always present. Sadly I haven't got patch for
> this.
Ah, good point. I'll just removed this for now.
-chuck
--
"But she goes not abroad in search of monsters to destroy." - John
Quincy Adams
More information about the dev
mailing list