Service überschreiben (contao.image.resizer)

[amahnert]

Hallo,

ich versuche contao.image.resizer - Service zu überschreiben. (um Bilder nach dem resizen zu komprimieren)

Ich gehe wie folgt vor:
Im eigenen Bundle überschreibe ich den contao.image.resize-Service.

config/services.yml

services:
    contao.image.resizer:
        class: Amahnert\CoreBundle\Image\ExtendedLegacyResizer
        arguments:
            - '%contao.image.target_dir%'
            - '@contao.image.resize_calculator'
            - '@filesystem'
            - '@contao.image.deferred_image_storage'
        public: true

classes/ExtendedLegacyResizer.php

<?php

namespace Amahnert\CoreBundle\Image;

use Contao\CoreBundle\Image\LegacyResizer;
use Contao\Image\ImageInterface;
use Contao\Image\ResizeConfiguration;
use Contao\Image\ResizeOptions;

class ExtendedLegacyResizer extends LegacyResizer
{

    public function resize(ImageInterface $image, ResizeConfiguration $config, ResizeOptions $options): ImageInterface
    {
        return parent::resize($image, $config, $options);
    }

}

und jetzt nach aufrufen einer beliebigen Seite bekomme ich den Fehler: "Call to a member function initialize() on null".
( in vendor/contao/core-bundle/src/Image/LegacyResizer.php (line 47) )

Was mache ich falsch ?
Komme leider seit 2 Tagen nicht weiter.

[webstar]

Da du einen Service eines anderen Bundles überschreibst, greift vermutlich das automatische Injecten des Contao-Frameworks nicht mehr (Symfony/Contao geht davon aus, dass es schon erledigt wurde).

Grundsätzlich ist das extenden hier nicht die saubere Lösung, vielmehr solltest du den Service dekorieren. Dies unterstützt der Symfony-Container auch: https://symfony.com/doc/current/serv...ecoration.html. Das sieht erstmal nach mehr Code aus, hat aber den entscheidenden Vorteil, dass dein Code völlig unabhängig ist von dem Resizer, den Contao (zukünftig) verwendet.


config/service.yml

amehnert.image.resizer:
  decorates: contao.image.resizer
  arguments:
    - @amehnert.image.resizer.inner
  public: true

class CompressingImageResizer implements DeferredImageInterface
{
    private DeferredImageInterface $resizer;
    
    public function __construct(DeferredImageInterface $resizer)
    {
        $this->resizer = $resizer;
    }

    public function getImagine(): ImagineInterface
    {
        return $this->resizer->getImagine();
    }

    public function getPath(): string
    {
        return $this->resizer->getPath();
    }

    public function getUrl(string $rootDir, string $prefix = ''): string
    {
        return $this->resizer->getUrl($rootDir, $prefix);
    }

    public function getDimensions(): ImageDimensions
    {
        return $this->resizer->getDimensions();
    }

    public function getImportantPart(): ImportantPart
    {
        return $this->resizer->getImportantPart();
    }

    public function setImportantPart(ImportantPart $importantPart = null): ImageInterface
    {
        return $this->resizer->setImportantPart($importantPart);
    }
} 

[amahnert]

Vielen Dank !
Es funktioniert. Ich habe meinen Fehler verstanden :thumbs-up:

[rory_ch]

Hallo zusammen

Ich würde gerne das gleiche machen und den contao.image.resizer Service dekorieren.
Aktuell habe ich folgendes:

services.yml

services:
    example.image.resizer:
        class: Vendor\ExampleBundle\Classes\ImageResizer
        decorates: contao.image.resizer
        arguments:
            - '@example.image.resizer.inner'
        public: true 

Classes\ImageResize.php

<?php

namespace Vendor\ExampleBundle\Classes;

use Contao\Image\DeferredImageInterface;
use Contao\CoreBundle\Image\LegacyResizer;
use Contao\Image\ImageInterface;
use Contao\Image\ResizeConfiguration;
use Contao\Image\ResizeOptions;
use Imagine\Image\ImagineInterface;
use Contao\Image\ImageDimensions;
use Contao\Image\ImportantPart;

class ImageResizer implements DeferredImageInterface
{
    private DeferredImageInterface $resizer;
    
    public function __construct(DeferredImageInterface $resizer)
    {
        $this->resizer = $resizer;
    }

    public function getImagine(): ImagineInterface
    {
        return $this->resizer->getImagine();
    }

    public function getPath(): string
    {
        return $this->resizer->getPath();
    }

    public function getUrl(string $rootDir, string $prefix = ''): string
    {
        return $this->resizer->getUrl($rootDir, $prefix);
    }

    public function getDimensions(): ImageDimensions
    {
        return $this->resizer->getDimensions();
    }

    public function getImportantPart(): ImportantPart
    {
        return $this->resizer->getImportantPart();
    }

    public function setImportantPart(ImportantPart $importantPart = null): ImageInterface
    {
        return $this->resizer->setImportantPart($importantPart);
    }
}

Dabei erhalte ich beim composer install / Cache leeren keine Fehlemeldung.
Aber beim öffnen des Backends, erscheint sofort diese Meldung:

Argument 1 passed to Vendor\ExampleBundle\Classes\ImageResizer::__construct() must implement interface Contao\Image\DeferredImageInterface, instance of Contao\CoreBundle\Image\LegacyResizer given, called in /home/var/cache/dev/ContainerH9ro3X5/getVendor_Image_ResizerService.php on line 22

Wenn ich nun die "DeferredImageInterface" Klasse bei der _construct Methode und dem typed Property entferne, erhalte ich einfach das nächste Problem:

Argument 1 passed to Contao\CoreBundle\Image\ImageFactory::__construct() must implement interface Contao\Image\ResizerInterface, instance of Vendor\ExampleBundle\Classes\ImageResizer given, called in /home/var/cache/dev/ContainerH9ro3X5/getContao_Image_ImageFactoryService.php on line 17

Mache ich was grundsätzliches Falsch?
Kenne mich mit dem dekorieren leider nicht aus.

Danke im Voraus