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

rr:Template does not URLfy strings #21

Closed
mariapoveda opened this issue Feb 14, 2019 · 5 comments
Closed

rr:Template does not URLfy strings #21

mariapoveda opened this issue Feb 14, 2019 · 5 comments
Assignees
Labels
bug Something isn't working next release The bug is fixed and the fix will be available in the next release.

Comments

@mariapoveda
Copy link

When using the template construct in a predicate object map the json values are not URLfied as when using the same expression in a subject map.

Predicate object map example:

rr:predicateObjectMap [
rr:predicate etvonto:hasForm;
rr:objectMap [
rr:template "http://easytv.linkeddata.es/resource/LinguisticExpressionItem/{Sls}{Url}{Word}";
rr:termType rr:IRI
]
]

Resulting Triple (see difference between subject and predicate generation):

http://easytv.linkeddata.es/resource/SignedLinguisticExpression/this%20movie%20three%20I%20have%20seen%20timeshttps%3A%2F%2Fmedia.spreadthesign.com%2Fvideo%2Fmp4%2F1%2F105750.mp4 https://w3id.org/def/easytv#decomposition <http://easytv.linkeddata.es/resource/LinguisticExpressionItem/this movie three I have seen timeshttps://media.spreadthesign.com/video/mp4/1/105750.mp41> null

@pheyvaer pheyvaer self-assigned this Feb 14, 2019
@pheyvaer pheyvaer added the bug Something isn't working label Feb 14, 2019
@pheyvaer
Copy link
Collaborator

Hi @mariapoveda,

We've confirmed the bug and fixed it locally. The fix will be available in the next release. We'll keep you up to date! Thanks for reporting 👍

@pheyvaer pheyvaer added the next release The bug is fixed and the fix will be available in the next release. label Feb 14, 2019
@mariapoveda
Copy link
Author

mariapoveda commented Feb 19, 2019

Thanks @pheyvaer

have you already planned the next release?

@pheyvaer
Copy link
Collaborator

Normally, we will have a new release next week 😃

@mariapoveda
Copy link
Author

Cool, thanks.

bjdmeest added a commit that referenced this issue Feb 27, 2019
fix Github issue 21

See merge request rml/proc/rmlmapper-java!39
@bjdmeest
Copy link
Collaborator

@mariapoveda , our new release (https://github.com/RMLio/rmlmapper-java/releases/tag/v4.3.2) contains the fix to this issue, thanks again for notifying!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working next release The bug is fixed and the fix will be available in the next release.
Projects
None yet
Development

No branches or pull requests

3 participants