Citations

[lorawoodford]

Include container information and resource identifier in PUI citations when available. This is an update/extension of the work begun in #1555 .

Description

Related JIRA Ticket or GitHub Issue

https://archivesspace.atlassian.net/browse/ANW-288
https://archivesspace.atlassian.net/browse/ANW-295

Motivation and Context

The archival object and resource citations in the PUI were missing both the resource identifier and physical storage information both of which should be included.

How Has This Been Tested?

Visually inspected and existing tests pass.

Screenshots (if appropriate):

Resource with no physical containers:

Series with no physical containers:

File with physical containers:

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

[cdibella]

@lorawoodford - looks good, but a few questions:

• What does it look like when there are multiple containers (per our discussion of including up to 5)
• What happens if there's no CUID?
• What happens if there's a preferred citation note?

[lorawoodford]

@cdibella

What does it look like when there are multiple containers (per our discussion of including up to 5)
With 5 instances:

With 6 instances:

What happens if there's no CUID?

What happens if there's a preferred citation note?

Preferred citation note ALWAYS overrides all of this. If a preferred citation note is found, that is what is displayed.

[cdibella]

Looks good! Thanks for the additional context.