Skip to content
This repository has been archived by the owner on Oct 24, 2022. It is now read-only.

METS author limits, date created, and TopicHub as agent #377

Merged
merged 2 commits into from
Oct 20, 2015

Conversation

JPrevost
Copy link
Member

This allows setting METS_RESTRICT_MAXIMUM_AUTHORS to an integer. If
there are more authors than that number, none are supplied in the METS
to prevent triggering a bug in our local DSpace installation.

Fixes a bug where we included a title instead of the date.

Adds dc.agent as a mechanism for detecting the package came from
TopicHub. If a better field is identified, this could be changed and we
can work with Carl to update the crosswalk that relies on this value.

closes #371
closes #370
closes #375

This allows setting METS_RESTRICT_MAXIMUM_AUTHORS to an integer. If
there are more authors than that number, none are supplied in the METS
to prevent triggering a bug in our local DSpace installation.

Fixes a bug where we included a title instead of the date.

Adds dc.agent as a mechanism for detecting the package came from
TopicHub. If a better field is identified, this could be changed and we
can work with Carl to update the crosswalk that relies on this value.

closes #371
closes #370
closes #375
@@ -214,6 +214,10 @@ case class Item(id: Int, // DB key
</epdcx:statement>
}

<epdcx:statement epdcx:propertyURI="http://purl.org/dc/elements/1.1/source">
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@richardrodgers can you confirm this is what we decided on? Thanks for the discussion and help determining a good choice for our needs here.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep - I believe it is. 👍

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@JPrevost
Copy link
Member Author

Before merging, I'm going to try to get an answer from Katharine about whether the catalogers prefer no authors or a limited subset when we pass the arbitrary trigger. If there is no clear answer, I'll merge as is and let them know to tell us if they want the behavior changed.

JPrevost added a commit that referenced this pull request Oct 20, 2015
METS author limits, date created, and TopicHub as agent
@JPrevost JPrevost merged commit 1deffa7 into master Oct 20, 2015
@JPrevost JPrevost deleted the 371_optionally_exclude_authors branch October 20, 2015 20:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants