ilch Forum » Ilch CMS 2.X » Kritik und Verbesserungen » [FIXED] PHP Versionsprüfung vor Installation

Geschlossen
  1. #1
    User Pic
    Mikro Mitglied
    Registriert seit
    05.10.2011
    Beiträge
    237
    Beitragswertungen
    21 Beitragspunkte
    Ilch 2.0 ist ja erst ab PHP-Version 5.4 Installierbar/lauffähig. Es wäre sinnvoller eine Prüfung der Version zu machen bevor jeglicher Code der 5.4 benötigt geladen wird. Wenn die Version nicht kompatibel ist sollte eine Meldung zu lesen sein ( nicht PHP error_reporting ). Denn ich sehe schon wieder X Einträge im Forum.


    Zuletzt modifiziert von Mikro am 18.01.2014 - 12:18:51
    Wer Rechtschreibfehler findet darf sie behalten!
    0 Mitglieder finden den Beitrag gut.
  2. #2
    User Pic
    Pion Hall Of Fame
    Registriert seit
    25.07.2007
    Beiträge
    870
    Beitragswertungen
    51 Beitragspunkte
    Danke für die "Kritik".

    Ich habe die Prüfung nun auch in die "index.php" eingebaut, mit dem Nachteil, dass diese Prüfung nun bei jedem Seitenaufruf angewandt wird.
    Falls dies im späteren Verlauf doch stört oder Perf. Probleme macht, kann man dies immer noch leicht rausmachen. Im Moment aber sinnvoll.

    Fehler wurde behoben, Änderung ist live.

    ( github.com/IlchCMS/Ilch-2.0/commit/1d3efd0ab46809480abf128380db1908feece266 )


    Zuletzt modifiziert von Pion am 18.01.2014 - 14:39:09
    0 Mitglieder finden den Beitrag gut.
  3. #3
    User Pic
    Siggi Hall Of Fame
    Registriert seit
    08.02.2007
    Beiträge
    6.558
    Beitragswertungen
    327 Beitragspunkte
    Ganz am Anfang war es doch mal so das die Installation gar nicht weiter geht wenn man nicht die gleiche oder höhere PHP Version hat.

    Wieso wurde dies nicht so gelassen?
    0 Mitglieder finden den Beitrag gut.
  4. #4
    User Pic
    Pion Hall Of Fame
    Registriert seit
    25.07.2007
    Beiträge
    870
    Beitragswertungen
    51 Beitragspunkte
    ZitatZitat geschrieben von Siggi
    Ganz am Anfang war es doch mal so das die Installation gar nicht weiter geht wenn man nicht die gleiche oder höhere PHP Version hat.

    Wieso wurde dies nicht so gelassen?


    Das Problem ist, dass Fehler bei 5.2 zb schon vor der Installationsrutine kommt. Außerdem muss er nicht mal die Installation machen, wenn er es gleich sieht. Hat also nur Vorteile
    0 Mitglieder finden den Beitrag gut.
  5. #5
    User Pic
    Saarlonz Hall Of Fame
    Registriert seit
    07.08.2008
    Beiträge
    3.003
    Beitragswertungen
    235 Beitragspunkte
    Ich denke unserem Pion fällt da schon noch eine gute Lösung ein zwinker
    0 Mitglieder finden den Beitrag gut.
  6. #6
    User Pic
    Mikro Mitglied
    Registriert seit
    05.10.2011
    Beiträge
    237
    Beitragswertungen
    21 Beitragspunkte
    Jo gerne, schau aber bitte nochmal über die index.php drüber.

    if(!version_compare('5.4.0', '5.4.0', '>='))
    Wer Rechtschreibfehler findet darf sie behalten!
    0 Mitglieder finden den Beitrag gut.
  7. #7
    User Pic
    Pion Hall Of Fame
    Registriert seit
    25.07.2007
    Beiträge
    870
    Beitragswertungen
    51 Beitragspunkte
    ZitatZitat geschrieben von Mikro
    Jo gerne, schau aber bitte nochmal über die index.php drüber.

    if(!version_compare('5.4.0', '5.4.0', '>='))


    Passt doch, damit hat man immer die richtige Version lachen

    Fehler wurde behoben, Änderung ist live.

    ( github.com/IlchCMS/Ilch-2.0/commit/98952bbce0f86c342d42174bcacb279ac82659e9 )
    0 Mitglieder finden den Beitrag gut.
Geschlossen

Zurück zu Kritik und Verbesserungen

Optionen: Bei einer Antwort zu diesem Thema eine eMail erhalten