Cache the output of the retrieved data

This makes sure we can call these retrieves in loops without huge performance changes.
madewithlove · Mar 19, 2020 · fbaf263 · fbaf263
1 parent 734eb8b
commit fbaf263
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 2 deletions.
diff --git a/src/Composer/DependencyTreeRetriever.php b/src/Composer/DependencyTreeRetriever.php
@@ -7,15 +7,23 @@
 
 class DependencyTreeRetriever
 {
+    private static $output;
+
     public function getDependencyTree(): string
     {
+        if (!is_null(self::$output)) {
+            return self::$output;
+        }
+
         $process = new Process(['composer', 'show', '-t', '-f', 'json']);
         $process->run();
 
         if (!$process->isSuccessful()) {
             throw new ProcessFailedException($process);
         }
 
-        return $process->getOutput();
+        self::$output = $process->getOutput();
+
+        return self::$output;
     }
 }
diff --git a/src/Composer/UsedLicensesRetriever.php b/src/Composer/UsedLicensesRetriever.php
@@ -7,15 +7,23 @@
 
 class UsedLicensesRetriever
 {
+    private static $output;
+
     public function getComposerLicenses(): string
     {
+        if (!is_null(self::$output)) {
+            return self::$output;
+        }
+
         $process = new Process(['composer', 'license', '-f', 'json']);
         $process->run();
 
         if (!$process->isSuccessful()) {
             throw new ProcessFailedException($process);
         }
 
-        return $process->getOutput();
+        self::$output = $process->getOutput();
+
+        return self::$output;
     }
 }