Fix crash due to missing @JvmStatic to convertToCase

cortinico · cortinico · commit 3252855e1d87 · 2024-07-01T14:27:55.000+01:00
Summary: Users are reporting that RN 0.75 is crashing due to us attempting to accessing a static method on `AndroidUnicodeUtils.convertToCase` which is not static anymore due to Kotlin conversion. Static access is inside Hermes codebase here: https://github.com/facebook/hermes/blob/f5c867514c71b25212eb3039230e0c095518b532/lib/Platform/Unicode/PlatformUnicodeJava.cpp#L107-L109 Changelog: [Android] [Fixed] - Fix crash due to missing JvmStatic to `convertToCase` Reviewed By: javache Differential Revision: D59218291 fbshipit-source-id: ac121a8bcd5fd917ee134d257f967c8e3e338ca5
diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/hermes/unicode/AndroidUnicodeUtils.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/hermes/unicode/AndroidUnicodeUtils.kt
@@ -39,6 +39,7 @@ public object AndroidUnicodeUtils {
   }
 
   @DoNotStrip
+  @JvmStatic
   public fun convertToCase(input: String, targetCase: Int, useCurrentLocale: Boolean): String {
     // Note Java's case conversions use the user's locale. For example "I".toLowerCase()
     // will produce a dotless i. From Java's docs: "To obtain correct results for locale

Original file line number	Diff line number	Diff line change
`@@ -39,6 +39,7 @@ public object AndroidUnicodeUtils {`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`@DoNotStrip`
	`42`	`+ @JvmStatic`
`42`	`43`	`public fun convertToCase(input: String, targetCase: Int, useCurrentLocale: Boolean): String {`
`43`	`44`	`// Note Java's case conversions use the user's locale. For example "I".toLowerCase()`
`44`	`45`	`// will produce a dotless i. From Java's docs: "To obtain correct results for locale`