felicienfrancois commented on PR #814: URL: https://github.com/apache/cordova-plugin-camera/pull/814#issuecomment-1321802644
@ochakov I tested your fork in my project and got 2 issues : 1°) At compilation time `Element uses-permission#android.permission.WRITE_EXTERNAL_STORAGE at AndroidManifest.xml:59:5-108 duplicated with element declared at AndroidManifest.xml:51:5-81` It seems that manifest merger fail to merge `<uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE" android:maxSdkVersion="32" />` with `<uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE" />` from my other plugins. => removing android:maxSdkVersion="32" solved the compilation issue. 2°) At runtime when trying to capture photo (either from camera or library), on Android 13, I always get permission refused. After adding some logs in onRequestPermissionResult, I saw that the only permission requested is READ_MEDIA_IMAGES and the result is [-1] (PackageManager.PERMISSION_DENIED). It seems that READ_MEDIA_IMAGES permission does not have to be requested. => Removing Manifest.permission.READ_MEDIA_IMAGES from requested permissions in CameraLauncher:129-130 solve the issue -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
