Merge pull request #980 from wizarrc/patch-8

Adds bounds to offset Fixes #978
swimlane · Sep 12, 2017 · e5dc5a0 · e5dc5a0
2 parents 80f13e0 + 78a6c46
commit e5dc5a0
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/src/components/datatable.component.ts b/src/components/datatable.component.ts
@@ -283,7 +283,12 @@ export class DatatableComponent implements OnInit, DoCheck, AfterViewInit {
    * The current offset ( page - 1 ) shown.
    * Default value: `0`
    */
-  @Input() offset: number = 0;
+  @Input() set offset(val: number) {
+    this._offset = val;
+  }
+  get offset(): number {
+    return Math.max(Math.min(this._offset, Math.ceil(this.rowCount / this.pageSize) - 1), 0);
+  }
 
   /**
    * Show the linear loading bar.
@@ -595,6 +600,7 @@ export class DatatableComponent implements OnInit, DoCheck, AfterViewInit {
 
   _limit: number | undefined;
   _count: number = 0;
+  _offset: number = 0;
   _rows: any[];
   _groupRowsBy: string;
   _internalRows: any[];