[CodingStyle] Skip by ref params on BinaryOpStandaloneAssignsToDirectRector (#7471)

* [CodingStyle] Skip by ref params on BinaryOpStandaloneAssignsToDirectRector

* [ci-review] Rector Rectify

* fix

---------

Co-authored-by: GitHub Action <[email protected]>

Author: samsonasik

rules-tests/CodingStyle/Rector/ClassMethod/BinaryOpStandaloneAssignsToDirectRector/Fixture/skip_by_ref_from_param_variable.php.inc

@@ -0,0 +1,14 @@
+<?php
+
+namespace Rector\Tests\CodingStyle\Rector\ClassMethod\BinaryOpStandaloneAssignsToDirectRector\Fixture;
+
+final class SkipByRefFromParamVariable
+{
+    public function run(&$first, &$second)
+    {
+        $first = 100;
+        $second = 200;
+
+        return $first <=> $second;
+    }
+}

rules/CodingStyle/Rector/ClassMethod/BinaryOpStandaloneAssignsToDirectRector.php

@@ -105,6 +105,15 @@ public function refactor(Node $node): ?Node
             return null;
         }
 
+        $resolveParamByRefVariables = $this->resolveParamByRefVariables($node);
+        if ($this->isNames($binaryOp->left, $resolveParamByRefVariables)) {
+            return null;
+        }
+
+        if ($this->isNames($binaryOp->right, $resolveParamByRefVariables)) {
+            return null;
+        }
+
         $binaryOp->left = $firstVariableAndExprAssign->getExpr();
         $binaryOp->right = $secondVariableAndExprAssign->getExpr();
 
@@ -117,6 +126,32 @@ public function provideMinPhpVersion(): int
         return PhpVersionFeature::VARIADIC_PARAM;
     }
 
+    /**
+     * @return string[]
+     */
+    private function resolveParamByRefVariables(ClassMethod|Function_|Closure $node): array
+    {
+        $paramByRefVariables = [];
+        foreach ($node->params as $param) {
+            if (! $param->var instanceof Variable) {
+                continue;
+            }
+
+            if (! $param->byRef) {
+                continue;
+            }
+
+            $paramName = $this->getName($param->var);
+            if ($paramName === null) {
+                continue;
+            }
+
+            $paramByRefVariables[] = $paramName;
+        }
+
+        return $paramByRefVariables;
+    }
+
     private function matchToVariableAssignExpr(Stmt $stmt): ?VariableAndExprAssign
     {
         if (! $stmt instanceof Expression) {