From 3eb112381d9d4f8fc1845f635c1c6f322200a00d Mon Sep 17 00:00:00 2001
From: Benoit Marty <benoit@matrix.org>
Date: Tue, 26 Sep 2023 15:01:30 +0200
Subject: [PATCH] Call `openFile` instead of `open` and avoid unwrapping the
 Activity context.

---
 .../messages/impl/media/local/AndroidLocalMediaActions.kt     | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/features/messages/impl/src/main/kotlin/io/element/android/features/messages/impl/media/local/AndroidLocalMediaActions.kt b/features/messages/impl/src/main/kotlin/io/element/android/features/messages/impl/media/local/AndroidLocalMediaActions.kt
index 3457ebc9ec0..ef93574134d 100644
--- a/features/messages/impl/src/main/kotlin/io/element/android/features/messages/impl/media/local/AndroidLocalMediaActions.kt
+++ b/features/messages/impl/src/main/kotlin/io/element/android/features/messages/impl/media/local/AndroidLocalMediaActions.kt
@@ -72,7 +72,7 @@ class AndroidLocalMediaActions @Inject constructor(
             if (activityResult.resultCode == Activity.RESULT_OK) {
                 pendingMedia?.let {
                     coroutineScope.launch {
-                        open(it)
+                        openFile(it)
                     }
                 }
             } else {
@@ -152,7 +152,7 @@ class AndroidLocalMediaActions @Inject constructor(
             .setFlags(Intent.FLAG_GRANT_READ_URI_PERMISSION)
             .setDataAndType(localMedia.toShareableUri(), localMedia.info.mimeType)
         withContext(coroutineDispatchers.main) {
-            activityContext!!.startActivity(openMediaIntent)
+            activityContext?.startActivity(openMediaIntent)
         }
     }