Unable to access invalid index: 1: invalid index referenced

[wondywang]

Hey folks,

I discovered that we get a internal error if we apply the following operations to a JSON:

[
    {
        "op":"remove",
        "path":"/foo/1"
    },
    {
        "op":"remove",
        "path":"/foo/2"
    }
]

Test case:

func TestApply(t *testing.T) {
	patch := `[ { "op": "remove", "path": "/foo/1" },{ "op": "remove", "path": "/foo/2" } ]`
	doc := `{ "foo": [ "bar", "qux", "baz" ] }`
	patchObj, err := DecodePatch([]byte(patch))
	if err != nil {
		t.Errorf("unexpected error: %+v", err)
	}
	patchedJS, err := patchObj.Apply([]byte(doc))
	if err != nil {
		t.Errorf("patchObj.Apply internal error occurred, %v", err)
	}
	t.Logf("patchedJS: %s", patchedJS)
}

Output:

patchObj.Apply internal error occurred, error in remove for path: '/foo/2': Unable to access invalid index: 2: invalid index referenced

If multiple paths are removed, this exception will be reported.

[wondywang]

it also mentioned here: mattbaird/jsonpatch#30

[wondywang]

it also mentioned here: mattbaird/jsonpatch#30

In the latest version, the remove operation has been reversed.