[#793] Export Metadata Subfields Description And Additional Descriptions #823
Conversation
| @@ -88,6 +88,8 @@ def invenio_metadata(gf) | |||
| "creators": creators(gf.creator), | |||
| "title": gf.title.first, | |||
| "additional_titles": gf.title.last(gf.title.size-1).map{ |title| {"title": title, "type": "alternative_title", "lang": "eng"} }, | |||
| "description": gf.description.first, | |||
There was a problem hiding this comment.
I'm a bit frustrated. The last time I looked at this I misunderstood how it was storing descriptions based on what I was reading back to CSV. I took it again today with file ids included to try and help understand the data better and realized it was storing descriptions as an array. I guess it work out in the end /shrug.
| @@ -88,6 +88,8 @@ def invenio_metadata(gf) | |||
| "creators": creators(gf.creator), | |||
| "title": gf.title.first, | |||
| "additional_titles": gf.title.last(gf.title.size-1).map{ |title| {"title": title, "type": "alternative_title", "lang": "eng"} }, | |||
| "description": gf.description.first, | |||
| "additional_descriptions": gf.description.last(gf.description.size-1).map{ |add_desc| {"description": add_desc, "type": "other", "lang": "eng"} }, | |||
There was a problem hiding this comment.
I think leaving it as "other" to begin with will be fine. It's seemingly a required field so we need to put something, but really this will need to be looked over by a librarian I think.
There was a problem hiding this comment.
This is an odd one, the way descriptions are stored. Anyway, what if there is only one description stored? Will this break?
There was a problem hiding this comment.
I'll check, but I think that ["entry"].last(0) returns [] which is then [].map{} which again returns []
There was a problem hiding this comment.
Sorry, forgot to reply. What I said above is the case.
|
Small and to the point. |
closes #793