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

[iOS] Handle error when fetching thumbnail #1149

Merged
merged 1 commit into from
Jun 22, 2024

Conversation

maksg
Copy link
Contributor

@maksg maksg commented Jun 21, 2024

One of my videos can't generate a thumbnail and it causes the error Domain=PHPhotosErrorDomain Code=3303, resulting in fetchThumb never calling back.
We should handle this case probably the same way as when imageData is nil.

I'm attaching the video I have issue with.

Desktop.2024.02.17.-.15.21.48.01.MP4

@AlexV525

This comment was marked as off-topic.

@maksg
Copy link
Contributor Author

maksg commented Jun 21, 2024

This is the video that I can't fetch thumbnail from. Add it to your library and call fetchThumb on it, you should get an error and no callback.

ios/Classes/core/PMManager.m Outdated Show resolved Hide resolved
ios/Classes/core/PMManager.m Show resolved Hide resolved
ios/Classes/core/PMManager.m Outdated Show resolved Hide resolved
@maksg maksg force-pushed the ios-handle-thumbnail-error branch 2 times, most recently from d2e728d to ca666b7 Compare June 21, 2024 16:09
@maksg maksg force-pushed the ios-handle-thumbnail-error branch from ca666b7 to 8116585 Compare June 21, 2024 16:45
@AlexV525 AlexV525 merged commit 8491c36 into fluttercandies:main Jun 22, 2024
12 of 13 checks passed
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.

2 participants