[Android] actionViewIntent - activity won't start on Android 11+

[Aure77]

If your app target Android API 30+ (and use Android 11 device),
using ReactNativeBlobUtil.android.actionViewIntent(path, "application/pdf"); may do nothing.

After investing code, startActivity is not called because resolveActivity always return null here:

react-native-blob-util/android/src/main/java/com/ReactNativeBlobUtil/ReactNativeBlobUtil.java

Lines 135 to 137 in 93059e0

	if (intent.resolveActivity(pm) != null) {
	this.getReactApplicationContext().startActivity(intent);
	}

This is due to new Android 11 package visibility.
I found a very interesting article that recommend do no not use resolveActivity - startActivity pattern to avoid potential issues.
See: https://cketti.de/2020/09/03/avoid-intent-resolveactivity/

Why not using try catch method for all Android version like here:

react-native-blob-util/android/src/main/java/com/ReactNativeBlobUtil/ReactNativeBlobUtil.java

Lines 145 to 150 in 93059e0

	try {
	this.getReactApplicationContext().startActivity(intent);
	promise.resolve(true);
	} catch (ActivityNotFoundException ex) {
	promise.reject("ENOAPP", "No app installed for " + mime);
	}

and that handle promise resolve/reject correctly.
(this workaround fix my issue with Android 11+ devices)

Full example to reproduce:

// url = url to a pdf file
const PDF_PATH = `${ReactNativeBlobUtil.fs.dirs.CacheDir}/pdfs/Document.pdf`;
const res = await ReactNativeBlobUtil.config({
  fileCache: true,
  appendExt: "pdf",
  path: PDF_PATH
}).fetch("GET", url);
if (res) {
  ReactNativeBlobUtil.android.actionViewIntent(
    res.path(),
    "application/pdf"
  );
}

Further more, actionViewIntent promise is only resolved for Android SDK < 24

react-native-blob-util/android/src/main/java/com/ReactNativeBlobUtil/ReactNativeBlobUtil.java

Line 147 in 93059e0

promise.resolve(true);

Is this normal ?

[Aure77]

I created a patch if someone want to test fix:
https://gist.github.com/Aure77/5b0f75aee85fa897d6a3469146a197df

I can create a PR if confirmed.

[rutgervanwilligen]

Successfully applied and tested this patch in my project. Using actionViewIntent with a PDF file.

[rutgervanwilligen]

However, using the solution described in this closed issue also fixes it for me. Not sure which of these is the way to go.

[RonRadtke]

@Aure77 would you be so kind to create a PR, I would merge it and create a new update for the lib then.

It's the better way to go. Adjusting the manifest is just a fix for some basically wrong behavior of the lib.