Fix for bug 251935: Remove $COOKIE from process_bug.cgi. r=joel, a=myk.

This commit is contained in:
kiko%async.com.br 2004-07-20 21:25:33 +00:00
Родитель e3e6913992
Коммит cd3bebfa78
1 изменённых файлов: 7 добавлений и 9 удалений

Просмотреть файл

@ -46,7 +46,6 @@ use Bugzilla::Flag;
use vars qw(%versions use vars qw(%versions
%components %components
%COOKIE
%legal_opsys %legal_opsys
%legal_platform %legal_platform
%legal_priority %legal_priority
@ -165,8 +164,8 @@ if (defined($::FORM{'id'})) {
# Set up the vars for nagiavtional <link> elements # Set up the vars for nagiavtional <link> elements
my $next_bug; my $next_bug;
if ($::COOKIE{"BUGLIST"} && $::FORM{'id'}) { if ($cgi->cookie("BUGLIST") && $::FORM{'id'}) {
my @bug_list = split(/:/, $::COOKIE{"BUGLIST"}); my @bug_list = split(/:/, $cgi->cookie("BUGLIST"));
$vars->{'bug_list'} = \@bug_list; $vars->{'bug_list'} = \@bug_list;
my $cur = lsearch(\@bug_list, $::FORM{"id"}); my $cur = lsearch(\@bug_list, $::FORM{"id"});
if ($cur >= 0 && $cur < $#bug_list) { if ($cur >= 0 && $cur < $#bug_list) {
@ -1284,7 +1283,7 @@ foreach my $id (@idlist) {
SendSQL("UNLOCK TABLES"); SendSQL("UNLOCK TABLES");
ThrowUserError('comment_required'); ThrowUserError('comment_required');
} else { } else {
AppendComment($id, $::COOKIE{'Bugzilla_login'}, $::FORM{'comment'}, AppendComment($id, Bugzilla->user->login, $::FORM{'comment'},
$::FORM{'commentprivacy'}, $timestamp, $::FORM{'work_time'}); $::FORM{'commentprivacy'}, $timestamp, $::FORM{'work_time'});
if ($::FORM{'work_time'}) { if ($::FORM{'work_time'}) {
LogActivityEntry($id, "work_time", "", $::FORM{'work_time'}, LogActivityEntry($id, "work_time", "", $::FORM{'work_time'},
@ -1702,7 +1701,7 @@ foreach my $id (@idlist) {
$vars->{'mailrecipients'} = { 'cc' => \@ccRemoved, $vars->{'mailrecipients'} = { 'cc' => \@ccRemoved,
'owner' => $origOwner, 'owner' => $origOwner,
'qa' => $origQaContact, 'qa' => $origQaContact,
'changer' => $::COOKIE{'Bugzilla_login'} }; 'changer' => Bugzilla->user->login };
$vars->{'id'} = $id; $vars->{'id'} = $id;
@ -1728,12 +1727,11 @@ foreach my $id (@idlist) {
SendSQL("INSERT INTO cc (who, bug_id) VALUES ($reporter, " . SqlQuote($duplicate) . ")"); SendSQL("INSERT INTO cc (who, bug_id) VALUES ($reporter, " . SqlQuote($duplicate) . ")");
} }
# Bug 171639 - Duplicate notifications do not need to be private. # Bug 171639 - Duplicate notifications do not need to be private.
AppendComment($duplicate, $::COOKIE{'Bugzilla_login'}, "*** Bug $::FORM{'id'} has been marked as a duplicate of this bug. ***", 0); AppendComment($duplicate, Bugzilla->user->login, "*** Bug $::FORM{'id'} has been marked as a duplicate of this bug. ***", 0);
CheckFormFieldDefined(\%::FORM,'comment'); CheckFormFieldDefined(\%::FORM,'comment');
SendSQL("INSERT INTO duplicates VALUES ($duplicate, $::FORM{'id'})"); SendSQL("INSERT INTO duplicates VALUES ($duplicate, $::FORM{'id'})");
$vars->{'mailrecipients'} = { 'changer' => $::COOKIE{'Bugzilla_login'} $vars->{'mailrecipients'} = { 'changer' => Bugzilla->user->login };
};
$vars->{'id'} = $duplicate; $vars->{'id'} = $duplicate;
$vars->{'type'} = "dupe"; $vars->{'type'} = "dupe";
@ -1746,7 +1744,7 @@ foreach my $id (@idlist) {
if ($check_dep_bugs) { if ($check_dep_bugs) {
foreach my $k (keys(%dependencychanged)) { foreach my $k (keys(%dependencychanged)) {
$vars->{'mailrecipients'} = { 'changer' => $::COOKIE{'Bugzilla_login'} }; $vars->{'mailrecipients'} = { 'changer' => Bugzilla->user->login };
$vars->{'id'} = $k; $vars->{'id'} = $k;
$vars->{'type'} = "dep"; $vars->{'type'} = "dep";