Fix name resolution in ArrayKeyFirstLastRector

packages/Php73/src/Rector/FuncCall/ArrayKeyFirstLastRector.php

@@ -10,6 +10,7 @@
 use Rector\Rector\AbstractRector;
 use Rector\RectorDefinition\CodeSample;
 use Rector\RectorDefinition\RectorDefinition;
+use Rector\ValueObject\PhpVersionFeature;
 
 /**
  * @see https://www.tomasvotruba.cz/blog/2018/08/16/whats-new-in-php-73-in-30-seconds-in-diffs/#2-first-and-last-array-key
@@ -121,13 +122,18 @@ public function refactor(Node $node): ?Node
 
     private function shouldSkip(FuncCall $funcCall): bool
     {
-        if ($this->isAtLeastPhpVersion('7.3')) {
+        if (! $this->isNames($funcCall, ['reset', 'end'])) {
+            return true;
+        }
+
+        if ($this->isAtLeastPhpVersion(PhpVersionFeature::ARRAY_KEY_FIRST_LAST)) {
             return false;
         }
 
         if (function_exists(self::ARRAY_KEY_FIRST) && function_exists(self::ARRAY_KEY_LAST)) {
             return false;
         }
-        return $this->isNames($funcCall, ['reset', 'end']);
+
+        return true;
     }
 }

packages/Php73/tests/Rector/FuncCall/ArrayKeyFirstLastRector/Fixture/skip_other_function_name.php.inc

@@ -0,0 +1,13 @@
+<?php
+
+namespace Rector\Php73\Tests\Rector\FuncCall\ArrayKeyFirstLastRector\Fixture;
+
+class SkipOtherFunctionName
+{
+    public function run()
+    {
+        $a = [];
+        ksort($a);
+        key($a);
+    }
+}

src/ValueObject/PhpVersionFeature.php

@@ -41,6 +41,11 @@ final class PhpVersionFeature
      */
     public const IS_COUNTABLE = '7.3';
 
+    /**
+     * @var string
+     */
+    public const ARRAY_KEY_FIRST_LAST = '7.3';
+
     /**
      * @var string
      */