EDI attr sets: fix behavior on row double-click

Extends fix for LH#11

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

diff --git a/Open-ILS/src/eg2/src/app/staff/admin/acq/edi_attr_set/edi-attr-sets.component.html b/Open-ILS/src/eg2/src/app/staff/admin/acq/edi_attr_set/edi-attr-sets.component.html
index 986a947..0230a0a 100644 (file)
--- a/Open-ILS/src/eg2/src/app/staff/admin/acq/edi_attr_set/edi-attr-sets.component.html
+++ b/Open-ILS/src/eg2/src/app/staff/admin/acq/edi_attr_set/edi-attr-sets.component.html
@@ -52,6 +52,7 @@
 <eg-grid #grid idlClass="{{idlClass}}" [dataSource]="dataSource" hideFields="{{hideGridFields}}"
     [sortable]="true" persistKey="{{persistKey}}" autoGeneratedColumnOrder="{{fieldOrder}}"
     [filterable]="true" [stickyHeader]="true"
+    (onRowActivate)="editSelected([$event])"
     [cellTextGenerator]="cellTextGenerator">
   <eg-grid-toolbar-button [disabled]="!canCreate" 
     label="New {{idlClassDef.label}}" i18n-label (onClick)="createNew()">

diff --git a/Open-ILS/src/eg2/src/app/staff/admin/acq/edi_attr_set/edi-attr-sets.component.ts b/Open-ILS/src/eg2/src/app/staff/admin/acq/edi_attr_set/edi-attr-sets.component.ts
index 805a683..fce5fdb 100644 (file)
--- a/Open-ILS/src/eg2/src/app/staff/admin/acq/edi_attr_set/edi-attr-sets.component.ts
+++ b/Open-ILS/src/eg2/src/app/staff/admin/acq/edi_attr_set/edi-attr-sets.component.ts
@@ -114,10 +114,6 @@ export class EdiAttrSetsComponent extends AdminPageComponent implements OnInit {
 
         this.classLabel = this.idlClassDef.label;
         this.includeOrgDescendants = true;
-        // TODO - this works, but doesn't clear the base component's subscription
-        // this.grid.onRowActivate.subscribe(
-        //     (idlThing: IdlObject) => this.editSelected([idlThing])
-        // );
     }
 
     countProviders(row: IdlObject): Observable<IdlObject> {