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

Update config.xqm #13

Merged
merged 2 commits into from
May 26, 2021
Merged

Update config.xqm #13

merged 2 commits into from
May 26, 2021

Conversation

lauml
Copy link
Contributor

@lauml lauml commented Mar 2, 2021

fix xpath to include children of surnames and forenames for issue #12

fix xpath to include children of surnames and forenames for issue ediarum#12
@MartinFechner MartinFechner self-assigned this Mar 18, 2021
@MartinFechner
Copy link
Contributor

In #12 steht, dass /tei:forename/text() geändert werden soll in /tei:forename//text(). Die Änderung hier ist allerdings //tei:forename/text().

@lauml Kannst Du das bitte nochmal prüfen?

to include child elements of surname (such as name[@type='used'])
@lauml
Copy link
Contributor Author

lauml commented May 25, 2021

Du hast Recht, Danke für den Hinweis! Habe es angepasst und getestet. Rufnamen werden mit dem neuen XPATH angezeigt. Für surname ist es meiner Meinung nach nicht notwendig. Habe es also nur für forename angepasst.

@MartinFechner MartinFechner merged commit 8870533 into ediarum:master May 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants