Add additional checks to ensure string is returned to avoid null bein…

…g returned
scoutapp · Apr 9, 2021 · 2fee014 · 2fee014
1 parent fea1102
commit 2fee014
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/src/Events/Metadata.php b/src/Events/Metadata.php
@@ -20,7 +20,9 @@
 use function class_exists;
 use function getenv;
 use function gethostname;
+use function is_array;
 use function is_string;
+use function method_exists;
 use function sprintf;
 
 use const PHP_VERSION;
@@ -129,8 +131,12 @@ private function rootPackageGitSha(): string
             return $herokuSlugCommit;
         }
 
-        if (class_exists(InstalledVersions::class)) {
-            return InstalledVersions::getRootPackage()['reference'];
+        if (class_exists(InstalledVersions::class) && method_exists(InstalledVersions::class, 'getRootPackage')) {
+            /** @var mixed $rootPackage */
+            $rootPackage = InstalledVersions::getRootPackage();
+            if (is_array($rootPackage) && array_key_exists('reference', $rootPackage) && is_string($rootPackage['reference'])) {
+                return $rootPackage['reference'];
+            }
         }
 
         return '';