[GitHub] [cordova-plugin-media-capture] breautek commented on a change in pull request #215: [Android] Store image/audio/video in FileProvider due to Android 11 updates

[GitBox <gi...@apache.org>]

breautek commented on a change in pull request #215:
URL: https://github.com/apache/cordova-plugin-media-capture/pull/215#discussion_r613227436



##########
File path: src/android/FileProvider.java
##########
@@ -18,4 +18,4 @@ Licensed to the Apache Software Foundation (ASF) under one
 */
 package org.apache.cordova.mediacapture;
 
-public class FileProvider extends androidx.core.content.FileProvider {}

Review comment:
       Same as above

##########
File path: src/android/Capture.java
##########
@@ -59,7 +59,7 @@ Licensed to the Apache Software Foundation (ASF) under one
 import android.net.Uri;
 import android.os.Environment;
 import android.provider.MediaStore;
-import androidx.core.content.FileProvider;

Review comment:
       Our next major version of cordova-android will force using AndroidX, so it's probably best if we stick with androidx here.
   
   Cordova-android@9 supports `AndroidX` via the `AndroidXEnabled` preference.




-- 
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.

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