Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GetImagesList "name must not be null" #53

Closed
amensia opened this issue Dec 27, 2022 · 2 comments · Fixed by #57
Closed

GetImagesList "name must not be null" #53

amensia opened this issue Dec 27, 2022 · 2 comments · Fixed by #57
Labels
bug Something isn't working

Comments

@amensia
Copy link

amensia commented Dec 27, 2022

Describe the bug
Hello, thanks again for the library, work really well ;)

I have a couple of crash on production linked to Extension.getImageList()
when name == null

I couldn't reproduce it, just a couple of stacktrace from crqashlytics

Fatal Exception: java.lang.NullPointerException: name must not be null at com.app.imagepickerlibrary.ExtensionsKt$getImagesList$2.invokeSuspend(Extensions.kt:253) at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33) at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:106) at kotlinx.coroutines.internal.LimitedDispatcher.run(LimitedDispatcher.kt:42) at kotlinx.coroutines.scheduling.TaskImpl.run(Tasks.kt:95) at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(CoroutineScheduler.java:570) at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.executeTask(CoroutineScheduler.kt:750) at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.runWorker(CoroutineScheduler.kt:677) at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(CoroutineScheduler.kt:664)

I didn't really dig into it, and I don't know how important the name is for a picture, but I was wondering if just avoiding picture if any params of the cursor is null could fix it without creating too many unwanted behavior

Regards

@priyankj-simform
Copy link
Collaborator

@amensia Thanks for the stack trace. Will check the issue and update you soon!

@priyankj-simform
Copy link
Collaborator

@amensia Closing the issue as new version 2.1 is released with the fix. Please feel free to reopen this if you still face the issue with the latest version of the library.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants