summaryrefslogtreecommitdiffstats
path: root/extensions
diff options
context:
space:
mode:
Diffstat (limited to 'extensions')
-rw-r--r--extensions/BMO/Extension.pm12
-rw-r--r--extensions/ComponentWatching/Extension.pm14
2 files changed, 14 insertions, 12 deletions
diff --git a/extensions/BMO/Extension.pm b/extensions/BMO/Extension.pm
index c800ade3d..1bb83c977 100644
--- a/extensions/BMO/Extension.pm
+++ b/extensions/BMO/Extension.pm
@@ -263,12 +263,12 @@ sub bugmail_recipients {
if (@$diffs) {
# Changed bug
foreach my $ref (@$diffs) {
- my ($who, $whoname, $what, $when,
- $old, $new, $attachid, $fieldname) = (@$ref);
-
- if ($fieldname eq "bug_group") {
- _cc_if_special_group($old, $recipients);
- _cc_if_special_group($new, $recipients);
+ next if !(exists $ref->{'new'}
+ && exists $ref->{'old'}
+ && exists $ref->{'field_name'});
+ if ($ref->{'field_name'} eq "bug_group") {
+ _cc_if_special_group($ref->{'old'}, $recipients);
+ _cc_if_special_group($ref->{'new'}, $recipients);
}
}
} else {
diff --git a/extensions/ComponentWatching/Extension.pm b/extensions/ComponentWatching/Extension.pm
index d73ce47cd..d39d9cd55 100644
--- a/extensions/ComponentWatching/Extension.pm
+++ b/extensions/ComponentWatching/Extension.pm
@@ -165,9 +165,10 @@ sub bugmail_recipients {
# we need the product to process the component, so scan for that first
my $product;
foreach my $ra (@$diffs) {
- my (undef, undef, undef, undef, $old, $new, undef, $field) = @$ra;
- if ($field eq 'product') {
- $product = Bugzilla::Product->new({ name => $old });
+ next if !(exists $ra->{'old'}
+ && exists $ra->{'field_name'});
+ if ($ra->{'field_name'} eq 'product') {
+ $product = Bugzilla::Product->new({ name => $ra->{'old'} });
$oldProductId = $product->id;
}
}
@@ -175,9 +176,10 @@ sub bugmail_recipients {
$product = Bugzilla::Product->new($oldProductId);
}
foreach my $ra (@$diffs) {
- my (undef, undef, undef, undef, $old, $new, undef, $field) = @$ra;
- if ($field eq 'component') {
- my $component = Bugzilla::Component->new({ name => $old, product => $product });
+ next if !(exists $ra->{'old'}
+ && exists $ra->{'field_name'});
+ if ($ra->{'field_name'} eq 'component') {
+ my $component = Bugzilla::Component->new({ name => $ra->{'old'}, product => $product });
$oldComponentId = $component->id;
}
}