From 7861c1d1d4a1b32619c0e36f985ecc4ff8c7ab8b Mon Sep 17 00:00:00 2001
From: dklawren <dklawren@users.noreply.github.com>
Date: Thu, 11 Jan 2018 09:55:24 -0500
Subject: [PATCH] Bug 1429600 - Update Revision.pm type checking to treat bug
 id as a simple string or undefined

---
 extensions/PhabBugz/lib/Revision.pm | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/extensions/PhabBugz/lib/Revision.pm b/extensions/PhabBugz/lib/Revision.pm
index 59312b39d..1c91e3d85 100644
--- a/extensions/PhabBugz/lib/Revision.pm
+++ b/extensions/PhabBugz/lib/Revision.pm
@@ -36,7 +36,7 @@ my $SearchResult = Dict[
         repositoryPHID    => Maybe[Str],
         status            => HashRef,
         summary           => Str,
-        "bugzilla.bug-id" => Str
+        "bugzilla.bug-id" => Maybe[Str]
     ],
     attachments => Dict[
         reviewers => Dict[
@@ -93,7 +93,10 @@ sub _load {
 
     # Some values in Phabricator for bug ids may have been saved
     # white whitespace so we remove any here just in case.
-    $result->{fields}->{'bugzilla.bug-id'} = trim($result->{fields}->{'bugzilla.bug-id'});
+    $result->{fields}->{'bugzilla.bug-id'}
+        = $result->{fields}->{'bugzilla.bug-id'}
+          ? trim($result->{fields}->{'bugzilla.bug-id'})
+          : "";
 
     return $result;
 }
-- 
2.47.3