Skip to content

Import all duplicates too#511

Merged
aurelienpierre merged 2 commits intoaurelienpierreeng:masterfrom
Jiyone:import_duplicate
May 13, 2025
Merged

Import all duplicates too#511
aurelienpierre merged 2 commits intoaurelienpierreeng:masterfrom
Jiyone:import_duplicate

Conversation

@Jiyone
Copy link
Copy Markdown
Collaborator

@Jiyone Jiyone commented Apr 9, 2025

This makes possible to import duplicates along with images if they exist.

  • When import & copy, it lists all XMPs with the image name and copy them to the destination directory.

    • in case of not found default XMP, the first duplicate found is copied as the default.
  • When import "in place", it just copy all XMPs with the image name.

    • in case of not found default XMP, It creates a new one.

At the end of each image import loop, it imports data from the XMPs found in the image directory, if they were not already imported.

I think I’ve got everything 🤔

@Jiyone Jiyone force-pushed the import_duplicate branch 6 times, most recently from fa9d4a2 to 8a6dbb5 Compare April 10, 2025 12:24
@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
1 Security Hotspot

See analysis details on SonarQube Cloud

Jiyone added 2 commits May 13, 2025 18:03
When import & copy, it lists all XMPs with the image name and copy them
to the destination directory.
- in case of not found default XMP, the first duplicate found is copied
  as the default.

if not importing by copy, it just copy all xmp with the image name.

at the end of each image import loop, it imports the XMPs found in the
image directory.
@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
1 Security Hotspot

See analysis details on SonarQube Cloud

@aurelienpierre aurelienpierre merged commit d42ff96 into aurelienpierreeng:master May 13, 2025
10 of 15 checks passed
@aurelienpierre
Copy link
Copy Markdown
Collaborator

Thanks !

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