-
Notifications
You must be signed in to change notification settings - Fork 9.8k
[image_picker] Fix a crash when picking video on iOS 13 and above. #2083
[image_picker] Fix a crash when picking video on iOS 13 and above. #2083
Conversation
e80fe21
to
637c4f8
Compare
What maintainers waiting for? Plugin is unusable on iOS 13 without this patch! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for providing this PR. I left some comments.
cbe9f36
to
e70e5f5
Compare
@miguelpruivo @cyanglaz please be aware that this fix introduce a new issue when source is camera(not gallery). |
Hi, thank you @ailic88, do you mind to provide more details? Edit: You are completely right, good catch @ailic88. It was because the recorded videos from camera don't actually need to be cached and thus the temporary path was exactly the same when caching (thus, an error was being thrown). Just committed a fix. Thank you. |
cf39c43
to
0991498
Compare
@miguelpruivo Please update the pubspec and changelog |
aa706f1
to
f7317b8
Compare
Done @cyanglaz. |
If only you guys knew how badly some of us flutter devs need the image_picker working properly on ios 13.Been waiting since September |
I still don’t get it why this is taking so long @trey-rosius. |
It's shame for official Flutter plugin to have CRITICAL issue are not fixed for such long time. Start thinking that Flutter is not right choice when you need reliability. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Let's fix the CI failure and land this.
@miguelpruivo Could you rebase the upstream/master, it might fix the CI. |
…gallery on iOS 13 or above
c678af6
to
c24ab81
Compare
Done @cyanglaz. |
@miguelpruivo Thanks, let's see if the CI passes. If not, I'll merge in #2275 and then we can try it again. |
Any Progress with this ? |
@cyanglaz is all good to go for a while. This should be merge ASAP to address severe crash issues on iOS 13 when picking from gallery. |
Description
Fixes an issue that could result in a crash when picking videos from gallery on iOS 13 or above.
Related Issues
#41046
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes (
[x]
). This will ensure a smooth and quick review process.///
).flutter analyze
) does not report any problems on my PR.Breaking Change