Hide error notifications when form is resubmitted

Author: spadgett

app/scripts/controllers/createRoute.js

@@ -46,6 +46,15 @@ angular.module('openshiftConsole')
       }
     ];
 
+    var hideErrorNotifications = function() {
+      NotificationsService.hideNotification("create-route-error");
+    };
+
+    $scope.cancel = function() {
+      hideErrorNotifications();
+      $window.history.back();
+    };
+
     ProjectsService
       .get($routeParams.project)
       .then(_.spread(function(project, context) {
@@ -82,6 +91,7 @@ angular.module('openshiftConsole')
 
         $scope.createRoute = function() {
           if ($scope.createRouteForm.$valid) {
+            hideErrorNotifications();
             $scope.disableInputs = true;
             var serviceName = $scope.routing.to.service.metadata.name;
             var labels = keyValueEditorUtils.mapEntries(keyValueEditorUtils.compactEntries($scope.labels));
@@ -111,6 +121,7 @@ angular.module('openshiftConsole')
                 $scope.disableInputs = false;
                 NotificationsService.addNotification({
                   type: "error",
+                  id: "create-route-error",
                   message: "An error occurred creating the route.",
                   details: $filter('getErrorDetails')(result)
                 });

app/scripts/controllers/edit/route.js

@@ -41,6 +41,10 @@ angular.module('openshiftConsole')
       title: "Edit"
     }];
 
+    $scope.hideErrorNotifications = function() {
+      NotificationsService.hideNotification("edit-route-error");
+    };
+
     ProjectsService
       .get($routeParams.project)
       .then(_.spread(function(project, context) {
@@ -158,19 +162,21 @@ angular.module('openshiftConsole')
 
         $scope.updateRoute = function() {
           if ($scope.form.$valid) {
+            $scope.hideErrorNotifications();
             $scope.disableInputs = true;
             var updated = updateRouteFields();
             DataService.update('routes', $scope.routeName, updated, context)
               .then(function() { // Success
                 NotificationsService.addNotification({
-                    type: "success",
-                    message: "Route " + $scope.routeName + " was successfully updated."
+                  type: "success",
+                  message: "Route " + $scope.routeName + " was successfully updated."
                 });
                 $location.path($scope.routeURL);
               }, function(response) { // Failure
                 $scope.disableInputs = false;
                 NotificationsService.addNotification({
                   type: "error",
+                  id: "edit-route-error",
                   message: "An error occurred updating route " + $scope.routeName + ".",
                   details: $filter('getErrorDetails')(response)
                 });

app/views/create-route.html

@@ -39,7 +39,7 @@ <h1>Create Route</h1>
                               ng-click="createRoute()"
                               ng-disabled="createRouteForm.$invalid || disableInputs || !createRoute"
                               value="">Create</button>
-                          <a class="btn btn-default btn-lg" href="#" back>Cancel</a>
+                          <a class="btn btn-default btn-lg" href="" ng-click="cancel()">Cancel</a>
                         </div>
                       </fieldset>
                     </div>

app/views/edit/route.html

@@ -30,7 +30,9 @@ <h1>Edit Route {{routeName}}</h1>
                         ng-click="updateRoute()"
                         ng-disabled="form.$invalid || disableInputs"
                         value="">Save</button>
-                    <a class="btn btn-default btn-lg" ng-href="{{routeURL}}">Cancel</a>
+                    <a class="btn btn-default btn-lg"
+                        ng-click="hideErrorNotifications()"
+                        ng-href="{{routeURL}}">Cancel</a>
                   </div>
                 </fieldset>
               </form>

dist/scripts/scripts.js

@@ -7894,7 +7894,7 @@ n(o + " could not be loaded.", m(a));
 }) :void j.toErrorPage("You do not have authority to update " + o + ".", "access_denied");
 }));
 } ]), angular.module("openshiftConsole").controller("EditRouteController", [ "$filter", "$location", "$routeParams", "$scope", "AlertMessageService", "AuthorizationService", "DataService", "Navigate", "NotificationsService", "ProjectsService", "RoutesService", function(a, b, c, d, e, f, g, h, i, j, k) {
-d.alerts = {}, d.renderOptions = {
+d.renderOptions = {
 hideFilterWidget:!0
 }, d.projectName = c.project, d.routeName = c.route, d.loading = !0, d.routeURL = h.resourceURL(d.routeName, "Route", d.projectName), d.breadcrumbs = [ {
 title:d.projectName,
@@ -7907,7 +7907,9 @@ title:d.routeName,
 link:d.routeURL
 }, {
 title:"Edit"
-} ], j.get(c.project).then(_.spread(function(e, j) {
+} ], d.hideErrorNotifications = function() {
+i.hideNotification("edit-route-error");
+}, j.get(c.project).then(_.spread(function(e, j) {
 if (d.project = e, d.breadcrumbs[0].title = a("displayName")(e), !f.canI("routes", "update", c.project)) return void h.toErrorPage("You do not have authority to update route " + c.routeName + ".", "access_denied");
 var l, m = a("orderByDisplayName");
 g.get("routes", d.routeName, j).then(function(a) {
@@ -7953,7 +7955,7 @@ weight:a.weight
 };
 d.updateRoute = function() {
 if (d.form.$valid) {
-d.disableInputs = !0;
+d.hideErrorNotifications(), d.disableInputs = !0;
 var c = n();
 g.update("routes", d.routeName, c, j).then(function() {
 i.addNotification({
@@ -7963,6 +7965,7 @@ message:"Route " + d.routeName + " was successfully updated."
 }, function(b) {
 d.disableInputs = !1, i.addNotification({
 type:"error",
+id:"edit-route-error",
 message:"An error occurred updating route " + d.routeName + ".",
 details:a("getErrorDetails")(b)
 });
@@ -8717,7 +8720,7 @@ details:c("getErrorDetails")(b)
 });
 }));
 } ]), angular.module("openshiftConsole").controller("CreateRouteController", [ "$filter", "$routeParams", "$scope", "$window", "ApplicationGenerator", "AuthorizationService", "DataService", "Navigate", "NotificationsService", "ProjectsService", "keyValueEditorUtils", function(a, b, c, d, e, f, g, h, i, j, k) {
-c.alerts = {}, c.renderOptions = {
+c.renderOptions = {
 hideFilterWidget:!0
 }, c.projectName = b.project, c.serviceName = b.service, c.labels = [], c.routing = {
 name:c.serviceName || ""
@@ -8729,11 +8732,17 @@ title:"Routes",
 link:"project/" + c.projectName + "/browse/routes"
 }, {
 title:"Create Route"
-} ], j.get(b.project).then(_.spread(function(j, l) {
+} ];
+var l = function() {
+i.hideNotification("create-route-error");
+};
+c.cancel = function() {
+l(), d.history.back();
+}, j.get(b.project).then(_.spread(function(j, m) {
 if (c.project = j, c.breadcrumbs[0].title = a("displayName")(j), !f.canI("routes", "create", b.project)) return void h.toErrorPage("You do not have authority to create routes in project " + b.project + ".", "access_denied");
-var m = a("orderByDisplayName");
-g.list("services", l).then(function(a) {
-c.services = m(a.by("metadata.name")), c.routing.to = {}, c.routing.to.service = _.find(c.services, function(a) {
+var n = a("orderByDisplayName");
+g.list("services", m).then(function(a) {
+c.services = n(a.by("metadata.name")), c.routing.to = {}, c.routing.to.service = _.find(c.services, function(a) {
 return !c.serviceName || a.metadata.name === c.serviceName;
 });
 }), c.copyServiceLabels = function() {
@@ -8746,22 +8755,23 @@ value:a
 });
 }, c.createRoute = function() {
 if (c.createRouteForm.$valid) {
-c.disableInputs = !0;
+l(), c.disableInputs = !0;
 var b = c.routing.to.service.metadata.name, f = k.mapEntries(k.compactEntries(c.labels)), h = e.createRoute(c.routing, b, f), j = _.get(c, "routing.alternateServices", []);
 _.isEmpty(j) || (h.spec.to.weight = _.get(c, "routing.to.weight"), h.spec.alternateBackends = _.map(j, function(a) {
 return {
 kind:"Service",
 name:_.get(a, "service.metadata.name"),
 weight:a.weight
 };
-})), g.create("routes", null, h, l).then(function() {
+})), g.create("routes", null, h, m).then(function() {
 i.addNotification({
 type:"success",
 message:"Route " + h.metadata.name + " was successfully created."
 }), d.history.back();
 }, function(b) {
 c.disableInputs = !1, i.addNotification({
 type:"error",
+id:"create-route-error",
 message:"An error occurred creating the route.",
 details:a("getErrorDetails")(b)
 });

dist/scripts/templates.js

@@ -4694,7 +4694,6 @@ angular.module('openshiftConsoleTemplates', []).run(['$templateCache', function(
     "<div class=\"row\">\n" +
     "<div class=\"col-md-10 col-md-offset-1\">\n" +
     "<breadcrumbs breadcrumbs=\"breadcrumbs\"></breadcrumbs>\n" +
-    "<alerts alerts=\"alerts\"></alerts>\n" +
     "<div class=\"mar-top-xl\">\n" +
     "<h1>Create Route</h1>\n" +
     "<div class=\"help-block\">\n" +
@@ -4711,7 +4710,7 @@ angular.module('openshiftConsoleTemplates', []).run(['$templateCache', function(
     "<a href=\"\" ng-click=\"copyServiceLabels()\">Copy Service Labels</a>\n" +
     "<div class=\"button-group gutter-top gutter-bottom\">\n" +
     "<button type=\"submit\" class=\"btn btn-primary btn-lg\" ng-click=\"createRoute()\" ng-disabled=\"createRouteForm.$invalid || disableInputs || !createRoute\" value=\"\">Create</button>\n" +
-    "<a class=\"btn btn-default btn-lg\" href=\"#\" back>Cancel</a>\n" +
+    "<a class=\"btn btn-default btn-lg\" href=\"\" ng-click=\"cancel()\">Cancel</a>\n" +
     "</div>\n" +
     "</fieldset>\n" +
     "</div>\n" +
@@ -9789,7 +9788,7 @@ angular.module('openshiftConsoleTemplates', []).run(['$templateCache', function(
     "</osc-routing>\n" +
     "<div class=\"button-group gutter-top gutter-bottom\">\n" +
     "<button type=\"submit\" class=\"btn btn-primary btn-lg\" ng-click=\"updateRoute()\" ng-disabled=\"form.$invalid || disableInputs\" value=\"\">Save</button>\n" +
-    "<a class=\"btn btn-default btn-lg\" ng-href=\"{{routeURL}}\">Cancel</a>\n" +
+    "<a class=\"btn btn-default btn-lg\" ng-click=\"hideErrorNotifications()\" ng-href=\"{{routeURL}}\">Cancel</a>\n" +
     "</div>\n" +
     "</fieldset>\n" +
     "</form>\n" +