From 892dce8e976039a1eaea6f92f3879167024d59b5 Mon Sep 17 00:00:00 2001
From: Alison <github@bankofknowledge.net>
Date: Fri, 16 Jun 2017 14:42:44 -0500
Subject: [PATCH 1/2] displays error & disables save when no dashboard name

fixes #76 and avoids the same error as #54 (text box not-save)
---
 client/app/pages/dashboards/dashboard.js             |  1 +
 .../app/pages/dashboards/edit-dashboard-dialog.html  | 12 ++++++++----
 client/app/pages/dashboards/edit-dashboard-dialog.js |  4 ++++
 3 files changed, 13 insertions(+), 4 deletions(-)

diff --git a/client/app/pages/dashboards/dashboard.js b/client/app/pages/dashboards/dashboard.js
index 20caeebec0..6b12126255 100644
--- a/client/app/pages/dashboards/dashboard.js
+++ b/client/app/pages/dashboards/dashboard.js
@@ -155,6 +155,7 @@ function DashboardCtrl($rootScope, $routeParams, $location, $timeout, $q, $uibMo
   };
 
   this.editDashboard = () => {
+    this.dashboard.existing_name = this.dashboard.name;
     $uibModal.open({
       component: 'editDashboardDialog',
       resolve: {
diff --git a/client/app/pages/dashboards/edit-dashboard-dialog.html b/client/app/pages/dashboards/edit-dashboard-dialog.html
index f680e2598a..ab0d7c0062 100644
--- a/client/app/pages/dashboards/edit-dashboard-dialog.html
+++ b/client/app/pages/dashboards/edit-dashboard-dialog.html
@@ -1,11 +1,15 @@
 <div class="modal-header">
-  <button type="button" class="close" ng-click="$ctrl.dismiss()" ng-disabled="$ctrl.saveInProgress" aria-hidden="true">&times;</button>
+  <button type="button" class="close" ng-click="$ctrl.closeWithoutSave()" ng-disabled="$ctrl.saveInProgress" aria-hidden="true">&times;</button>
   <h4 class="modal-title">Edit: {{$ctrl.dashboard.name}}</h4>
 </div>
 <div class="modal-body">
   <p>
-    <input type="text" class="form-control" placeholder="Dashboard Name" ng-model="$ctrl.dashboard.name" required>
+    <form name="myForm">
+    <input name="dashboardName" type="text" class="form-control" placeholder="Dashboard Name" ng-model="$ctrl.dashboard.name" required>
+    </form>
   </p>
+  <p ng-if="myForm.dashboardName.$invalid" class="alert alert-warning">A name is required.</p>
+
 
   <div gridster="$ctrl.gridsterOptions" ng-if="$ctrl.items | notEmpty">
     <ul>
@@ -16,6 +20,6 @@ <h4 class="modal-title">Edit: {{$ctrl.dashboard.name}}</h4>
   </div>
 </div>
 <div class="modal-footer">
-  <button type="button" class="btn btn-default" ng-disabled="$ctrl.saveInProgress" ng-click="$ctrl.dismiss()">Close</button>
-  <button type="button" class="btn btn-primary" ng-disabled="$ctrl.saveInProgress" ng-click="$ctrl.saveDashboard()">Save</button>
+  <button type="button" class="btn btn-default" ng-disabled="$ctrl.saveInProgress" ng-click="$ctrl.closeWithoutSave()">Close</button>
+  <button type="button" class="btn btn-primary" ng-disabled="$ctrl.saveInProgress || myForm.dashboardName.$invalid"  ng-click="$ctrl.saveDashboard()">Save</button> 
 </div>
diff --git a/client/app/pages/dashboards/edit-dashboard-dialog.js b/client/app/pages/dashboards/edit-dashboard-dialog.js
index 5d0cc6ee13..70eab4bafd 100644
--- a/client/app/pages/dashboards/edit-dashboard-dialog.js
+++ b/client/app/pages/dashboards/edit-dashboard-dialog.js
@@ -93,6 +93,10 @@ const EditDashboardDialog = {
         Events.record('create', 'dashboard');
       }
     };
+    this.closeWithoutSave = () => {
+      this.dashboard.name = this.dashboard.existing_name;
+      this.dismiss();
+    };
   },
 };
 

From 26ebb54d8147046720c277b88de35849993a1c1f Mon Sep 17 00:00:00 2001
From: Alison <github@bankofknowledge.net>
Date: Fri, 16 Jun 2017 18:49:15 -0500
Subject: [PATCH 2/2] indent update based on PR comment

---
 client/app/pages/dashboards/edit-dashboard-dialog.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/client/app/pages/dashboards/edit-dashboard-dialog.html b/client/app/pages/dashboards/edit-dashboard-dialog.html
index ab0d7c0062..48be84d4e2 100644
--- a/client/app/pages/dashboards/edit-dashboard-dialog.html
+++ b/client/app/pages/dashboards/edit-dashboard-dialog.html
@@ -5,7 +5,7 @@ <h4 class="modal-title">Edit: {{$ctrl.dashboard.name}}</h4>
 <div class="modal-body">
   <p>
     <form name="myForm">
-    <input name="dashboardName" type="text" class="form-control" placeholder="Dashboard Name" ng-model="$ctrl.dashboard.name" required>
+      <input name="dashboardName" type="text" class="form-control" placeholder="Dashboard Name" ng-model="$ctrl.dashboard.name" required>
     </form>
   </p>
   <p ng-if="myForm.dashboardName.$invalid" class="alert alert-warning">A name is required.</p>