i can't really get excited one way or another. if you want a universal way to get session id, call session_id(). your patch is a very slight improvement, i suppose. On 1/16/08, Ivan Sergio Borgonovo <mail@webthatworks.it> wrote:
On Wed, 16 Jan 2008 15:04:03 -0500 "Moshe Weitzman" <weitzman@tejasa.com> wrote:
If people find it a good idea to be able to get $user->sid even for not authenticated users... I don't mind about the implementation ;) If people think it is a bad idea, I'd like to know why.
i can't make out what your proposal is. please post a patch.
Index: session.inc =================================================================== --- session.inc (revisione 419) +++ session.inc (copia locale) @@ -48,6 +48,7 @@ else { $session = isset($user->session) ? $user->session : ''; $user = drupal_anonymous_user($session); + $user->sid=session_id(); }
return $user->session;
-- Ivan Sergio Borgonovo http://www.webthatworks.it