ShareIntentReceiverActivity - Filter media types to allow only images…

… and videos
wordpress-mobile · Dec 7, 2023 · c5f7c68 · c5f7c68
1 parent 7482ba3
commit c5f7c68
Showing 1 changed file with 11 additions and 2 deletions.
diff --git a/WordPress/src/main/java/org/wordpress/android/ui/ShareIntentReceiverActivity.java b/WordPress/src/main/java/org/wordpress/android/ui/ShareIntentReceiverActivity.java
@@ -97,14 +97,23 @@ private void downloadExternalMedia() {
         if (Intent.ACTION_SEND_MULTIPLE.equals(getIntent().getAction())) {
             ArrayList<Uri> externalUris = getIntent().getParcelableArrayListExtra((Intent.EXTRA_STREAM));
             for (Uri uri : externalUris) {
-                mLocalMediaUris.add(MediaUtils.downloadExternalMedia(this, uri));
+                if (isAllowedMediaType(uri)) {
+                    mLocalMediaUris.add(MediaUtils.downloadExternalMedia(this, uri));
+                }
             }
         } else if (Intent.ACTION_SEND.equals(getIntent().getAction())) {
             Uri externalUri = getIntent().getParcelableExtra(Intent.EXTRA_STREAM);
-            mLocalMediaUris.add(MediaUtils.downloadExternalMedia(this, externalUri));
+            if (isAllowedMediaType(externalUri)) {
+                mLocalMediaUris.add(MediaUtils.downloadExternalMedia(this, externalUri));
+            }
         }
     }
 
+    private boolean isAllowedMediaType(@NonNull Uri uri) {
+        String filePath = MediaUtils.getRealPathFromURI(this, uri);
+        return MediaUtils.isValidImage(filePath) || MediaUtils.isVideo(filePath);
+    }
+
     private void initShareFragment() {
         ShareIntentReceiverFragment shareIntentReceiverFragment = ShareIntentReceiverFragment
                 .newInstance(!isSharingText(), loadLastUsedBlogLocalId());