🎁 IiifPrint Enhancements to clean up split works as PDF fileset is deleted

[ShanaLMoore]

Note: using this update requires running rails iiif_print:install:migrations in your application to pull in the additional database migration.

related to

• 🎁 add helpful query attributes to iiif_print_pending_relationships adventist-dl#656

• Add parent_model, child_model, and file_id to the iiif_print_pending_relationships table for easier queries.
  The goal here is twofold:

  1. Add file_id of the triggering PDF's fileset, to be able to tie pending relationships to a specific PDF file, rather than a work, as one work may have multiple PDF attachments begin split.
  2. Add parent & child model, to have everything needed in the PendingRelationship table to rerun a relationship job which may have run out of retries

• Add service to destroy child works and pending relationships as a PDF fileset is destroyed.
  Future enhancements could include:

  1. Submitting the service as a background job
  2. Create a way to do the PDF splitting separate from an import, using this service to do the cleanup.

• Removes IiifPrint::PendingRelationships table entries when relationships are formed without errors, regardless whether the count matches.
  Errors are still reported on the job, so followup can be done if needed, but the relationship table entries are unnecessary in this situation.

• Destroys spawned child works as file_set is destroyed. Child works which were NOT created as part of PDF splitting will not be affected by this.

Screenshots

Work with two PDF filesets, both split into children

Work after one of the filesets is deleted through the UI

NOTES

ref: https://docs.google.com/document/d/1QPeT3bLHQkWWkhX5SQ9Pntgtlm-bqlUxoz5E5Xt1tRI/edit

[ShanaLMoore]

TODO: Add file set id to pending table

shared work on pull request

[ShanaLMoore]

maybe consider renaming this file since it's doing more than setting the child flag now. I suppose that would be a breaking change so we could also cut a new release?

Now that I say that, maybe that can be a TODO for later given the urgency 😅

[laritakr]

lol yeah I considered that but since it's included in EVERY repo that uses IiifPrint, it's def a breaking change which is why I left it. Plus it's still a type of child flag if you stretch hard enough. ;)