FriendsOfREDAXO / minify_images

Optimiert Bilder mittels Media-Manager-Effekt über verschiedene Dienste (TinyPNG, ImageMagick, u.a.)
MIT License
32 stars 1 forks source link

Umstellung auf 'includeCurrentPageSubPath' #4

Closed christophboecker closed 5 years ago

christophboecker commented 6 years ago

Ziel des PR: Umstellung auf aktuelle rex-Methoden und einheitliche Vorgehensweise in möglichst allen AddOns.

Wie ich letztens gelernt habe, gibt es eine alte und eine aktuelle Methode, die Sub-Pages einzubinden. Die neue Methode ist seit REDAXO 5.1 verfügbar.

Die alte Methode funktioniert weiterhin, sollte aber wegen verbesserter Interoperabilität zwischen AddOns auf die neue Version umgestellt werden (Hinweis von gharlan). In den Core-Addons ist das bereits geschehen, desgleichen bei einige FOR- und Y-Addons. Bezogen auf das Addon selbst sind beide Varianten derzeit gleichwertig. Die neue Methode erlaubt z.B. eine verbesserte Einbindung von Pages aus anderen Addons, die dann im Context des Quell-Addons laufen und nicht des Ziel-Addons.

Betroffen sind rund 40 AddOns (FOR/Y), konkret deren Script pages/index.php, das wie folgt geändert werden sollte:

Alt:

    echo rex_view::title(.....);
    include rex_be_controller::getCurrentPageObject()->getSubPath();

ab REDAXO 5.1:

    echo rex_view::title(.....);
    rex_be_controller::includeCurrentPageSubPath();