VectorGOLEM/includes
Timo Tijhof d56792addb Fix <h1> to be present even if title is "0"
Regression from 74b9803d9a, caused by a bug in LightNCandy which
caused {{foo}} to render "0", and {{#foo}} to pass as true with "0", but then
in {{#foo}}<b>{{foo}}</b>{{/foo} render as empty string producing "<b></b>".
In other words, the conditional is passing and the inner block is executed,
but the placeholder is mistakenly converting "0" => null => "" (empty string),
causing the <h1> to render but without any text in it.

Work around this bug by simply removing the conditional. Several other skins
already don't have this conditional and it's unclear why or in what
situation MediaWiki would send OutputPage to SkinTemplate without a title.

I think it would make sense in such rare case to still have a consistent
layout for extensions and gadgets to interact with and not omit the H1
element, but render it with the value that OutputPage gave it, even if it
is the empty string.

Bug: T219864
Change-Id: I6e04b512d2fe2e949ff5385cb38ceebe392fb255
2019-04-02 19:47:38 +01:00
..
templates Fix <h1> to be present even if title is "0" 2019-04-02 19:47:38 +01:00
Hooks.php Move class files to includes/ 2018-03-12 16:52:48 -07:00
ResourceLoaderLessModule.php Move class files to includes/ 2018-03-12 16:52:48 -07:00
SkinVector.php Add method scope visibility 2018-11-01 21:01:32 +01:00
VectorTemplate.php Fix <h1> to be present even if title is "0" 2019-04-02 19:47:38 +01:00