From 1f9c1f2e3bcbb8f54e8f2724da3e21b1495cb7c7 Mon Sep 17 00:00:00 2001
From: Samuel Padgett <spadgett@redhat.com>
Date: Tue, 31 Jan 2017 15:43:17 -0500
Subject: [PATCH] Don't link to image stream tags that haven't synced yet

Don't link to image stream tags that haven't synced or had an error
syncing. The page will give a not found error.
---
 app/views/browse/imagestream.html | 5 ++++-
 dist/scripts/templates.js         | 5 ++++-
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/app/views/browse/imagestream.html b/app/views/browse/imagestream.html
index 6f66e8d02d..bfa7aaa80a 100644
--- a/app/views/browse/imagestream.html
+++ b/app/views/browse/imagestream.html
@@ -68,7 +68,10 @@ <h1 class="contains-actions">
                 </tbody>
                 <tbody ng-if="(tagsByName | hashSize) > 0">
                   <tr ng-repeat="tag in tagsByName | orderBy : 'name'">
-                    <td data-title="Tag"><a href="{{imageStream | navigateResourceURL}}/{{tag.name}}">{{tag.name}}</a></td>
+                    <td data-title="Tag">
+                      <a ng-if="tag.status" ng-href="{{imageStream | navigateResourceURL}}/{{tag.name}}">{{tag.name}}</a>
+                      <span ng-if="!tag.status">{{tag.name}}</span>
+                    </td>
                     <td data-title="From">
                       <!-- this value can get long when its an ImageStreamImage. Use max width and class truncate to elide some
                            of the SHA1 for long values. They can still be copied even when elided. -->
diff --git a/dist/scripts/templates.js b/dist/scripts/templates.js
index 64678ed771..030147a6b7 100644
--- a/dist/scripts/templates.js
+++ b/dist/scripts/templates.js
@@ -3171,7 +3171,10 @@ angular.module('openshiftConsoleTemplates', []).run(['$templateCache', function(
     "</tbody>\n" +
     "<tbody ng-if=\"(tagsByName | hashSize) > 0\">\n" +
     "<tr ng-repeat=\"tag in tagsByName | orderBy : 'name'\">\n" +
-    "<td data-title=\"Tag\"><a href=\"{{imageStream | navigateResourceURL}}/{{tag.name}}\">{{tag.name}}</a></td>\n" +
+    "<td data-title=\"Tag\">\n" +
+    "<a ng-if=\"tag.status\" ng-href=\"{{imageStream | navigateResourceURL}}/{{tag.name}}\">{{tag.name}}</a>\n" +
+    "<span ng-if=\"!tag.status\">{{tag.name}}</span>\n" +
+    "</td>\n" +
     "<td data-title=\"From\">\n" +
     "\n" +
     "<span ng-if=\"!tag.spec.from\"><em>pushed</em></span>\n" +