-
-
Notifications
You must be signed in to change notification settings - Fork 300
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
add an option to the sync command to 're-sync' images that already exist #588
add an option to the sync command to 're-sync' images that already exist #588
Conversation
$info = $this->metadataExtractor->bare(); | ||
$this->metadataExtractor->size($info, $path); | ||
$this->metadataExtractor->validate($info); | ||
$info['type'] = 'raw'; |
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.
#574 changes support of raw files, i.e. metadata is also extracted for raw files if possible. See
Lychee/app/ModelFunctions/PhotoFunctions.php
Lines 286 to 289 in 32b460e
$info = $this->metadataExtractor->extract($path, $mimeType); | |
if ($kind == 'raw') { | |
$info['type'] = 'raw'; | |
} |
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.
@tmp-hallenser I adjusted my code to match the #574
It looks like this may have been an issue downloading EXIFtool and not the code changes? |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Since XMP sidecars are now allowed, images with the same checksum will not get updated metadata if the sidecar changes but the checksum remains the same. This adds an optional flag to the
lychee:sync
command that forces existing images with no change in checksum to get updated metadata from XMP files (if they exist). It only updates the image if the metadata read in differs from the metadata that the image aleady has.