Signature for compression only requires array of mimetypes, if empty, there will be no conversion

Author: tijmenbruggeman

src/class-tiny-compress-client.php

@@ -88,7 +88,7 @@ protected function validate() {
 		}
 	}
 
-	protected function compress( $input, $resize_opts, $preserve_opts, $convert_opts ) {
+	protected function compress( $input, $resize_opts, $preserve_opts, $convert_to ) {
 		try {
 			$this->last_error_code = 0;
 			$this->set_request_options( \Tinify\Tinify::getClient() );
@@ -121,8 +121,7 @@ protected function compress( $input, $resize_opts, $preserve_opts, $convert_opts
 			$buffer = $compress_result->toBuffer();
 			$result = array( $buffer, $meta, null );
 
-			if ( isset( $convert_opts['convert'] ) && true == $convert_opts['convert'] ) {
-				$convert_to = $convert_opts['convert_to'];
+			if ( count( $convert_to ) > 0 ) {
 				$convert_source = $source->convert( array(
 					'type' => $convert_to,
 				) );

src/class-tiny-compress-fopen.php

@@ -82,7 +82,7 @@ protected function validate() {
 		}
 	}
 
-	protected function compress( $input, $resize_opts, $preserve_opts, $convert_opts ) {
+	protected function compress( $input, $resize_opts, $preserve_opts, $convert_to ) {
 		$params = $this->request_options( 'POST', $input );
 		list($details, $headers, $status_code) = $this->request( $params );
 
@@ -146,8 +146,7 @@ protected function compress( $input, $resize_opts, $preserve_opts, $convert_opts
 
 		$convert = null;
 
-		if ( isset( $convert_opts['convert'] ) && true === $convert_opts['convert'] ) {
-			$convert_to = $convert_opts['convert_to'];
+		if ( count( $convert_to ) > 0 ) {
 			$convert_params = $this->request_options(
 				'POST',
 				array(

src/class-tiny-compress.php

@@ -99,14 +99,14 @@ public function get_status() {
 	 * @param [type] $file
 	 * @param array $resize_opts
 	 * @param array $preserve_opts
-	 * @param array{ convert: bool, convert_to: string } conversion options
+	 * @param array{ string } conversion options
 	 * @return void
 	 */
 	public function compress_file(
 		$file,
 		$resize_opts = array(),
 		$preserve_opts = array(),
-		$convert_opts = array()
+		$convert_to = array()
 	) {
 		if ( $this->get_key() == null ) {
 			throw new Tiny_Exception( self::KEY_MISSING, 'KeyError' );
@@ -131,7 +131,7 @@ public function compress_file(
 				$file_data,
 				$resize_opts,
 				$preserve_opts,
-				$convert_opts
+				$convert_to
 			);
 		} catch ( Tiny_Exception $err ) {
 			$this->call_after_compress_callback();
@@ -172,7 +172,7 @@ protected abstract function compress(
 		$input,
 		$resize_options,
 		$preserve_options,
-		$convert_opts
+		$convert_to
 	);
 
 	protected static function identifier() {

test/unit/TinyCliTest.php

@@ -56,21 +56,15 @@ public function test_will_compress_attachments_given_in_params()
 			'file' => "vfs://root/wp-content/uploads/2025/07/test.png",
 			'resize' => false,
 			'preserve' => array(),
-			'convert_opts' => array(
-				'convert' => false,
-				'convert_to' => array(
-					'image/avif',
-					'image/webp'
-				)
-			),
+			'convert_to' => array(),
 		);
 		$mockCompressor->expects($this->once())
 			->method('compress_file')
 			->with(
 				$expected['file'],
 				$expected['resize'],
 				$expected['preserve'],
-				$expected['convert_opts']
+				$expected['convert_to']
 			);
 		$settings->set_compressor($mockCompressor);