Improved jsonpath for array elements

Author: bixuehujin

src/Validator.php

@@ -44,7 +44,8 @@ protected function getBuiltInTypeValidators()
                 }
 
                 $givenType = $this->getType($value);
-                $path = implode('.', $this->path);
+
+                $path = $this->getNormalizedPath();
 
                 $this->addError($path, "The path of '$path' requires to be a $type, $givenType is given");
 
@@ -101,6 +102,14 @@ public function addType(string $type, $definition)
     protected $errors = [];
     protected $path = ['$'];
 
+    protected function getNormalizedPath()
+    {
+        return strtr(implode('.', $this->path), [
+            '.[' => '[',
+            '].' => ']',
+        ]);
+    }
+
     public function getErrors()
     {
         return $this->errors;
@@ -152,7 +161,7 @@ protected function matchArray(array $data, $definition)
         $definition = $definition[0];
 
         foreach ($data as $index => $row) {
-            array_push($this->path, $index);
+            array_push($this->path, '[' . $index . ']');
             $result = $this->matchInternal($row, $definition);
             array_pop($this->path);
 

tests/JsonTypeTest.php

@@ -153,8 +153,8 @@ public function typeErrorMessages()
             [
                 [1, 2, 3],
                 ['string'],
-                '$.0',
-                "The path of '$.0' requires to be a string, integer is given",
+                '$[0]',
+                "The path of '$[0]' requires to be a string, integer is given",
             ],
             [
                 [
@@ -163,8 +163,8 @@ public function typeErrorMessages()
                 [
                     'foo' => ['string'],
                 ],
-                '$.foo.0',
-                "The path of '$.foo.0' requires to be a string, integer is given",
+                '$.foo[0]',
+                "The path of '$.foo[0]' requires to be a string, integer is given",
             ],
         ];
     }