From 28c489579fd0ffe37acf3c10917118bcc320d69f Mon Sep 17 00:00:00 2001
From: Tim Pope <code@tpope.net>
Date: Tue, 1 Dec 2015 03:34:10 -0500
Subject: [PATCH] Add another guard to :Gbrowse support

Now that I've actually had time to assess things, it's clear this
parameter is only used as a way to convey the branch name and makes zero
sense.  So, it's going to get dropped sooner or later.
---
 autoload/rhubarb.vim | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/autoload/rhubarb.vim b/autoload/rhubarb.vim
index d2d2dd3..efa1036 100644
--- a/autoload/rhubarb.vim
+++ b/autoload/rhubarb.vim
@@ -184,7 +184,7 @@ endfunction
 " Section: Fugitive :Gbrowse support
 
 function! rhubarb#fugitive_url(opts, ...) abort
-  if a:0 || type(a:opts) != type({}) || !has_key(a:opts, 'repo')
+  if a:0 || type(a:opts) != type({}) || !has_key(a:opts, 'repo') || !has_key(a:opts, 'revision')
     return ''
   endif
   let root = s:homepage_for_url(get(a:opts, 'remote'))