XH Core-Styles - Systemschriftart

[olape]

https://github.com/cmsimple-xh/cmsimple ... w.php#L181

Code: Select all

            if (substr($this->currentDirectory, 0, strlen($folder)) !== $folder) {

zu

Code: Select all

            if (substr($this->currentDirectory, 0, strlen("$folder/")) !== "$folder/") {

Probiere mal!

[frase]

Probiere mal!

Yep, das funktioniert.
Aber - hatten wir das nicht schonmal ein paar Zeilen weiter oben?
https://github.com/cmsimple-xh/cmsimple ... 1b91cae67f

Machst du einen PR?

[olape]

Aber - hatten wir das nicht schonmal ein paar Zeilen weiter oben?

Ja

Machst du einen PR?

Wohin? Gibt es den Filebrowser denn überhaupt einzeln?
Wenn wir keine 1.7.5 wollen, dann, denke ich jedenfalls, müsste erst mal eine 1.8 angelegt werden.
Es macht sicher keinen Sinn, das jetzt noch auf die 1.7 zu machen.

[frase]

Wohin? Gibt es den Filebrowser denn überhaupt einzeln?

Nein.

Wenn wir keine 1.7.5 wollen, dann, denke ich jedenfalls, müsste erst mal eine 1.8 angelegt werden.
Es macht sicher keinen Sinn, das jetzt noch auf die 1.7 zu machen.

Stimmt.
Dann wenigstens ein Issue, damit es nicht vergessen wird.

[cmb]

Oh, tatsächlich, das hatte ich übersehen. Kann gerne gleich für den "master" Branch gefixt werden.

Und substr() sollte man für Stringvergleiche eigentlich gar nicht verwenden, weil substr() eben einen neuen String erzeugt, was unnötig ist. Ab PHP 8.0.0 geht es sehr elegant mit:

Code: Select all

str_starts_with($this->currentDirectory, "$folder/")

Für ältere Version könnte man es so ersetzen:

Code: Select all

strncmp($this->currentDirectory, "$folder/", strlen("$folder/") === 0

Oder vielleicht gar einen str_starts_with() Fallback implementieren (wird, glaube ich, auch an diversen anderen Stellen genutzt, häufig mit strpos(...) === 0 gelöst).

[olape]

Kann gerne gleich für den "master" Branch gefixt werden.

Na dann ...