Don't fail Expm.Package.public_homepage if there is no homepage or re…

…po URL
yrashk · Jun 14, 2013 · 2c7c8ab · 2c7c8ab
1 parent f97cc34
commit 2c7c8ab
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/lib/package.ex b/lib/package.ex
@@ -143,15 +143,19 @@ defrecord Expm.Package,
       github_repo = Enum.find(repositories(package), fn(r) -> 
                                 Regex.match?(%r(.*github.com/.+), r[:git])
                               end)
-      path = Regex.replace(%r{.*github.com/(.+)$},github_repo[:r],"\\1")
-      path = Regex.replace(%r{(.+)(\.git)$}, path, "\\1")
-      github_repo = Keyword.put github_repo, :github, path
+      unless nil?(github_repo) do
+        path = Regex.replace(%r{.*github.com/(.+)$},github_repo[:r],"\\1")
+        path = Regex.replace(%r{(.+)(\.git)$}, path, "\\1")
+        github_repo = Keyword.put github_repo, :github, path
+      end
     end
     cond do
      not nil?(homepage(package)) ->
        homepage(package)
      not nil?(github_repo) ->
        "https://github.com/#{github_repo[:github]}"
+     true ->
+       ""
     end
   end