tito10047 / progressive-image-bundle / 31819863337

96%

Build Type

push

github

Committed by Jozef Mostka

Commit Message

fix: POI crop/thumbnail filter ordering when imageConfigs has pre-existing thumbnail

PHP keeps existing array keys at their original position when a value is
overwritten. If imageConfigs already contained filters.thumbnail, assigning
crop then thumbnail produced [thumbnail, crop] — causing LiipImagine to
apply thumbnail (centred on image centre) before crop (POI region), so the
focal point was ignored for vertical centering.

Fix: unset both keys before reassigning to guarantee crop precedes thumbnail
in array-insertion order. Covered by a new unit test that passes imageConfigs
with a pre-existing thumbnail filter and asserts crop index < thumbnail index.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Coverage Stats

0 of 1 new or added line in 1 file covered. (0.0%)

163 existing lines in 10 files now uncovered.

910 of 1004 relevant lines covered (90.64%)

414.26 hits per line