Fix: File is not a picture (on Android 10 or above)

[jmcrox]

• Fix getCaptureImageOutputUri(Context): Uri to return same file in each call on Android 10 or above.

close #21

Bug

Cause:

On Android 10 or above there is a problem when ask Uri to dump camera image. The Uri passed to the camera intent via MediaStore.EXTRA_OUTPUT is different to the Uri when we are going to read the image. This is because the method was creating a temporary file in each call.

Reproduce

Run project test module on any Android 10 or above and select camere to obtain the image.

How the bug was solved:

The bug was solved changing the Uri returned when on Android 10 or above. Now, the Uri returned is build like Android 9 or lower. That is, the returned Uri is new File(getImage.getPath(), "pickImageResult.jpeg")

[Canato]

@jmcrox Thanks for the fix!!!

Sadly I could not reproduce the bug to test the fix.
Device: Pixel 3, OS 11

• Run test module
• Click Start Activity
• Choose from images and crop
• Result: Image cropped, not issues.

Maybe someone else could take a look?
@VEIKKO99 @CanHub/imagecropcollaborators

Or maybe you can add a bug or specific emulator I can test =D

[jmcrox]

Sorry, I forgot to mention that the problem is when you use the camera option because getCaptureImageOutputUri(Context): Uri is called twice (before and after camera capture) with different result. I have edited my first post.

[VEIKKO99]

Maybe someone else could take a look?
@VEIKKO99 @CanHub/imagecropcollaborators

Made a quick test and this fix seems to correct the bug. Thanks @jmcrox

@Canato Please make a release with this fix included. I think there are lot of devs waiting for this. Thanks!

[Canato]

Maybe someone else could take a look?
@VEIKKO99 @CanHub/imagecropcollaborators

Made a quick test and this fix seems to correct the bug. Thanks @jmcrox

@Canato Please make a release with this fix included. I think there are lot of devs waiting for this. Thanks!

\o/

@jmcrox please add the fix to the changelog and we can merge for the release

[jmcrox]

Ok @Canato. CHANGELOG.md updated.

Thanks.