quietly_check_login() needs to be called in colchange.cgi

RESOLVED FIXED in Bugzilla 2.16

Status

()

Bugzilla
User Interface
RESOLVED FIXED
16 years ago
5 years ago

People

(Reporter: dkl, Assigned: dkl)

Tracking

({regression})

2.16
Bugzilla 2.16
x86
Linux
regression

Details

Attachments

(1 attachment)

(Assignee)

Description

16 years ago
Currently in colchange.cgi, quietly_check_login or confirm_login is not being
called which means the footer reports that the user is not logged in. Adding
quietly_check_login() to the top fixes this problem. Attaching patch.
(Assignee)

Comment 1

16 years ago
Created attachment 86803 [details] [diff] [review]
Fix to add login check to colchange.cgi (v1)
Comment on attachment 86803 [details] [diff] [review]
Fix to add login check to colchange.cgi (v1)

2xr= justdave

tested on syndicomm (landfill is down), both on trunk and the 2.16 branch, this
fixes it.
Attachment #86803 - Flags: review+
Keywords: patch, regression, review
Target Milestone: --- → Bugzilla 2.16
-> patch author
Assignee: myk → dkl
(Assignee)

Comment 4

16 years ago
BUGZILLA-2_16-BRANCH
Checking in colchange.cgi;
/cvsroot/mozilla/webtools/bugzilla/colchange.cgi,v  <--  colchange.cgi
new revision: 1.24.2.1; previous revision: 1.24
done

HEAD
Checking in colchange.cgi;
/cvsroot/mozilla/webtools/bugzilla/colchange.cgi,v  <--  colchange.cgi
new revision: 1.25; previous revision: 1.24
done

Resolving fixed.
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.