fix: support splice with multiple items

Author: farfromrefug

src/collectionview/collectionview.android.ts

@@ -712,11 +712,17 @@ export class CollectionView extends CollectionViewBase {
                 return;
             }
             case ChangeType.Splice: {
-                if (event.addedCount > 0) {
-                    this._listViewAdapter.notifyItemRangeInserted(event.index, event.addedCount);
-                }
-                if (event.removed && event.removed.length > 0) {
-                    this._listViewAdapter.notifyItemRangeRemoved(event.index, event.removed.length);
+                const added = event.addedCount;
+                const removed = (event.removed && event.removed.length) || 0;
+                if (added > 0 && added === removed) {
+                    this._listViewAdapter.notifyItemRangeChanged(event.index, added);
+                } else {
+                    if (event.addedCount > 0) {
+                        this._listViewAdapter.notifyItemRangeInserted(event.index, event.addedCount);
+                    }
+                    if (event.removed && event.removed.length > 0) {
+                        this._listViewAdapter.notifyItemRangeRemoved(event.index, event.removed.length);
+                    }
                 }
                 return;
             }

src/collectionview/collectionview.ios.ts

@@ -438,25 +438,35 @@ export class CollectionView extends CollectionViewBase {
             }
             case ChangeType.Splice: {
                 view.performBatchUpdatesCompletion(() => {
-                    if (event.addedCount > 0) {
-                        const indexes = NSMutableArray.alloc<NSIndexPath>().init();
-                        for (let index = 0; index < event.addedCount; index++) {
-                            indexes.addObject(NSIndexPath.indexPathForItemInSection(event.index + index, 0));
-                        }
-                        view.insertItemsAtIndexPaths(indexes);
-                    }
-                    if (event.removed && event.removed.length > 0) {
+                    const added = event.addedCount;
+                    const removed = (event.removed && event.removed.length) || 0;
+                    if (added > 0 && added === removed) {
                         const indexes = NSMutableArray.new<NSIndexPath>();
-                        for (let index = 0; index < event.removed.length; index++) {
-                            indexes.addObject(NSIndexPath.indexPathForItemInSection(event.index + index, 0));
+                        for (let index = 0; index < added; index++) {
+                            indexes.addObject(NSIndexPath.indexPathForRowInSection(event.index + index, 0));
+                        }
+                        view.reloadItemsAtIndexPaths(indexes);
+                    } else {
+                        if (event.addedCount > 0) {
+                            const indexes = NSMutableArray.alloc<NSIndexPath>().init();
+                            for (let index = 0; index < event.addedCount; index++) {
+                                indexes.addObject(NSIndexPath.indexPathForItemInSection(event.index + index, 0));
+                            }
+                            view.insertItemsAtIndexPaths(indexes);
                         }
-                        this.unbindUnusedCells(event.removed);
-                        if (Trace.isEnabled()) {
-                            CLog(CLogTypes.info, 'deleteItemsAtIndexPaths', indexes.count);
+                        if (event.removed && event.removed.length > 0) {
+                            const indexes = NSMutableArray.new<NSIndexPath>();
+                            for (let index = 0; index < event.removed.length; index++) {
+                                indexes.addObject(NSIndexPath.indexPathForItemInSection(event.index + index, 0));
+                            }
+                            this.unbindUnusedCells(event.removed);
+                            if (Trace.isEnabled()) {
+                                CLog(CLogTypes.info, 'deleteItemsAtIndexPaths', indexes.count);
+                            }
+                            view.performBatchUpdatesCompletion(() => {
+                                view.deleteItemsAtIndexPaths(indexes);
+                            }, null);
                         }
-                        view.performBatchUpdatesCompletion(() => {
-                            view.deleteItemsAtIndexPaths(indexes);
-                        }, null);
                     }
                 }, null);
                 return;
@@ -712,7 +722,7 @@ export class CollectionView extends CollectionViewBase {
         cellView['iosIgnoreSafeArea'] = true;
         View.layoutChild(this, cellView, 0, 0, cellSize[0], cellSize[1]);
         if (Trace.isEnabled()) {
-            CLog(CLogTypes.log, 'layoutCell', index, cellView, cellView.getMeasuredWidth(), cellView.getMeasuredHeight());
+            CLog(CLogTypes.log, 'layoutCell', index, cellView.getMeasuredWidth(), cellView.getMeasuredHeight());
         }
     }