[GitHub] [cordova-plugin-file] MauriceFrank commented on pull request #566: Add READ_MEDIA - permissions for Android 13+

["MauriceFrank (via GitHub)" <gi...@apache.org>]

MauriceFrank commented on PR #566:
URL: https://github.com/apache/cordova-plugin-file/pull/566#issuecomment-1531308665

   > I enabled test but they will likely fail on Android because it will depend on the platform using Compile SDK 33. The Android should be resolved when cordova-android@12 is released.
   
   Actually I just retested and it works fine while just requesting one permission at a time (READ_MEDIA_IMAGES), but it fails reading (file.listDir() with folder "DCIM") when trying to request all at once. Might be a syntax mistake in the PR or another issue. 
   
   Shall I update it or close it? For my own usage, it is enough to request the READ_MEDIA_IMAGES. However, for other users it might not be enough, since it should work with all types of Media. That's why I have tried with this PR to request all needed permissions but obviously it isn't as simple.
   
   As you suggested, the better approach for accessing external storage would be to not use the file-plugin at all anymore and rather use (if existing) a cordova plugin using the MediaStore API.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org