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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a SMT for OCFL path #2079

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Add a SMT for OCFL path #2079

wants to merge 4 commits into from

Conversation

whikloj
Copy link
Collaborator

@whikloj whikloj commented Apr 30, 2024

JIRA Ticket: https://fedora-repository.atlassian.net/browse/FCREPO-3935

What does this Pull Request do?

Generates a Server Managed Triple fedora:ocflPath with the full path to the base of the resource.

How should this be tested?

This is a bit hacky as the only way to know if something is in staging or the repo is to get that information from fcrepo-storage-ocfl and pass it back as part of the Resource Headers (I think).

That seemed like a large change, which I think involves to change the resource headers object (in fcrepo-storage-ocfl) to store the path and change the conversion code from OCFL resource headers to Fedora resource headers to set this path. Then change the ResourceFactory to transfer the path from the resource headers to the FedoraResource. Then we could still use the ManagedPropertiesService to set the triple using the path.

This makes some assumptions, which if we ever make the path configurable would make it invalid. 馃し

To test, build and run this code. Make an object in Fedora, view the metadata for that object and find a new http://fedora.info/definitions/v4/repository#ocflPath triple with the full path on disk.

For Archival Groups and their parts, they all point to the path of the Archival Group.

Interested parties

Tag (@ mention) interested parties or, if unsure, @fcrepo/committers

@whikloj whikloj marked this pull request as ready for review June 20, 2024 15:29
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.

None yet

1 participant