diff options
author | mkanat%bugzilla.org <> | 2006-10-21 05:50:30 +0200 |
---|---|---|
committer | mkanat%bugzilla.org <> | 2006-10-21 05:50:30 +0200 |
commit | 0b954e5dc3a6e199252c97d876113b013eb3d3bd (patch) | |
tree | 5e2bcc48c4cc2bb02dbd8583e423d5917ac30278 /process_bug.cgi | |
parent | 810ac042bacc992cc1cb3ae324b0d61b0615b697 (diff) | |
download | bugzilla-0b954e5dc3a6e199252c97d876113b013eb3d3bd.tar.gz bugzilla-0b954e5dc3a6e199252c97d876113b013eb3d3bd.tar.xz |
Bug 348923: Clean up a few more variable-scoping issues for mod_perl
Patch By Max Kanat-Alexander <mkanat@bugzilla.org> r=LpSolit, a=myk
Diffstat (limited to 'process_bug.cgi')
-rwxr-xr-x | process_bug.cgi | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/process_bug.cgi b/process_bug.cgi index dbe62f606..33a5c75ec 100755 --- a/process_bug.cgi +++ b/process_bug.cgi @@ -71,7 +71,7 @@ $vars->{'use_keywords'} = 1 if Bugzilla::Keyword::keyword_count(); my @editable_bug_fields = editable_bug_fields(); my $requiremilestone = 0; -my $PrivilegesRequired = 0; +local our $PrivilegesRequired = 0; ###################################################################### # Subroutines @@ -729,7 +729,7 @@ sub ChangeStatus { } sub ChangeResolution { - my ($str) = (@_); + my ($bug, $str) = (@_); my $dbh = Bugzilla->dbh; my $cgi = Bugzilla->cgi; @@ -1053,7 +1053,7 @@ SWITCH: for ($cgi->param('knob')) { last SWITCH; }; /^clearresolution$/ && CheckonComment( "clearresolution" ) && do { - ChangeResolution(''); + ChangeResolution($bug, ''); last SWITCH; }; /^(resolve|change_resolution)$/ && CheckonComment( "resolve" ) && do { @@ -1081,7 +1081,7 @@ SWITCH: for ($cgi->param('knob')) { ChangeStatus('RESOLVED'); } - ChangeResolution($cgi->param('resolution')); + ChangeResolution($bug, $cgi->param('resolution')); last SWITCH; }; /^reassign$/ && CheckonComment( "reassign" ) && do { @@ -1125,7 +1125,7 @@ SWITCH: for ($cgi->param('knob')) { }; /^reopen$/ && CheckonComment( "reopen" ) && do { ChangeStatus('REOPENED'); - ChangeResolution(''); + ChangeResolution($bug, ''); last SWITCH; }; /^verify$/ && CheckonComment( "verify" ) && do { @@ -1184,7 +1184,7 @@ SWITCH: for ($cgi->param('knob')) { _remove_remaining_time(); ChangeStatus('RESOLVED'); - ChangeResolution('DUPLICATE'); + ChangeResolution($bug, 'DUPLICATE'); my $comment = $cgi->param('comment'); $comment .= "\n\n" . get_text('bug_duplicate_of', { dupe_of => $duplicate }); |