From: myk%mozilla.org <>
Date: Tue, 5 Mar 2002 07:38:55 +0000 (+0000)
Subject: Fix for bug 128784: Eliminates redundant function call in userprefs.cgi.
X-Git-Tag: bugzilla-2.16rc1~193
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=cd081e358a07abf94e730de47c05d5be82d07bfe;p=thirdparty%2Fbugzilla.git

Fix for bug 128784: Eliminates redundant function call in userprefs.cgi.
Patch by Myk Melez <myk@mozilla.org>
r=gerv x 2
---

diff --git a/userprefs.cgi b/userprefs.cgi
index 301b90b1e9..be8adb3480 100755
--- a/userprefs.cgi
+++ b/userprefs.cgi
@@ -38,9 +38,7 @@ sub sillyness {
 }
 
 # Use global template variables.
-use vars qw($template $vars);
-
-my $userid;
+use vars qw($template $vars $userid);
 
 # The default email flags leave all email on.
 my $defaultflagstring = "ExcludeSelf~on~";
@@ -305,8 +303,6 @@ confirm_login();
 
 GetVersionTable();
 
-$userid = DBNameToIdAndCheck($::COOKIE{'Bugzilla_login'});
-
 $vars->{'login'} = $::COOKIE{'Bugzilla_login'};
 $vars->{'changes_saved'} = $::FORM{'dosave'};