process articles with multiple imprints in preview#638
process articles with multiple imprints in preview#638SteffenAstheimer wants to merge 9 commits intodevelopfrom
Conversation
| declare function bibl:printJournalCitation($monogr as element(tei:monogr), $wrapperElement as element(), $lang as xs:string) as element()? { | ||
| if (count($monogr/tei:imprint) = 1) then | ||
| bibl:print-journal-citation-from-imprint($monogr, $monogr/tei:imprint, $wrapperElement, $lang) | ||
| else () |
There was a problem hiding this comment.
I'm not sure whether this is really what we want since this will only work for journal citations (in general) when used with the preview-biblio.html where the missing imprints are output.
All other calls to this function (e.g. from letter editorials) will have empty imprints.
I know this is tricky to solve, I just wanted to make sure about the consequences …
There was a problem hiding this comment.
Yeah, that's definitely a catch. I guess so far it works, since we only have multiple imprints in our bibliography data, and nowhere else. But in theory it should be allowed elsewhere.
We could call the new helper function bibl:printJournalCitationsByImprint from within document.html where the sources are created. This is the only other place where we might want to display multiple imprints that I can think of (apart from the not yet existing single view html page):
WeGA-WebApp/templates/document.html
Lines 226 to 230 in 6365337
There was a problem hiding this comment.
How about this:
- We return to having one
bibl:printJournalCitation()function which handles all occurrences of journal citations, both with single or multiple imprints. Handling multiple imprints should be seen as fallback behaviour. All imprints simply get concatenated and the whole journal citation is classified with a "deleteme" class (for step 2):
Lines 177 to 193 in c77eea0
- In
app:preview-citation()we erase all deleteme attributes to prevent both journal title and issues to appear in the citation string:
Lines 1977 to 1989 in c77eea0
- We use the bug of leftover "in: " strings as a feature and print all issues of the serial publication as list items without the leading "erschienen in: " header:
Alternatively we could surely get rid of the "in: " and return to having a section header "erschienen in: ".
What do you think, @peterstadler?
This will handle articles with multiple imprints, omit the journal title from the citation and add a list of all journal and issues in which the text was published.
Here an example:

And one example where we even have links to the texts and might think about adding the links to the actual issue titles in the future:
