Redundant use of DBNameToIDAndCheck in userprefs.cgi

RESOLVED FIXED in Bugzilla 2.16

Status

()

Bugzilla
User Accounts
RESOLVED FIXED
16 years ago
5 years ago

People

(Reporter: justdave, Assigned: myk)

Tracking

2.15
Bugzilla 2.16
PowerPC
Mac System 9.x

Details

Attachments

(1 attachment)

The new userprefs.cgi that was recently checked in contains the following code:

confirm_login();
    
GetVersionTable();

$userid = DBNameToIdAndCheck($::COOKIE{'Bugzilla_login'});

The last of these three lines is unnecessary.  confirm_login, two lines above,
already does this.  (Including setting the value of $::userid).

We shouldn't need to have $userid declared as a local variable here.
(Assignee)

Comment 1

16 years ago
Created attachment 72442 [details] [diff] [review]
patch v1: fixes problem
Comment on attachment 72442 [details] [diff] [review]
patch v1: fixes problem

That'll do :-) rx2=gerv.

Gerv
Attachment #72442 - Flags: review+
(Assignee)

Comment 3

16 years ago
Checking in userprefs.cgi;
/cvsroot/mozilla/webtools/bugzilla/userprefs.cgi,v  <--  userprefs.cgi
new revision: 1.30; previous revision: 1.29
done
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
fixing incorrect milestones on fixed bugs.
Target Milestone: --- → Bugzilla 2.16
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.