summaryrefslogtreecommitdiffstats
path: root/extensions/Push/lib
diff options
context:
space:
mode:
authorDavid Walsh <davidwalsh83@gmail.com>2017-07-19 21:06:36 +0200
committerDylan William Hardison <dylan@hardison.net>2017-07-19 21:06:36 +0200
commitbd5ae5e8e17f448e77766bbb1fdd09ba5e9230ec (patch)
tree807f23f00d525b51d267287d3317b516a1c8b441 /extensions/Push/lib
parentfa9970f7a6545fd9ed81a2fef23e5453db46bbec (diff)
downloadbugzilla-bd5ae5e8e17f448e77766bbb1fdd09ba5e9230ec.tar.gz
bugzilla-bd5ae5e8e17f448e77766bbb1fdd09ba5e9230ec.tar.xz
Bug 1374396 - Add initial push connector for Phabricator
Diffstat (limited to 'extensions/Push/lib')
-rw-r--r--extensions/Push/lib/Connector/Phabricator.pm16
1 files changed, 2 insertions, 14 deletions
diff --git a/extensions/Push/lib/Connector/Phabricator.pm b/extensions/Push/lib/Connector/Phabricator.pm
index 89ca41a2d..5c9cabe8b 100644
--- a/extensions/Push/lib/Connector/Phabricator.pm
+++ b/extensions/Push/lib/Connector/Phabricator.pm
@@ -37,25 +37,13 @@ sub should_send {
return 0 unless Bugzilla->params->{phabricator_enabled};
- if (!( $message->routing_key
- =~ /^(?:attachment|bug)\.modify:.*\bbug_group\b/
- )
- )
- {
- return 0;
- }
+ return 0 unless $message->routing_key =~ /^(?:attachment|bug)\.modify:.*\bbug_group\b/;
my $data = $message->payload_decoded;
my $bug_data = $self->_get_bug_data($data) || return 0;
my $bug = Bugzilla::Bug->new( { id => $bug_data->{id}, cache => 1 } );
- my $has_phab_stub_attachment
- = $bug->has_attachment_with_mimetype(PHAB_CONTENT_TYPE);
-
- if ($has_phab_stub_attachment) {
- return 1;
- }
- return 0;
+ return $bug->has_attachment_with_mimetype(PHAB_CONTENT_TYPE);
}
sub send {