TASK: servername auch im title tag im backend fragment ergänzt #132
Conversation
fragments/maintenance/backend.php
Outdated
| <title>Maintenance</title> | ||
| <title> | ||
| <?php | ||
| if (rex_addon::get('yrewrite')->isAvailable() && rex_yrewrite::getCurrentDomain()->getName() !== '') { |
There was a problem hiding this comment.
| if (rex_addon::get('yrewrite')->isAvailable() && rex_yrewrite::getCurrentDomain()->getName() !== '') { | |
| if (rex_addon::get('yrewrite')->isAvailable() && rex_yrewrite::getCurrentDomain()->getName() !== null) { |
@alxndr-w Wir hatten es Ja neulich im Slack darüber ob getName() auch Null zurückgeben kann. Dann würde es eventuell sinn machen, anstatt auf leer, auf null zu prüfen.
Wie siehst du das?
There was a problem hiding this comment.
| if (rex_addon::get('yrewrite')->isAvailable() && rex_yrewrite::getCurrentDomain()->getName() !== '') { | |
| if (rex_addon::get('yrewrite')->isAvailable() && rex_yrewrite::getCurrentDomain() !== null && rex_yrewrite::getCurrentDomain()->getName() !== '') { |
There was a problem hiding this comment.
Oder so.. doppelt hält besser ;)
There was a problem hiding this comment.
So meinte ich. Du musst sicherstellen, dass bei rex_yrewrite::getCurrentDomain() auch wirklich ein Objekt zurückkommt, bevor du die Methode ->getName() ausführen kannst.
Bzw. es gibt auch noch eine andere Schreibweise:
rex_yrewrite::getCurrentDomain()?->getName()
^
ich weiß nicht, ob die bereits in PHP 8.1 verfügbar ist. Hier könntest du mal recherchieren:
There was a problem hiding this comment.
Kam wohl mit PHP 8.0 hinzu: https://php.watch/versions/8.0/null-safe-operator
Dann gibt es diese auch mit PHP 8.1 und Höher.
Co-authored-by: Alexander Walther <[email protected]>
crydotsnake
force-pushed
the
task/servername-auch-im-backend-fragment-title-tag-eingebunden
branch
from
7c3307e to
aeae8a9
Compare
…le-tag-eingebunden
Beschreibung / Description
Ergänzt denn Servernamen der Seite noch im Fragment für die Wartungsmodusmaske im Backend.