Vorhandensein einer Grafik über die UUID prüfen

[pottkind]

Hallo liebes Forum, ich brauche Hilfe bei der Einbindung eines Bildes über die UUID.
Ich binde in meinem Controller über den FigureBuilder ein Bild ein. Das funktioniert auch.

        if($model->singleSRC){

            $figureBuilder = \System::getContainer()
                   ->get(\Contao\CoreBundle\Image\Studio\Studio::class)
                   ->createFigureBuilder();

            $imageUuid = \StringUtil::binToUuid($model->singleSRC);

            $figureBuilder
                ->fromUuid($imageUuid)
                ->setSize($model->size);

            $figure = $figureBuilder->build();

            $figure->applyLegacyTemplateData($template);

        } 

Wenn dieses Bild jetzt jedoch über die Dateiverwaltung gelöscht wird (aus welchem Grund auch immer), wird verständlicherweise eine Fehlermeldung generiert (Es ist ein Fehler aufgetreten...):

Im Log steht folgendes:

Uncaught PHP Exception Contao\CoreBundle\Exception\InvalidResourceException: "DBAFS item with UUID "d6166254-e8f2-11ed-b87e-95a26d3ce6ca" could not be found." at ...

Die UUID wird weiterhin übergeben, weshalb ich nicht auf das Existieren der UUID prüfen kann.
Aber das Bild selbst existiert ja nicht mehr.

Wie prüfe ich am besten, ob das Bild überhaupt existiert und verhindere, dass die Seite die Fehlermeldung ausgibt.
Im besten Fall wird einfach das Bild nicht angezeigt.

[Spooky]

$figure = $this->imageStudio->createFigureBuilder()
    ->from($model->singleSRC)
    ->setSize($model->size)
    ->buildIfResourceExists()
;

if ($figure) {
    $figure->applyLegacyTemplateData($template);
} 

[pottkind]

$figure = $this->imageStudio->createFigureBuilder()
    ->from($model->singleSRC)
    ->setSize($model->size)
    ->buildIfResourceExists()
;

if ($figure) {
    $figure->applyLegacyTemplateData($template);
} 

Hallo Spooky, danke für die schnelle Antwort.
Muss ich nicht über die UUID gehen?

$imageUuid = \StringUtil::binToUuid($model->singleSRC) 

Kann mein Code mit deinem einfach ersetzt werden?
Hier wirft dann allerdings

$figure = $this->imageStudio->createFigureBuilder() 

einen Fehler

Call to a member function createFigureBuilder() on null

use Contao\CoreBundle\Image\Studio; hatte ich oben eingebunden.

[Spooky]

$model->singleSRC ist bereits die UUID.

[pottkind]

$model->singleSRC ist bereits die UUID.

Ok. Danke für den Hinweis

Ich habe jetzt einfach den ursprünglichen Code etwas nach deiner Vorlage erweitert.

$figureBuilder in die Variable $figure gepackt.
UUID durch ->from($model->singleSRC) ersetzt, ->buildIfResourceExists hinzugefügt und unten auf Gültigkeit von $figure geprüft.

            $figureBuilder = \System::getContainer()
                   ->get(\Contao\CoreBundle\Image\Studio\Studio::class)
                   ->createFigureBuilder();

            $figure = $figureBuilder
                ->from($model->singleSRC)
                ->setSize($model->size)
               ->buildIfResourceExists();
            
            if ($figure) {
                $figure = $figureBuilder->build();
                $figure->applyLegacyTemplateData($template);
            } 

So funktioniert es!

Danke dir!

[Spooky]

Statt \System::getContainer() solltest du Dependency Injection verwenden.

[pottkind]

Statt \System::getContainer() solltest du Dependency Injection verwenden.

Hallo Spooky, jetzt wollte ich es noch auf die Dependency Injection hin anpassen.
Wenn ich mir die Doku ansehe, dann müsste es per Dependency Injection so eingebunden werden:

$figureBuilder = $this->studio->createFigureBuilder(); 

Auf diese Weise ist jedoch "$this->studio" eine undefined property.
Fehler im Log:

Call to a member function createFigureBuilder() on null

Das use statement habe ich drin.

use Contao\CoreBundle\Image\Studio;

Was fehlt bei mir?

[Spooky]

Das use Statement alleine reicht natürlich nicht - das expandiert dir ja nur den FQCN von einem Klassennamen. Für Dependency Injection muss:

• Deine Klasse als Service registriert sein.
• Über die Service Konfiguration injectest du dir das Image Studio.
• Im Konstruktor kannst du dann auf die Services, die du dir injected hast, zugreifen.

Für die App brauchst du unter PHP 8.1+ nur folgendes machen, weil der Rest bereits automatisch konfiguriert ist:

namespace App;

use Contao\CoreBundle\Image\Studio\Studio;

class MyService
{
    public function __construct(private readonly Studio $studio)
    {
    }
    
    public function __invoke(): void
    {
        // $this->studio->…;
    }
} 

[pottkind]

Das use Statement alleine reicht natürlich nicht - das expandiert dir ja nur den FQCN von einem Klassennamen. Für Dependency Injection muss:

• Deine Klasse als Service registriert sein.
• Über die Service Konfiguration injectest du dir das Image Studio.
• Im Konstruktor kannst du dann auf die Services, die du dir injected hast, zugreifen.

Für die App brauchst du unter PHP 8.1+ nur folgendes machen, weil der Rest bereits automatisch konfiguriert ist:

namespace App;

use Contao\CoreBundle\Image\Studio\Studio;

class MyService
{
    public function __construct(private readonly Studio $studio)
    {
    }
    
    public function __invoke(): void
    {
        // $this->studio->…;
    }
} 

Super, danke dir!

Ich muss das Projekt dann erst einmal auf PHP 8.1 heben und testen.
Dann versuche ich es mal nach deiner Anleitung.

Danke noch mal für die Hilfe!

[Spooky]

Unter PHP 7 wäre es so:

namespace App;

use Contao\CoreBundle\Image\Studio\Studio;

class MyService
{
    private Studio $studio;

    public function __construct(Studio $studio)
    {
        $this->studio = $studio;
    }
    
    public function __invoke(): void
    {
        // $this->studio->…;
    }
} 

Siehe https://wiki.php.net/rfc/constructor_promotion bzw. https://www.php.net/manual/en/langua...ctor.promotion