[dev] [commits] Horde branch master updated. bf04645b471373b610d645ac1da27e4b88a9437c

[Jan Schneider]

Zitat von Michael M Slusarz <slusarz at horde.org>:

> commit 05f2a53df4b555f1fdaded5213af80fed4bc0907
> Author: Michael M Slusarz <slusarz at curecanti.org>
> Date:   Fri Jul 9 11:15:14 2010 -0600
>
>     I *think* we always want Horde::url() to use Horde webroot
>
>     All of a sudden, I was getting weird redirects to URLs like
>     example.com/imp/imp during logins - because webroot was example.com/imp
>
>     This might be an artifact of new sidebar code (?)
>
>     If you want the application's webroot, I believe this is what
>     Horde::applicationUrl() is for.
>
>  framework/Core/lib/Horde.php |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> http://git.horde.org/diff.php/framework/Core/lib/Horde.php?rt=horde-git&r1=2d9e92a12f14de23c0db49575296a890da533583&r2=05f2a53df4b555f1fdaded5213af80fed4bc0907

No, Horde::url() is supposed to work with the current application,  
thus this change is wrong. I think we could get rid off  
applicationUrl() completely though, to simplify things.

Jan.

-- 
Do you need professional PHP or Horde consulting?
http://horde.org/consulting/