From 214344d7b71264f37905de19a0be2bf4b3cd922c Mon Sep 17 00:00:00 2001
From: Jason Etheridge <jason@EquinoxInitiative.org>
Date: Mon, 11 Nov 2019 13:53:54 -0500
Subject: [PATCH] some form validation

Signed-off-by: Jason Etheridge <jason@EquinoxInitiative.org>
---
 .../src/app/staff/acq/search/picklist-clone-dialog.component.html   | 6 +++++-
 .../src/app/staff/acq/search/picklist-create-dialog.component.html  | 6 +++++-
 .../src/app/staff/acq/search/picklist-merge-dialog.component.html   | 5 ++++-
 3 files changed, 14 insertions(+), 3 deletions(-)

diff --git a/Open-ILS/src/eg2/src/app/staff/acq/search/picklist-clone-dialog.component.html b/Open-ILS/src/eg2/src/app/staff/acq/search/picklist-clone-dialog.component.html
index 460e0c0061..662aaca27b 100644
--- a/Open-ILS/src/eg2/src/app/staff/acq/search/picklist-clone-dialog.component.html
+++ b/Open-ILS/src/eg2/src/app/staff/acq/search/picklist-clone-dialog.component.html
@@ -1,4 +1,5 @@
 <ng-template #dialogContent>
+<form class="form-validated">
   <div class="modal-header bg-info">
     <h3 class="modal-title" i18n>Clone Selection List: {{leadListName}}</h3>
     <button type="button" class="close"
@@ -8,15 +9,18 @@
   </div>
   <div class="modal-body">
     <h4 i18n>Selection list name:</h4>
-    <input type="text" id="create-picklist-name"
+    <input type="text" id="create-picklist-name" required
+      [ngModelOptions]="{standalone: true}"
       class="form-control" [(ngModel)]="selectionListName">
   </div>
   <div class="modal-footer">
     <button type="button" class="btn btn-success"
+      [disabled]="!selectionListName"
       (click)="cloneList()" i18n>Clone</button>
     <button type="button" class="btn btn-warning"
       (click)="close()" i18n>Cancel</button>
   </div>
+</form>
 </ng-template>
 <eg-alert-dialog #fail i18n-dialogBody
   dialogBody="Could not create this selection list.">
diff --git a/Open-ILS/src/eg2/src/app/staff/acq/search/picklist-create-dialog.component.html b/Open-ILS/src/eg2/src/app/staff/acq/search/picklist-create-dialog.component.html
index 5a130891b0..b4ac8ad665 100644
--- a/Open-ILS/src/eg2/src/app/staff/acq/search/picklist-create-dialog.component.html
+++ b/Open-ILS/src/eg2/src/app/staff/acq/search/picklist-create-dialog.component.html
@@ -1,4 +1,5 @@
 <ng-template #dialogContent>
+<form class="form-validated">
   <div class="modal-header bg-info">
     <h3 class="modal-title" i18n>Create New Selection List</h3>
     <button type="button" class="close"
@@ -8,15 +9,18 @@
   </div>
   <div class="modal-body">
     <h4 i18n>Selection list name:</h4>
-    <input type="text" id="create-picklist-name"
+    <input type="text" id="create-picklist-name" required
+      [ngModelOptions]="{standalone: true}" required
       class="form-control col-lg-7" [(ngModel)]="selectionListName">
   </div>
   <div class="modal-footer">
     <button type="button" class="btn btn-success"
+      [disabled]="!selectionListName"
       (click)="createList()" i18n>Create</button>
     <button type="button" class="btn btn-warning"
       (click)="close()" i18n>Cancel</button>
   </div>
+</form>
 </ng-template>
 <eg-alert-dialog #fail i18n-dialogBody
   dialogBody="Could not create this selection list.">
diff --git a/Open-ILS/src/eg2/src/app/staff/acq/search/picklist-merge-dialog.component.html b/Open-ILS/src/eg2/src/app/staff/acq/search/picklist-merge-dialog.component.html
index 0569d65509..6cdf54e72f 100644
--- a/Open-ILS/src/eg2/src/app/staff/acq/search/picklist-merge-dialog.component.html
+++ b/Open-ILS/src/eg2/src/app/staff/acq/search/picklist-merge-dialog.component.html
@@ -1,4 +1,5 @@
 <ng-template #dialogContent>
+<form class="form-validated">
   <div class="modal-header bg-info">
     <h3 class="modal-title" i18n>Merge Selection Lists</h3>
     <button type="button" class="close"
@@ -8,7 +9,7 @@
   </div>
   <div class="modal-body">
     <h4 i18n>Lead Selection List:</h4>
-      <select [(ngModel)]="leadList">
+      <select [(ngModel)]="leadList" [ngModelOptions]="{standalone: true}" required>
         <option *ngFor="let list of selectedLists"
           value="{{list.id()}}">{{list.name()}}</option>
       </select>
@@ -19,10 +20,12 @@
   </div>
   <div class="modal-footer">
     <button type="button" class="btn btn-success"
+      [disabled]="!leadList"
       (click)="mergeLists()" i18n>Merge</button>
     <button type="button" class="btn btn-warning"
       (click)="close()" i18n>Cancel</button>
   </div>
+</form>
 </ng-template>
 <eg-alert-dialog #fail i18n-dialogBody
   dialogBody="Could not merge the selection lists.">
-- 
2.11.0