diff hgext/phabricator.py @ 41164:43fd1947301d

phabricator: teach {phabreview} to work without --amend

There's some possibility of this being wrong (e.g. if the url config was
changed).  But commit messages can be amended too, so that isn't bulletproof
either.  Having something seems better than just returning None.

--- a/hgext/phabricator.py	Sat Jan 05 15:20:33 2019 -0500
+++ b/hgext/phabricator.py	Sat Jan 05 19:50:30 2019 -0500
@@ -997,3 +997,17 @@
             b'url': m.group(b'url'),
             b'id': b"D{}".format(m.group(b'id')),
         })
+    else:
+        tags = ctx.repo().nodetags(ctx.node())
+        for t in tags:
+            if _differentialrevisiontagre.match(t):
+                url = ctx.repo().ui.config(b'phabricator', b'url')
+                if not url.endswith(b'/'):
+                    url += b'/'
+                url += t
+
+                return templateutil.hybriddict({
+                    b'url': url,
+                    b'id': t,
+                })
+    return None