MM: Unnötige Cache-Dateien vermeiden

[gharlan]

Wenn ein MM-Typ keinerlei Effekte hat, wird die Originaldatei ausgeliefert. Die braucht eigentlich dann auch nicht gecacht werden.

Genauso wenn man einen "virtuellen" MM-Typ aufruft, und der EP MEDIA_MANAGER_FILTERSET keine Effekte liefert.

[staabm]

kann man das unit-testen?

[gharlan]

kann man das unit-testen?

Zumindest für rex_media_manager::create habe ich nun einen Test ergänzt.
Für den normalen Ablauf über rex_media_manager::init() (mit sendMedia()) aber nicht, da dort exit() mit drin steckt.

[staabm]

Ungetestet

[bloep]

Ungetestet

[alxndr-w]

Was ist der Usecase?

[gharlan]

Was ist der Usecase?

Es geht darum unnötige Cache-Dateien zu vermeiden, wenn eigentliche gar keine Effekte angewandt werden.

[alxndr-w]

Gar keine Effekte oder keine Bildeffekte?

[gharlan]

Gar keine Effekte.

[alxndr-w]

Da verstehe ich immer noch nicht den Use case. Wann will man ein Profil anlegen, ohne Effekte hinzuzufügen?

[gharlan]

So wirklich sehe ich da auch keinen Use case für. Aber wir haben diese Möglichkeit ja nicht jetzt frisch eingeführt, sondern nur dafür gesorgt, wenn jemand das nutzt (was vorher auch schon ging), dass dann nicht unnötig Cache-Dateien angelegt werden.

Wo man es eventuell nutzen könnte: Man weiß schon, dass man den MM-Typ mal brauchen wird, die Effekte sind aber noch nicht vorhanden oder zumindest noch nicht eingerichtet. Aber den Typ möchte man schon mal verwenden.
Oder man hat mal einen Typ verwendet, braucht den nun aber eigentlich nicht mehr, da man die Originaldateien an den Stellen stattdessen möchte. Die Aufrufe sollen aber ggf weiter funktionieren.

Irgendwie so vielleicht.
Wobei ich auch nicht unbedingt was dagegen hab, wenn wir die Möglichkeit abschaffen würden, und stattdessen einen Fehler werfen würden oder so, wenn gar keine Effekte angewandt werden.
Für den Moment ging es mir aber vor allem darum diese unnötigen Cache-Dateien zu verhindern, ohne am Verhalten nach Außen etwas zu verändern.

[tbaddade]

siehe auch https://github.com/yakamara/redaxo_yrewrite/blob/master/setup/.htaccess#L97