[Phplib-trackers] [ phplib-Patches-501754 ] removed call-by-ref from user.inc

[<no...@so...>]

Patches item #501754, was opened at 2002-01-10 04:41
You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=403613&aid=501754&group_id=31885

Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Gaetano Giunta (ggiunta)
Assigned to: Nobody/Anonymous (nobody)
Summary: removed call-by-ref from user.inc

Initial Comment:
modified calls to session->serialize() that will give warning 
messages with allow-call-by-refernce set to Off in php.ini

----------------------------------------------------------------------

>Comment By: Gaetano Giunta (ggiunta)
Date: 2002-01-14 03:35

Message:
Logged In: YES 
user_id=114789

DOH! forgot to mention: patch is against version 7.4rc1

in this 
release definition of sess->serialize in session.inc is ok! I think user-
>freeze was added to override session->freeze (since user is a descendent 
of session this is correct); the main difference is that if user==nobody 
(not authenticated user), session data is not frozen. Actually code 
could be simpler: if user!= nobody then call inherited 
method...

Gaetano

----------------------------------------------------------------------

Comment By: Dieter Steinwedel (dsteinwe)
Date: 2002-01-11 08:53

Message:
Logged In: YES 
user_id=373951

You must also change the function header of "serialize" in 
session.inc. Add a "&" before the param "$str". Otherwise 
it doesn't work!

Why do you put "freeze" to the class "user"? It is a part 
of "session".

----------------------------------------------------------------------

You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=403613&aid=501754&group_id=31885