feat(sort): emit previous and current change on sort event

BREAKING CHANGE: The sort event now emits a NgsgOrderChange instead of an array

Author: nivekcode

projects/ng-sortgrid-demo/src/app/introduction/examples/async-pipe/async-pipe-memory.component.html

@@ -1,9 +1,13 @@
 <h5 style="margin-bottom: 20px">4. Load items and use them with the async pipe</h5>
 <button style="margin-bottom: 20px" class="btn btn-primary" (click)="loadItems()">Load items</button>
 <div class="card border-primary mb-3">
+  <div class="card-body" *ngIf="previousSortOrder.length > 0">
+    <h1 class="card-title">Previous sort order</h1>
+    <h2 class="card-text">{{ previousSortOrder }}</h2>
+  </div>
   <div class="card-body">
-    <h1 class="card-title">Sort order</h1>
-    <h2 class="card-text">{{ sortOrder }}</h2>
+    <h1 class="card-title">Current sort order</h1>
+    <h2 class="card-text">{{ currentSortOrder }}</h2>
   </div>
 </div>
 <div class="example-container">

projects/ng-sortgrid-demo/src/app/introduction/examples/async-pipe/async-pipe-memory.component.ts

@@ -1,17 +1,25 @@
-import {Component} from '@angular/core';
+import {Component, OnInit} from '@angular/core';
 import {Observable, of} from 'rxjs';
 import {delay, tap} from 'rxjs/operators';
 
+import {NgsgOrderChange} from '../../../../../../ng-sortgrid/src/lib/shared/ngsg-order-change.model';
+
 @Component({
   selector: 'ngsg-demo-async',
   templateUrl: './async-pipe-memory.component.html',
   styleUrls: ['./async-pipe-memory.component.css']
 })
-export class AsyncPipeMemoryComponent {
+export class AsyncPipeMemoryComponent implements OnInit {
 
   item$: Observable<number[]>;
   loading = false;
-  public sortOrder: number[];
+  public currentSortOrder: number[];
+  public previousSortOrder: number[];
+
+  ngOnInit(): void {
+    this.previousSortOrder = [];
+    this.currentSortOrder = [];
+  }
 
   public loadItems(): void {
     this.loading = true;
@@ -21,8 +29,8 @@ export class AsyncPipeMemoryComponent {
     );
   }
 
-  public applyOrder(newOrder: number[]): void {
-    this.sortOrder = newOrder;
+  public applyOrder(orderChange: NgsgOrderChange<number>): void {
+    this.currentSortOrder = orderChange.currentOrder;
+    this.previousSortOrder = orderChange.previousOrder;
   }
-
 }

projects/ng-sortgrid-demo/src/app/introduction/examples/react-on-changes/react-on-changes-memory.component.html

@@ -1,17 +1,21 @@
 <div class="card border-primary mb-3">
+  <div class="card-body" *ngIf="previousSortOrder.length > 0">
+    <h1 class="card-title">Previous sort order</h1>
+    <h2 class="card-text">{{ previousSortOrder }}</h2>
+  </div>
   <div class="card-body">
-    <h1 class="card-title">Sort order</h1>
-    <h2 class="card-text">{{ sortOrder }}</h2>
+    <h1 class="card-title">Current sort order</h1>
+    <h2 class="card-text">{{ currentSortOrder }}</h2>
   </div>
 </div>
 <div class="example-container">
   <ngsg-card *ngFor="let item of items"
-       ngSortgridItem
-       ngSortGridGroup="react-on-changes"
-       [item]="item"
-       [ngSortGridItems]="items"
-       (sorted)="applyOrder($event)"
-       class="example-box">
+             ngSortgridItem
+             ngSortGridGroup="react-on-changes"
+             [item]="item"
+             [ngSortGridItems]="items"
+             (sorted)="applyOrder($event)"
+             class="example-box">
     {{ item }}
   </ngsg-card>
 </div>

projects/ng-sortgrid-demo/src/app/introduction/examples/react-on-changes/react-on-changes-memory.component.ts

@@ -1,4 +1,5 @@
 import {Component, OnInit} from '@angular/core';
+import {NgsgOrderChange} from '../../../../../../ng-sortgrid/src/lib/shared/ngsg-order-change.model';
 
 @Component({
   selector: 'ngsg-demo-react-on-changes-memory',
@@ -7,14 +8,17 @@ import {Component, OnInit} from '@angular/core';
 export class ReactOnChangesMemoryComponent implements OnInit {
 
   public items = [1, 2, 3, 4, 5, 6, 7, 8, 9, 10];
-  public sortOrder: number[];
+  public currentSortOrder: number[];
+  public previousSortOrder: number[];
 
   ngOnInit(): void {
-    this.sortOrder = [...this.items];
+    this.currentSortOrder = [...this.items];
+    this.previousSortOrder = [];
   }
 
-  public applyOrder(newOrder: number[]): void {
-    this.sortOrder = newOrder;
+  public applyOrder(orderChange: NgsgOrderChange<number>): void {
+    this.currentSortOrder = orderChange.currentOrder;
+    this.previousSortOrder = orderChange.previousOrder;
   }
 
 }

projects/ng-sortgrid/src/lib/ngsg-item.directive.spec.ts

@@ -7,6 +7,7 @@ import {NgsgSelectionService} from './mutliselect/ngsg-selection.service';
 import {NgsgReflectService} from './sort/reflection/ngsg-reflect.service';
 import {NgsgStoreService} from './store/ngsg-store.service';
 import {NgsgEventsService} from './shared/ngsg-events.service';
+import {NgsgOrderChange} from './shared/ngsg-order-change.model';
 
 describe('NgsgItemDirective', () => {
   let sut: NgsgItemDirective;
@@ -24,7 +25,8 @@ describe('NgsgItemDirective', () => {
     'resetSelectedItems',
     'hasGroup',
     'hasItems',
-    'setItems'
+    'setItems',
+    'getItems'
   ]);
   const ngsgEventService = new NgsgEventsService();
   const scrollHelperService = {
@@ -116,6 +118,7 @@ describe('NgsgItemDirective', () => {
 
   it('should sort if the group contains selectedItems', () => {
     ngsgStore.hasSelectedItems.and.returnValue(true);
+    ngsgStore.getItems.and.returnValue([]);
     ngsgStore.hasItems.and.returnValue(true);
     sut.drop();
     expect(ngsgSortService.endSort).toHaveBeenCalled();
@@ -125,34 +128,41 @@ describe('NgsgItemDirective', () => {
     const group = 'test-group';
     sut.ngSortGridGroup = group;
     ngsgStore.hasSelectedItems.and.returnValue(true);
+    ngsgStore.getItems.and.returnValue([]);
 
     sut.drop();
     expect(ngsgReflectService.reflectChanges).toHaveBeenCalledWith(group, elementRef.nativeElement);
   });
 
-  it('should get the reflected changes from the reflection service and emit them', done => {
+  it('should emit a OrderChange containing the previous item order and the new itemorder', done => {
     const group = 'test-group';
-    const reflectedChanges = ['item two', 'item one', 'item three'];
+    const currentItemOrder = ['item one', 'item two', 'item three'];
+    const newItemOrder = ['item two', 'item one', 'item three'];
+    const expectedOrderChange: NgsgOrderChange<string> = {previousOrder: currentItemOrder, currentOrder: newItemOrder};
 
     ngsgStore.hasSelectedItems.and.returnValue(true);
     ngsgStore.hasItems.and.returnValue(true);
-    ngsgReflectService.reflectChanges.and.returnValue(reflectedChanges);
+    ngsgStore.getItems.and.returnValue(currentItemOrder);
+    ngsgReflectService.reflectChanges.and.returnValue(newItemOrder);
     sut.ngSortGridGroup = group;
 
-    sut.sorted.subscribe(changes => {
-      expect(reflectedChanges).toEqual(changes);
+    sut.sorted.subscribe((orderChange: NgsgOrderChange<string>) => {
+      expect(orderChange).toEqual(expectedOrderChange);
       done();
     });
     sut.drop();
-    expect(ngsgReflectService.reflectChanges).toHaveBeenCalledWith(group, elementRef.nativeElement);
   });
 
   it('should reset the selected items on drop', () => {
+    ngsgStore.hasSelectedItems.and.returnValue(true);
+    ngsgStore.hasItems.and.returnValue(true);
     sut.drop();
     expect(ngsgStore.resetSelectedItems).toHaveBeenCalled();
   });
 
   it('should stream the dropped event on the eventservice', done => {
+    ngsgStore.hasSelectedItems.and.returnValue(true);
+    ngsgStore.hasItems.and.returnValue(true);
     ngsgEventService.dropped$.subscribe(() => done());
     sut.drop();
   });

projects/ng-sortgrid/src/lib/ngsg-item.directive.ts

@@ -12,14 +12,16 @@ import {
   SimpleChanges
 } from '@angular/core';
 
-import {fromEvent, Observable, Subject} from 'rxjs';
+import {fromEvent, Subject} from 'rxjs';
 import {takeUntil, takeWhile, throttleTime} from 'rxjs/operators';
+
 import {NgsgSortService} from './sort/sort/ngsg-sort.service';
 import {NgsgSelectionService} from './mutliselect/ngsg-selection.service';
 import {NgsgReflectService} from './sort/reflection/ngsg-reflect.service';
 import {NgsgStoreService} from './store/ngsg-store.service';
 import {NgsgEventsService} from './shared/ngsg-events.service';
 import {ScrollHelperService} from './helpers/scroll/scroll-helper.service';
+import {NgsgOrderChange} from './shared/ngsg-order-change.model';
 
 const selector = '[ngSortgridItem]';
 
@@ -32,11 +34,10 @@ export class NgsgItemDirective implements OnInit, OnChanges, AfterViewInit, OnDe
   @Input() scrollSpeed;
   @Input() autoScroll = false;
 
-  @Output() sorted = new EventEmitter<any>();
+  @Output() sorted = new EventEmitter<NgsgOrderChange<any>>();
 
   private selected = false;
   private destroy$ = new Subject();
-  private drag$: Observable<Event>;
 
   constructor(
     public el: ElementRef,
@@ -124,10 +125,10 @@ export class NgsgItemDirective implements OnInit, OnChanges, AfterViewInit, OnDe
       otherwhise the ordered items can not be emitted in the (sorted) event`);
       return;
     }
-
+    const previousOrder = [...this.ngsgStore.getItems(this.ngSortGridGroup)];
     this.sortService.endSort();
-    const reflectedChanges = this.reflectService.reflectChanges(this.ngSortGridGroup, this.el.nativeElement);
-    this.sorted.next(reflectedChanges);
+    const currentOrder = this.reflectService.reflectChanges(this.ngSortGridGroup, this.el.nativeElement);
+    this.sorted.next({previousOrder, currentOrder});
     this.ngsgStore.resetSelectedItems(this.ngSortGridGroup);
     this.ngsgEventService.dropped$.next();
   }

projects/ng-sortgrid/src/lib/shared/ngsg-order-change.model.ts

@@ -0,0 +1,4 @@
+export interface NgsgOrderChange<T> {
+  previousOrder: T[];
+  currentOrder: T[];
+}

projects/ng-sortgrid/src/public-api.ts

@@ -3,3 +3,4 @@
  */
 export * from './lib/ngsg-item.directive';
 export * from './lib/ngsg.module';
+export * from './lib/shared/ngsg-order-change.model';