search

Callidon / sparql-engine

🚂 A framework for building SPARQL query engines in Javascript/Typescript
https://callidon.github.io/sparql-engine
MIT License
99 stars 14 forks source link

fixes issue #16 #32

Closed dwhitney closed 5 years ago

dwhitney commented 5 years ago

This PR fixes #16, when the name for a GRAPH clause is bound in a variable. The method is to go through binding, by binding looking for the bound variable, and pulling the related graph from the dataset or creating one using the dataset's graph factory if one is set. The search for the variable is only done if the graph's name is a varaible - otherwise the existing method is followed.

dwhitney commented 5 years ago

I ran the linter because my editor defaults are different than the projects. There were several files that I did not edit that were updated by the linter. The only ones I made functional edits to are src/engine/stages/bgp-stage-builder.ts and test/sparql/graph-test.js

Callidon commented 5 years ago

Nice, thank you very much for this. It's another step towards full SPARQl 1.1 compliance for the framework. Only a couple left to go :)