PHP Versionscheck

[redaxo]

PHP Versionscheck

[dergel]

am anfang des codes eine Meldung ala 'php version >=5.3 needed' reicht.. dann gehts halt noch gar nicht weiter, was auch ok ist .. der andere weg wäre edler, aber muss für den moment nicht sein.

[staabm]

+1 fuer gleich abbrechen

[gharlan]

Scheinbar wird der Check gar nicht ausgeführt, da die master.inc.php ein Parse Error wirft. Siehe http://www.redaxo.org/de/forum/entwicklungen-f39/redaxo-5-noch-diesen-oktober-t18539.html#p102164

[gharlan]

Ich hab mir die master.inc.php noch mal angeschaut, und gehe davon aus, dass der Fehler nur bei PHP 4 kommt. Denn die Datei enthält keinen Code, der erst ab 5.3 läuft, und in der Zeile, die den Fehler wirft, wird eine Klassenkonstante benutzt, die es in PHP 4 noch nicht gab. In PHP 5 (< 5.3) wird daher wohl die oben eingebaute Fehlermeldung korrekt erscheinen.

Wollen wir PHP 4 vernachlässigen, oder an der Stelle statt der Konstante direkt den Wert der Konstante übergeben? Oder habt ihr noch andere Ideen?

[staabm]

Kommt die Fehlermeldung des eigentlichen versioncheck dann auch nicht?

[gharlan]

Ich vermute nein. Beim Parsen der Datei entsteht ja schon der Fehler, und dann wird die Datei wohl gar nicht ausgeführt.

[staabm]

Es geht um folgende zeile https://github.com/redaxo/redaxo/blob/master/redaxo/src/core/master.inc.php#L69 ?

Meinst du das php gar nicht anfaengt zu parsen und daher abbricht bevor es ganz oben auf die phpversions abfrage läuft?

[gharlan]

Genau, um die Zeile geht es. Und ja, ich vermute, dass es so abläuft.

[Koala]

PHP4 sollte jetzt aber wirklich ignoriert werden. Keine Gnade :-)

[gharlan]

Scheinbar ist meine Vermutung falsch, und der Parse Error kommt auch bei PHP 5.2, siehe http://www.redaxo.org/de/forum/entwicklungen-f39/redaxo-5-noch-diesen-oktober-t18539.html#p102200

[staabm]

http://php.net/manual/de/language.oop5.constants.php

[gharlan]

Und was sagt die Seite aus? Mir erschließt es sich dadurch immer noch nicht. Denn die Konstanten gibt es ja seit 5.0, nur der Zugriff per $class::constant (also die Klasse als Variable) gibt es erst ab 5.3, aber das nutzen wir ja gar nicht.

[xrstf]

http://3v4l.org/dQPhB

[gharlan]

Cool, danke! Also doch nur bei PHP 4, da muss sich lichtenstein im Forum wohl vertan haben mit der Version.

Ich schließe mich dann mal @Koala an und bin für ignorieren. @dergel @tbaddade @staabm ?

[staabm]

👍

[tbaddade]

👍

[dergel]

👍