Add ImageMagick support for image processing with GD fallback

[Arro38]

Questions	Answers
Branch?	develop
Description?	Add ImageMagick (Imagick) as alternative image processing engine with GD fallback, and fix missing AVIF format in validation constants
Type?	new feature
Category?	CO
BC breaks?	no
UI tests	https://github.com/SiraDIOP/ga.tests.ui.pr/actions/runs/21628357206
Deprecations?	no
Fixed ticket?	N/A
How to test?	See test plan below

Summary

• Fix missing AVIF format in ImageManager::MIME_TYPE_SUPPORTED and EXTENSIONS_SUPPORTED constants — AVIF uploads were rejected by validation
• Add ImageMagick (Imagick) as alternative image processing engine with automatic detection and GD fallback. Imagick provides higher quality resizing (Lanczos filter) and broader AVIF support
• Update AvifExtensionChecker to detect Imagick AVIF support first, then fall back to GD
• Add unit tests for all new Imagick methods (@requires extension imagick)

Changes

classes/ImageManager.php

• Add image/avif to MIME_TYPE_SUPPORTED and avif to EXTENSIONS_SUPPORTED
• Add isImagickAvailable() — cached static check for Imagick extension
• Add resizeWithImagick() — full resize pipeline using Imagick with Lanczos filter, autoOrient(), canvas compositing (transparent for png/webp/avif, white for jpg)
• Add writeImagick() — format-specific output with quality from PS_*_QUALITY config, progressive JPEG
• Add getImagickSourceFileType() — maps Imagick format strings to IMAGETYPE_* constants
• Extract calculateResizeDimensions() — shared between GD and Imagick paths
• Extract getDestinationFileType() — shared file type resolution
• Dispatch in resize(): try Imagick first, catch exceptions → fall back to GD

src/Core/Image/AvifExtensionChecker.php

• Check Imagick AVIF support first (more efficient), then fall back to GD check

Tests

• ImageManagerTest.php: add AVIF cases to existing data providers + isImagickAvailable test
• ImageManagerImagickTest.php (new): 8 tests covering resizeWithImagick, writeImagick, calculateResizeDimensions, getImagickSourceFileType, auto-orientation

Architecture decision

Uses dispatch pattern (not strategy) because ImageManagerCore is a legacy static class with 22+ static callers. Zero API changes — all callers continue working unchanged. Same pattern as PrestaShop's Db class.

Test plan

1. With Imagick installed: Upload a product image → verify jpg/webp/avif thumbnails generated with correct dimensions
2. Without Imagick: Disable Imagick in php.ini → same test → verify GD fallback works transparently
3. AVIF upload: Upload a .avif file as product image → should be accepted (was rejected before due to missing constant)
4. Unit tests: composer run unit-tests — 32 tests, 53 assertions pass on PHP 8.4 + Imagick

[ps-jarvis]

Hi, thanks for this contribution!

Please consider opening an issue before submitting a Pull Request:

• If it's a bug fix, it helps maintainers verify that the bug is effectively due to a defect in the code, and that it hasn't been fixed already.
• It can help trigger a discussion about the best implementation path before a single line of code is written.
• It may lead the Core Product team to mark that issue as a priority, further attracting the maintainers' attention.

(Note: this is an automated message, but answering it will reach a real human)

[ps-jarvis]

Hello @Arro38!

This is your first pull request on PrestaShop repository of the PrestaShop project.

Thank you, and welcome to this Open Source community!

[Hlavtox]

Hello @Arro38, what a great job!!!

I will approve, just one wish. Please keep the comments I added regarding the JPG/PNG combo and transparecy. 🙏

/*
         * If the filetype is not forced and we are requesting a JPG file, we will adjust the format inside
         * the image according to PS_IMAGE_QUALITY in some cases.
         */

            // If PS_IMAGE_QUALITY is set to png_all, we will use PNG file no matter the 

            // If PS_IMAGE_QUALITY is set to png (optional), we will use PNG if the original format could support transparency.

[Hlavtox]

💣💣💣

[Touxten]

Hello @Arro38

I did a little test, and here's what came out in the video.

When I don't have Imagick, I can still import an AVIF image with an error.
The image isn't created, but there is data in the database.

It works very well when I have Imagick.

2026-01-30.mp4

[Touxten]

@Arro38
Thank you!
It seems to be working now, I'll resubmit a request for reviews.

2026-02-02.mp4

[Touxten]

cc @kpodemski for wording.

[SiraDIOP]

Hello @Arro38,

Thanks for your PR,

When imagick installed:

Enregistrement.de.l.ecran.2026-02-03.a.14.16.07.mov

When imagick not installed

Enregistrement.de.l.ecran.2026-02-03.a.14.27.41.mov

Thanks

I'm waiting automated tests to approve the PR
https://github.com/SiraDIOP/ga.tests.ui.pr/actions/runs/21628357206

[Touxten]

Thank you @SiraDIOP
You did not test the .avif format in the video.

[SiraDIOP]

Hello @Touxten,

My bad i forget the video

Enregistrement.de.l.ecran.2026-02-04.a.14.44.44.mov

When imagick is desactived

Enregistrement.de.l.ecran.2026-02-04.a.14.50.08.mov

Thank you

[SiraDIOP]

Hello @Arro38,

I ran the automated tests, but they are still red.
After discussing with the automated QA team, it seems that the problem comes from this file:

``tests/UI/campaigns/functional/BO/15_header/07_myProfile.ts

Can you please check it?
Thanks in advance! 🙏

[Arro38]

Hello @Arro38,

I ran the automated tests, but they are still red. After discussing with the automated QA team, it seems that the problem comes from this file:

``tests/UI/campaigns/functional/BO/15_header/07_myProfile.ts

Can you please check it? Thanks in advance!

Hi @SiraDIOP !

Yes, I checked the test. The failure is expected because this PR adds AVIF support, which modifies the list of
supported MIME types in ImageManager::MIME_TYPE_SUPPORTED.

The issue:

• The error message now includes image/avif in the allowed formats list
• The test expects the old message (without AVIF) from boMyProfilePage.errorInvalidFormatImageMessage in the
  @prestashop-core/ui-testing package

The solution:
I can fix this test in two ways:

1. Quick fix (in this PR): Update the test to check only the beginning of the error message, making it
   independent of the exact format list:
   expect(textResult).to.contains('Image format not recognized, allowed formats are:');

2. Long-term fix: Update the @prestashop-core/ui-testing package to include AVIF in
   errorInvalidFormatImageMessage, but this requires a separate PR on that repository.

Which approach do you prefer? I'd suggest option 1 to unblock this PR quickly.

[Arro38]

Hi team! 👋 @Codencode @Touxten @SiraDIOP

Just a friendly bump on this PR. All 42 CI checks are now passing (including
the myProfile test that was previously failing).

This PR adds:

• Imagick as an image processing engine (with GD fallback)
• AVIF format support for both upload and thumbnail generation

Would appreciate a review when you get a chance. Thanks!

[Touxten]

Hello @Arro38

@Codencode and I are external to PrestaShop. I will contact the PrestaShop team again.

Thank you for your patience.