Merge pull request #940 from reitermarkus/fix-untapped-installation

Fix cask installation from untapped Tap not working.

Library/Homebrew/cask/lib/hbc/source/tapped_qualified.rb

@@ -2,11 +2,21 @@ require "hbc/source/tapped"
 
 class Hbc::Source::TappedQualified < Hbc::Source::Tapped
   def self.me?(query)
-    !Hbc::QualifiedToken.parse(query).nil? && path_for_query(query).exist?
+    return if (tap = tap_for_query(query)).nil?
+
+    tap.installed? && path_for_query(query).exist?
+  end
+
+  def self.tap_for_query(query)
+    qualified_token = Hbc::QualifiedToken.parse(query)
+    return if qualified_token.nil?
+
+    user, repo, token = qualified_token
+    Tap.fetch(user, repo)
   end
 
   def self.path_for_query(query)
     user, repo, token = Hbc::QualifiedToken.parse(query)
-    Tap.new(user, repo).cask_dir.join(token.sub(%r{(\.rb)?$}i, ".rb"))
+    Tap.fetch(user, repo).cask_dir.join(token.sub(%r{(\.rb)?$}i, ".rb"))
   end
 end

Library/Homebrew/cask/lib/hbc/source/untapped_qualified.rb

@@ -1,11 +1,10 @@
 require "hbc/source/tapped_qualified"
 
 class Hbc::Source::UntappedQualified < Hbc::Source::TappedQualified
-  def self.path_for_query(query)
+  def self.me?(query)
-    user, repo, token = Hbc::QualifiedToken.parse(query)
+    return if (tap = tap_for_query(query)).nil?
 
-    tap = Tap.fetch(user, repo)
+    tap.install
-    tap.install unless tap.installed?
+    tap.installed? && path_for_query(query).exist?
-    tap.cask_dir.join(token.sub(%r{(\.rb)?$}i, ".rb"))
   end
 end