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

Fix display video duration cause app crash when targeting to sdk 29 and fix display media file under root in folder mode #331

Merged
merged 3 commits into from Mar 2, 2021

Conversation

wiwuwa
Copy link
Contributor

@wiwuwa wiwuwa commented Feb 18, 2021

Fix Android Q cannot open gallery when the directory contains video

@wiwuwa wiwuwa changed the title Fix display video duration cause app crash when targeting to sdk 29 Fix display video duration cause app crash when targeting to sdk 29 and fix display media file under root in folder mode Feb 24, 2021
@wiwuwa
Copy link
Contributor Author

wiwuwa commented Feb 24, 2021

Fix cannot display media file under root in folder mode
#333 (comment)

@bipinvaylu
Copy link

@esafirm I am also facing related issue in latest version 2.4.2 version as well. For now, I have to fix this issue by using this open PR code. It would be nice if you merge this PR or just notify once this PR target issue fixed. So i shall move back to dependency instead of using as sub-module (with this PR code).

cc: @hjchin @sr1dh4r @xpdlf1004 @LinX64

@esafirm
Copy link
Owner

esafirm commented Mar 2, 2021

Thanks @wiwuwa LGTM 👍

@esafirm esafirm merged commit a26b0f6 into esafirm:main Mar 2, 2021
@siva-komaragiri
Copy link

This issue still happened with 2.4.5, retriever.extractMetadata(MediaMetadataRetriever.METADATA_KEY_DURATION) is getting null which resulted in NumberFormatException with Long.parseLong

Running on Android 10 with app targeting SDK version 30

@esafirm
Copy link
Owner

esafirm commented Apr 23, 2021

@siva-komaragiri will add the fix to version 3.0.0

@tiny2021
Copy link

tiny2021 commented May 7, 2021

There have other issue related to this
#357

india2sarthak pushed a commit to india2sarthak/android-image-picker that referenced this pull request Sep 1, 2023
…nd fix display media file under root in folder mode (esafirm#331)

* Fix display video duration app crash bug for targeting to sdk 29

* Fix cannot display media file right under root directory in Folder mode (esafirm#333)

* simplify further null-proof the code (suggested by esafirm)

Co-authored-by: sli <sli@nexify.com.hk>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants