Fix simple literals returned as NULL using SERVICE (issue #1278)

rdflib/plugins/sparql/evaluate.py

@@ -381,18 +381,22 @@ def _yieldBindingsFromServiceCallResult(ctx: QueryContext, r, variables):
     res_dict: Dict[Variable, Identifier] = {}
     for var in variables:
         if var in r and r[var]:
-            if r[var]["type"] == "uri":
-                res_dict[Variable(var)] = URIRef(r[var]["value"])
-            elif r[var]["type"] == "bnode":
-                res_dict[Variable(var)] = BNode(r[var]["value"])
-            elif r[var]["type"] == "literal" and "datatype" in r[var]:
+            d = r[var]
+            t = d["type"]
+            if t == "uri":
+                res_dict[Variable(var)] = URIRef(d["value"])
+            elif t == "literal":
                 res_dict[Variable(var)] = Literal(
-                    r[var]["value"], datatype=r[var]["datatype"]
+                    d["value"], datatype=d.get("datatype"), lang=d.get("xml:lang")
                 )
-            elif r[var]["type"] == "literal" and "xml:lang" in r[var]:
+            elif t == "typed-literal":
                 res_dict[Variable(var)] = Literal(
-                    r[var]["value"], lang=r[var]["xml:lang"]
+                    d["value"], datatype=URIRef(d["datatype"])
                 )
+            elif t == "bnode":
+                res_dict[Variable(var)] = BNode(d["value"])
+            else:
+                raise NotImplementedError("json term type %r" % t)
     yield FrozenBindings(ctx, res_dict)
 
 

test/test_issues/test_issue1278.py

@@ -0,0 +1,19 @@
+from test.utils import helper
+
+from rdflib import Graph, Literal, Variable
+
+
+def test():
+    """Test service returns simple literals not as NULL.
+
+    Issue: https://github.com/RDFLib/rdflib/issues/1278
+    """
+
+    g = Graph()
+    q = """SELECT ?s ?p ?o
+WHERE {
+    SERVICE <https://DBpedia.org/sparql> {
+        VALUES (?s ?p ?o) {(<http://example.org/a> <http://example.org/b> "c")}
+    }
+}"""
+    assert results.bindings[0].get(Variable("o")) == Literal("c")