Merge pull request #21 from johanfforsberg/array-move

Moving array item into other array item

Author: stefankoegl

jsonpatch.py

@@ -466,7 +466,7 @@ def apply(self, obj):
         subobj, part = from_ptr.to_last(obj)
         value = subobj[part]
 
-        if self.pointer.contains(from_ptr):
+        if isinstance(subobj, dict) and self.pointer.contains(from_ptr):
             raise JsonPatchException('Cannot move values into its own children')
 
         obj = RemoveOperation({

tests.py

@@ -88,6 +88,12 @@ def test_move_array_item(self):
         res = jsonpatch.apply_patch(obj, [{'op': 'move', 'from': '/foo/1', 'path': '/foo/3'}])
         self.assertEqual(res, {'foo': ['all', 'cows', 'eat', 'grass']})
 
+    def test_move_array_item_into_other_item(self):
+        obj = [{"foo": []}, {"bar": []}]
+        patch = [{"op": "move", "from": "/0", "path": "/0/bar/0"}]
+        res = jsonpatch.apply_patch(obj, patch)
+        self.assertEqual(res, [{'bar': [{"foo": []}]}])
+
     def test_copy_object_key(self):
         obj = {'foo': {'bar': 'baz', 'waldo': 'fred'},
                'qux': {'corge': 'grault'}}