[dev] [commits] Horde branch master updated. 7b56cd8b6078f3ebae95ff272e24694c48ceb37c
Jan Schneider
jan at horde.org
Fri Feb 11 18:35:13 UTC 2011
Zitat von Michael M Slusarz <slusarz at horde.org>:
> -----------------------------------------------------------------------
>
> commit 96c8725bfe9d6746252dd74adabf0ad9753b469d
> Author: Michael M Slusarz <slusarz at curecanti.org>
> Date: Fri Feb 11 10:45:47 2011 -0700
>
> Revert "Actually use $app_webroot/fileroot for the Horde stanza."
>
> This reverts commit 86be1eafeff8848940928febf5a93ea2d2f7b071.
>
> This is incorrect usage of $app_webroot. It is *NOT* the location of
> the base of horde. It is the directory that contains the horde apps
> (e.g. the base git directory).
>
> HORDE_BASE already contains the correct horde webroot. No need to
> calculate again.
>
> framework/Core/lib/Horde/Registry.php | 8 ++------
> 1 files changed, 2 insertions(+), 6 deletions(-)
>
> http://git.horde.org/horde-git/-/commit/96c8725bfe9d6746252dd74adabf0ad9753b469d
You are mixing things here. $app_webroot has nothing to do with
directories, it't the URL's root, and without that, _detectWebroot()
is used for horde's 'webroot' setting, which is wrong.
The same is true for $app_fileroot though. Try a fresh git install
with a pristine registry.php. It won't load.
> -----------------------------------------------------------------------
>
> commit 7b56cd8b6078f3ebae95ff272e24694c48ceb37c
> Author: Michael M Slusarz <slusarz at curecanti.org>
> Date: Fri Feb 11 10:49:19 2011 -0700
>
> Revert "No idea what happened to getLoginParams(), but it
> doesn't exist anymore."
>
> This reverts commit cf2d5a60366246e3a91c2a7847568651b51a27f6.
>
> getLoginParams() exists (Horde_Core_Auth_Application), and this commit
> breaks things like logging into IMP for me.
>
> horde/login.php | 4 ----
> 1 files changed, 0 insertions(+), 4 deletions(-)
>
> http://git.horde.org/horde-git/-/commit/7b56cd8b6078f3ebae95ff272e24694c48ceb37c
But that's the only driver the has this method. And it's passing it
through the $_base driver, but no other driver has this methods. I
don't even get to the login screen at the moment, because i get a
fatal error of this missing method. I can log in to IMP just fine.
Jan.
--
Do you need professional PHP or Horde consulting?
http://horde.org/consulting/
More information about the dev
mailing list