Add caching support to Filesystem getRequire

Author: amir9480

src/Illuminate/Filesystem/Filesystem.php

@@ -2,6 +2,7 @@
 
 namespace Illuminate\Filesystem;
 
+use Closure;
 use ErrorException;
 use FilesystemIterator;
 use Illuminate\Contracts\Filesystem\FileNotFoundException;
@@ -18,6 +19,13 @@ class Filesystem
 {
     use Conditionable, Macroable;
 
+    /**
+     * Array of cached require values.
+     *
+     * @var array
+     */
+    protected array $cachedRequirements = [];
+
     /**
      * Determine if a file or directory exists.
      *
@@ -107,21 +115,32 @@ public function sharedGet($path)
      *
      * @param  string  $path
      * @param  array  $data
+     * @param  bool|Closure  $cache
      * @return mixed
      *
      * @throws \Illuminate\Contracts\Filesystem\FileNotFoundException
      */
-    public function getRequire($path, array $data = [])
+    public function getRequire($path, array $data = [], bool|Closure $cache = false)
     {
+        if ($cache && isset($this->cachedRequirements[$path])) {
+            return $this->cachedRequirements[$path];
+        }
+
         if ($this->isFile($path)) {
             $__path = $path;
             $__data = $data;
 
-            return (static function () use ($__path, $__data) {
+            $require = (static function () use ($__path, $__data) {
                 extract($__data, EXTR_SKIP);
 
                 return require $__path;
             })();
+
+            if (is_callable($cache) ? call_user_func($cache, $require) : $cache) {
+                $this->cachedRequirements[$path] = $require;
+            }
+
+            return $require;
         }
 
         throw new FileNotFoundException("File does not exist at path {$path}.");

tests/Filesystem/FilesystemTest.php

@@ -344,6 +344,17 @@ public function testGetRequireThrowsExceptionNonExistingFile()
         $files->getRequire(self::$tempDir.'/file.php');
     }
 
+    public function testGetRequireCachedReturnsProperly()
+    {
+        file_put_contents(self::$tempDir.'/file.php', '<?php return "Howdy?"; ?>');
+        $files = new Filesystem;
+        $files->getRequire(self::$tempDir.'/file.php', cache: true);
+        file_put_contents(self::$tempDir.'/file.php', '<?php return "Hey!"; ?>');
+
+        $this->assertSame('Hey!', $files->getRequire(self::$tempDir.'/file.php'));
+        $this->assertSame('Howdy?', $files->getRequire(self::$tempDir.'/file.php', cache: true));
+    }
+
     public function testJsonReturnsDecodedJsonData()
     {
         file_put_contents(self::$tempDir.'/file.json', '{"foo": "bar"}');