json-patch-php

json-patch-php copied to clipboard

Thanks so much for making this, it worked right out of the box and was crazy easy to use. I ended up having to use a different package because it...

joeynovak

count(): Parameter must be an array or an object that implements Countable

3

src: `{"bankruptcy":null}` ``` $src = {array} [1] bankruptcy = null ``` dst: `{"bankruptcy":24}` ``` $dst = {array} [1] bankruptcy = {int} 24 ``` ``` { "message": "count(): Parameter must be...

4n70w4

mjanser

Fix array diff issue, as positioning is not preserved, its generated in reverse breaking JSON-Patch 4.1 ADD: "The specified index MUST NOT be greater than the number of elements in...

joseph-montanez

Incorrect diff result

2

I added some objects to array as values and execute diff function. Returns patches which sorted descending, that's why function patch crashed because greater indexes in patches are above than...

Dimona

Could it be possible to tag a version that contains the count issue fix.

2

The library I [develop](https://github.com/makasim/yadm/blob/master/composer.json#L17) uses this one and as for now there is no way to require the version with the [fix for count issue](https://github.com/mikemccabe/json-patch-php/pull/17) other then set dev-master dep.

makasim

src or dst can be null

1

src / dst can also be null, if something is added / removed.

dbu

makasim

If you had value with type array in source array, then you try to change it to null it will create incorrect patches and it this is accompanied by many...

Dimona

Here is the code: ``` ``` The second test fails with following error: > Fatal error: Uncaught mikemccabe\JsonPatch\JsonPatchException: Can't operate outside of array bounds in C:\xampp\htdocs\web\vendor\mikemccabe\json-patch-php\src\JsonPatch.php:540 Stack trace: #0 C:\xampp\htdocs\web\vendor\mikemccabe\json-patch-php\src\JsonPatch.php(514):...

pavlexander