From 01924c8a2b0ecef63cade97ee2bb904fa2813f4e Mon Sep 17 00:00:00 2001
From: ADmad <admad.coder@gmail.com>
Date: Sun, 27 Oct 2024 22:56:10 +0530
Subject: [PATCH] Optimize ImageInterface instance creation.

- Remove unneeded temp image creation
- Specify only the required binary image decoder class for ImageManager::read()
---
 src/Api/Api.php       |  3 ++-
 src/Server.php        | 13 +------------
 tests/Api/ApiTest.php |  5 ++++-
 tests/ServerTest.php  |  7 ++-----
 4 files changed, 9 insertions(+), 19 deletions(-)

diff --git a/src/Api/Api.php b/src/Api/Api.php
index 45be6f9..8c0f662 100644
--- a/src/Api/Api.php
+++ b/src/Api/Api.php
@@ -2,6 +2,7 @@
 
 namespace League\Glide\Api;
 
+use Intervention\Image\Decoders\BinaryImageDecoder;
 use Intervention\Image\ImageManager;
 use Intervention\Image\Interfaces\ImageInterface;
 use League\Glide\Manipulators\ManipulatorInterface;
@@ -88,7 +89,7 @@ public function getManipulators(): array
      */
     public function run(string $source, array $params): string
     {
-        $image = $this->imageManager->read($source);
+        $image = $this->imageManager->read($source, BinaryImageDecoder::class);
 
         foreach ($this->manipulators as $manipulator) {
             $manipulator->setParams($params);
diff --git a/src/Server.php b/src/Server.php
index dbc4de4..fcc34d6 100644
--- a/src/Server.php
+++ b/src/Server.php
@@ -620,24 +620,13 @@ public function makeImage(string $path, array $params): string
             throw new FilesystemException('Could not read the image `'.$sourcePath.'`.');
         }
 
-        // We need to write the image to the local disk before
-        // doing any manipulations. This is because EXIF data
-        // can only be read from an actual file.
-        $tmp = tempnam($this->tempDir, 'Glide');
-
-        if (false === file_put_contents($tmp, $source)) {
-            throw new FilesystemException('Unable to write temp file for `'.$sourcePath.'`.');
-        }
-
         try {
             $this->cache->write(
                 $cachedPath,
-                $this->api->run($tmp, $this->getAllParams($params))
+                $this->api->run($source, $this->getAllParams($params))
             );
         } catch (FilesystemV2Exception $exception) {
             throw new FilesystemException('Could not write the image `'.$cachedPath.'`.');
-        } finally {
-            unlink($tmp);
         }
 
         return $cachedPath;
diff --git a/tests/Api/ApiTest.php b/tests/Api/ApiTest.php
index c55b2c1..9b36670 100644
--- a/tests/Api/ApiTest.php
+++ b/tests/Api/ApiTest.php
@@ -79,6 +79,9 @@ public function testRun()
 
         $api = new Api($manager, [$manipulator]);
 
-        $this->assertEquals('encoded', $api->run(dirname(__FILE__, 2).'/files/red-pixel.png', []));
+        $this->assertEquals('encoded', $api->run(
+            file_get_contents(dirname(__FILE__, 2).'/files/red-pixel.png'),
+            []
+        ));
     }
 }
diff --git a/tests/ServerTest.php b/tests/ServerTest.php
index 07b7b8c..387fba8 100644
--- a/tests/ServerTest.php
+++ b/tests/ServerTest.php
@@ -2,7 +2,6 @@
 
 namespace League\Glide;
 
-use Hamcrest\Matchers;
 use League\Glide\Filesystem\FileNotFoundException;
 use League\Glide\Filesystem\FilesystemException;
 use PHPUnit\Framework\TestCase;
@@ -522,8 +521,7 @@ public function testMakeImageFromSource()
         }));
 
         $this->server->setApi(\Mockery::mock('League\Glide\Api\ApiInterface', function ($mock) {
-            $tmpDirPattern = Matchers::matchesPattern('~\/?'.sys_get_temp_dir().'.*~');
-            $mock->shouldReceive('run')->withArgs([$tmpDirPattern, []])->andReturn('content')->once();
+            $mock->shouldReceive('run')->withArgs(['content', []])->andReturn('content')->once();
         }));
 
         $this->assertEquals(
@@ -546,8 +544,7 @@ public function testMakeImageFromSourceWithCustomTmpDir()
 
         $this->server->setTempDir(__DIR__);
         $this->server->setApi(\Mockery::mock('League\Glide\Api\ApiInterface', function ($mock) {
-            $tmpDirPattern = Matchers::matchesPattern('~^'.__DIR__.'.*~');
-            $mock->shouldReceive('run')->with($tmpDirPattern, [])->andReturn('content')->once();
+            $mock->shouldReceive('run')->with('content', [])->andReturn('content')->once();
         }));
 
         $this->assertEquals(