tbaddade / redaxo_url

REDAXO 5 AddOn zur URL-Generierung für eigene AddOns (ehemals Url Control, ehemals Frau Schultze)
MIT License
46 stars 22 forks source link

update psr/http-message #287

Open rene-mueller opened 9 months ago

rene-mueller commented 9 months ago

Please update psr/http-message to latest version. The old Version is not compatible with newer PHP versions.

alxndr-w commented 9 months ago

https://packagist.org/packages/psr/http-message#1.1

Es gibt zwar eine 2.0, jedoch erwartet riimu/kit-urlparser Version ^1, aber die 1.1 wäre mir höheren Versionen kompatibel.

Das gesagt wäre mir kein Fehler aufgefallen im Zusammenhang mit dem URL-Addon und PHP 8.2

rene-mueller commented 9 months ago

@alxndr-w es wird direkt auch kein Fehler geworfen, aber die Typensicherung ist nicht gewährleistet. Andere Addons die auch ^1.0 haben werden bei einem composer update somit zusammen nicht funktionieren.

tbaddade commented 9 months ago

Das Issue gehört für mich eher in den Vendor. https://github.com/riimu/kit-urlparser

rene-mueller commented 9 months ago

@tbaddade der Vendor kann hier nichts machen, da ihr den Vendor-Ordner mit im GIT habt. Die composer.json von riimu/kit-urlparser sagt, dass die Version ^1.0 benötigt wird. Somit ist hier die aktuellste Version die 1.1. Somit bitte ich um die Aktualisierung des Vendor-Ordners mittels composer update. PS: Besser wäre es, wenn ihr über den Vendor-Ordner mittels Github-Actions bei jedem Release generiert und dann daran anhängt.

tbaddade commented 9 months ago

Das URL AddOn hat als Vendor riimu/kit-urlparser dabei. Und in diesem Verndor sollte auch das Update von psr/http-message erfolgen. Sobald das geschehen ist, kann das URL AddOn nachziehen.

Ich möchte nicht Vendoren von Vendoren durchgehen und diese noch innerhalb vom Vendor testen.

alxndr-w commented 9 months ago

@tbaddade ich weiß nicht, ob ein Missverständnis vorliegt, aber riimu/kit-urlparser hat doch als Anforderung nur ^1.0, also müsste sich doch über ein composer update die Version von psr/http-message auf 1.1 erhöhen lassen.

https://github.com/Riimu/Kit-UrlParser/blob/483d9807582b2f524564bcbe58017d985e1907e0/composer.json#L23

tbaddade commented 9 months ago

Ich hatte nicht genau gelesen und dachte ihr wollt psr/http-message in Version 2.0 haben.

rene-mueller commented 9 months ago

Danke :)

alxndr-w commented 9 months ago

Passiert, danke auch!