js:fix list renderBunchedItems array size <0

doric-pub · Mar 31, 2020 · 5d3016f · 5d3016f
1 parent a265371
commit 5d3016f
Show file tree

Hide file tree

Showing 6 changed files with 6 additions and 6 deletions.
diff --git a/doric-js/bundle/doric-lib.es5.js b/doric-js/bundle/doric-lib.es5.js
@@ -1839,7 +1839,7 @@ var List = /** @class */ (function (_super) {
     List.prototype.renderBunchedItems = function (start, length) {
         var _this = this;
         this.ignoreDirtyCallOnce = true;
-        return new Array(Math.min(length, this.itemCount - start)).fill(0).map(function (_, idx) {
+        return new Array(Math.max(0, Math.min(length, this.itemCount - start))).fill(0).map(function (_, idx) {
             var listItem = _this.getItem(start + idx);
             return listItem.toModel();
         });

diff --git a/doric-js/bundle/doric-lib.js b/doric-js/bundle/doric-lib.js
@@ -1360,7 +1360,7 @@ class List extends Superview {
     }
     renderBunchedItems(start, length) {
         this.ignoreDirtyCallOnce = true;
-        return new Array(Math.min(length, this.itemCount - start)).fill(0).map((_, idx) => {
+        return new Array(Math.max(0, Math.min(length, this.itemCount - start))).fill(0).map((_, idx) => {
             const listItem = this.getItem(start + idx);
             return listItem.toModel();
         });

diff --git a/doric-js/bundle/doric-vm.js b/doric-js/bundle/doric-vm.js
@@ -2819,7 +2819,7 @@ class List extends Superview {
     }
     renderBunchedItems(start, length) {
         this.ignoreDirtyCallOnce = true;
-        return new Array(Math.min(length, this.itemCount - start)).fill(0).map((_, idx) => {
+        return new Array(Math.max(0, Math.min(length, this.itemCount - start))).fill(0).map((_, idx) => {
             const listItem = this.getItem(start + idx);
             return listItem.toModel();
         });

diff --git a/doric-js/lib/src/widget/list.js b/doric-js/lib/src/widget/list.js
@@ -67,7 +67,7 @@ export class List extends Superview {
     }
     renderBunchedItems(start, length) {
         this.ignoreDirtyCallOnce = true;
-        return new Array(Math.min(length, this.itemCount - start)).fill(0).map((_, idx) => {
+        return new Array(Math.max(0, Math.min(length, this.itemCount - start))).fill(0).map((_, idx) => {
             const listItem = this.getItem(start + idx);
             return listItem.toModel();
         });

diff --git a/doric-js/src/widget/list.ts b/doric-js/src/widget/list.ts
@@ -106,7 +106,7 @@ export class List extends Superview implements IList {
 
     private renderBunchedItems(start: number, length: number) {
         this.ignoreDirtyCallOnce = true
-        return new Array(Math.min(length, this.itemCount - start)).fill(0).map((_, idx) => {
+        return new Array(Math.max(0, Math.min(length, this.itemCount - start))).fill(0).map((_, idx) => {
             const listItem = this.getItem(start + idx)
             return listItem.toModel()
         })

diff --git a/doric-web/dist/index.js b/doric-web/dist/index.js