refactor(grid): finished main refactoring

Author: DDavidkov

projects/igniteui-angular/src/lib/grid-common/api.service.ts

@@ -5,15 +5,15 @@ import { IFilteringExpression, FilteringLogic } from '../data-operations/filteri
 import { ISortingExpression, SortingDirection } from '../data-operations/sorting-expression.interface';
 import { IgxGridCellComponent } from './cell.component';
 import { IgxColumnComponent } from './column.component';
-import { IgxRowBaseComponent } from './row-base.component';
+import { IgxRowComponent } from './row.component';
 import { IFilteringOperation, FilteringExpressionsTree, IFilteringExpressionsTree } from '../../public_api';
 import { IGridEditEventArgs, IGridComponent } from './grid-interfaces';
 
 /**
  *@hidden
  */
 @Injectable()
-export abstract class IGridAPIService <T extends IGridComponent> {
+export class IGridAPIService <T extends IGridComponent> {
 
     public change: Subject<any> = new Subject<any>();
     protected state: Map<string, T> = new Map<string, T>();
@@ -106,7 +106,7 @@ export abstract class IGridAPIService <T extends IGridComponent> {
         }
     }
 
-    public get_row_by_key(id: string, rowSelector: any): IgxRowBaseComponent<IGridComponent>  {
+    public get_row_by_key(id: string, rowSelector: any): IgxRowComponent<IGridComponent>  {
         const primaryKey = this.get(id).primaryKey;
         if (primaryKey !== undefined && primaryKey !== null) {
             return this.get(id).dataRowList.find((row) => row.rowData[primaryKey] === rowSelector);
@@ -115,7 +115,7 @@ export abstract class IGridAPIService <T extends IGridComponent> {
         }
     }
 
-    public get_row_by_index(id: string, rowIndex: number): IgxRowBaseComponent<IGridComponent> {
+    public get_row_by_index(id: string, rowIndex: number): IgxRowComponent<IGridComponent> {
         return this.get(id).rowList.find((row) => row.index === rowIndex);
     }
 
@@ -347,7 +347,12 @@ export abstract class IGridAPIService <T extends IGridComponent> {
         });
     }
 
-    protected abstract remove_grouping_expression(id: string, fieldName: string);
-    public abstract refreshSearch(id: string, updateHighlight?: boolean);
+    protected remove_grouping_expression(id: string, fieldName: string) {
+
+    }
+
+    public refreshSearch(id: string, updateHighlight?: boolean) {
+
+    }
 
 }

projects/igniteui-angular/src/lib/grid/column-pinning.component.html renamed to projects/igniteui-angular/src/lib/grid-common/column-pinning/column-pinning.component.html

@@ -1,11 +1,11 @@
 
 import { CommonModule } from '@angular/common';
-import { Directive, Component, EventEmitter, Output, NgModule, ChangeDetectorRef } from '@angular/core';
+import { Directive, Component, NgModule, ChangeDetectorRef } from '@angular/core';
 import { FormsModule } from '@angular/forms';
-import { IgxCheckboxModule } from '../checkbox/checkbox.component';
-import { ColumnChooserBase } from '../grid-common/column-hiding/column-chooser-base';
-import { ColumnChooserItemBase } from '../grid-common/column-hiding/column-chooser-item-base';
-import { IgxInputGroupModule } from '../input-group/input-group.component';
+import { IgxCheckboxModule } from '../../checkbox/checkbox.component';
+import { ColumnChooserBase } from '../column-hiding/column-chooser-base';
+import { ColumnChooserItemBase } from '../column-hiding/column-chooser-item-base';
+import { IgxInputGroupModule } from '../../input-group/input-group.component';
 
 @Directive({
     selector: '[igxColumnPinningItem]'

projects/igniteui-angular/src/lib/grid/column-pinning.component.ts renamed to projects/igniteui-angular/src/lib/grid-common/column-pinning/column-pinning.component.ts

@@ -17,8 +17,8 @@ import { DataType } from '../data-operations/data-util';
 import { IgxTextHighlightDirective } from '../directives/text-highlight/text-highlight.directive';
 import { IGridAPIService } from './api.service';
 import { IgxGridCellComponent } from './cell.component';
-import { IgxDateSummaryOperand, IgxNumberSummaryOperand, IgxSummaryOperand } from './grid-summary';
-import { IgxRowBaseComponent } from './row-base.component';
+import { IgxDateSummaryOperand, IgxNumberSummaryOperand, IgxSummaryOperand } from './summaries/grid-summary';
+import { IgxRowComponent } from './row.component';
 import {
     IgxCellEditorTemplateDirective,
     IgxCellFooterTemplateDirective,
@@ -599,7 +599,7 @@ export class IgxColumnComponent implements AfterContentInit {
      * @memberof IgxColumnComponent
      */
     get cells(): IgxGridCellComponent[] {
-        return this.grid.rowList.filter((row) => row instanceof IgxRowBaseComponent)
+        return this.grid.rowList.filter((row) => row instanceof IgxRowComponent)
             .map((row) => row.cells.filter((cell) => cell.columnIndex === this.index))
             .reduce((a, b) => a.concat(b), []);
     }

projects/igniteui-angular/src/lib/grid-common/column.component.ts

@@ -15,12 +15,12 @@ import {
     AfterViewInit
 } from '@angular/core';
 import { Subject } from 'rxjs';
-import { DataType } from '../data-operations/data-util';
-import { IGridAPIService } from './api.service';
-import { IGridComponent } from './grid-interfaces';
-import { IFilteringExpression } from '../data-operations/filtering-expression.interface';
-import { FilteringExpressionsTree } from '../data-operations/filtering-expressions-tree';
-import { IFilteringOperation } from '../data-operations/filtering-condition';
+import { DataType } from '../../data-operations/data-util';
+import { IGridAPIService } from '../api.service';
+import { IGridComponent } from '../grid-interfaces';
+import { IFilteringExpression } from '../../data-operations/filtering-expression.interface';
+import { FilteringExpressionsTree } from '../../data-operations/filtering-expressions-tree';
+import { IFilteringOperation } from '../../data-operations/filtering-condition';
 
 
 /**

projects/igniteui-angular/src/lib/grid-common/grid-filtering-expression.component.html renamed to projects/igniteui-angular/src/lib/grid-common/filtering/grid-filtering-expression.component.html

@@ -14,18 +14,17 @@ import {
     ViewChild,
     ViewChildren
 } from '@angular/core';
-import { Subject, Subscription } from 'rxjs';
-import { DataType } from '../data-operations/data-util';
-import { IgxToggleDirective, IgxOverlayOutletDirective } from '../directives/toggle/toggle.directive';
-import { IGridAPIService } from './api.service';
-import { IGridComponent } from './grid-interfaces';
-import { FilteringExpressionsTree } from '../data-operations/filtering-expressions-tree';
-import { IgxButtonGroupComponent } from '../buttonGroup/buttonGroup.component';
+import { Subscription } from 'rxjs';
+import { DataType } from '../../data-operations/data-util';
+import { IgxToggleDirective } from '../../directives/toggle/toggle.directive';
+import { IGridAPIService } from '../api.service';
+import { IGridComponent } from '../grid-interfaces';
+import { FilteringExpressionsTree } from '../../data-operations/filtering-expressions-tree';
+import { IgxButtonGroupComponent } from '../../buttonGroup/buttonGroup.component';
 import { IgxGridFilterExpressionComponent } from './grid-filtering-expression.component';
-import { FilteringLogic, IFilteringExpression } from '../data-operations/filtering-expression.interface';
-import { OverlaySettings, HorizontalAlignment } from '../services/overlay/utilities';
-import { ConnectedPositioningStrategy } from '../services/overlay/position/connected-positioning-strategy';
-import { IgxBooleanFilteringOperand } from '../data-operations/filtering-condition';
+import { FilteringLogic, IFilteringExpression } from '../../data-operations/filtering-expression.interface';
+import { OverlaySettings, HorizontalAlignment } from '../../services/overlay/utilities';
+import { ConnectedPositioningStrategy } from '../../services/overlay/position/connected-positioning-strategy';
 /**
  *@hidden
  */
@@ -272,14 +271,18 @@ export class IgxGridFilterComponent implements OnInit, OnDestroy, DoCheck {
     }
 
     protected isFilteringApplied(): boolean {
-        const expr = this.gridAPI.get(this.gridID).filteringExpressionsTree.find(this.column.field);
+        const grid = this.gridAPI.get(this.gridID);
+        if (grid) {
+            const expr = this.gridAPI.get(this.gridID).filteringExpressionsTree.find(this.column.field);
 
-        if (expr) {
-            if (expr instanceof FilteringExpressionsTree) {
-                return expr.filteringOperands.length > 0;
+            if (expr) {
+                if (expr instanceof FilteringExpressionsTree) {
+                    return expr.filteringOperands.length > 0;
+                }
+                return true;
             }
-            return true;
         }
+
         return false;
     }