refactor type check

1 files changed, 5 insertions, 4 deletions

diff --git a/jsonpatch.py b/jsonpatch.py
index ae2fbef..c9dcd3c 100644
--- a/jsonpatch.py
+++ b/jsonpatch.py
@@ -366,6 +366,11 @@ class AddOperation(PatchOperation):
         value = self.operation["value"]
         subobj, part = self.pointer.to_last(obj)
 
+        # type is already checked in to_last(), so we assert here
+        # for consistency
+        assert isinstance(subobj, list) or isinstance(subobj, dict), \
+            "invalid document type %s" (type(doc),)
+
         if isinstance(subobj, list):
 
             if part == '-':
@@ -385,10 +390,6 @@ class AddOperation(PatchOperation):
             else:
                 subobj[part] = value
 
-        else:
-            raise JsonPatchConflict("can't add to type '%s'"
-                                    "" % subobj.__class__.__name__)
-
         return obj