search

tbaddade / redaxo_sprog

Platzhalter ersetzen
MIT License
30 stars 15 forks source link

Vendor: Reduzierung & Aktualisierung #77

Closed gharlan closed 2 years ago

gharlan commented 2 years ago

Der Core liefert bereits das Polyfill-Paket für php80 mit. Und das ctype-polyfill-paket ist überflüssig, da der Core die ctype-Extension erfordert (und vorher das Polyfill enthielt). Daher wird nun das Laden der beiden Pakete über den replace-Bereich in der composer.json verhindert.

Außerdem wird nach jedem composer update/install die unnötigen autoload-Dateien entfernt, da wir die vendors über REDAXO laden.

Der PR aktualisiert auch das serializer-Paket.

Getestet habe ich den CSV-Export im Addon.

tbaddade commented 2 years ago

Der Core liefert bereits das Polyfill-Paket für php80 mit. Und das ctype-polyfill-paket ist überflüssig, da der Core die ctype-Extension erfordert (und vorher das Polyfill enthielt).

Müsste die REDAXO Version mit angehoben werden? https://github.com/tbaddade/redaxo_sprog/blob/master/package.yml#L7

gharlan commented 2 years ago

Müsste die REDAXO Version mit angehoben werden?

Hmm guter Punkt. Also nicht wegen ctype, denn da hatten wir auch damals schon das Polyfill-Paket. Aber das polyfill-php80 ist erst mit R5.11 hinzugekommen. Ich habe die Mindestversion mal entsprechend erhöht. Falls du das eigentlich nicht willst, könnte ich aber auch das Paket doch wieder hinzufügen hier.