[Php73] Handle named argument on JsonThrowOnErrorRector (#1021)

* Test case for Incorrect behavior of JsonThrowOnErrorRector #6752

* Closes #1020 Fixes rectorphp/rector#6752

* allow named on json or value key

* not allowed named argument

* clean up

* clean up

Co-authored-by: Björn Bösel <bjoernboesel@gmail.com>

rules-tests/Php73/Rector/FuncCall/JsonThrowOnErrorRector/Fixture/skip_named_argument_flags.php.inc

@@ -0,0 +1,8 @@
+<?php
+
+namespace Rector\Tests\Php73\Rector\FuncCall\JsonThrowOnErrorRector\Fixture;
+
+function skipNamedArgumentFlags($str){
+    json_decode($str, flags: JSON_THROW_ON_ERROR);
+}
+

rules-tests/Php73/Rector/FuncCall/JsonThrowOnErrorRector/Fixture/skip_named_argument_value_or_json.php.inc

@@ -0,0 +1,8 @@
+<?php
+
+namespace Rector\Tests\Php73\Rector\FuncCall\JsonThrowOnErrorRector\Fixture;
+
+function skipNamedArgumentValueOrJson($str, array $data){
+    json_decode(json: $str);
+    json_encode(value: $data);
+}

rules/Php73/Rector/FuncCall/JsonThrowOnErrorRector.php

@@ -8,6 +8,7 @@
 use PhpParser\Node\Arg;
 use PhpParser\Node\Expr\ConstFetch;
 use PhpParser\Node\Expr\FuncCall;
+use PhpParser\Node\Identifier;
 use PhpParser\Node\Name;
 use PhpParser\Node\Scalar\LNumber;
 use Rector\Core\Rector\AbstractRector;
@@ -82,6 +83,13 @@ private function shouldSkip(FuncCall $funcCall): bool
             return true;
         }
 
+        $args = $funcCall->getArgs();
+        foreach ($args as $arg) {
+            if ($arg->name instanceof Identifier) {
+                return true;
+            }
+        }
+
         return (bool) $this->betterNodeFinder->findFirstNext($funcCall, function (Node $node): bool {
             if (! $node instanceof FuncCall) {
                 return false;