-
Notifications
You must be signed in to change notification settings - Fork 8
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Some minor RIF-CS feed modifications #400
Comments
Is this something for the new company or is it urgent and needs my attention? |
Nope, not urgent. Terem can take care of it. |
Jono, I noticed that RDA isstill picking up only 245 of our 338 open collections. Can you please confirm that the above work in this ticket has been done and that al our collections are being served via RIF-CS? |
Can you please confirm that the above work in this ticket has been done and that al our collections are being served via RIF-CS? |
RDA is picking up all of our collections now |
The RIF-CS feed is currently broken as far as I can tell. The feed at: http://catalog.paradisec.org.au/oai/collection?verb=ListRecords&metadataPrefix=rif ...currently returns a 500 Internal Error Code. The harvest of records into research data Australia has been failing for quite a while now because of this. (Started poking around because RDA records are marked up with schema.org, and hence crawled by the new Google Datasets search. Unfortunately, not all PARADISEC records are showing up. So it would be good to fix this so that latest PARADISEC records show up.) |
supplanted by new issue #679 |
The suggestions that @jangari makes at the top of this issue still hold has sensible additions to the RIF-CS feed and are unrelated (as far as I can tell) to the issue of the feed breaking. There are more that could be added to this too. For instance, the inclusion of the DOI as an identifier for the collection. |
@nthieberger Are the initial improvements still worth implementing? |
We need to modify the code that generates the rif-cs feed to include more fields, and to account for some changes to our collections in RDA.
There are
threefour main changes:I'll go through these in detail below
The OAI-PMH feed currently maps the Data access rights field (of collections) to the RIF-CS element
/OAI-PMH/ListRecords/record/metadata/registryObjects/registryObject/collection/rights/accessRights
. We need to include the next field in the database, Data access details (which is the free text form of the access restrictions, like 'access by permission of depositor only', or whatever) in the element /rights/rightsStatement. As such, the rights element should look like this:Importantly, this should only happen if the Data access details field has content. If not, then do not supply the rightsStatement element.
I've created records in RDA for the NABU catalog, and one for the whole Paradisec collection which will have all collections as parts. As such, this chunk needs to be added to the OAI feed for every collection:
Ethnologue have changed their permalink structure and while the old structure automatically redirects to the new links, we shouldn't always expect that to be the case, and should update our links to reflect it.
Ethnologue links are generated from the ethnologue codes in the catalogue, as well as the standard language name (for the link title). The current structure is:
This should change to:
As we've moved offices from Sydney University to the Conservatorium, all records are actually outdated. The chunk below (which is in
/OAI-PMH/ListRecords/record/metadata/registryObjects/registryObject/collection/location/address
)needs to be updated to our current address. So change:to:
Similarly for the postal addresses of depositors(
/OAI-PMH/ListRecords/record/metadata/registryObjects/registryObject/party/location/address/physical
), which currently look like:Should be changed to look like:
All of the above is generated by collection.rb, and once you find the rif-cs section of that file (line 243 onwards) it should all be fairly straightforward.
If I come across any more changes, I'll add them to this issue.
The text was updated successfully, but these errors were encountered: