Merge pull request #67 from wmde/fix-remove-operation

Fix `JsonPointer::remove()` not handling sequential arrays correctly

Author: vearutop

src/JsonPointer.php

@@ -289,6 +289,7 @@ public static function remove(&$holder, $pathItems, $flags = 0)
                             $isAssociative = true;
                             break;
                         }
+                        $i++;
                     }
                 }
 

tests/src/JsonPointerTest.php

@@ -96,6 +96,13 @@ public function testGetSetDeleteObject()
         $this->assertEquals(null, $s->one->two);
     }
 
+    public function testSequentialArrayIsPreserved()
+    {
+		$json = [ 'l1' => [ 0 => 'foo', 1 => 'bar', 2 => 'baz' ] ];
+		JsonPointer::remove($json, ['l1', '1'], JsonPointer::TOLERATE_ASSOCIATIVE_ARRAYS);
+		$this->assertSame('{"l1":["foo","baz"]}', json_encode($json));
+    }
+
 }
 
 class Sample