From c4c1eb9f453c49f6aa24e9f771ebf060a2c3ac43 Mon Sep 17 00:00:00 2001 From: dklawren Date: Wed, 3 Jan 2018 10:34:11 -0500 Subject: Bug 1427743 - legacy phabbugz API code errors when trying to set an inactive review flag --- extensions/PhabBugz/lib/WebService.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'extensions') diff --git a/extensions/PhabBugz/lib/WebService.pm b/extensions/PhabBugz/lib/WebService.pm index f8c109612..80a7df9b1 100644 --- a/extensions/PhabBugz/lib/WebService.pm +++ b/extensions/PhabBugz/lib/WebService.pm @@ -183,7 +183,7 @@ sub update_reviewer_statuses { my (@denied_flags, @new_flags, @removed_flags, %accepted_done, $flag_type); foreach my $flag (@{ $attachment->flags }) { next if $flag->type->name ne 'review'; - $flag_type = $flag->type; + $flag_type = $flag->type if $flag->type->is_active; if (any { $flag->setter->id == $_ } @$denied_user_ids) { push(@denied_flags, { id => $flag->id, setter => $flag->setter, status => 'X' }); } @@ -196,7 +196,7 @@ sub update_reviewer_statuses { } } - $flag_type ||= first { $_->name eq 'review' } @{ $attachment->flag_types }; + $flag_type ||= first { $_->name eq 'review' && $_->is_active } @{ $attachment->flag_types }; # Create new flags foreach my $user_id (@$accepted_user_ids) { -- cgit v1.2.3-24-g4f1b