Skip to content

Check more carefully for 'isFavorite' since it's not always there#1060

Merged
AndreyNikiforov merged 1 commit intoicloud-photos-downloader:masterfrom
euank:isFavoriteCheck
Jan 18, 2025
Merged

Check more carefully for 'isFavorite' since it's not always there#1060
AndreyNikiforov merged 1 commit intoicloud-photos-downloader:masterfrom
euank:isFavoriteCheck

Conversation

@euank
Copy link
Copy Markdown
Contributor

@euank euank commented Jan 18, 2025

Fixes #1058, see that issue for an example of a file where this would throw an exception before.

I've added a unit test which errored before this change as well.

Fixes icloud-photos-downloader#1058, see that issue for an example of a file where this would
throw an exception before.
@AndreyNikiforov AndreyNikiforov merged commit 01a921c into icloud-photos-downloader:master Jan 18, 2025
rhoopr referenced this pull request in rhoopr/kei Feb 8, 2026
…58 (#1060)

Fixes #1058, see that issue for an example of a file where this would
throw an exception before.
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.

Crash accessing isFavorite for --xmp-sidecar

2 participants