Skip to content

Draft: replace mishal/iless with mediawiki/less.php, increase php platform to most current 7.2.34, fixes #2381

Jan-Hendrik Willms requested to merge biest-2381 into main

Der MR tauscht die Bibliothek mishal/iless gegen mediawiki/less.php aus und passt die Stellen im Kern entsprechend an. Dieser Schritt ist notwendig, da iless nicht mehr aktiv gepflegt wird und wir die "Probleme" mit den Warnungen nicht gut in den Griff kriegen können. Es gibt eine Anpassung, dass die Warnungen immer unterdrückt werden, aber schön ist das nicht.

Mit der Lösung gibt es noch folgende Probleme bzw. Besonderheiten:

  • Die vorherige Version hat leider für LESS invalide Strings als korrekte Variablen übernommen. Das wird einem hier nun vor die Füße fallen, wenn man zum Beispiel eigene Variablen für Pfade ohne umgebende Quotes übergibt. Bei mir geht dabei aktuell nur ein einziges Plugin kaputt, aber man weiss ja nie.
  • Unsere in der package.json festgezurrte PHP-Version ist 7.2.5. Dadurch bedingt würden wir nur Version 2.x von mediawiki/less.php nutzen können. Durch Erhöhung auf die aktuellste PHP 7.2-Version 7.2.34 können wir Version 3.x nutzen. Ich finde die Anpassung unproblematisch, aber sie sollte uns bewusst sein.

Ich bin mir ingesamt nicht sicher, ob sich die Umstellung lohnt, da wir ja eh von LESS wegwollen und ob das hier wirklich ein BIEST sein sollte oder vielleicht eher ein TIC. Oder wir lassen das komplett sein und deprecaten LESS einfach in der nächsten Version und entfernen den Support zur 6.0.

Merge request reports

Loading