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

Add support for the short form of xsd values #34

Merged
merged 2 commits into from Feb 15, 2018
Merged

Add support for the short form of xsd values #34

merged 2 commits into from Feb 15, 2018

Conversation

niklas88
Copy link
Member

Instead of giving the full URL xsd values may use a short form "123"^^xsd:integer, add support for this. As before we don't really care for the URL but support some base types.

Tests are included

This requires searching for the ':' from the right because it may appear
in dateTime strings. Do the same for the '#' to keep it symmetric.
previously this error didn't have any consequences but with support for short
xsd names it's no longer just ignored.
@Buchhold
Copy link
Member

Hey Niklas, thanks for the commit. Can you merge this pull request yourself? If not, do you know which additional permissions I have to give to you and where I can do this?

@niklas88
Copy link
Member Author

You're welcome, I can merge this myself. Just want to still do it with Pull Requests to make it clear where the changes are coming from and giving @floriankramer a chance to comment.

@niklas88 niklas88 merged commit 77f0cc0 into ad-freiburg:master Feb 15, 2018
@niklas88 niklas88 deleted the short_xsd_support branch March 3, 2018 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants