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

add an option to the sync command to 're-sync' images that already exist #588

Merged
merged 5 commits into from
May 26, 2020

Conversation

alex-phillips
Copy link
Contributor

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.

@codecov-commenter
Copy link

codecov-commenter commented May 23, 2020

Codecov Report

Merging #588 into master will decrease coverage by 0.11%.
The diff coverage is 41.17%.

Comment on lines 812 to 815
$info = $this->metadataExtractor->bare();
$this->metadataExtractor->size($info, $path);
$this->metadataExtractor->validate($info);
$info['type'] = 'raw';
Copy link
Contributor

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

$info = $this->metadataExtractor->extract($path, $mimeType);
if ($kind == 'raw') {
$info['type'] = 'raw';
}

Copy link
Contributor Author

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

@alex-phillips
Copy link
Contributor Author

It looks like this may have been an issue downloading EXIFtool and not the code changes?

@sonarcloud
Copy link

sonarcloud bot commented May 26, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
2.4% 2.4% Duplication

@ildyria ildyria merged commit 737c7a0 into LycheeOrg:master May 26, 2020
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.

4 participants