Make ClipboardModule TurboModule-compatible

Summary:
We forgot to make ClipboardModule TurboModule compatible. I think this was most likely because our codemods targeted all Java classes that extended ReactContextBaseJavaModule. The ClipboardModule extends ContextBaseJavaModule instead.

There are no other NativeModules that extend ContextBaseJavaModule.

Changelog: [Internal]

Reviewed By: sshic

Differential Revision: D31291293

fbshipit-source-id: cf5d21898101699f8c349b013a77e8329339a8d3
This commit is contained in:
Ramanpreet Nara
2021-09-30 20:48:29 -07:00
committed by Facebook GitHub Bot
parent 95187f7ebc
commit 40a367e0a3
3 changed files with 16 additions and 10 deletions
@@ -1,4 +1,4 @@
load("//tools/build_defs/oss:rn_defs.bzl", "react_native_dep", "react_native_target", "rn_android_library")
load("//tools/build_defs/oss:rn_defs.bzl", "react_native_dep", "react_native_root_target", "react_native_target", "rn_android_library")
rn_android_library(
name = "clipboard",
@@ -16,4 +16,7 @@ rn_android_library(
react_native_target("java/com/facebook/react/common:common"),
react_native_target("java/com/facebook/react/module/annotations:annotations"),
],
exported_deps = [
react_native_root_target(":FBReactNativeSpec"),
],
)
@@ -9,17 +9,16 @@ package com.facebook.react.modules.clipboard;
import android.content.ClipData;
import android.content.ClipboardManager;
import android.content.Context;
import com.facebook.react.bridge.ContextBaseJavaModule;
import com.facebook.fbreact.specs.NativeClipboardSpec;
import com.facebook.react.bridge.Promise;
import com.facebook.react.bridge.ReactMethod;
import com.facebook.react.bridge.ReactApplicationContext;
import com.facebook.react.module.annotations.ReactModule;
/** A module that allows JS to get/set clipboard contents. */
@ReactModule(name = ClipboardModule.NAME)
public class ClipboardModule extends ContextBaseJavaModule {
public class ClipboardModule extends NativeClipboardSpec {
public ClipboardModule(Context context) {
public ClipboardModule(ReactApplicationContext context) {
super(context);
}
@@ -31,10 +30,12 @@ public class ClipboardModule extends ContextBaseJavaModule {
}
private ClipboardManager getClipboardService() {
return (ClipboardManager) getContext().getSystemService(getContext().CLIPBOARD_SERVICE);
return (ClipboardManager)
getReactApplicationContext()
.getSystemService(getReactApplicationContext().CLIPBOARD_SERVICE);
}
@ReactMethod
@Override
public void getString(Promise promise) {
try {
ClipboardManager clipboard = getClipboardService();
@@ -50,7 +51,7 @@ public class ClipboardModule extends ContextBaseJavaModule {
}
}
@ReactMethod
@Override
public void setString(String text) {
ClipData clipdata = ClipData.newPlainText(null, text);
ClipboardManager clipboard = getClipboardService();