- Issue created by @ericgsmith
- 🇳🇿New Zealand ericgsmith
ericgsmith → changed the visibility of the branch 3476288-check-file-exists to hidden.
- 🇳🇿New Zealand ericgsmith
ericgsmith → changed the visibility of the branch 3476288-check-file-exists to active.
- 🇳🇿New Zealand ericgsmith
ericgsmith → changed the visibility of the branch 3476288-on-3343547 to hidden.
- Merge request !4Issue #3476288: Return 404 if image derivative does not exist → (Open) created by ericgsmith
- 🇳🇿New Zealand ericgsmith
Updated to revert previous attempt (since all this did was upgrade a warning to an error) and throw a 404 exception + explanation of why we don't want to fall back to generating the image style here.
Added branch 3476288-on-3343547 with this change onto 🐛 Make compatible with Drupal 9/10 RTBC since this module really needs that as well to work.
- 🇳🇿New Zealand klidifia
I tested the updated patch with the scenario outlined and confirm no warnings/errors and instead a 404 now -- looks good.
The code looks good and the comment explains nicely. Good stuff!