From 039975deef0fae8e9d64db7b237e262e4f095eff Mon Sep 17 00:00:00 2001
From: dklawren <dklawren@users.noreply.github.com>
Date: Tue, 9 Jan 2018 12:23:18 -0500
Subject: Bug 1429110: Update Revision.pm type checking to treat bug id as a
 simple string

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

(limited to 'extensions')

diff --git a/extensions/PhabBugz/lib/Revision.pm b/extensions/PhabBugz/lib/Revision.pm
index e229043e2..59312b39d 100644
--- a/extensions/PhabBugz/lib/Revision.pm
+++ b/extensions/PhabBugz/lib/Revision.pm
@@ -22,12 +22,6 @@ use Bugzilla::Extension::PhabBugz::Util qw(
 use Types::Standard -all;
 use Type::Utils;
 
-my $EmptyStr = declare "EmptyStr",
-    as Str,
-    where { length($_) == 0 },
-    inline_as { $_[0]->parent->inline_check($_) . " && length($_) == 0" },
-    message { "String is not empty" };
-
 my $SearchResult = Dict[
     id     => Int,
     type   => Str,
@@ -42,7 +36,7 @@ my $SearchResult = Dict[
         repositoryPHID    => Maybe[Str],
         status            => HashRef,
         summary           => Str,
-        "bugzilla.bug-id" => Int | $EmptyStr,
+        "bugzilla.bug-id" => Str
     ],
     attachments => Dict[
         reviewers => Dict[
@@ -97,6 +91,10 @@ sub _load {
         $result = $result->{result}->{data}->[0];
     }
 
+    # 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'});
+
     return $result;
 }
 
-- 
cgit v1.2.3-24-g4f1b