bug(file-picker): type `application/json` is ignored on Android

[robingenz]

Plugin version:

• 0.6.1

Platform(s):

• Android

Current behavior:

Some users reported that application/json as a type is ignored by Android (see here).

Expected behavior:

application/json should not be ignored. Try to find a solution/workaround or document it.

Steps to reproduce:

Related code:

insert short code snippets here

Other information:

Capacitor doctor:

insert the output from `npx cap doctor` here

[dogoku]

@robingenz I may have found something related to this

I noticed that the docs for types prop have a mistake. They say explicitly that "this option is ignored, if limit is set."

But looking at the code, types is applied only when limit == 1

capacitor-plugins/packages/file-picker/android/src/main/java/io/capawesome/capacitorjs/plugins/filepicker/FilePickerPlugin.java

Line 102 in f95a3ba

if (limit == 1 && parsedTypes != null && parsedTypes.length > 0) {

Given that the default value of limit is 0 and given this warning in the docs, my initial assumption was that, if i set any value other than 0, then types will not work, when in reality, it's the opposite.

Once I set limit=1, then application/pdf filter works as intended.

So this lead me to believe that the above issue, could be just a misunderstanding due to the typo in the docs.

[robingenz]

@dogoku Feel free to give it a try but last time I tested it this was a real issue and unrelated to limit.

[dogoku]

I just realized that I misread the title. This issue is about application/json, not application/pdf.

Although my comment about the docs being incorrect is still valid, it's not the reason for this specific bug.

I'll try and send a small PR for changing the docs, if u are ok with it, so that others will not get confused like I did.

[robingenz]

@dogoku Sure, that would be great!