From 31d872084497dd4b3c1e0eb81f1626a70783bf41 Mon Sep 17 00:00:00 2001
From: Bill Erickson <berickxx@gmail.com>
Date: Thu, 3 May 2018 12:32:56 -0400
Subject: [PATCH] LP#1775466 Billing types; grid additions, misc

Signed-off-by: Bill Erickson <berickxx@gmail.com>
---
 .../eg2/src/app/share/dialog/dialog.component.ts   |  1 +
 .../app/share/fm-editor/fm-editor.component.html   |  5 +++--
 .../src/app/share/grid/grid-toolbar.component.html |  8 +++++++
 .../src/app/share/grid/grid-toolbar.component.ts   |  3 +++
 .../src/eg2/src/app/share/grid/grid.component.html |  5 +++--
 .../src/eg2/src/app/share/grid/grid.component.ts   |  4 +++-
 Open-ILS/src/eg2/src/app/share/grid/grid.module.ts |  7 +++---
 .../src/eg2/src/app/share/grid/grid.service.ts     |  4 ++++
 .../app/share/org-select/org-select.component.ts   |  3 +++
 .../eg2/src/app/share/string/string.component.ts   |  3 +++
 .../server/config/billing_type.component.html      | 10 ++++++++-
 .../admin/server/config/billing_type.component.ts  | 26 +++++++++++++++++-----
 12 files changed, 64 insertions(+), 15 deletions(-)

diff --git a/Open-ILS/src/eg2/src/app/share/dialog/dialog.component.ts b/Open-ILS/src/eg2/src/app/share/dialog/dialog.component.ts
index 1f979fcd7e..e1b3e42103 100644
--- a/Open-ILS/src/eg2/src/app/share/dialog/dialog.component.ts
+++ b/Open-ILS/src/eg2/src/app/share/dialog/dialog.component.ts
@@ -56,6 +56,7 @@ export class EgDialogComponent implements OnInit {
                     this.modalRef = null;
                 },
                 (result) => {
+                    console.log('dialog closed with ' + result);
                     reject(result);
                     this.modalRef = null;
                 }
diff --git a/Open-ILS/src/eg2/src/app/share/fm-editor/fm-editor.component.html b/Open-ILS/src/eg2/src/app/share/fm-editor/fm-editor.component.html
index f3e7dadc53..e509ce1255 100644
--- a/Open-ILS/src/eg2/src/app/share/fm-editor/fm-editor.component.html
+++ b/Open-ILS/src/eg2/src/app/share/fm-editor/fm-editor.component.html
@@ -8,7 +8,7 @@
     </button>
   </div>
   <div class="modal-body">
-    <form role="form" class="form-validated">
+    <form #fmEditForm="ngForm" role="form" class="form-validated">
       <div class="form-group row" *ngFor="let field of fields">
         <div class="col-lg-3 offset-lg-1">
           <label for="rec-{{field.name}}">{{field.label}}</label>
@@ -118,7 +118,8 @@
   <div class="modal-footer">
     <button type="button" class="btn btn-success" *ngIf="mode == 'view'"
       (click)="close()" i18n>Close</button>
-    <button type="button" class="btn btn-info" *ngIf="mode != 'view'"
+    <button type="button" class="btn btn-info" 
+      [disabled]="fmEditForm.invalid" *ngIf="mode != 'view'"
       (click)="save()" i18n>Save</button>
     <button type="button" class="btn btn-warning ml-2" *ngIf="mode != 'view'"
       (click)="cancel()" i18n>Cancel</button>
diff --git a/Open-ILS/src/eg2/src/app/share/grid/grid-toolbar.component.html b/Open-ILS/src/eg2/src/app/share/grid/grid-toolbar.component.html
index 4604fd89dd..77f38c29f9 100644
--- a/Open-ILS/src/eg2/src/app/share/grid/grid-toolbar.component.html
+++ b/Open-ILS/src/eg2/src/app/share/grid/grid-toolbar.component.html
@@ -1,6 +1,14 @@
 
 <div class="eg-grid-toolbar">
 
+  <div class="btn-toolbar" *ngIf="toolbarButtons.length">
+    <div class="btn-grp">
+      <button *ngFor="let btn of toolbarButtons" class="btn btn-light" (click)="btn.action()">
+        {{btn.label}}
+      </button>
+    </div>
+  </div>
+
   <!-- push everything else to the right -->
   <div class="flex-1"></div>
 
diff --git a/Open-ILS/src/eg2/src/app/share/grid/grid-toolbar.component.ts b/Open-ILS/src/eg2/src/app/share/grid/grid-toolbar.component.ts
index 01b5880695..5920d978e9 100644
--- a/Open-ILS/src/eg2/src/app/share/grid/grid-toolbar.component.ts
+++ b/Open-ILS/src/eg2/src/app/share/grid/grid-toolbar.component.ts
@@ -1,6 +1,8 @@
 import {Component, Input, OnInit} from '@angular/core';
 import {EgGridDataSource} from './grid-data-source';
 import {Pager} from '@eg/share/util/pager';
+import {EgGridService, EgGridColumn, EgGridColumnSet, EgGridToolbarButton} 
+  from '@eg/share/grid/grid.service';
 
 @Component({
   selector: 'eg-grid-toolbar',
@@ -11,6 +13,7 @@ export class EgGridToolbarComponent implements OnInit {
 
     @Input() dataSource: EgGridDataSource;
     @Input() pager: Pager;
+    @Input() toolbarButtons: EgGridToolbarButton[];
 
     ngOnInit() {
 
diff --git a/Open-ILS/src/eg2/src/app/share/grid/grid.component.html b/Open-ILS/src/eg2/src/app/share/grid/grid.component.html
index 4c5835c36d..60a48fc807 100644
--- a/Open-ILS/src/eg2/src/app/share/grid/grid.component.html
+++ b/Open-ILS/src/eg2/src/app/share/grid/grid.component.html
@@ -1,10 +1,10 @@
 
 <div class="eg-grid">
-  <eg-grid-toolbar [dataSource]="dataSource" [pager]="pager"></eg-grid-toolbar>
+  <eg-grid-toolbar [dataSource]="dataSource" [pager]="pager" 
+    [toolbarButtons]="toolbarButtons"></eg-grid-toolbar>
   <eg-grid-header [columnSet]="columnSet"></eg-grid-header>
 
   <div class="eg-grid-row eg-grid-body-row"
-    (dblclick)="onRowDblClick(row)"
     [ngClass]="{'eg-grid-row-selected': selector[idx]}"
     *ngFor="let row of dataSource.getPageOfRows(pager); let idx = index">
 
@@ -15,6 +15,7 @@
       {{pager.rowNumber(idx)}}
     </div>
     <div class="eg-grid-cell eg-grid-body-cell" [ngStyle]="{flex:col.flex}"
+      (dblclick)="onRowDblClick(row)"
       *ngFor="let col of columnSet.displayColumns()">
       {{getDisplayValue(row, col)}}
     </div>
diff --git a/Open-ILS/src/eg2/src/app/share/grid/grid.component.ts b/Open-ILS/src/eg2/src/app/share/grid/grid.component.ts
index 570b542d12..01a2aa4efd 100644
--- a/Open-ILS/src/eg2/src/app/share/grid/grid.component.ts
+++ b/Open-ILS/src/eg2/src/app/share/grid/grid.component.ts
@@ -3,7 +3,7 @@ import {EgGridDataSource} from './grid-data-source';
 import {EgIdlService} from '@eg/core/idl.service';
 import {EgOrgService} from '@eg/core/org.service';
 import {Pager} from '@eg/share/util/pager';
-import {EgGridService, EgGridColumn, EgGridColumnSet} 
+import {EgGridService, EgGridColumn, EgGridColumnSet, EgGridToolbarButton} 
   from '@eg/share/grid/grid.service';
 
 @Component({
@@ -23,12 +23,14 @@ export class EgGridComponent implements OnInit {
     columnSet: EgGridColumnSet;
     selector: {[idx:number] : boolean};
     onRowDblClick$: EventEmitter<any>;
+    toolbarButtons: EgGridToolbarButton[];
     
     constructor(private gridSvc: EgGridService) {
         this.pager = new Pager();
         this.selector = {};
         this.pager.limit = 10; // TODO
         this.onRowDblClick$ = new EventEmitter<any>();
+        this.toolbarButtons = [];
     }
 
     ngOnInit() {
diff --git a/Open-ILS/src/eg2/src/app/share/grid/grid.module.ts b/Open-ILS/src/eg2/src/app/share/grid/grid.module.ts
index a664b43e95..cbba8dda36 100644
--- a/Open-ILS/src/eg2/src/app/share/grid/grid.module.ts
+++ b/Open-ILS/src/eg2/src/app/share/grid/grid.module.ts
@@ -4,9 +4,9 @@ import {FormsModule} from '@angular/forms';
 import {EgGridComponent} from './grid.component';
 import {EgGridColumnComponent} from './grid-column.component';
 import {EgGridHeaderComponent} from './grid-header.component';
-import {EgGridBodyComponent} from './grid-body.component';
 import {EgGridToolbarComponent} from './grid-toolbar.component';
 import {EgGridService} from './grid.service';
+import {EgGridToolbarButtonComponent} from './grid-toolbar-button.component';
 
 @NgModule({
     declarations: [
@@ -14,8 +14,8 @@ import {EgGridService} from './grid.service';
         EgGridComponent,
         EgGridColumnComponent,
         EgGridHeaderComponent,
-        EgGridBodyComponent,
-        EgGridToolbarComponent
+        EgGridToolbarComponent,
+        EgGridToolbarButtonComponent
     ],
     imports: [
         CommonModule,
@@ -25,6 +25,7 @@ import {EgGridService} from './grid.service';
         // public components
         EgGridComponent,
         EgGridColumnComponent,
+        EgGridToolbarButtonComponent
     ],
     providers: [
         EgGridService
diff --git a/Open-ILS/src/eg2/src/app/share/grid/grid.service.ts b/Open-ILS/src/eg2/src/app/share/grid/grid.service.ts
index 41bd1bac8e..2bab414700 100644
--- a/Open-ILS/src/eg2/src/app/share/grid/grid.service.ts
+++ b/Open-ILS/src/eg2/src/app/share/grid/grid.service.ts
@@ -105,5 +105,9 @@ export class EgGridColumnSet {
     }
 }
 
+export class EgGridToolbarButton {
+    label: string;
+    action: () => any;
+}
 
 
diff --git a/Open-ILS/src/eg2/src/app/share/org-select/org-select.component.ts b/Open-ILS/src/eg2/src/app/share/org-select/org-select.component.ts
index 627dd4e803..6f76875e4b 100644
--- a/Open-ILS/src/eg2/src/app/share/org-select/org-select.component.ts
+++ b/Open-ILS/src/eg2/src/app/share/org-select/org-select.component.ts
@@ -120,7 +120,10 @@ export class EgOrgSelectComponent implements OnInit {
     }
 
     // Fired by the typeahead to inform us of a change.
+    // TODO: this does not fire when the value is cleared :( -- implement
+    // change detection on this.selected to look specifically for NULL.
     orgChanged(selEvent: NgbTypeaheadSelectItemEvent) {
+        console.log('change occurred ' + selEvent.item);
         this.onChange.emit(this.org.get(selEvent.item.id));
     }
 
diff --git a/Open-ILS/src/eg2/src/app/share/string/string.component.ts b/Open-ILS/src/eg2/src/app/share/string/string.component.ts
index 8c81898b73..f6a858a64f 100644
--- a/Open-ILS/src/eg2/src/app/share/string/string.component.ts
+++ b/Open-ILS/src/eg2/src/app/share/string/string.component.ts
@@ -44,6 +44,9 @@ export class EgStringComponent implements OnInit {
 
     // Apply the new context if provided, give our container a 
     // chance to update, then resolve with the current string.
+    // NOTE: talking to the native DOM element is not so great, but
+    // hopefully we can retire the EgString* code entirely once
+    // in-code translations are supported (Ang6?)
     current(ctx?: any): Promise<string> {
         if (ctx) this.ctx = ctx;
         return new Promise(resolve => {
diff --git a/Open-ILS/src/eg2/src/app/staff/admin/server/config/billing_type.component.html b/Open-ILS/src/eg2/src/app/staff/admin/server/config/billing_type.component.html
index 25023e77bb..684883c6e8 100644
--- a/Open-ILS/src/eg2/src/app/staff/admin/server/config/billing_type.component.html
+++ b/Open-ILS/src/eg2/src/app/staff/admin/server/config/billing_type.component.html
@@ -2,9 +2,17 @@
 </eg-staff-banner>
 
 <eg-grid #btGrid idlClass="cbt" [dataSource]="dataSource">
+  <eg-grid-toolbar-button label="New Billing Type" i18n-label [action]="createBillingType">
+  </eg-grid-toolbar-button>
 </eg-grid>
 
-<fm-record-editor #btEditDialog idlClass="cbt">
+<fm-record-editor #btEditDialog idlClass="cbt" requiredFields="name,org_unit">
 </fm-record-editor>
 
+<ng-template #successStrTmpl i18n>Billing Type Update Succeeded</ng-template>
+<eg-string #successString [template]="successStrTmpl"></eg-string>
+
+<ng-template #createStrTmpl i18n>Billing Type Succeessfully Created</ng-template>
+<eg-string #createString [template]="createStrTmpl"></eg-string>
+
 
diff --git a/Open-ILS/src/eg2/src/app/staff/admin/server/config/billing_type.component.ts b/Open-ILS/src/eg2/src/app/staff/admin/server/config/billing_type.component.ts
index 2fde658dff..dce24dfde7 100644
--- a/Open-ILS/src/eg2/src/app/staff/admin/server/config/billing_type.component.ts
+++ b/Open-ILS/src/eg2/src/app/staff/admin/server/config/billing_type.component.ts
@@ -5,6 +5,7 @@ import {EgToastService} from '@eg/share/toast/toast.service';
 import {Pager} from '@eg/share/util/pager';
 import {EgPcrudService} from '@eg/core/pcrud.service';
 import {FmRecordEditorComponent} from '@eg/share/fm-editor/fm-editor.component';
+import {EgStringComponent} from '@eg/share/string/string.component';
 
 @Component({
     templateUrl: './billing_type.component.html'
@@ -15,12 +16,27 @@ export class BillingTypeComponent implements OnInit {
     dataSource: EgGridDataSource;
     @ViewChild('btGrid') btGrid: EgGridComponent;
     @ViewChild('btEditDialog') btEditDialog: FmRecordEditorComponent;
+    @ViewChild('successString') successString: EgStringComponent;
+    @ViewChild('createString') createString: EgStringComponent;
+    createBillingType: () => any;
 
     constructor(
         private pcrud: EgPcrudService,
         private toast: EgToastService
     ) {
         this.dataSource = new EgGridDataSource();
+
+        this.createBillingType = () => {
+            this.btEditDialog.mode = 'create';
+            this.btEditDialog.open().then(
+                ok => {
+                    this.createString.current()
+                        .then(str => this.toast.success(str));
+                    this.btGrid.reload();
+                },
+                err => { }
+            );
+        }
     }
 
     ngOnInit() {
@@ -39,13 +55,11 @@ export class BillingTypeComponent implements OnInit {
                 this.btEditDialog.recId = bt.id();
                 this.btEditDialog.open().then(
                     ok => {
-                        // TODO: i18n
-                        this.toast.success('Billing Type Update Succeeded'),
-                        this.btGrid.reload()
+                        this.successString.current()
+                            .then(str => this.toast.success(str));
+                        this.btGrid.reload();
                     },
-                    err => {
-                        this.toast.warning('Billing Type Update Canceled')
-                    }
+                    err => { }
                 );
             }
         );
-- 
2.11.0