Merge pull request #63 from wmde/fix-op-boolean-true

Fix error if "op" is boolean true

Author: vearutop

src/JsonPatch.php

@@ -70,6 +70,13 @@ public static function import(array $data)
                 throw new MissingFieldException('path', $operation);
             }
 
+            if (!is_string($operation->op)) {
+                throw new InvalidFieldTypeException('op', 'string', $operation);
+            }
+            if (!is_string($operation->path)) {
+                throw new InvalidFieldTypeException('path', 'string', $operation);
+            }
+
             $op = null;
             switch ($operation->op) {
                 case Add::OP:
@@ -91,17 +98,8 @@ public static function import(array $data)
                     $op = new Test();
                     break;
                 default:
-                    if (!is_string($operation->op)) {
-                        throw new InvalidFieldTypeException('op', 'string', $operation);
-                    }
-
                     throw new UnknownOperationException($operation);
             }
-
-            if (!is_string($operation->path)) {
-                throw new InvalidFieldTypeException('path', 'string', $operation);
-            }
-            
             $op->path = $operation->path;
             if ($op instanceof OpPathValue) {
                 if (property_exists($operation, 'value')) {

tests/src/JsonPatchTest.php

@@ -137,7 +137,7 @@ public function provideInvalidFieldType()
     {
         return [
             '"op" invalid type' => [
-                (object)array('op' => array('foo' => 'bar'), 'path' => '/123', 'value' => 'test'),
+                (object)array('op' => true, 'path' => '/123', 'value' => 'test'),
                 'Invalid field type - "op" should be of type: string',
                 'op',
                 'string'