Bommels05/Mozilla
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 163114 - Templatise all calls to DisplayError. Patch C. Patch by gerv; r=burnus.

Browse Source 
git-svn-id: svn://10.0.0.236/trunk@130932 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
gerv%gerv.net
2002-10-01 22:41:09 +00:00
parent 7d339091a7
commit f5ff5cdd26
7 changed files with 75 additions and 71 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
mozilla/webtools/bugzilla/process_bug.cgi
View File

@@ -84,9 +84,7 @@ if (defined $::FORM{'id'}) {
}
# Make sure there are bugs to process.
scalar(@idlist)
|| DisplayError("You did not select any bugs to modify.")
&& exit;
scalar(@idlist) || ThrowUserError("no_bugs_chosen");
# If we are duping bugs, let's also make sure that we can change
# the original. This takes care of issue A on bug 96085.
@@ -616,11 +614,9 @@ foreach my $field ("rep_platform", "priority", "bug_severity",
my $prod_id; # Remember, can't use this for mass changes
if ($::FORM{'product'} ne $::dontchange) {
$prod_id = get_product_id($::FORM{'product'});
if (! $prod_id) {
DisplayError("The <tt>" . html_quote($::FORM{'product'}) .
"</tt> product doesn't exist.");
exit;
}
$prod_id ||
ThrowUserError("invalid_product_name", {product => $::FORM{'product'});
DoComma();
$::query .= "product_id = $prod_id";
} else {
@@ -637,12 +633,10 @@ if ($::FORM{'component'} ne $::dontchange) {
}
$comp_id = get_component_id($prod_id,
$::FORM{'component'});
if (! $comp_id) {
DisplayError("The <tt>" . html_quote($::FORM{'component'}) .
"</tt> component doesn't exist in the <tt>" .
html_quote($::FORM{'product'}) . "</tt> product");
exit;
}
$comp_id || ThrowCodeError("invalid_component",
{component => $::FORM{'component'},
product => $::FORM{'product'}});
DoComma();
$::query .= "component_id = $comp_id";
}