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

Properly validate context and support additional context definitions #12

Closed
acka47 opened this issue Sep 2, 2020 · 4 comments · Fixed by #16
Closed

Properly validate context and support additional context definitions #12

acka47 opened this issue Sep 2, 2020 · 4 comments · Fixed by #16
Assignees

Comments

@acka47
Copy link
Member

acka47 commented Sep 2, 2020

I think these are currently not validated as incorrect.

@acka47
Copy link
Member Author

acka47 commented Sep 8, 2020

Das Problem ist, dass bisher der Context nicht vernünftig validiert wird. Fehlen z.B. die SKOS properties ("prefLabel": "skos:prefLabel" & "inScheme": "skos:inScheme"), dann wird das bei der Validierung nicht entdeckt.

Eine Möglichkeit, damit umzugehen, ist eine Aktualisierung des Schemas wie in ca2236b.

Eine zweite – eventuell elegantere – Möglichkeit ist die Publikation eines externen Kontext, der dann im Schema referenziert wird und entsprechend validiert wird.

Eine dritte Möglichkeit ist dann natürlich, beide Optionen zu erlauben.

acka47 added a commit that referenced this issue Sep 8, 2020
updating schema and examples,
ref #12
acka47 added a commit that referenced this issue Sep 8, 2020
@acka47
Copy link
Member Author

acka47 commented Sep 8, 2020

Im Branch 12b-contextTest habe ich mal die Lösung mit einem externen @context-Dokument umgesetzt und zwar so, dass optional auch ein weiterer Kontext referenziert werden kann oder auch nur ein einzelnes Key-URI-Mapping ergänzt werden kann wie in Beispiel 20 in der 1.1 Spec.

Das wäre meine präferierte Lösung.

@acka47 acka47 changed the title Add test file with missing skos properties in context Properly validate context and support additional context definitions Sep 8, 2020
@acka47
Copy link
Member Author

acka47 commented Sep 9, 2020

Mir fiel gerade auf, dass ja im Schema nur Concept and inSchemeaus dem SKOS Namespace erwähnt werden. Dementsprechend sollten wir prefLabel aus dem Context entfernen. Dies würde es Anwendungen auch überlassen, ob sie dafür kein Sprachtag, das Default-Sprachtag oder eine language map benutzen wollen.

@acka47
Copy link
Member Author

acka47 commented Sep 10, 2020

Dementsprechend sollten wir prefLabel aus dem Context entfernen.

Um die Anforderungen in #18 zu erfüllen, bleibt wird prefLabel im Kontext bleiben und um "@container": "@language" ergänzt, siehe 12499f9#diff-447f8484035cf0d63b4dea54fed16bec.

@acka47 acka47 added this to AMB Jul 8, 2024
@acka47 acka47 moved this to Done in AMB Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
1 participant