Bug 1509142 - Should not display the 'Reveal Secret' link when secrets without 'data' field

Author: jhadvig

app/views/browse/secret.html

@@ -16,7 +16,7 @@ <h1 class="contains-actions">
               We'll use this as a best-effort check to see if we should show the
               button.
             -->
-            <button ng-if="project.metadata.name | canIAddToProject"
+            <button ng-if="(project.metadata.name | canIAddToProject) && secret.data"
                     type="button"
                     class="btn btn-default hidden-xs"
                     ng-click="addToApplication()">
@@ -32,7 +32,7 @@ <h1 class="contains-actions">
                class="dropdown-toggle actions-dropdown-kebab visible-xs-inline"
                data-toggle="dropdown"><i class="fa fa-ellipsis-v" aria-hidden="true"></i><span class="sr-only">Actions</span></a>
             <ul class="dropdown-menu dropdown-menu-right actions action-button">
-              <li ng-if="project.metadata.name | canIAddToProject" class="visible-xs">
+              <li ng-if="(project.metadata.name | canIAddToProject) && secret.data" class="visible-xs">
                 <a href="" role="button" ng-click="addToApplication()">Add to Application</a>
               </li>
               <li ng-if="'secrets' | canI : 'update'">
@@ -61,7 +61,7 @@ <h1 class="contains-actions">
           <div class="resource-details">
             <h2 class="mar-top-none">
               {{secret.type}}
-              <small class="mar-left-sm"><a href="" ng-click="view.showSecret = !view.showSecret">{{view.showSecret ? "Hide" : "Reveal"}} Secret</a></small>
+              <small class="mar-left-sm"><a href="" ng-if="secret.data" ng-click="view.showSecret = !view.showSecret">{{view.showSecret ? "Hide" : "Reveal"}} Secret</a></small>
             </h2>
             <dl class="secret-data left">
               <div ng-repeat="(secretDataName, secretData) in decodedSecretData" class="image-source-item">
@@ -90,6 +90,10 @@ <h2 class="mar-top-none">
                 </div>
               </div>
             </dl>
+            <div ng-if="!secret.data" class="empty-state-message text-center">
+              <h2>No data.</h2>
+              <p>This secret has no data.</p>
+            </div>
             <annotations annotations="secret.metadata.annotations"></annotations>
           </div>
         </div><!-- /col-* -->

dist/scripts/templates.js

@@ -3576,7 +3576,7 @@ angular.module('openshiftConsoleTemplates', []).run(['$templateCache', function(
     "<h1 class=\"contains-actions\">\n" +
     "<div class=\"pull-right dropdown\">\n" +
     "\n" +
-    "<button ng-if=\"project.metadata.name | canIAddToProject\" type=\"button\" class=\"btn btn-default hidden-xs\" ng-click=\"addToApplication()\">\n" +
+    "<button ng-if=\"(project.metadata.name | canIAddToProject) && secret.data\" type=\"button\" class=\"btn btn-default hidden-xs\" ng-click=\"addToApplication()\">\n" +
     "Add to Application\n" +
     "</button>\n" +
     "<button type=\"button\" class=\"dropdown-toggle btn btn-default actions-dropdown-btn hidden-xs\" data-toggle=\"dropdown\" ng-hide=\"!('secrets' | canIDoAny)\">\n" +
@@ -3586,7 +3586,7 @@ angular.module('openshiftConsoleTemplates', []).run(['$templateCache', function(
     "\n" +
     "<a href=\"\" ng-if=\"project.metadata.name | canIAddToProject\" class=\"dropdown-toggle actions-dropdown-kebab visible-xs-inline\" data-toggle=\"dropdown\"><i class=\"fa fa-ellipsis-v\" aria-hidden=\"true\"></i><span class=\"sr-only\">Actions</span></a>\n" +
     "<ul class=\"dropdown-menu dropdown-menu-right actions action-button\">\n" +
-    "<li ng-if=\"project.metadata.name | canIAddToProject\" class=\"visible-xs\">\n" +
+    "<li ng-if=\"(project.metadata.name | canIAddToProject) && secret.data\" class=\"visible-xs\">\n" +
     "<a href=\"\" role=\"button\" ng-click=\"addToApplication()\">Add to Application</a>\n" +
     "</li>\n" +
     "<li ng-if=\"'secrets' | canI : 'update'\">\n" +
@@ -3611,7 +3611,7 @@ angular.module('openshiftConsoleTemplates', []).run(['$templateCache', function(
     "<div class=\"resource-details\">\n" +
     "<h2 class=\"mar-top-none\">\n" +
     "{{secret.type}}\n" +
-    "<small class=\"mar-left-sm\"><a href=\"\" ng-click=\"view.showSecret = !view.showSecret\">{{view.showSecret ? \"Hide\" : \"Reveal\"}} Secret</a></small>\n" +
+    "<small class=\"mar-left-sm\"><a href=\"\" ng-if=\"secret.data\" ng-click=\"view.showSecret = !view.showSecret\">{{view.showSecret ? \"Hide\" : \"Reveal\"}} Secret</a></small>\n" +
     "</h2>\n" +
     "<dl class=\"secret-data left\">\n" +
     "<div ng-repeat=\"(secretDataName, secretData) in decodedSecretData\" class=\"image-source-item\">\n" +
@@ -3636,6 +3636,10 @@ angular.module('openshiftConsoleTemplates', []).run(['$templateCache', function(
     "</div>\n" +
     "</div>\n" +
     "</dl>\n" +
+    "<div ng-if=\"!secret.data\" class=\"empty-state-message text-center\">\n" +
+    "<h2>No data.</h2>\n" +
+    "<p>This secret has no data.</p>\n" +
+    "</div>\n" +
     "<annotations annotations=\"secret.metadata.annotations\"></annotations>\n" +
     "</div>\n" +
     "</div>\n" +