Skip to content

Conversation

MPvHarmelen
Copy link

@MPvHarmelen MPvHarmelen commented Dec 22, 2022

When serialising the following query, the surrounding ( ) are forgotten, causing a syntactically invalid query to be generated:

ASK WHERE { ?s ^(^<a:a>) ?o. }

becomes:

ASK WHERE { ?s ^^<a:a> ?o. }

which is syntactically invalid.

This pull-request contains a failing test for this case.

@jeswr
Copy link
Contributor

jeswr commented Dec 28, 2022

@MPvHarmelen I just had a glance at the files changed and there is only a commit adding the test file; did you forget to commit the actual code changes?

I'm not a maintainer so I can't run the workflow to see if the tests are currently failing or not.

@MPvHarmelen
Copy link
Author

Sorry, no! I encountered this issue and wanted to create an issue/pull-request combination to improve the DX for debugging the problem, but I forgot the issue half of it...

@RubenVerborgh
Copy link
Owner

(Quickly rebased this one to trigger CI.)

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.

3 participants