You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
#104 reveals that the current SPARQL serializer will always add prefix declarations when calling $a->as_sparql(namespaces => $map), even when $a isn't a top-level query object. The tests included in #104 include examples of serializing bare IRIs, and getting:
This needs to be changed so that prefix declarations are only emitted for top-level queries. Special care should be taken to avoid emitting new declarations for sub-queries.
I'm not sure if this fix needs to happen in AtteanX::Serializer::SPARQL, Attean::API::SPARQLSerializable->sparql_tokens, or Attean::API::SPARQLSerializable->as_sparql.
The text was updated successfully, but these errors were encountered:
#104 reveals that the current SPARQL serializer will always add prefix declarations when calling
$a->as_sparql(namespaces => $map)
, even when$a
isn't a top-level query object. The tests included in #104 include examples of serializing bare IRIs, and getting:This needs to be changed so that prefix declarations are only emitted for top-level queries. Special care should be taken to avoid emitting new declarations for sub-queries.
I'm not sure if this fix needs to happen in
AtteanX::Serializer::SPARQL
,Attean::API::SPARQLSerializable->sparql_tokens
, orAttean::API::SPARQLSerializable->as_sparql
.The text was updated successfully, but these errors were encountered: