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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

attributes lost on name[type='corporate'] #2

Closed
holfordm opened this issue Apr 25, 2017 · 10 comments
Closed

attributes lost on name[type='corporate'] #2

holfordm opened this issue Apr 25, 2017 · 10 comments
Assignees

Comments

@holfordm
Copy link
Collaborator

name type="corporate" has lost other attributes (e.g. @ref, @key, @role) on conversion to orgName. Example: Bodley_0065 at <origPlace>.

@jamescummings
Copy link
Contributor

Yeah, same problem as the persName issue, with the same question. I'll treat them alike.

@jamescummings
Copy link
Contributor

Ok, this is done in my script. I notice a related issue where there are lots of <persName> inside <name type="corporate">. I've not found one yet that I would call a person. Am I right that these should just disappear?
i.e.

<name type="corporate" role="formerOwner"><persName>Merton College</persName></name>

become

<orgName role="formerOwner">Merton College</name>

@holfordm
Copy link
Collaborator Author

holfordm commented Apr 25, 2017 via email

@jamescummings
Copy link
Contributor

Should I be getting rid of persName inside author? Most of the time there is only a persName (if there is one) in 7 cases it is also got another element there (often title.... which seems weird to me) i.e.

<author ref="http://viaf.org/viaf/99942145">
       <persName>Livy</persName>
       <title>Ab urbe condita</title>
 </author>

Should I try to move title outside of author? i.e.

<author ref="http://viaf.org/viaf/99942145">Livy</author>
<title>Ab urbe condita</title>

@holfordm
Copy link
Collaborator Author

holfordm commented Apr 25, 2017 via email

@jamescummings
Copy link
Contributor

Easy enough to add to the conversion a rule for author elements that have a title that moves it outside, and a rule for title elements inside author elements that makes them (otherwise) vanish. So no worries... just wanted to check it was the right thing to do.

@holfordm
Copy link
Collaborator Author

Might be worth me double checking in case there are other elements in there as well? e.g. note?

@jamescummings
Copy link
Contributor

Notes inside author don't really bother me.... but titles inside author are just nonsensical in my opinion. So I'm running the conversion again with that updated and hopefully the persName and orgName things straightened out.

@holfordm
Copy link
Collaborator Author

holfordm commented Apr 26, 2017 via email

@jamescummings jamescummings self-assigned this May 3, 2017
@jamescummings
Copy link
Contributor

I believe this is done, so I'm closing it. re-open if it is still a problem.

holfordm pushed a commit that referenced this issue Jun 3, 2019
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

No branches or pull requests

2 participants