From 01de717dae2b0dfd671db8bc9e306296000abdf2 Mon Sep 17 00:00:00 2001 From: Pieter De Baets Date: Thu, 6 Mar 2025 04:16:53 -0800 Subject: [PATCH] Correctly propagate prepareToRecycleView return value Summary: `prepareToRecycleView` returns nullable, since we may decide a view is not recyclable. We should respect that and return the view returned by super. Changelog: [Internal] Reviewed By: fabriziocucci Differential Revision: D70696246 --- .../com/facebook/react/views/text/ReactTextViewManager.java | 2 +- .../java/com/facebook/react/views/view/ReactViewManager.kt | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/text/ReactTextViewManager.java b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/text/ReactTextViewManager.java index 83d7f3cf34af..a16cc2ff5ec5 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/text/ReactTextViewManager.java +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/text/ReactTextViewManager.java @@ -74,7 +74,7 @@ public ReactTextViewManager(@Nullable ReactTextViewManagerCallback reactTextView // Defaults from ReactTextAnchorViewManager setSelectionColor(preparedView, null); } - return view; + return preparedView; } @Override diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/view/ReactViewManager.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/view/ReactViewManager.kt index c17d7627442e..233be99a71e0 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/view/ReactViewManager.kt +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/view/ReactViewManager.kt @@ -72,11 +72,11 @@ public open class ReactViewManager : ReactClippingViewManager() override fun prepareToRecycleView( reactContext: ThemedReactContext, view: ReactViewGroup - ): ReactViewGroup { + ): ReactViewGroup? { // BaseViewManager val preparedView = super.prepareToRecycleView(reactContext, view) preparedView?.recycleView() - return view + return preparedView } @ReactProp(name = "accessible")