Merge pull request #1 from masaki/normalize_job_url

job url should not contain "//"
kentaro · Nov 8, 2012 · 1fb4b45 · 1fb4b45
2 parents 341351a + 283cd86
commit 1fb4b45
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/lib/ikachan/message.rb b/lib/ikachan/message.rb
@@ -11,7 +11,9 @@ def message
 
       instance = Java::jenkins::model::Jenkins.instance
       root_url = (instance && instance.root_url) || ''
-      msg << " (#{root_url}/#{build.url})"
+
+      url = [ root_url, build.url ].join(root_url[-1] == '/' ? '' : '/')
+      msg << " (#{url})"
 
       "\x03%s%s" % [code.to_s, msg]
     end