andileni / media_negotiator Goto Github PK

In der package.yml steht noch 4.0.0, obwohl es ja bereits 4.0.2 ist.

Ich erhalte ein:

Call to undefined function imageavif()
Datei: redaxo/src/addons/media_manager/lib/managed_media.php:338 

redaxo/redaxo#5605

Es gibt auch den Header Eintrag Accept: image/avif,*/*.
Aktuell wird */* nicht als avif oder webp ausgeliefert sonder im default format.
Wie sollte man das am besten angehen? Wieso ist er nicht image/*?

Es scheint auch so, als würden manche Browser (z.b. Firefox 113) nur den neuesten und aktuellsten accept header angeben.
So ist dieser oftmals nur image/avif und image/webp fehlt.

Wann wird die methode eigentlich angepasst? :)

In der media_manager.php

Hallo,
bekomme bei Aufruf der config Fehlermeldung auf meinem Testserver. Server kann kein avif mit Imagick. Das sollte gecheckt werden, bevor damit ein Bild zu erzeugen versucht wird.

ImagickException thrown with message "Unable to set image format"

Stacktrace:
#15 ImagickException in /Users/Shared/Dropbox/htdocs/martha-labil-2023/redaxo/src/addons/media_negotiator/pages/config.php:110
#14 Imagick:setImageFormat in /Users/Shared/Dropbox/htdocs/martha-labil-2023/redaxo/src/addons/media_negotiator/pages/config.php:110
#13 require in /Users/Shared/Dropbox/htdocs/martha-labil-2023/redaxo/src/core/lib/packages/package.php:233
#12 rex_package:includeFile in /Users/Shared/Dropbox/htdocs/martha-labil-2023/redaxo/src/core/lib/be/controller.php:503
#11 rex_be_controller:{closure} in /Users/Shared/Dropbox/htdocs/martha-labil-2023/redaxo/src/core/lib/util/timer.php:62
#10 rex_timer:measure in /Users/Shared/Dropbox/htdocs/martha-labil-2023/redaxo/src/core/lib/be/controller.php:485
#9 rex_be_controller:includePath in /Users/Shared/Dropbox/htdocs/martha-labil-2023/redaxo/src/core/lib/be/controller.php:453
#8 rex_be_controller:includeCurrentPageSubPath in /Users/Shared/Dropbox/htdocs/martha-labil-2023/redaxo/src/addons/media_negotiator/pages/index.php:4
#7 require in /Users/Shared/Dropbox/htdocs/martha-labil-2023/redaxo/src/core/lib/packages/package.php:233
#6 rex_package:includeFile in /Users/Shared/Dropbox/htdocs/martha-labil-2023/redaxo/src/core/lib/be/controller.php:503
#5 rex_be_controller:{closure} in /Users/Shared/Dropbox/htdocs/martha-labil-2023/redaxo/src/core/lib/util/timer.php:62
#4 rex_timer:measure in /Users/Shared/Dropbox/htdocs/martha-labil-2023/redaxo/src/core/lib/be/controller.php:485
#3 rex_be_controller:includePath in /Users/Shared/Dropbox/htdocs/martha-labil-2023/redaxo/src/core/lib/be/controller.php:436
#2 rex_be_controller:includeCurrentPage in /Users/Shared/Dropbox/htdocs/martha-labil-2023/redaxo/src/core/backend.php:241
#1 require in /Users/Shared/Dropbox/htdocs/martha-labil-2023/redaxo/src/core/boot.php:155
#0 require in /Users/Shared/Dropbox/htdocs/martha-labil-2023/redaxo/index.php:9

Das Update auf 2.2.1 brachte keine Verbesserung.

Komischerweise werden die Bilder durch Aufruf im img tag nicht ausgegeben (frontend und backend).
Ein direkter Aufruf per url funktioniert aber... ?!?!

Der Effekt hat glaube ich eine Caching-Problematik. Nicht selbst getestet, aber folgendes Szenario:

• MediaManager-Cache-Dateien existieren noch nicht
• User1 öffnet Website in modernem Browser
• Bei den Bildern wird als Accept-Header also auch avif, webp etc geschickt
• Dein Effekt setzt das Format auf avif oder webp
• Bild wird erzeugt und in den MM-Cache gelegt
• --
• User2 öffnet nun die Website in altem Browser, der kein avif/webp kann
• Bild wird angefragt mit Accept-Header ohne avif/webp
• Trotzdem liefert der MM das Bild aus dem Cache, also das avif- oder webp-Bild
• Die Effekte werden überhaupt nicht durchlaufen, wenn das Bild schon im Cache ist

Spontan habe ich dafür keine leichte Lösung. Es kann im MM aktuell eigentlich pro Typ/Bild-Kombi nur eine Cache-Datei geben.

jetzt hab ich (leider) einen weiteren interessanten Fall auf einem anderen Server mit media_negotiator 4.0.0alpha1

• gd hat sowohl webp als auch avif enabled und wird accepted
• media_negotiator konvertiert in avif, allerdings wird kein Bild generiert -> Error:

imageavif(): avif error - Could not encode image: No codec available
Datei: redaxo/src/addons/media_manager/lib/managed_media.php:343

Über den Media Manager habe ich getestet, ob die Konvertierung über den default-Konvertierungs-Effekt funktioniert. Ergebnis: webp wird konvertiert, avif nicht.

Es wäre daher vielleicht sinnvoll im Addon die Möglichkeit der Codec-Auswahl einzubauen.
EDIT: hab jetzt als workaround die Erkennung von avif in der Helper.php auskommentiert, damit meine Seite funktioniert.

Ein etwas spezieller Fall:
auf einem Shared-Webhosting eines Kunden funktioniert das Addon nicht, es werden einfach leere AVIF Dateien geschrieben.
Das liegt daran, dass die Funktion imageavif zwar existiert, aber in GD mutmasslich kein Codec da ist, um in AVIF umzuwandeln. Das Resultat ist, dass die resultierende AVIF Datei korrupt ist bzw. einfach leer bleibt.

Liegt natürlich nicht am Addon, aber vielleicht gibt es eine Möglichkeit diesen Edge-Case abzufangen, so dass wenigstens die Originaldatei angezeigt wird statt gar nichts? Habe aber gerade keine Idee wie, ausser ev. die Dateigrösse abzufragen ...

Ich schreib mal dem Support des Hosters ... kann ja irgendwie nicht sein.

ich betreibe ein paar Seiten die fast ausschließlich aus Bildern bestehen, dabei musste ich extrem längere Ladezeiten feststellen im Vergleich zur normalen Jpeg Auslieferung, sodass die geringere Dateigröße keinerlei Vorteil mehr bringt, auch wenn die Bildern bereits gecacht wurden. ich verwende lazysizes als lazyloader.

Es stellt sich die Frage was passiert mit gif und PNG, sollten Sie an den Effekt übergeben werden?

Ich bekomme immer JPG

vlt. siehe auch Diskussion redaxo/redaxo#5498