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

Disallow the selection of variables in a negative branch #507

Closed
Marie-Sparna opened this issue May 17, 2023 · 1 comment
Closed

Disallow the selection of variables in a negative branch #507

Marie-Sparna opened this issue May 17, 2023 · 1 comment
Labels

Comments

@Marie-Sparna
Copy link
Contributor

Error 500 when activating the eye on "enableNegation" parameter. Not very useful to activate the eye in that case but we don't really understand the error when we run the query then :

image

@tfrancart
Copy link
Contributor

Not very useful to activate

I would even say that selecting something which be definition does not exist is a logical non-sense :-)

but we don't really understand the error

The error message is given by Virtuoso, not by Sparnatural. Other triplestores would not return an error, and simply not bind the variable. We can't do anything about the error message.

What we can do however is to:

  • if a "negation" is selected, then unselect all variables in that branch
  • disable the selection of variable in a branch that is within a negation

I think this was working this way before the port to Typescript

@tfrancart tfrancart changed the title Bug with activating the "eye" on enableNegation parameter Disallow the selection of variables in a negative branch May 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants