[dev] Re: [cvs] commit: imp test.php

[Jan Schneider]

Quoting Chuck Hagenbuch <chuck at horde.org>:

> Quoting Michael M Slusarz <slusarz at bigworm.colorado.edu>:
>
> >   Log:
> >   Use Horde_Test::.
>
> Can you add a check such that if IMP can't find the Horde_Test class, it
> tells the user to install/configure Horde first? I'm not opposed to what
> you're doing, but the test files were designed to have no external
> dependancies, so that they could be most useful in a broken
> installation...
>
> While you're at it, though, want to move checking for config files into
> the
> test files also?
>
> I don't think we should necessarily take that check out of application
> index.php files, though I'm open to that argument, but maybe we can make
> it
> a session flag, so it's only done once per session per app?

I don't think that makes much sense because the index.php files are only
accessed if the users switch the application, never while they use an
application.

Jan.

--
http://www.horde.org - The Horde Project
http://www.ammma.de - discover your knowledge
http://www.tip4all.de - Deine private Tippgemeinschaft