From: Kohei Yoshino Date: Wed, 10 Apr 2019 13:38:45 +0000 (-0400) Subject: no bug - Fix Triage Owner list throwing if predefined product is not found X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=3879da6db991de02b93c6bd659d9301a2e30423b;p=thirdparty%2Fbugzilla.git no bug - Fix Triage Owner list throwing if predefined product is not found --- diff --git a/extensions/BMO/lib/Reports/Triage.pm b/extensions/BMO/lib/Reports/Triage.pm index d19f459e8..6e78ba3b2 100644 --- a/extensions/BMO/lib/Reports/Triage.pm +++ b/extensions/BMO/lib/Reports/Triage.pm @@ -57,11 +57,9 @@ sub unconfirmed { ? $input->{'component'} : [$input->{'component'}]; foreach my $component_name (@$ra_components) { - my $component - = Bugzilla::Component->new({name => $component_name, product => $product}) - || ThrowUserError('invalid_object', - {object => 'Component', value => $component_name}); - push @component_ids, $component->id; + next unless my $component = Bugzilla::Component->new( + {name => $component_name, product => $product, cache => 1}); + push(@component_ids, $component->id); } } @@ -261,7 +259,8 @@ sub owners { my @product_names = $input->{product} ? ($input->{product}) : DEFAULT_OWNER_PRODUCTS; foreach my $name (@product_names) { - push(@products, Bugzilla::Product->check({name => $name})); + next unless my $product = Bugzilla::Product->new({name => $name, cache => 1}); + push(@products, $product); } } @@ -272,9 +271,9 @@ sub owners { ? $input->{'component'} : [$input->{'component'}]; foreach my $component_name (@$ra_components) { - my $component = Bugzilla::Component->check( - {name => $component_name, product => $products[0]}); - push @component_ids, $component->id; + next unless my $component = Bugzilla::Component->new( + {name => $component_name, product => $products[0], cache => 1}); + push(@component_ids, $component->id); } }