From 1485ef3f2db7ccc8bfdb8fdc5ae2bfde65685c79 Mon Sep 17 00:00:00 2001
From: Zankaria <zankaria.auxa@mailu.io>
Date: Mon, 17 Mar 2025 22:21:38 +0100
Subject: [PATCH] ImageFormatReader: add iamge format readers

---
 inc/Service/Media/GdImageFormatReader.php     | 18 ++++++++++
 inc/Service/Media/ImageFormatReader.php       | 11 +++++++
 inc/Service/Media/MagickImageFormatReader.php | 33 +++++++++++++++++++
 3 files changed, 62 insertions(+)
 create mode 100644 inc/Service/Media/GdImageFormatReader.php
 create mode 100644 inc/Service/Media/ImageFormatReader.php
 create mode 100644 inc/Service/Media/MagickImageFormatReader.php

diff --git a/inc/Service/Media/GdImageFormatReader.php b/inc/Service/Media/GdImageFormatReader.php
new file mode 100644
index 00000000..de20435c
--- /dev/null
+++ b/inc/Service/Media/GdImageFormatReader.php
@@ -0,0 +1,18 @@
+<?php
+namespace Vichan\Service\Media;
+
+
+class GdImageFormatReader implements ImageFormatReader {
+	/**
+	 * getimagesize() is agnostic of any image metadata.
+	 * If e.g. the Exif Orientation flag is set to a value which rotates the image by 90 or 270 degress, index 0 and 1
+	 * are swapped, i.e. the contain the height and width, respectively.
+	 */
+	public function getSizes(string $file_path): array {
+		$ret = \getimagesize($file_path);
+		if ($ret === false) {
+			throw new \RuntimeException("Could not read image sizes of '$file_path'");
+		}
+		return $ret;
+	}
+}
diff --git a/inc/Service/Media/ImageFormatReader.php b/inc/Service/Media/ImageFormatReader.php
new file mode 100644
index 00000000..2aff3b24
--- /dev/null
+++ b/inc/Service/Media/ImageFormatReader.php
@@ -0,0 +1,11 @@
+<?php
+namespace Vichan\Service\Media;
+
+
+interface ImageFormatReader {
+	/**
+	 * @param string $file_path Image file path.
+	 * @return array An array with width and height.
+	 */
+	public function getSizes(string $file_path): array;
+}
diff --git a/inc/Service/Media/MagickImageFormatReader.php b/inc/Service/Media/MagickImageFormatReader.php
new file mode 100644
index 00000000..6484b2f6
--- /dev/null
+++ b/inc/Service/Media/MagickImageFormatReader.php
@@ -0,0 +1,33 @@
+<?php
+namespace Vichan\Service\Media;
+
+
+class MagickImageFormatReader implements ImageFormatReader {
+	private string $prefix;
+
+	public static function createImageMagickReader(): MagickImageFormatReader {
+		return new self('');
+	}
+
+	public static function createGraphicsMagickReader(): MagickImageFormatReader {
+		return new self('gm ');
+	}
+
+	private function __construct(string $prefix) {
+		$this->prefix = $prefix;
+	}
+
+	public function getSizes(string $file_path): array {
+		$arg = escapeshellarg("$file_path[0]");
+		$ret_exec = shell_exec_error("{$this->prefix}identify -format \"%w %h\" $arg");
+
+		if (!\is_string($ret_exec)) {
+			throw new \RuntimeException("Error while executing identify");
+		}
+		$ret_match = \preg_match('/^(\d+) (\d+)$/', $ret_exec, $m);
+		if (!$ret_match) {
+			throw new \RuntimeException("Could not parse identify output");
+		}
+		return [ $m[1], $m[2] ];
+	}
+}