Closed alxndr-w closed 3 years ago
gharlan
commented
3 years ago @gharlan wäre das so richtig?
Nein, doppelte Keys sind nicht möglich. Ich glaube es müsste >=5.5, <8 heißen. Wobei ich mir mit dem Komma gerade nicht sicher bin.
Ich möchte das Addon aber eigentlich nicht komplett inkompatibel zu php 8 setzen. Es lässt sich ja teilweise nutzen, und ist auch nur ein Entwicklungswerkzeug, somit finde ich es nicht dramatisch, wenn Teilbereiche nicht funktionieren in PHP 8.
alxndr-w
commented
3 years ago @gharlan ich kann's gar nicht mehr richtig benutzen, keine einzige Datentabelle kann ich mir anschauen. Ich würde halt die, die das installieren, gerne davor bewahren, solange es kein Update gibt.
staabm
commented
3 years ago ich kann's gar nicht mehr richtig benutzen, keine einzige Datentabelle kann ich mir anschauen
das ist in meinen augen das perfekte beispiel warum man bei der php version requirement nach oben nicht offen sein sollte, wie es gregor aber gerne nutzt.
selbst wenn du diesen PR jetzt hier jetzt mit einem konstraint so einschränkst, dass php8 nicht mehr erlaubt ist, kannst du dieses requirement für alle vergangenen releases nicht mehr ändern.
d.h. dadurch dass du das nächste releaes (angenommen 1.8.3) jetzt auf <php8 einschränkst, würde ein ordentlicher depenency resolver jetzt sagen: na da nehm ich eben die 1.8.2, die war ja kompatibel mit php8.
-> man hat ein unlösbares problem bzw. müsste alle alten releases anpassen
gharlan
commented
3 years ago Wobei ich die Problematik nicht verneine, sie aber deutlich niedriger einordne, als du. Also in der Gesamtabwägung nehme ich die Problematik in Kauf.
Hinzu kommt: Öfters meint man, man wäre nun kompatibel zu der neuen Version, ist es dann aber doch noch nicht. So auch hier. adminer hatte im Changelog "Support PHP 8". Ich hatte adminer geupdatet, und hatte es auch mit PHP 8 grob "getestet". Dabei habe ich aber gar nicht bis zu der Datenansicht geklickt. Ich bin mir aber sicher, wenn ich die PHP-Version nach oben eingeschränkt gehabt hätte, hätte ich sie in der letzen Version bereits auf PHP 9 hochgesetzt (im Vertrauen zu deren Changelog und meinem Grobtest), und wir hätten das Problem jetzt genauso.
Zusätzlich kommt hinzu, zumindest bei REDAXO: Meist wird man die PHP-Version einfach hochsetzen (oder wird vom Hoster hochgesetzt) und dann schaut man, ob es funktioniert. Bei dem Vorgang kommen die Versionsbedingungen in der package.yml also gar nicht zum Tragen. Und neu installieren wird man sowieso meist die neuste Version. (Die hoffentlich die neuste PHP-Version unterstützt, in dem konkreten Fall hier halt leider auch noch nicht ganz).
@gharlan ich kann's gar nicht mehr richtig benutzen, keine einzige Datentabelle kann ich mir anschauen. Ich würde halt die, die das installieren, gerne davor bewahren, solange es kein Update gibt.
Man kann die Struktur einsehen und ändern, den rex_sql_table-Code rauskopieren, etc. Das möchte ich den PHP8-Nutzern (u.a. mir selbst) nicht verwehren, nur weil ein anderer Part (wenn auch ebenso wichtiger Part) leider noch nicht funktioniert.
36
@gharlan wäre das so richtig?