Exception when serializing QuotedGraphs to N3

[ericpeden]

Attempting to N3-serialize a graph with a formula in it (as shown by this quick and dirty test case) results in the following error:

Traceback (most recent call last):
  File "/home/eric/src/rdflib/test/test_n3.py", line 208, in testModel
    g.serialize(format="n3")
  File "/home/eric/src/rdflib/rdflib/graph.py", line 903, in serialize
    serializer.serialize(stream, base=base, encoding=encoding, **args)
  File "/home/eric/src/rdflib/rdflib/plugins/serializers/turtle.py", line 206, in serialize
    subjects_list = self.orderSubjects()
  File "/home/eric/src/rdflib/rdflib/plugins/serializers/turtle.py", line 64, in orderSubjects
    recursable.sort()
  File "/home/eric/src/rdflib/rdflib/term.py", line 168, in __lt__
    return _ORDERING[type(self)] < _ORDERING[type(other)]
KeyError: <class 'rdflib.graph.QuotedGraph'>

3.4.0 doesn't exhibit this bug.

[gromgull]

Well spotted, QuotedGraph must be added to this list:

_ORDERING = dict(map(reversed, enumerate([BNode, Variable, URIRef, Literal])))

(https://github.com/RDFLib/rdflib/blob/master/rdflib/term.py#L1566)

I am slightly worried that none of our tests tried to serialize any N3 though...

[gweis]

Not only quoted graph, but all direct subclasses of Node should be in this
list.

Sent from my iPad

On 24/05/2013, at 10:23 PM, Gunnar Aastrand Grimnes <
notifications@github.com> wrote:

Well spotted, QuotedGraph must be added to this list:

_ORDERING = dict(map(reversed, enumerate([BNode, Variable, URIRef,
Literal])))

(https://github.com/RDFLib/rdflib/blob/master/rdflib/term.py#L1566)

I am slightly worried that none of our tests tried to serialize any N3
though...

—
Reply to this email directly or view it on
GitHubhttps://github.com//issues/291#issuecomment-18401636
.