-
-
Notifications
You must be signed in to change notification settings - Fork 38
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
author pages should link to works? #79
Comments
I've implemented this on the QA site: http://medieval-qa.bodleian.ox.ac.uk/catalog/person_84812388 It was also a request during the Fihrist training but I'm currently overhauling the indexing scripts for Medieval, which I'll then adapt for Fihrist and the rest. It is only with the more efficient XQuery developed for #148 that this is feasible. If an author has only got one work then the link takes you to a page with the same list of manuscripts (or a subset if they have another role as well as author) but with the changes for #34 there is a chance of there being notes, external links or a bibliography on the work page (e.g. http://medieval-qa.bodleian.ox.ac.uk/catalog/person_72430265.) And at the other extreme, for the most prolific authors, it pushes the links to manuscripts a long way down the page: http://medieval-qa.bodleian.ox.ac.uk/catalog/person_66806872 The reverse - links to authors from works - could also be added. |
Works now link to authors on QA. |
Get associations from works authority files as well as manuscript TEI files. See #79.
I am waiting for feedback from Martin on this and #34.
|
Sorting of works has been by ID (alphabetically, so work_1 and work_100 came before work_2 and then work_200) but I've recently figured out how to stop Blacklight doing that, and can now control the order of all such lists in the XQuery. So now they're sorted alphabetically by the link text (which is the The list is generated by combining both the association of authors to works in the works.xml authority file and the association of authors to works by being in the same I've set it up that way because then it will work for other catalogues, which don't have any author-to-work associations in their authority files. But I can change it for Medieval so that only the authority files are used. Doing so would reduce the total number of work links on all the person pages from 120,926 to 120,333 (and the total number of author links on work pages from 151,751 to 151,707.) |
I think approach (b) just using the works files is preferable; it will miss a few things (but many of them are not true 'works' in any case), but it will avoid unexpected combinations, such as the co-occurence of Virgil and Pseudo Ovid in this record (http://medieval-qa.bodleian.ox.ac.uk/catalog/manuscript_3921), item 2, which turns Ps-Ovid into an author of the Georgics here (http://medieval-qa.bodleian.ox.ac.uk/catalog/person_285292080) |
I've made that change. Should people be likewise only included as an Author in the Roles filter if they're in the works.xml authority file? For example, Corsini has no works listed despite being included as an author in the role filter of the people search because he has been tagged as |
Roles should contine to be based on the manuscript records - because, as you say, there are some people identified as authors in the manuscript records who don't, for various reasons, have works authority files |
One additional question is whether |
OK, I'll add "Translator of" lists. |
This has been implemented on QA. For example... http://medieval-qa.bodleian.ox.ac.uk/catalog/person_35253721 ...has a "Translator of" field with a link to... http://medieval-qa.bodleian.ox.ac.uk/catalog/work_375 ...which has a "Translator(s)" field with a link back again. As discussed above with authors, this is based solely on |
This would be helpful, since it is currently possible to miss index entries for works due to variant spellings.
E.g., search for "mirfield" brings up the entry for "Mirfeld, John, -1407" (as the variant spelling is captured there), but not the work entry Mirfeld, John, -1407: Breuiarium bartholomaei [Latin] (because the variant spelling is not captured there)
The text was updated successfully, but these errors were encountered: