ACTION_GET_CONTENT with multiple MIME types

[moffatman]

Some apps provide multiple MIME types separated by either comma or pipe.

Without this, image/*,video/* only shows images.

Fixes #403

What is it?

• Bugfix
• Feature
• Codebase improvement

Description of the changes in your PR

• Parse ACTION_GET_CONTENT compound type
• Handle case of mIsGetImageContentIntent && mIsGetVideoContentIntent

Acknowledgement

• I read the contribution guidelines.

[naveensingh]

Before fixing a bug or adding a new feature, ensure it’s already reported or requested in the repository, open, and doesn’t have the needs triage label. If the issue is closed or has needs triage label, your PR will likely be rejected. The only exceptions are critical bugs that we weren't able to classify yet.

Thanks for the PR! You should create an issue first as stated in the contribution guidelines.

Some apps provide multiple MIME types separated by either comma or pipe.

Can you name some apps that actually do this?

[moffatman]

I created #403. I am opening this PR since a user of my app (https://github.com/moffatman/chan) complained about the media picker only showing images.

I use the Flutter file_picker plugin which offers a "media" filetype which turns into the "image/,video/" MIME.

You can see this technique is used in other places too

https://github.com/search?q=%22image%2F*%2Cvideo%2F*%22+language%3Ajava&type=code

[naveensingh]

Thanks!

Flutter file_picker plugin

I'm curious, is there no way to supply the types in a standard way like one can do with Intent.EXTRA_MIME_TYPES?

[moffatman]

In my opinion there are way too many intents and ways to do this on Android. ACTION_GET_CONTENT, ACTION_OPEN_DOCUMENT, ACTION_OPEN_DOCUMENT_TREE of parent dir, ACTION_PICK. This way I've found works quite well with the exception of your gallery app. Maybe EXTRA_MIME_TYPES could work everywhere too. But your app doesn't implement it either. So I prefer to leave it this way....