FriendsOfREDAXO / developer

Ermöglicht die Bearbeitung von Templates, Modulen und Aktionen im Dateisystem (also mit einem beliebigen Editor).
MIT License
90 stars 11 forks source link

Module/Templates via file includen statt streamwrapper #70

Closed staabm closed 5 years ago

staabm commented 6 years ago

Wäre es möglich, dass redaxo bei installiertem developer die module und templates aus dem filesystem included?

Solange php bei stream wrappeen noch keinen opcache verwendet könnte dies einen performance vorteil darstellen (these die noch zu validieren ist)

gharlan commented 6 years ago

Die Dateien im Developer-Addon sind die ungeparsten (REX_VARs), die können also nicht einfach genommen werden. Beispiel Moduloutput: Pro Slice muss es eine separate PHP-Datei geben, da die REX_VALUEs ja unterschiedlich sind. Zumindest wenn man nicht auch an den REX_VARs auch noch was ändert.

rex_stream hat übrigens schon einen alternativen Modus, wo mit realen Dateien gearbeitet wird: https://github.com/redaxo/redaxo/blob/master/redaxo/src/core/lib/util/stream.php#L49-L50 Aktuell wird der Modus aber nur verwendet, wenn die suhosin-Extension installiert ist, und da rex:// nicht freigegeben ist.

Ich denke, dass in diesem Modus der Opcache bereits funktionieren müsste. Und ich habe auch schon ab und zu überlegt, ob man den Modus nicht nur noch verwenden sollte.

Vorteil:

Nachteile:

staabm commented 6 years ago

ok, da muss man mal blackfire messen mit. danke für den input.

danach kann man dann entscheiden wie es weiter geht.

gharlan commented 5 years ago

Das Developer-Addon kann hier aus meiner Sicht nichts tun. So oder so müssen die Dateien erst geparst werden (REX_VARs), und das ist bereits nicht mehr Part vom Developer-Addon. Ob somit anschließend mit einem Stream oder mit echten Dateien gearbeitet wird, müsste im Core oder im structure-Addon entschieden werden.