From 40f6998b6766e8aa3c038a1416e5c62cbafca109 Mon Sep 17 00:00:00 2001 From: Ziqi Chen Date: Fri, 27 Jul 2018 15:13:08 -0700 Subject: [PATCH] added a null check on accessibilityStates Summary: Added a check for null on `accessibilityStates`. Now, if a null value is passed in to the prop, it simply does not do anything. Reviewed By: achen1 Differential Revision: D9034714 fbshipit-source-id: caffa41a1d8b08d5358d085e4e268e8e6f1c9344 --- .../java/com/facebook/react/uimanager/BaseViewManager.java | 3 +++ 1 file changed, 3 insertions(+) diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java index fe13dd51f911dd..e2c57825fd4485 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java +++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java @@ -128,6 +128,9 @@ public void setAccessibilityRole(T view, String accessibilityRole) { public void setViewStates(T view, ReadableArray accessibilityStates) { view.setSelected(false); view.setEnabled(true); + if (accessibilityStates == null) { + return; + } for (int i = 0; i < accessibilityStates.size(); i++) { String state = accessibilityStates.getString(i); if (state.equals("selected")) {