fix: RecursiveSerializer- outputs undeclared prefix for predicates that contains the base as a substring #3267
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary of changes
Fixes: #3160
Affected serializers:
turtle
longturtle
n3
trig
The serializer creates two auto-generated prefix names (
ns1
,ns2
, etc) for the same IRI because therelativize
method in the superclass returns a "new" IRI. This indirectly creates another prefix name for the same term, but for its relative form. We override therelative
method to prevent it from creating these relative terms unless the base namespace matches exactly.Checklist
the same change.
./examples
.so maintainers can fix minor issues and keep your PR up to date.