fix-style

Author: tranvantri

src/Helpers.php

@@ -4,6 +4,7 @@
 namespace Rebing\GraphQL;
 
 use Closure;
+use OutOfBoundsException;
 
 class Helpers
 {
@@ -26,29 +27,28 @@ public static function applyEach(Closure $callback, $valueOrValues)
 
     /**
      * Check compatible ability to use thecodingmachine/safe.
-     *
-     * @param string $methodName
-     * @return bool
      */
     public static function shouldUseSafe(string $methodName): bool
     {
-        $safeVersion = \Composer\InstalledVersions::getVersion('thecodingmachine/safe');
+        $packageName = 'thecodingmachine/safe';
+        $safeVersion = \Composer\InstalledVersions::getVersion($packageName);
+
+        if (!$safeVersion) {
+            throw new OutOfBoundsException("Package {$packageName} is being replaced or provided but is not really installed");
+        }
 
         $skipFunctions = [
             'uksort',
         ];
 
         // Version 2.
-        if (version_compare($safeVersion, '2', '>='))
-        {
-            if (in_array($methodName, $skipFunctions))
-            {
+        if (version_compare($safeVersion, '2', '>=')) {
+            if (\in_array($methodName, $skipFunctions)) {
                 return false;
             }
         }
 
-        if (!is_callable('\\Safe\\' . $methodName))
-        {
+        if (!\is_callable('\\Safe\\' . $methodName)) {
             return false;
         }
 

src/Support/AliasArguments/ArrayKeyChange.php

@@ -27,9 +27,13 @@ private function orderPaths(array $paths): array
             return $this->pathLevels($b) <=> $this->pathLevels($a);
         };
 
-        if (Helpers::shouldUseSafe('uksort')) {
+        $shouldUseSafe = Helpers::shouldUseSafe('uksort');
+
+        if ($shouldUseSafe && \is_callable('\\Safe\\uksort')) {
             \Safe\uksort($paths, $callback);
-        } else {
+        }
+
+        if (!$shouldUseSafe) {
             uksort($paths, $callback);
         }