fetch: Gracefully error if task has no git URI set

Rather than aborting with a string processing exception.

Reviewed-by: Thibault Saunier <tsaunier@gnome.org>
Differential Revision: https://phabricator.freedesktop.org/D1237

1 files changed, 6 insertions, 3 deletions

diff --git a/git-phab b/git-phab
index b89f204..aa668ce 100755
--- a/git-phab
+++ b/git-phab
@@ -1487,9 +1487,12 @@ Paste API Token from that page and press <enter>: """ % self.phabricator_uri)
 
         reply = self.phabricator.maniphest.query(ids=[int(self.task[1:])])
 
-        props = list(reply.values())[0]
-        uri = props['auxiliary']['std:maniphest:git:uri-branch']
-        remote, branch = uri.split('#')
+        try:
+            props = list(reply.values())[0]
+            uri = props['auxiliary']['std:maniphest:git:uri-branch']
+            remote, branch = uri.split('#')
+        except:
+            self.die("Git URI is not set on task; cannot fetch it.")
 
         print("Git URI: %s, branch: %s" % (remote, branch))
         self.repo.git.fetch(remote, "%s" % branch)