-
Notifications
You must be signed in to change notification settings - Fork 642
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
ShaclParseException: No sh:path on a property shape #2129
Comments
https://www.w3.org/TR/shacl/#property-shapes
A property shape must have an The stacktrace is at parse time. Line 25 (which should be indented?) has sh:property with no sh:path. The Maybe |
From that same paragraph, the next sentence:
I thought this meant that it's fine without I would like to hear @HolgerKnublauch's opinon. From what I understand, the shapes from the
This my actually work. I always forget that skipping |
That text isn't wrong but it does not override or modify the previous text. "Exactly one" would be better. Elsewhere it says
If the code walks from the target start point it is encountered as a node shape. I can't see text that then makes the Jena parses all the shapes, not just reachable ones from targets, or ones encountered while validating the data.of the data. |
Jena fails to parse SHACL shapes due to missing `path`: ``` No sh:path on a property shape: node=<https://semiceu.github.io/DCAT-AP/releases/3.0.0/html/shacl/shapes.ttl#DatasetSeries_Shape> sh:property _:B1f2b5d2b47a093d15d2dd175eb75559e ``` Probably related to apache/jena#2129 Intellij also report an error on those rules: ``` Invalid cardinality: expected min 1: Got count = 0 Broader shape: shsh:PropertyShapeShape Shape: [__:o:9683 in shsh.ttl] Path: sh:path SHACL: sh:path Component: sh:MinCountConstraintComponent ``` For the series, I'm not 100% sure `sh:inversePath dcat:inSeries;` and `sh:path [ sh:inversePath dcat:inSeries; ];` are equivalent. Also when validating a series only, this rule will always fails when the dataset is not in the graph - but that's another discussion.
SEMICeu/DCAT-AP has made changes. |
Version
4.10
What happened?
When validating SHACL, Jena appears too strict when parsing property shapes, as it requires
sh:path
in all casesSee https://s.zazuko.com/3rcogrc. It should find a violation where the
schema:name
is commented out in data graph.Relevant output and stacktrace
The text was updated successfully, but these errors were encountered: