Protostar: "Footer" in "error.php" und "index.php" unterschiedlich

Gerry77 · 4. April 2020

Und nochmal ein herzliches Hallo an die Community,

Mir ist aufgefallen, dass sich der "Footer" in der Datei "index.php" des Templates Protostar anders verhält als der von/in der Datei "error.php", also auf der Fehlerseite. Dort wird nämlich der Sprung nach oben nicht animiert, nachdem man auf "Nach oben" geklickt hat.

Und tatsächlich gibt es da ein paar Unterschiede:

protostar/index.php

PHP

<footer class="footer" role="contentinfo">
    <div class="container<?php echo ($params->get('fluidContainer') ? '-fluid' : ''); ?>">
        <hr />
        <jdoc:include type="modules" name="footer" style="none" />
        <p class="pull-right">
            <a href="#top" id="back-top">
                <?php echo JText::_('TPL_PROTOSTAR_BACKTOTOP'); ?>
            </a>
        </p>
        <p>
            &copy; <?php echo date('Y'); ?> <?php echo $sitename; ?>
        </p>
    </div>
</footer>

Alles anzeigen

protostar/error.php

PHP

<div class="footer">
    <div class="container<?php echo ($params->get('fluidContainer') ? '-fluid' : ''); ?>">
        <hr />
        <?php echo $this->getBuffer('modules', 'footer', array('style' => 'none')); ?>
        <p class="pull-right">
            <a href="#top" id="back-top">
                <?php echo JText::_('TPL_PROTOSTAR_BACKTOTOP'); ?>
            </a>
        </p>
        <p>
            &copy; <?php echo date('Y'); ?> <?php echo $sitename; ?>
        </p>
    </div>
</div>

Alles anzeigen

Sollten die "Footer" nicht einheitlich sein bzw. dasselbe Verhalten haben?

Liebe Grüße, Gerry77

Harmageddon · 4. April 2020

Hi! Das ist eine sehr gute Beobachtung! Da wurden im Lauf der Zeit einige Sachen in der index.php geändert, die es nicht in die error.php geschafft haben. Ich habe hier mal einen Pull Request erstellt, um das zu ändern: https://github.com/joomla/joomla-cms/pull/28573

Gerry77 · 4. April 2020

Hey Harmageddon,

Boahh, echt top, bin begeistert! Vielen Dank!!

In Deinem PR für "error.php" ist diese Zeile nicht drin:

Code

<jdoc:include type="modules" name="footer" style="none" />

Stattdessen steht da noch:

PHP

<?php echo $this->getBuffer('modules', 'footer', array('style' => 'none')); ?>

Korrekt oder nur ein Versehen?

Liebe Grüße, Gerald

Harmageddon · 4. April 2020

Das ist korrekt, da die jdoc-Anweisungen nur in der normalen Template-Datei vom System ersetzt werden. Die Errorseite ist eine sehr reduzierte Version, in der man solche Sachen manuell nachladen muss. Das wird in der von dir angegebenen Zeile gemacht, genauso bei anderen Modulpositionen.

Gerry77 · 4. April 2020

Ah, OK, vielen Dank für die Info und vor allem für die wahnsinnig schnelle Reaktion!!

Liebe Grüße, Gerald

Gerry77 · 5. April 2020

Hey Harmageddon ,

Ich habe die Datei "error.php" nochmal mit den anderen Dateien verglichen, ich glaube, in Bezug auf HTML5 fehlt da vielleicht noch was:

Code

// Output as HTML5
$this->setHtml5(true);

Falls nicht, bitte um Entschuldigung.

Liebe Grüße, Gerald

Harmageddon · 5. April 2020

Hallo Gerald! Soweit ich das sehe, existiert diese Methode für die Klasse ErrorDocument nicht. Die einzige Stellen, an denen der damit gesetzte Wert überprüft wird, sind im HeadRenderer, also da, wo der Inhalt für <jdoc:include type="head" /> erzeugt wird. Das fällt für die error.php aber flach. Sollte daher in meinen Augen so passen.

Gerry77 · 5. April 2020

Vielen Dank, Harmageddon, für die Information! Ich werde die Änderungen im PR für unser Template, das auf Protostar basiert, nachziehen, sobald sie offiziell übernommen worden sind.

Liebe Grüße, Gerald

Harmageddon · 8. April 2020

Der Vollständigkeit halber: Die Änderung ist eingebaut und wird in Version 3.9.17 verfügbar sein. Danke fürs Melden!

Gerry77 · 8. April 2020

Zitat von Harmageddon

Die Änderung ist eingebaut und wird in Version 3.9.17 verfügbar sein.

Super, dann werde ich die Änderung auch an unserem Template vornehmen!

Zitat von Harmageddon

Danke fürs Melden!

Aber gerne, ich bedanke mich fürs Beheben!

Liebe Grüße, Gerald