From dcebd61ddec2a5b19524ac88b60c2f75ce0f171a Mon Sep 17 00:00:00 2001
From: Joseph Schorr <josephschorr@users.noreply.github.com>
Date: Tue, 26 Jul 2016 16:06:41 -0700
Subject: [PATCH] Add support for custom commit links in the build UI

Fixes #1251
---
 static/directives/triggered-build-description.html |  4 ++--
 static/js/services/trigger-service.js              | 11 +++++++++++
 2 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/static/directives/triggered-build-description.html b/static/directives/triggered-build-description.html
index 6936012a4..c2fb109de 100644
--- a/static/directives/triggered-build-description.html
+++ b/static/directives/triggered-build-description.html
@@ -27,8 +27,8 @@
     <!-- Full Commit Information -->
     <div class="tbd-content" ng-switch-when="fullcommit">
       <div class="commit-message">
-        <span class="anchor" is-only-text="!TriggerService.getFullLinkTemplate(build, 'commit')"
-              ng-safenewtab href="{{ TriggerService.getFullLinkTemplate(build, 'commit').replace('{sha}', TriggerService.getCommitSHA(build.trigger_metadata)) }}">
+        <span class="anchor" is-only-text="!TriggerService.getCommitUrl(build)"
+              ng-safenewtab href="{{ TriggerService.getCommitUrl(build) }}">
           {{ getMessageSummary(build.trigger_metadata.commit_info.message) }}
         </span>
         <span class="long-description-toggle"
diff --git a/static/js/services/trigger-service.js b/static/js/services/trigger-service.js
index 5826804bc..9c02526a6 100644
--- a/static/js/services/trigger-service.js
+++ b/static/js/services/trigger-service.js
@@ -135,6 +135,17 @@ angular.module('quay').factory('TriggerService', ['UtilService', '$sanitize', 'K
     });
   };
 
+  triggerService.getCommitUrl = function(build) {
+    // Check for a predefined URL first.
+    if (build.trigger_metadata && build.trigger_metadata.commit_info &&
+        build.trigger_metadata.commit_info.url) {
+      return build.trigger_metadata.commit_info.url;
+    }
+
+    return triggerService.getFullLinkTemplate(build, 'commit')
+                         .replace('{sha}', triggerService.getCommitSHA(build.trigger_metadata))
+  };
+
   triggerService.getFullLinkTemplate = function(build, templateName) {
     var name = build.trigger.service;
     var type = triggerTypes[name];