bondjimbond / islandora_westvault

Automated preservation for Islandora objects
GNU General Public License v3.0
1 stars 1 forks source link

Use traversal method from islandora_xacml_editor #9

Closed bondjimbond closed 5 years ago

bondjimbond commented 5 years ago

Trying now to copy the traversal settings from islandora_xacml_editor in the management form.

First goal is to actually get the options to appear on the form. So far not much success.

bondjimbond commented 5 years ago

@mjordan Did you see the invitation link by the way? You still aren't a collaborator.

mjordan commented 5 years ago

The form is showing up for me, in the traversal branch:

westvault_children

bondjimbond commented 5 years ago

@mjordan The comment at the start of this issue is somewhat outdated. :)

Right now we have very basic child object preservation on collections: https://github.com/bondjimbond/islandora_westvault/blob/traversal/includes/management_form.inc#L77-L85

Next steps required:

bondjimbond commented 5 years ago

I'm thinking about my strategy for dealing with collections. Here's my idea...

  1. When a collection is preserved, collection gets a PRESERVATION datastream. So do all its current children, if they haven't already got one. (Done.)
  2. Using hook_islandora_object_ingested(): when a new object is ingested, check its parent collection. IF parent has a PRESERVATION datastream, create a PRESERVATION datastream (run function islandora_westvault_add_preservation_datastream()).

Once the above is working, tweak somewhat:

  1. When preserving an object, IF it's a collection, check type of preservation (new children only, current children only, old and new children).
  2. Add a line into the PRESERVATION datastream: <collectionPreservationMethod>(SELECTED STRATEGY)</collectionPreservationMethod>
  3. Check preservation method before sending children through the preservation process
    1. When new objects are ingested, check parent for PRESERVATION datastream. IF datastream exists, read the preservation method and proceed accordingly.

Does that all make sense to you, @mjordan?

bondjimbond commented 5 years ago

Addressed with #20 - ignored islandora_xacml_editor and created my own approach.