[imp] problem with logout

[Patrick Boutilier]

Try changing lines 52 to 57 of imp/templates/menu/menu.inc from:


/* Logout */
if ($registry->getMethod('auth/logout') == 'imp') {
     Menu::printItem(Horde::url($registry->getParam('webroot', 'horde') 
. '/login.php?reason=logout'), _("Logout"), 'logout.gif', 
$registry->getParam('graphics', 'horde'));
} else {
     Menu::printItem(Horde::applicationUrl('login.php?reason=logout'), 
_("Logout"), 'logout.gif', $registry->getParam('graphics', 'horde'));
}


to:


/* Logout */
if ($registry->getMethod('auth/logout') == 'imp') {
     Menu::printItem(Horde::applicationUrl('login.php?reason=logout'), 
_("Logout"), 'logout.gif', $registry->getParam('graphics', 'horde'));

} else {
Menu::printItem(Horde::url($registry->getParam('webroot', 'horde') . 
'/login.php?reason=logout'), _("Logout"), 'logout.gif', 
$registry->getParam('graphics', 'horde'));
}


In other words reverse the logic.


The logic was changed from 3.1 to 3.2RCx . Somebody asked about this on 
the list but I never saw a reply as to why it was changed.




Craig White wrote:
> Sorry - previous message had all that in there - was trimmed out in
> Eric's reply
> 
> horde RELENG_2
> imp RELENG_3
> 
> updated this am
> 
> RH 8.0 - all errata applied (I don't think that this affects anything)
> Apache 2.0x
> PHP 4.12.x
> 
> Craig
> 
> On Tue, 2003-02-11 at 09:24, Patrick Boutilier wrote:
> 
>>What version of IMP?
>>
>>
>>
>>Craig White wrote:
>>
>>>On Mon, 2003-02-10 at 21:33, Eric Rostetter wrote:
>>>
>>>
>>>>Quoting Craig White <craigwhite at azapple.com>:
>>>>
>>>>
>>>>
>>>>>when I log out,
>>>>
>>>>>From IMP's logout button (top menu in IMP), or the Horde logout
>>>
>>>button
>>>
>>>
>>>>(on summary page or link at the bottom of the page), or both?
>>>
>>>----
>>>MUST be horde summary page
>>>
>>>no bottom menu on that page - only buttons are options & logout
>>>----
>>>
>>>
>>>>
>>>>>If I could locate buttons for mail & addressbook on that screen, I
>>>
>>>could
>>>
>>>
>>>>>live with it, but the only way for me to get back to email is to
>>>
>>>logout
>>>
>>>
>>>>>and go back in via long address or redirect that I use.
>>>>
>>>>Normally you should have a horde menu at the bottom of the screen with
>>>>all your applications in it.  Only reasons I can think of why you
>>>
>>>don't
>>>
>>>
>>>>is you disabled it, or you go directly to IMP rather than Horde when
>>>>you start the session...
>>>>
>>>
>>>----
>>>I can't believe that I disabled it
>>>----
>>>
>>>
>>>>>Why doesn't logging back in from the logout screen take be back into
>>>>>IMP?
>>>>
>>>>Don't know.  Really, no idea.  In fact, I personally want it to go to
>>>>the summary page.  Though I must say I use a CVS HEAD setup so my 
>>>>summary page may be slightly better than yours?  Not sure...
>>>>
>>>>Anyway, if you want people to go to IMP when they login, for what ever
>>>>reasons, try editing horde/config/prefs.php and changing the "value"
>>>>for "initial_application" to be "imp" instead of the empty value.
>>>
>>>-----
>>>I did change horde/config/prefs.php to value imp for initial_application
>>>and it didn't do anything - I found that already ;-)
>>>
>>>Anyway, maybe the problem is my methodology...
>>>
>>>I have set up Apache to redirect...
>>>
>>>/mail/ https://www.aidssc.org/horde/imp
>>>
>>>so if they go to http://www.aidssc.org/mail they are automatically
>>>redirected to using SSL & the imp/login.php
>>>
>>>but when you click the logout button on imp, you are directed to
>>>https://www.aidssc.org/horde/imp/login.php?reason=logout&url=https%3A%2F%2Fwww.aidssc.org%2Fhorde%2Flogin.php
>>>
>>>note that the imp directory isn't included on the path - when I change
>>>it by adding imp%2F before login.php, it works like I would expect. So I
>>>would guess that I would want the logout to include the imp path in the
>>>URL wouldn't I? If so, how would I accomplish that? It would seem that I
>>>should be able to show the buttons for imp & turba on the horde summary
>>>page easier.
>>>
>>>Thanks,
>>>
>>>Craig
>>>
>>>
>>>
>>
>>
>>-- 
>>IMP mailing list
>>Frequently Asked Questions: http://horde.org/faq/
>>To unsubscribe, mail: imp-unsubscribe at lists.horde.org
> 
> 
>