Skip to content

Commit da1ef0d

Browse files
fabriziocuccimeta-codesync[bot]
authored andcommitted
Simplify hasNativeModule boolean ternary (#55490)
Summary: Pull Request resolved: #55490 Changelog: [Internal] Simplified a redundant boolean ternary expression in CatalystInstanceImpl.hasNativeModule(). The expression `condition ? true : fallback` is equivalent to `condition || fallback`. This change improves code readability without changing behavior. Reviewed By: cortinico Differential Revision: D92836584 fbshipit-source-id: 2946c779e73c06554e04c1df75ddccc5f9d048ff
1 parent cb24531 commit da1ef0d

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridge/CatalystInstanceImpl.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -416,9 +416,8 @@ public <T extends JavaScriptModule> T getJSModule(Class<T> jsInterface) {
416416
@Override
417417
public <T extends NativeModule> boolean hasNativeModule(Class<T> nativeModuleInterface) {
418418
String moduleName = getNameFromAnnotation(nativeModuleInterface);
419-
return getTurboModuleRegistry() != null && getTurboModuleRegistry().hasModule(moduleName)
420-
? true
421-
: mNativeModuleRegistry.hasModule(moduleName);
419+
return (getTurboModuleRegistry() != null && getTurboModuleRegistry().hasModule(moduleName))
420+
|| mNativeModuleRegistry.hasModule(moduleName);
422421
}
423422

424423
@Override

0 commit comments

Comments
 (0)