Merge pull request #605 from pchampin/fix-issue-604

Collection.__setitem__ now clears old value, fixes issue #604

Author: joernhees

rdflib/collection.py

@@ -129,7 +129,7 @@ def __setitem__(self, key, value):
         """TODO"""
         c = self._get_container(key)
         if c:
-            self.graph.add((c, RDF.first, value))
+            self.graph.set((c, RDF.first, value))
         else:
             raise IndexError(key)
 

test/test_issue604.py

@@ -0,0 +1,17 @@
+from __future__ import print_function
+
+from rdflib import *
+from rdflib.collection import Collection
+
+def test_issue604():
+
+    EX = Namespace('http://ex.co/')
+    g = Graph()
+    bn = BNode()
+    g.add((EX.s, EX.p, bn))
+    c = Collection(g, bn, map(Literal, [1,2,4]))
+    c[2] = Literal(3)
+    got = list(g.objects(bn, RDF.rest/RDF.rest/RDF.first))
+    expected = [ Literal(3) ]
+    assert got == [ Literal(3) ], got
+