Ensure unsorted order of previous dimensions is retained. Ensure original collection is not poluted.

Author: MKirova

projects/igniteui-angular/src/lib/grids/pivot-grid/pivot-grid.pipes.ts

@@ -4,11 +4,12 @@ import { DataUtil } from '../../data-operations/data-util';
 import { FilteringExpressionsTree, IFilteringExpressionsTree } from '../../data-operations/filtering-expressions-tree';
 import { IFilteringStrategy } from '../../data-operations/filtering-strategy';
 import { IPivotConfiguration, IPivotDimension, IPivotKeys } from './pivot-grid.interface';
-import { DefaultPivotSortingStrategy, PivotColumnDimensionsStrategy, PivotRowDimensionsStrategy } from '../../data-operations/pivot-strategy';
+import { DefaultPivotSortingStrategy,
+     PivotColumnDimensionsStrategy, PivotRowDimensionsStrategy } from '../../data-operations/pivot-strategy';
 import { PivotUtil } from './pivot-util';
 import { FilteringLogic } from '../../data-operations/filtering-expression.interface';
 import { IGridSortingStrategy } from '../../data-operations/sorting-strategy';
-import { ISortingExpression } from '../../data-operations/sorting-expression.interface';
+import { ISortingExpression, SortingDirection } from '../../data-operations/sorting-expression.interface';
 import { GridBaseAPIService, IgxGridBaseDirective } from '../hierarchical-grid/public_api';
 import { GridType } from '../common/grid.interface';
 /**
@@ -154,12 +155,18 @@ export class IgxPivotGridSortingPipe implements PipeTransform {
                     fieldName: x.memberName,
                     strategy: DefaultPivotSortingStrategy.instance()
                 });
+            } else {
+                expressions.push({
+                    dir: SortingDirection.None,
+                    fieldName: x.memberName,
+                    strategy: DefaultPivotSortingStrategy.instance()
+                });
             }
         });
         if (!expressions.length) {
             result = collection;
         } else {
-            result = DataUtil.sort(cloneArray(collection), expressions, sorting, this.gridAPI.grid);
+            result = DataUtil.sort(cloneArray(collection, true), expressions, sorting, this.gridAPI.grid);
         }
 
         return result;

src/app/pivot-grid/pivot-grid.sample.ts

@@ -36,23 +36,14 @@ export class PivotGridSampleComponent {
                 enabled: true
             },
             {
-                memberName: 'AllDate',
-                memberFunction: () => 'AllDate',
-                enabled: true,
-                childLevel: {
-                    memberName: 'Years',
-                    memberFunction: (rec) => {
-                        const recordValue = rec['Date'];
-                        return recordValue ? (new Date(recordValue)).getFullYear().toString() : rec['Years'];
-                        },
-                        enabled: true,
-                        childLevel: {
-                            memberName: 'Date',
-                            dataType: GridColumnDataType.Date,
-                            enabled: true
-                        }
-                }
+                memberName: 'SellerName',
+                enabled: true
             },
+            {
+                memberName: 'Date',
+                dataType: GridColumnDataType.Date,
+                enabled: true
+            }
         ],
         values: [
             {
@@ -98,6 +89,10 @@ export class PivotGridSampleComponent {
             ProductCategory: 'Accessories', UnitPrice: 85.58, SellerName: 'David Haley',
             Country: 'UK', City: 'London', Date: '04/07/2012', UnitsSold: 293
         },
+        {
+            ProductCategory: 'Accessories', UnitPrice: 85.58, SellerName: 'David Haley',
+            Country: 'UK', City: 'London', Date: '04/07/2011', UnitsSold: 293
+        },
         {
             ProductCategory: 'Components', UnitPrice: 18.13, SellerName: 'John Smith',
             Country: 'Japan', City: 'Yokohama', Date: '12/08/2012', UnitsSold: 240