fix(android): resolve Scrapped or attached views may not be recycled crash

Author: MrRefactor

android/src/main/java/com/reactnativepagerview/PagerViewViewManager.kt

@@ -2,6 +2,7 @@ package com.reactnativepagerview
 
 import android.view.View
 import android.view.ViewGroup
+import androidx.recyclerview.widget.RecyclerView
 import androidx.viewpager2.widget.ViewPager2
 import androidx.viewpager2.widget.ViewPager2.OnPageChangeCallback
 import com.facebook.infer.annotation.Assertions
@@ -88,6 +89,21 @@ class PagerViewViewManager : ViewGroupManager<NestedScrollableHost>(), RNCViewPa
         return host
     }
 
+    private fun stopScrollIfNeeded(host: NestedScrollableHost) {
+        val recyclerView = (host.getChildAt(0) as? ViewPager2)?.getChildAt(0) as? RecyclerView
+        recyclerView?.stopScroll()
+    }
+
+    override fun onDropViewInstance(view: NestedScrollableHost) {
+        val viewPager = view.getChildAt(0) as? ViewPager2
+        val recyclerView = viewPager?.getChildAt(0) as? RecyclerView
+        recyclerView?.let {
+            it.stopScroll()
+            it.swapAdapter(null, false)
+        }
+        super.onDropViewInstance(view)
+    }
+
     override fun addView(host: NestedScrollableHost, child: View, index: Int) {
         PagerViewViewManagerImpl.addView(host, child, index)
     }
@@ -99,14 +115,17 @@ class PagerViewViewManager : ViewGroupManager<NestedScrollableHost>(), RNCViewPa
     }
 
     override fun removeView(parent: NestedScrollableHost, view: View) {
+        stopScrollIfNeeded(parent)
         PagerViewViewManagerImpl.removeView(parent, view)
     }
 
     override fun removeAllViews(parent: NestedScrollableHost) {
+        stopScrollIfNeeded(parent)
         PagerViewViewManagerImpl.removeAllViews(parent)
     }
 
     override fun removeViewAt(parent: NestedScrollableHost, index: Int) {
+        stopScrollIfNeeded(parent)
         PagerViewViewManagerImpl.removeViewAt(parent, index)
     }
 
@@ -206,4 +225,4 @@ class PagerViewViewManager : ViewGroupManager<NestedScrollableHost>(), RNCViewPa
                 PageScrollStateChangedEvent.EVENT_NAME, MapBuilder.of("registrationName", "onPageScrollStateChanged"),
                 PageSelectedEvent.EVENT_NAME, MapBuilder.of("registrationName", "onPageSelected"))
     }
-}
+}

android/src/main/java/com/reactnativepagerview/ViewPagerAdapter.kt

@@ -17,19 +17,28 @@ class ViewPagerAdapter() : Adapter<ViewPagerViewHolder>() {
   override fun onBindViewHolder(holder: ViewPagerViewHolder, index: Int) {
     val container: FrameLayout = holder.container
     val child = getChildAt(index)
-    holder.setIsRecyclable(false)
 
     if (container.childCount > 0) {
       container.removeAllViews()
     }
 
     if (child.parent != null) {
-      (child.parent as FrameLayout).removeView(child)
+      (child.parent as ViewGroup).removeView(child)
     }
 
     container.addView(child)
   }
 
+  override fun onViewRecycled(holder: ViewPagerViewHolder) {
+    super.onViewRecycled(holder)
+    holder.container.removeAllViews()
+  }
+
+  override fun onFailedToRecycleView(holder: ViewPagerViewHolder): Boolean {
+    holder.container.removeAllViews()
+    return true
+  }
+
   override fun getItemCount(): Int {
     return childrenViews.size
   }
@@ -45,17 +54,16 @@ class ViewPagerAdapter() : Adapter<ViewPagerViewHolder>() {
 
   fun removeChild(child: View) {
     val index = childrenViews.indexOf(child)
-    
-    if(index > -1) {
+
+    if (index > -1) {
       removeChildAt(index)
     }
   }
 
   fun removeAll() {
-    for (index in 1..childrenViews.size) {
-      val child = childrenViews[index-1]
-      if (child.parent?.parent != null) {
-        (child.parent.parent as ViewGroup).removeView(child.parent as View)
+    for (child in childrenViews) {
+      if (child.parent != null) {
+        (child.parent as ViewGroup).removeView(child)
       }
     }
     val removedChildrenCount = childrenViews.size
@@ -64,7 +72,11 @@ class ViewPagerAdapter() : Adapter<ViewPagerViewHolder>() {
   }
 
   fun removeChildAt(index: Int) {
-    if (index >= 0 && index < childrenViews.size) { 
+    if (index >= 0 && index < childrenViews.size) {
+      val child = childrenViews[index]
+      if (child.parent != null) {
+        (child.parent as ViewGroup).removeView(child)
+      }
       childrenViews.removeAt(index)
       notifyItemRemoved(index)
     }