- π«π·France DrDam
I agreed for breaking the loop for performance issues.
If the style have 5 effects and if the crop are the first, there are no use to check all others. - Merge request !19Issue #3227105 by Stefdewa: No crop found in getCropFromImageStyleId if the... β (Open) created by DrDam
- Status changed to RTBC
4 months ago 9:57am 4 September 2024 - Status changed to Needs work
about 2 months ago 7:23pm 24 November 2024 - π·πΊRussia zniki.ru
Thanks for the MR, please fix CS https://git.drupalcode.org/issue/crop-3227105/-/jobs/2644451.
And it would be great to add tests, to display improvement.This change will also made change to existing logic of getCropFromImageStyleId().
It would be great if we can inform users about ongoing change. - π«π·France DrDam
The question is whether the current behavior is βintentionalβ or a case that was not anticipated.
- π·πΊRussia zniki.ru
This need to be fixed, and in order to prevent same issue come back, we need add test case.