- Issue created by @jsacksick
- Merge request !24Issue #3485811 by jsacksick: Too few arguments to function Drupal\Core\Form\ConfigFormBase::__construct() → (Open) created by jsacksick
- Status changed to Needs review
4 days ago 8:05pm 9 June 2025 - 🇪🇪Estonia tormi Tallinn
I've applied the patch from #3 and identified a small issue that needed correction. The patch had an inconsistent service reference in
src/Form/ImageOptimizerForm.php
.Issue found:
Line 57 in the patch was using$instance->webhook = $container->get('fastly.webhook');
but this service doesn't exist. All other form files correctly usefastly.services.webhook
. - 🇮🇱Israel jsacksick
This no longer applies due to 🐛 The website encountered an unexpected error. Try again later. Active being committed. So perhaps we should mark this one as outdated...
- 🇮🇱Israel jsacksick
I would have prefered to see this patch so there is no need to keep tracking changes to the parent constructor, but it is what it is...