Android-Image-Cropper icon indicating copy to clipboard operation
Android-Image-Cropper copied to clipboard

2.8.0 requires project to use androidx

Open matthewkrueger opened this issue 5 years ago • 21 comments

There is a bug/crash in version 2.7.0 that's affecting the production version of our app. I read that this bug was fixed in 2.8.0.

I have tried to update to the 2.8.0 version of Android-Image-Cropper, but it appears you switched to androidx, which would require my entire project to be upgraded to androidx. I cannot do that at this time, but the crash in the 2.7.0 image cropper code is causing big issues in production.

There is nothing in your documentation about requiring androidx to use this library. That's the first step - please update your docs as this took quite a while to narrow down.

Secondly, how can I get this library to work alongside other non-androidx projects?

matthewkrueger avatar Dec 05 '18 14:12 matthewkrueger

+1 here

manuelrego27 avatar Dec 05 '18 17:12 manuelrego27

I love this library and all the work you've done on it, but forcing established apps to upgrade to androidx may not be the right call, especially since 2.8.0 included a critical bug fix that is NEEDED for Android Pie users to use the library without crashing. If there's a way to make it co-exist with the old support library, I think that would be a much better option. I can't imagine that most app developers can upgrade big apps to androidx yet.

Over the last week in production, we've had over 150 users experience a crash with the image cropper. If there is no easy solution, I'm going to have to fork this library, or switch to a different library. Thanks for your time to read and consider this.

matthewkrueger avatar Dec 05 '18 18:12 matthewkrueger

i share your opinion @matthewkrueger. i was working on a project and had to pause because i traveled for some work. i tried continuing my project this morning and i'm still stuck with android manifest errors. upgrading my entire project to androidx will be difficult for now.

MBEDOBE avatar Dec 10 '18 13:12 MBEDOBE

Hi, anyone with a workaround, just in case?

unludo avatar Dec 23 '18 21:12 unludo

you can use the previous version though it's not as the latest version.

On Sun, 23 Dec 2018 21:28 unludo <[email protected] wrote:

Hi, anyone with a workaround, just in case?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ArthurHub/Android-Image-Cropper/issues/633#issuecomment-449664292, or mute the thread https://github.com/notifications/unsubscribe-auth/AYEtYJ-gWG8kVtc0-_reSH49uHTmXItRks5u7_WYgaJpZM4ZC47Z .

MBEDOBE avatar Dec 23 '18 21:12 MBEDOBE

No, it's crashing on android pie, it's blocking.

unludo avatar Dec 23 '18 21:12 unludo

that's true. with Android pie, the previous version crashes

On Sun, 23 Dec 2018 21:36 unludo <[email protected] wrote:

No, it's crashing on android pie, it's blocking.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ArthurHub/Android-Image-Cropper/issues/633#issuecomment-449664637, or mute the thread https://github.com/notifications/unsubscribe-auth/AYEtYBOoeDbj5nyesp8A8bGO5uu7dvJkks5u7_dJgaJpZM4ZC47Z .

MBEDOBE avatar Dec 23 '18 21:12 MBEDOBE

try: implementation 'com.github.svran:Android-Image-Cropper:2.8.0.4' i forked this project and change to "com.android.support" libary.

svran avatar Dec 26 '18 09:12 svran

ok let me try it out. I'll get back to you😊

On Wed, 26 Dec 2018 09:30 LiulianRan <[email protected] wrote:

try: implementation 'com.github.svran:Android-Image-Cropper:2.8.0.2' i forked this project and change to "com.android.support" libary.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ArthurHub/Android-Image-Cropper/issues/633#issuecomment-449938025, or mute the thread https://github.com/notifications/unsubscribe-auth/AYEtYBnsGg9K_2AC7u00MQT7d3lvez5Lks5u80GpgaJpZM4ZC47Z .

MBEDOBE avatar Dec 26 '18 09:12 MBEDOBE

sync failed: failed to resolve: 'com.github.svran:Android-Image-Cropper:2.8.0.2'

On Wed, 26 Dec 2018 09:30 LiulianRan <[email protected] wrote:

try: implementation 'com.github.svran:Android-Image-Cropper:2.8.0.2' i forked this project and change to "com.android.support" libary.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ArthurHub/Android-Image-Cropper/issues/633#issuecomment-449938025, or mute the thread https://github.com/notifications/unsubscribe-auth/AYEtYBnsGg9K_2AC7u00MQT7d3lvez5Lks5u80GpgaJpZM4ZC47Z .

MBEDOBE avatar Dec 26 '18 09:12 MBEDOBE

add this allprojects { repositories { jcenter() maven { url "https://jitpack.io" } } }

svran avatar Dec 26 '18 09:12 svran

gradle synced successfully but using it in the app throws a build error: package com.thartofdev.edmodo.cropper does not exist. I think the problem lies in the import statement.

On Wed, 26 Dec 2018 09:44 LiulianRan <[email protected] wrote:

add this allprojects { repositories { jcenter() maven { url "https://jitpack.io" } } }

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ArthurHub/Android-Image-Cropper/issues/633#issuecomment-449940206, or mute the thread https://github.com/notifications/unsubscribe-auth/AYEtYLe_uI8dENtJUvkazeOWaD_Ab34uks5u80UWgaJpZM4ZC47Z .

MBEDOBE avatar Dec 26 '18 09:12 MBEDOBE

i only change androidX to support libary ,Did the original author modify the api?

svran avatar Dec 26 '18 10:12 svran

no did you test it in an app?

On Wed, 26 Dec 2018 10:09 LiulianRan <[email protected] wrote:

i only change androidX to support libary ,Did the original author modify the api?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ArthurHub/Android-Image-Cropper/issues/633#issuecomment-449943659, or mute the thread https://github.com/notifications/unsubscribe-auth/AYEtYMIjPeHpOhKPp0h7zOUH9Yof5MV9ks5u80rxgaJpZM4ZC47Z .

MBEDOBE avatar Dec 26 '18 10:12 MBEDOBE

this is what happens when gradle syncs successfully. image attached below.

On Wed, 26 Dec 2018 10:09 LiulianRan <[email protected] wrote:

i only change androidX to support libary ,Did the original author modify the api?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ArthurHub/Android-Image-Cropper/issues/633#issuecomment-449943659, or mute the thread https://github.com/notifications/unsubscribe-auth/AYEtYMIjPeHpOhKPp0h7zOUH9Yof5MV9ks5u80rxgaJpZM4ZC47Z .

MBEDOBE avatar Dec 26 '18 10:12 MBEDOBE

now try: implementation 'com.github.svran:Android-Image-Cropper:2.8.0.4' Sorry, I made a mistake in uploading the libary file.

svran avatar Dec 26 '18 11:12 svran

yh it's working well now! gracias

On Wed, 26 Dec 2018 11:14 LiulianRan <[email protected] wrote:

now try: implementation 'com.github.svran:Android-Image-Cropper:2.8.0.4' Sorry, I made a mistake in uploading the libary file.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ArthurHub/Android-Image-Cropper/issues/633#issuecomment-449951581, or mute the thread https://github.com/notifications/unsubscribe-auth/AYEtYDNzC4JWfAWhTVwTNpRlZzar3sRuks5u81oFgaJpZM4ZC47Z .

MBEDOBE avatar Dec 26 '18 12:12 MBEDOBE

now try: implementation 'com.github.svran:Android-Image-Cropper:2.8.0.4' Sorry, I made a mistake in uploading the libary file.

when I build , it need com.android.support:exifinterface:28.0.0,when I add api 'com.android.support:exifinterface:28.0.0', This bitch get this error :Unable to resolve dependency for ':app@releaseUnitTest/compileClasspath': Could not download exifinterface.aar, so how to reslove this ?

dazeGitHub avatar Dec 28 '18 02:12 dazeGitHub

now try: implementation 'com.github.svran:Android-Image-Cropper:2.8.0.4' Sorry, I made a mistake in uploading the libary file.

when I build , it need com.android.support:exifinterface:28.0.0,when I add api 'com.android.support:exifinterface:28.0.0', This bitch get this error :Unable to resolve dependency for ':app@releaseUnitTest/compileClasspath': Could not download exifinterface.aar, so how to reslove this ?

if only could not download ,add "google()" in your project build.gradle file. if exifinterface the version conflicts. use: implementation ( 'com.github.svran:Android-Image-Cropper:2.8.0.4' ) { exclude group: 'com.android.support' ,module: 'exifinterface'} And use the exifinterface version of your project. implementation 'com.android.support:exifinterface:your_project_support-libary_version'

svran avatar Dec 28 '18 02:12 svran

Adding implementation com.github.svran:Android-Image-Cropper:2.8.0.4 solved the problem for me, thanks @svran

raphaelbgr avatar Feb 12 '19 18:02 raphaelbgr

now try: implementation 'com.github.svran:Android-Image-Cropper:2.8.0.4' Sorry, I made a mistake in uploading the libary file.

Problem solved without migrating to AndroidX! Thanks @svran

davidalonso avatar Oct 09 '19 16:10 davidalonso