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

adds plugin CopyArchiveTags #1021

Merged
merged 9 commits into from
Jul 28, 2024
Merged

adds plugin CopyArchiveTags #1021

merged 9 commits into from
Jul 28, 2024

Conversation

IceBreeze
Copy link
Contributor

I took advantage of the issue #966 to try some suggestions like:

  • exception handling for the plugins
  • named parameters

They are implemented inside the plugin because I think they are too invasive to be put elsewhere at the moment.

Also I added a little helper method to retrieve the tags from an archive and I slightly reworked various get_archive* methods internally, but I couldn't come up with a better name for h_get_archive for the one that needs the Redis instance to be passed.

lib/LANraragi/Plugin/Metadata/CopyArchiveTags.pm Outdated Show resolved Hide resolved
lib/LANraragi/Utils/Database.pm Outdated Show resolved Hide resolved
lib/LANraragi/Plugin/Metadata/CopyArchiveTags.pm Outdated Show resolved Hide resolved
lib/LANraragi/Plugin/Metadata/CopyArchiveTags.pm Outdated Show resolved Hide resolved
lib/LANraragi/Plugin/Metadata/CopyArchiveTags.pm Outdated Show resolved Hide resolved
lib/LANraragi/Utils/Database.pm Outdated Show resolved Hide resolved
lib/LANraragi/Utils/Database.pm Outdated Show resolved Hide resolved
Copy link

holopin-bot bot commented Jul 28, 2024

Congratulations @IceBreeze, the maintainer of this repository has issued you a holobyte! Here it is: https://holopin.io/holobyte/clz4te7lo43430ckztuus8vlq

This badge can only be claimed by you, so make sure that your GitHub account is linked to your Holopin account. You can manage those preferences here: https://holopin.io/account.
Or if you're new to Holopin, you can simply sign up with GitHub, which will do the trick!

@Difegue Difegue merged commit 1ed1757 into Difegue:dev Jul 28, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Introduce a plugin that can copy metadata from another archive in the db.
2 participants