fix: add FragmentActivity check and error handling in RNPaywallsModule (

fixes RevenueCat#1127)
NosimusAI · Feb 22, 2025 · 57d14fb · 57d14fb
1 parent 23db42a
commit 57d14fb
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/...urchases-ui/android/src/main/java/com/revenuecat/purchases/react/ui/PaywallViewManager.kt b/...urchases-ui/android/src/main/java/com/revenuecat/purchases/react/ui/PaywallViewManager.kt
@@ -18,9 +18,12 @@ internal class PaywallViewManager : BasePaywallViewManager<PaywallView>() {
     }
 
     override fun createViewInstance(themedReactContext: ThemedReactContext): PaywallView {
-        return PaywallView(themedReactContext).also { view ->
-            view.setPaywallListener(createPaywallListenerWrapper(themedReactContext, view))
-            view.setDismissHandler(getDismissHandler(themedReactContext, view))
+    return PaywallView(themedReactContext).apply {
+        // Ensure the view is properly initialized before setting listeners
+        post {
+            setPaywallListener(createPaywallListenerWrapper(themedReactContext, this))
+            setDismissHandler(getDismissHandler(themedReactContext, this))
+            }
         }
     }
 

diff --git a/...-purchases-ui/android/src/main/java/com/revenuecat/purchases/react/ui/RNPaywallsModule.kt b/...-purchases-ui/android/src/main/java/com/revenuecat/purchases/react/ui/RNPaywallsModule.kt
@@ -26,6 +26,7 @@ internal class RNPaywallsModule(reactContext: ReactApplicationContext) :
                 is FragmentActivity -> currentActivity
                 else -> {
                     Log.e(NAME, "RevenueCat paywalls require application to use a FragmentActivity")
+                    promise.reject("E_ACTIVITY_NOT_FRAGMENT", "Current activity is not a FragmentActivity")
                     null
                 }
             }