From 54a75de467432018040179bf2a4bbf6fe096d4e9 Mon Sep 17 00:00:00 2001 From: Riccardo Cipolleschi Date: Tue, 25 Jul 2023 02:54:32 -0700 Subject: [PATCH] Return NONE accessibility role if `null` is passed. Summary: The `value` string of the ReactAccessibilityDelegate's `fromValue` method is marked as `Nullable`, however the `null` case is not handled. This change handle the `null` case setting the accesssibility role to NONE. ## Changelog: [Android][Fixed] - Set the accessibility role to `NONE` when a `null` string is passed to `fromValue` Differential Revision: D47752098 fbshipit-source-id: fd76a42493e404fe0b75f064954962635532b6e8 --- .../facebook/react/uimanager/ReactAccessibilityDelegate.java | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java index b8b08c9bf74e8c..eed04a1c711f7f 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java @@ -295,6 +295,10 @@ public static String getValue(AccessibilityRole role) { } public static AccessibilityRole fromValue(@Nullable String value) { + if (value == null) { + return NONE; + } + for (AccessibilityRole role : AccessibilityRole.values()) { if (role.name().equalsIgnoreCase(value)) { return role;