Merge pull request #186 from abf/rosa-build:184-undefined-method-gsub

#184: NoMethodError: undefined method `gsub' for nil:NilClass

app/helpers/git_helper.rb

@@ -5,7 +5,10 @@ module GitHelper
     # node.url(treeish) looks like:
     # - http://0.0.0.0:3000/abf/git@abf.rosalinux.ru:abf/rhel-scripts.git
     # - git://github.com/avokhmin/mdv-scripts.git
-    url = node.url(treeish).gsub(/.git$/, '')
+    # - empty string if ".gitmodules" does not exist
+    url = node.url(treeish)
+    return nil if url.blank?
+    url.gsub!(/.git$/, '')
     if url =~ /^git:/
       url.gsub!(/^git/, 'http')
     elsif str = /git@.*:.*/.match(url)

app/views/projects/git/trees/_show.html.haml

@@ -26,10 +26,12 @@
           - if node.is_a? Grit::Submodule
             .pic= image_tag 'folder-submodule.png'
             .name
-              - url = submodule_url node, @treeish
-              = link_to(node.name, url, :class => 'files-see')
-              = '@'
-              = link_to(node.id[0..6], "#{url}/tree/#{node.id}", :class => 'files-see')
+              - if url = submodule_url(node, @treeish)
+                = link_to(node.name, url, :class => 'files-see')
+                = '@'
+                = link_to(node.id[0..6], "#{url}/tree/#{node.id}", :class => 'files-see')
+              - else
+                = "#{node.name} @ #{node.id[0..6]}"
           - else
             - options = [@project, @treeish, node_path]
             - if node.is_a?(Grit::Tree)