[imp] re: /horde/lib/Prefs.php

[Bill Neely]

John:
All I know is that when the ! is present, the following error appears when you 
shoose logout from a loged in session.

Warning: Constant sid already defined 
in /usr/home/horde/public_html/horde/lib/Prefs.php on line 109

Warning: Cannot send session cache limiter - headers already sent (output 
started at /usr/home/horde/public_html/horde/lib/Prefs.php:109) 
in /usr/home/horde/public_html/horde/lib/Prefs.php on line 109

Removing the ! remedies the situation. In older versions of the CVS, the code 
was without the !
-- 
Bill


Quoting Jon Parise <jon@csh.rit.edu>:

> On Fri, Jun 22, 2001 at 09:46:55PM -0700, Bill Neely wrote:
> 
> > I believe there is a typo in horde/lib/Prefs.php (todays CVS)
> > 
> > Line reads:
> > if (!session_is_registered('prefs_cache')) {
> > 
> > shouldn't it read:
> > if (session_is_registered('prefs_cache')) {
> 
> I'm assuming you mean line 100 of Prefs.php.
>  
> > The first version causes error message to spill on login page when logour
> 
> > button is pressed in mailbox.
> 
> No, that line reads correctly.  It makes sure the session
> variable 'prefs_cache' hasn't already been registered before we
> initialize it.  Elsewhere in that file, we perform the inverse
> test to determine whether 'prefs_cache' has been registered so
> that we know we can use the preferences cache.
> 
> -- 
> Jon Parise (jon@csh.rit.edu)  .  Rochester Inst. of Technology
> http://www.csh.rit.edu/~jon/  :  Computer Science House Member
> 
> -- 
> IMP mailing list: http://horde.org/imp/
> Archive: http://marc.theaimsgroup.com/?l=imp&r=1&w=2
> Frequently Asked Questions: http://horde.org/faq/
> To unsubscribe, mail: imp-unsubscribe@lists.horde.org
> 
>