Updated blancks/fast-jsonpatch-php to v2

Author: blancks

README.md

@@ -44,7 +44,7 @@ The compliance test strictly checks if the output json of each library is consis
 
 | Library / Status                                                                                                                                             | Version    |
 |--------------------------------------------------------------------------------------------------------------------------------------------------------------|------------|
-| <div align="right">![blancks/fast-jsonpatch-php](https://github.com/blancks/php-jsonpatch-benchmarks/workflows/blancks/fast-jsonpatch-php/badge.svg)</div>   | v1.2.3     |
+| <div align="right">![blancks/fast-jsonpatch-php](https://github.com/blancks/php-jsonpatch-benchmarks/workflows/blancks/fast-jsonpatch-php/badge.svg)</div>   | v2.0       |
 | <div align="right">![remorhaz/php-json-patch](https://github.com/blancks/php-jsonpatch-benchmarks/workflows/remorhaz/php-json-patch/badge.svg)</div>         | v0.6.1     |
 | <div align="right">![mikemccabe/json-patch-php](https://github.com/blancks/php-jsonpatch-benchmarks/workflows/mikemccabe/json-patch-php/badge.svg)</div>     | dev-master |
 | <div align="right">![php-jsonpatch/php-jsonpatch](https://github.com/blancks/php-jsonpatch-benchmarks/workflows/php-jsonpatch/php-jsonpatch/badge.svg)</div> | v4.1.0     |
@@ -61,15 +61,18 @@ The compliance test strictly checks if the output json of each library is consis
 The following table shows the average time each library took to apply a patch with 1000 operations to a target document as summary of the performance. 
 The actual benchmark data is available [here](https://docs.google.com/spreadsheets/d/1ZTDWh1k-zzhYHqZB3JMD2WRV0bPRIWUMRbLiMJhMLHk/edit?usp=sharing).
 
-| Library                     | Microseconds |
+| Library (fully RFC compliant only) | Microseconds |
+|------------------------------------|--------------|
+| blancks/fast-jsonpatch-php         | 4511         |
+| remorhaz/php-json-patch            | 870711       |
+
+| Library (others)            | Microseconds |
 |-----------------------------|--------------|
-| blancks/fast-jsonpatch-php  | 2903         |
 | mikemccabe/json-patch-php   | 3355         |
 | swaggest/json-diff          | 3638         |
 | gamringer/php-json-patch    | 7276         |
 | xp-forge/json-patch         | 8534         |
 | php-jsonpatch/php-jsonpatch | 10970        |
-| remorhaz/php-json-patch     | 870711       |
 
 > These results are indicative and may vary depending on the specific use case and system environment.
 

composer.json

@@ -16,7 +16,7 @@
         }
     ],
     "require": {
-        "blancks/fast-jsonpatch-php": "^1.2",
+        "blancks/fast-jsonpatch-php": "dev-dev-v2",
         "mikemccabe/json-patch-php": "dev-master",
         "php-jsonpatch/php-jsonpatch": "^4.1",
         "xp-forge/json-patch": "^2.1",

composer.lock

@@ -4,11 +4,12 @@
 
 function applyJsonPatch(string &$json, string $patch): float
 {
+    static $FastJsonPatch = null;
     $jsonDecoded = json_decode($json);
-    $patchDecoded = json_decode($patch);
+    $FastJsonPatch ??= new \blancks\JsonPatch\FastJsonPatch($jsonDecoded);
 
     $microtime = microtime(true);
-    \blancks\JsonPatch\FastJsonPatch::applyByReference($jsonDecoded, $patchDecoded);
+    $FastJsonPatch->apply($patch);
     $output = microtime(true) - $microtime;
 
     $json = json_encode($jsonDecoded);

src/lib/blancks_fast-jsonpatch.php

@@ -1,33 +1,35 @@
 <?php declare(strict_types=1);
 
 namespace blancks\JsonPatchBenchmarkTests;
+use blancks\JsonPatch\exceptions\FastJsonPatchException;
+use blancks\JsonPatch\FastJsonPatch;
 use PHPUnit\Framework\Attributes\DataProvider;
 
 class Blancks_fast_jsonpatchTest extends JsonPatchCompliance
 {
     #[DataProvider('validOperationsProvider')]
     public function testJsonPatchCompliance(string $json, string $patch, string $expected): void
     {
+        $document = json_decode($json);
+        (new FastJsonPatch($document))->apply($patch);
+
         $this->assertSame(
-            json_encode(json_decode($expected, false, 512, JSON_THROW_ON_ERROR)),
-            \blancks\JsonPatch\FastJsonPatch::apply($json, $patch)
+            $this->normalizeJson($expected),
+            $this->normalizeJson(json_encode($document))
         );
     }
 
     #[DataProvider('atomicOperationsProvider')]
     public function testAtomicOperations(string $json, string $patch, string $expected): void
     {
-        $document = json_decode($json);
+        $FastJsonPatch = FastJsonPatch::fromJson($json);
 
-        try {
-            \blancks\JsonPatch\FastJsonPatch::applyByReference($document, json_decode($patch));
-        } catch (\Throwable) {
-            // expecting some error
-        }
+        $this->expectException(FastJsonPatchException::class);
+        $FastJsonPatch->apply($patch);
 
         $this->assertSame(
             $this->normalizeJson($expected),
-            $this->normalizeJson(json_encode($document))
+            $this->normalizeJson($this->jsonEncode($FastJsonPatch->getDocument()))
         );
     }
 }