Missing bits of fix for bug 236678: Clean up access to COOKIE global.

Murder the last remaining places in the tree where COOKIE is used;
includes a rather thorough cleanup of Bugzilla::Bug->user and a minor
doc update. r=joel, a=justdave.

diff --git a/Bugzilla/Auth/Login/WWW.pm b/Bugzilla/Auth/Login/WWW.pm
index 2c45562d260ffbd69cf903f99922ff1730b0553b..d18c758d9a41bc9643957f8f213eff89cb2e2cbb 100644 (file)
--- a/Bugzilla/Auth/Login/WWW.pm
+++ b/Bugzilla/Auth/Login/WWW.pm
@@ -43,9 +43,6 @@ sub login {
 
     # Avoid double-logins, which may confuse the auth code
     # (double cookies, odd compat code settings, etc)
-    # This is particularly important given the munging for
-    # $::COOKIE{'Bugzilla_login'} from a userid to a loginname
-    # (for backwards compat)
     if (defined $user) {
         return $user;
     }
@@ -71,11 +68,6 @@ sub login {
 
         # Compat stuff
         $::userid = $userid;
-
-        # Evil compat hack. The cookie stores the id now, not the name, but
-        # old code still looks at this to get the current user's email
-        # so it needs to be set.
-        $::COOKIE{'Bugzilla_login'} = $user->login;
     } else {
         Bugzilla->logout_request();
     }