@vaadin/component-base 24.4.0-dev.223e39f050 → 24.4.0-dev.b3e1d14600

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vaadin/component-base",
-  "version": "24.4.0-dev.223e39f050",
+  "version": "24.4.0-dev.b3e1d14600",
   "publishConfig": {
     "access": "public"
   },
@@ -42,5 +42,5 @@
     "@vaadin/testing-helpers": "^0.6.0",
     "sinon": "^13.0.2"
   },
-  "gitHead": "5e2e3bfc811c95aed9354235fab93fdbf43eb354"
+  "gitHead": "502d4f5b03f770a83d270d98078cde230254dd0e"
 }

package/src/data-provider-controller/cache.js

@@ -201,7 +201,10 @@ export class Cache {
   setPage(page, items) {
     const startIndex = page * this.pageSize;
     items.forEach((item, i) => {
-      this.items[startIndex + i] = item;
+      const itemIndex = startIndex + i;
+      if (itemIndex < this.size) {
+        this.items[itemIndex] = item;
+      }
     });
   }
 

package/src/data-provider-controller/data-provider-controller.js

@@ -258,14 +258,14 @@ export class DataProviderController extends EventTarget {
         return;
       }
 
-      cache.setPage(page, items);
-
       if (size !== undefined) {
         cache.size = size;
       } else if (params.parentItem) {
         cache.size = items.length;
       }
 
+      cache.setPage(page, items);
+
       this.recalculateFlatSize();
 
       this.hasData = true;
@@ -286,6 +286,6 @@ export class DataProviderController extends EventTarget {
 
   /** @private */
   __isPlaceholder(item) {
-    return this.placeholder ? item === this.placeholder : !item;
+    return item === this.placeholder;
   }
 }