Commit 25c00ad2 authored by lpsolit%gmail.com's avatar lpsolit%gmail.com

Bug 303694: Eliminate deprecated Bugzilla::DB routines from enter_bug.cgi -…

Bug 303694: Eliminate deprecated Bugzilla::DB routines from enter_bug.cgi - Patch by Teemu Mannermaa <wicked@etlicon.fi> r=LpSolit a=justdave
parent 256ddf6e
...@@ -475,15 +475,18 @@ if ( ($cloned_bug_id) && ...@@ -475,15 +475,18 @@ if ( ($cloned_bug_id) &&
$default{'version'} = $vars->{'version'}->[$#{$vars->{'version'}}]; $default{'version'} = $vars->{'version'}->[$#{$vars->{'version'}}];
} }
# Only used with placeholders below
trick_taint($product);
# Get list of milestones. # Get list of milestones.
if ( Param('usetargetmilestone') ) { if ( Param('usetargetmilestone') ) {
$vars->{'target_milestone'} = $::target_milestone{$product}; $vars->{'target_milestone'} = $::target_milestone{$product};
if (formvalue('target_milestone')) { if (formvalue('target_milestone')) {
$default{'target_milestone'} = formvalue('target_milestone'); $default{'target_milestone'} = formvalue('target_milestone');
} else { } else {
SendSQL("SELECT defaultmilestone FROM products WHERE " . $default{'target_milestone'} =
"name = " . SqlQuote($product)); $dbh->selectrow_array('SELECT defaultmilestone FROM products
$default{'target_milestone'} = FetchOneColumn(); WHERE name = ?', undef, $product);
} }
} }
...@@ -498,9 +501,9 @@ my @status; ...@@ -498,9 +501,9 @@ my @status;
# confirmation, user cannot confirm UNCONFIRMED # confirmation, user cannot confirm UNCONFIRMED
# confirmation, user can confirm NEW, UNCONFIRMED. # confirmation, user can confirm NEW, UNCONFIRMED.
SendSQL("SELECT votestoconfirm FROM products WHERE name = " . my $votestoconfirm = $dbh->selectrow_array('SELECT votestoconfirm FROM products
SqlQuote($product)); WHERE name = ?', undef, $product);
if (FetchOneColumn()) { if ($votestoconfirm) {
if (UserInGroup("editbugs") || UserInGroup("canconfirm")) { if (UserInGroup("editbugs") || UserInGroup("canconfirm")) {
push(@status, "NEW"); push(@status, "NEW");
} }
...@@ -520,17 +523,19 @@ if (formvalue('bug_status') && (lsearch(\@status, formvalue('bug_status')) >= 0) ...@@ -520,17 +523,19 @@ if (formvalue('bug_status') && (lsearch(\@status, formvalue('bug_status')) >= 0)
$default{'bug_status'} = $status[0]; $default{'bug_status'} = $status[0];
} }
SendSQL("SELECT DISTINCT groups.id, groups.name, groups.description, " . my $grouplist = $dbh->selectall_arrayref(
"membercontrol, othercontrol " . q{SELECT DISTINCT groups.id, groups.name, groups.description,
"FROM groups LEFT JOIN group_control_map " . membercontrol, othercontrol
"ON group_id = id AND product_id = $product_id " . FROM groups
"WHERE isbuggroup != 0 AND isactive != 0 ORDER BY description"); LEFT JOIN group_control_map
ON group_id = id AND product_id = ?
WHERE isbuggroup != 0 AND isactive != 0
ORDER BY description}, undef, $product_id);
my @groups; my @groups;
while (MoreSQLData()) { foreach my $row (@$grouplist) {
my ($id, $groupname, $description, $membercontrol, $othercontrol) my ($id, $groupname, $description, $membercontrol, $othercontrol) = @$row;
= FetchSQLData();
# Only include groups if the entering user will have an option. # Only include groups if the entering user will have an option.
next if ((!$membercontrol) next if ((!$membercontrol)
|| ($membercontrol == CONTROLMAPNA) || ($membercontrol == CONTROLMAPNA)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment