v1.5.1-b2

- Changed: Decimal places are not truncated when entering the maximum file size.
- Changed: array_map() replaced by array_column().
- Changed: Set the maximum number of colours for PNG files to 128.
- Added: The file will not be changed if the iMagick image is larger than the original image.

Author: IMC-GER

imcger/imgupload/adm/style/acp_imgupload_body.html

@@ -68,7 +68,7 @@ <h1>{{ lang('ACP_IMCGER_IMGUPLOAD_TITLE') }}</h1>
 		<dl>
 			<dt><label for="imcger_imgupload_max_filesize">{{ lang('ACP_IMCGER_FILESIZE_MAX') ~ lang('COLON') }}</label><br><span>{{ lang('ACP_IMCGER_FILESIZE_MAX_DESC') }}</span></dt>
 			<dd>
-				{{ common.number('imcger_imgupload_max_filesize', IMCGER_FILESIZE_MAX, 0, 9999999999) }}
+				{{ common.number('imcger_imgupload_max_filesize', IMCGER_FILESIZE_MAX, 0, 9999999999, 'any') }}
 				{% set units = {'BYTES': 'b', 'KIB': 'kb', 'MIB': 'mb', } %}
 				{{ common.select('unit_select', IMCGER_FILESIZE_UNIT, units) }}
 			</dd>

imcger/imgupload/composer.json

@@ -3,8 +3,8 @@
     "type": "phpbb-extension",
     "description": "Using ImageMagick php librarie for resize image attachments and creating thumbnails.",
     "homepage": "https://github.com/IMC-GER/phpBB-Image-upload-use-ImageMagick/tags",
-    "version": "1.5.0",
-    "time": "2025-06-16",
+    "version": "1.5.1-b2",
+    "time": "2025-11-25",
     "license": "GPL-2.0-only",
     "authors": [
         {

imcger/imgupload/controller/admin_controller.php

@@ -87,7 +87,7 @@ public function display_options(): void
 	protected function set_variable(): void
 	{
 		$unit_select  = $this->request->variable('unit_select', 'b');
-		$max_filesize = $this->request->variable('imcger_imgupload_max_filesize', 0);
+		$max_filesize = $this->request->variable('imcger_imgupload_max_filesize', 0.0);
 		$max_filesize = ($unit_select == 'kb') ? round($max_filesize * 1024) : (($unit_select == 'mb') ? round($max_filesize * 1048576) : $max_filesize);
 
 		$this->config->set('imcger_imgupload_tum_quality', $this->request->variable('imcger_imgupload_tum_quality', 80));

imcger/imgupload/event/main_listener.php

@@ -98,11 +98,7 @@ public function set_template_vars(): void
 		$this->db->sql_freeresult();
 
 		// Convert 2 dimensional array into simple array
-		$allowed_images = array_map(
-								function ($n)
-								{
-									return $n['extension'];
-								}, $ext_ary);
+		$allowed_images = array_column($ext_ary, 'extension');
 
 		$metadata_manager = $this->ext_manager->create_extension_metadata_manager('imcger/imgupload');
 
@@ -250,6 +246,15 @@ public function imcger_modify_uploaded_file(object $event): void
 				$write_image = true;
 			}
 
+			$image_filesize = strlen($image->getImageBlob());
+
+			if ($image_filesize > $filesize)
+			{
+				$image->clear();
+
+				return;
+			}
+
 			// store the image when attribute change
 			if ($write_image || $img['changed'])
 			{
@@ -262,7 +267,7 @@ public function imcger_modify_uploaded_file(object $event): void
 					$filedata_array = $event['filedata'];
 
 					// set new file size
-					$filedata_array['filesize'] = $filesize ? $filesize : $image->getImageBlob();
+					$filedata_array['filesize'] = $image_filesize;
 
 					// if image format change to JPEG set filedata to JPEG
 					if ($image_format == 'JPEG' && $event['filedata']['mimetype'] != 'image/jpeg')
@@ -504,11 +509,13 @@ public function set_image_compression(object $image, int $quality = 80): void
 			break;
 
 			case 'PNG':
-				if ($image->getImageColors() > 256)
+				if ($image->getImageColors() > 128)
 				{
-					$image->quantizeImage(256, \Imagick::COLORSPACE_SRGB, 16, false, false);
+					$image->quantizeImage(128, \Imagick::COLORSPACE_SRGB, 8, false, false);
 					$image->setImageType(\Imagick::IMGTYPE_TRUECOLORMATTE);
 				}
+
+				$image->setCompressionQuality(95);
 			break;
 
 			case 'WEBP':