René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"JsonDiff bug found","articleBody":"Consider the following Jsons:\r\n\r\n**Source:**\r\n```\r\n{\r\n    \"store\": {\r\n        \"a\": [\r\n            \"v1\",\r\n            \"v2\"\r\n        ]\r\n    }\r\n}\r\n```\r\n\r\n**Target:**\r\n```\r\n{\r\n    \"store\": {\r\n        \"a\": []\r\n    }\r\n}\r\n```\r\n\r\nrunning `JsonDiff.asJson(source, target)` returns:\r\n\r\n```\r\n[ {\r\n  \"op\" : \"remove\",\r\n  \"path\" : \"/store/a/0\"\r\n}, {\r\n  \"op\" : \"remove\",\r\n  \"path\" : \"/store/a/0\"\r\n} ]\r\n```\r\n\r\nwhich as it is seen both path is same but actually the next path should be `\"/store/a/1\"`. The reason is there is a bug in **JsonDiff** class in function **generateArrayDiffs**. There is a for loop as:\r\n\r\n```\r\nfor(index = size; index \u003c firstSize; ++index) {\r\n   processor.valueRemoved(pointer.append(size), source.get(index));\r\n}\r\n```\r\n\r\nWhich should be changed to:\r\n\r\n```\r\nfor(index = size; index \u003c firstSize; ++index) {\r\n   processor.valueRemoved(pointer.append(index), source.get(index));\r\n}\r\n```","author":{"url":"https://github.com/fmoghaddam","@type":"Person","name":"fmoghaddam"},"datePublished":"2024-10-01T10:47:21.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":1},"url":"https://github.com/163/json-patch/issues/163"}