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: Do not panic on incorrect date format in EXIF #4

Closed
wants to merge 1 commit into from
Closed

Fix: Do not panic on incorrect date format in EXIF #4

wants to merge 1 commit into from

Conversation

egorz-eng
Copy link
Contributor

@TheLastGimbus thank you for the script, it helped me a lot!
I've successfully migrated 200GB of files from Google Photos.
There is one very annoying bug I've encountered: if the date format in EXIF is incorrect for some reason, the script exits without trying other sources.
This simple fix worked for me.

@TheLastGimbus
Copy link
Owner

TheLastGimbus commented Nov 25, 2020

Could you somehow send me certain folder/photo that caused you this problem?

If I remember (and wrote it) correctly, script should only quit when it sees incorrect album folder name, nof exif data. Does it also crash on wrong exif?

Because if it is about the folder name, then it is intentional feature, and it could cause a lot of mess if it didn't quit

@egorz-eng
Copy link
Contributor Author

Sure! I've tried my best to find the funniest one. crash.zip

@TheLastGimbus
Copy link
Owner

I think I resolved the problem other way, check #7, I mentioned you there. But you surely helped me in resolving this 😁

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

2 participants