From 1c45941711f237cc75819e44c2611d0f84a6e5b8 Mon Sep 17 00:00:00 2001
From: Oliver Vogel <oliver@olivervogel.com>
Date: Wed, 4 Oct 2023 16:29:28 +0200
Subject: [PATCH] Add file pointer image decoder

---
 .../Gd/Decoders/FilePointerImageDecoder.php   | 25 +++++++++++++++++++
 src/Drivers/Gd/InputHandler.php               |  1 +
 .../Decoders/FilePointerImageDecoder.php      | 25 +++++++++++++++++++
 src/Drivers/Imagick/InputHandler.php          |  1 +
 .../Decoders/FilePointerImageDecoderTest.php  | 25 +++++++++++++++++++
 .../Decoders/FilePointerImageDecoderTest.php  | 25 +++++++++++++++++++
 6 files changed, 102 insertions(+)
 create mode 100644 src/Drivers/Gd/Decoders/FilePointerImageDecoder.php
 create mode 100644 src/Drivers/Imagick/Decoders/FilePointerImageDecoder.php
 create mode 100644 tests/Drivers/Gd/Decoders/FilePointerImageDecoderTest.php
 create mode 100644 tests/Drivers/Imagick/Decoders/FilePointerImageDecoderTest.php

diff --git a/src/Drivers/Gd/Decoders/FilePointerImageDecoder.php b/src/Drivers/Gd/Decoders/FilePointerImageDecoder.php
new file mode 100644
index 00000000..80860797
--- /dev/null
+++ b/src/Drivers/Gd/Decoders/FilePointerImageDecoder.php
@@ -0,0 +1,25 @@
+<?php
+
+namespace Intervention\Image\Drivers\Gd\Decoders;
+
+use Intervention\Image\Exceptions\DecoderException;
+use Intervention\Image\Interfaces\ColorInterface;
+use Intervention\Image\Interfaces\ImageInterface;
+
+class FilePointerImageDecoder extends BinaryImageDecoder
+{
+    public function decode($input): ImageInterface|ColorInterface
+    {
+        if (!is_resource($input) || !in_array(get_resource_type($input), ['file', 'stream'])) {
+            throw new DecoderException('Unable to decode input');
+        }
+
+        $contents = '';
+        rewind($input);
+        while (!feof($input)) {
+            $contents .= fread($input, 8192);
+        }
+
+        return parent::decode($contents);
+    }
+}
diff --git a/src/Drivers/Gd/InputHandler.php b/src/Drivers/Gd/InputHandler.php
index 89dccc71..c100808c 100644
--- a/src/Drivers/Gd/InputHandler.php
+++ b/src/Drivers/Gd/InputHandler.php
@@ -8,6 +8,7 @@ class InputHandler extends AbstractInputHandler
 {
     protected $decoders = [
         Decoders\ImageObjectDecoder::class,
+        Decoders\FilePointerImageDecoder::class,
         Decoders\ArrayColorDecoder::class,
         Decoders\HtmlColorNameDecoder::class,
         Decoders\RgbStringColorDecoder::class,
diff --git a/src/Drivers/Imagick/Decoders/FilePointerImageDecoder.php b/src/Drivers/Imagick/Decoders/FilePointerImageDecoder.php
new file mode 100644
index 00000000..6806a699
--- /dev/null
+++ b/src/Drivers/Imagick/Decoders/FilePointerImageDecoder.php
@@ -0,0 +1,25 @@
+<?php
+
+namespace Intervention\Image\Drivers\Imagick\Decoders;
+
+use Intervention\Image\Exceptions\DecoderException;
+use Intervention\Image\Interfaces\ColorInterface;
+use Intervention\Image\Interfaces\ImageInterface;
+
+class FilePointerImageDecoder extends BinaryImageDecoder
+{
+    public function decode($input): ImageInterface|ColorInterface
+    {
+        if (!is_resource($input) || !in_array(get_resource_type($input), ['file', 'stream'])) {
+            throw new DecoderException('Unable to decode input');
+        }
+
+        $contents = '';
+        rewind($input);
+        while (!feof($input)) {
+            $contents .= fread($input, 8192);
+        }
+
+        return parent::decode($contents);
+    }
+}
diff --git a/src/Drivers/Imagick/InputHandler.php b/src/Drivers/Imagick/InputHandler.php
index 05a4100e..3c751a29 100644
--- a/src/Drivers/Imagick/InputHandler.php
+++ b/src/Drivers/Imagick/InputHandler.php
@@ -8,6 +8,7 @@ class InputHandler extends AbstractInputHandler
 {
     protected $decoders = [
         Decoders\ImageObjectDecoder::class,
+        Decoders\FilePointerImageDecoder::class,
         Decoders\ArrayColorDecoder::class,
         Decoders\HexColorDecoder::class,
         Decoders\HtmlColorNameDecoder::class,
diff --git a/tests/Drivers/Gd/Decoders/FilePointerImageDecoderTest.php b/tests/Drivers/Gd/Decoders/FilePointerImageDecoderTest.php
new file mode 100644
index 00000000..5de29a12
--- /dev/null
+++ b/tests/Drivers/Gd/Decoders/FilePointerImageDecoderTest.php
@@ -0,0 +1,25 @@
+<?php
+
+namespace Intervention\Image\Tests\Drivers\Gd\Decoders;
+
+use Intervention\Image\Drivers\Gd\Decoders\FilePointerImageDecoder;
+use Intervention\Image\Drivers\Gd\Image;
+use Intervention\Image\Tests\TestCase;
+use Intervention\Image\Tests\Traits\CanCreateGdTestImage;
+
+/**
+ * @requires extension gd
+ * @covers \Intervention\Image\Drivers\Gd\Decoders\FilePointerImageDecoder
+ */
+class FilePointerImageDecoderTest extends TestCase
+{
+    use CanCreateGdTestImage;
+
+    public function testDecode(): void
+    {
+        $decoder = new FilePointerImageDecoder();
+        $fp = fopen($this->getTestImagePath('test.jpg'), 'r');
+        $result = $decoder->decode($fp);
+        $this->assertInstanceOf(Image::class, $result);
+    }
+}
diff --git a/tests/Drivers/Imagick/Decoders/FilePointerImageDecoderTest.php b/tests/Drivers/Imagick/Decoders/FilePointerImageDecoderTest.php
new file mode 100644
index 00000000..aeb4bc89
--- /dev/null
+++ b/tests/Drivers/Imagick/Decoders/FilePointerImageDecoderTest.php
@@ -0,0 +1,25 @@
+<?php
+
+namespace Intervention\Image\Tests\Drivers\Imagick\Decoders;
+
+use Intervention\Image\Drivers\Imagick\Decoders\FilePointerImageDecoder;
+use Intervention\Image\Drivers\Imagick\Image;
+use Intervention\Image\Tests\TestCase;
+use Intervention\Image\Tests\Traits\CanCreateImagickTestImage;
+
+/**
+ * @requires extension imagick
+ * @covers \Intervention\Image\Drivers\Imagick\Decoders\FilePointerImageDecoder
+ */
+class FilePointerImageDecoderTest extends TestCase
+{
+    use CanCreateImagickTestImage;
+
+    public function testDecode(): void
+    {
+        $decoder = new FilePointerImageDecoder();
+        $fp = fopen($this->getTestImagePath('test.jpg'), 'r');
+        $result = $decoder->decode($fp);
+        $this->assertInstanceOf(Image::class, $result);
+    }
+}