[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