[CodeQuality] Skip empty array push args and no 2nd arg on ChangeArra…

…yPushToArrayAssignRector (#2266)

* [CodeQuality] Skip empty array push args and no 2nd arg on ChangeArrayPushToArrayAssignRector

* Fixed 🎉

rules-tests/CodeQuality/Rector/FuncCall/ChangeArrayPushToArrayAssignRector/Fixture/skip_empty_args.php.inc

@@ -0,0 +1,13 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\FuncCall\ChangeArrayPushToArrayAssignRector\Fixture;
+
+class SkipEmptyArgs
+{
+    public function run()
+    {
+        array_push();
+
+        echo 'test';
+    }
+}

rules-tests/CodeQuality/Rector/FuncCall/ChangeArrayPushToArrayAssignRector/Fixture/skip_no_second_arg.php.inc

@@ -0,0 +1,11 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\FuncCall\ChangeArrayPushToArrayAssignRector\Fixture;
+
+class SkipNoSecondArg
+{
+    public function run(array $items)
+    {
+        array_push($items);
+    }
+}

rules/CodeQuality/Rector/FuncCall/ChangeArrayPushToArrayAssignRector.php

@@ -70,9 +70,17 @@ public function refactor(Node $node): ?array
 
         $args = $funcCall->getArgs();
 
+        if ($args === []) {
+            return null;
+        }
+
         /** @var Arg $firstArg */
         $firstArg = array_shift($args);
 
+        if ($args === []) {
+            return null;
+        }
+
         $arrayDimFetch = new ArrayDimFetch($firstArg->value);
 
         $newStmts = [];