Jdlrobson has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/383179 )
Change subject: Check config variables are set before applying ...................................................................... Check config variables are set before applying Bug: T169732 Change-Id: I979210617b594da60a883371c6d7073341add81f --- M wmf-config/CommonSettings.php 1 file changed, 11 insertions(+), 5 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/operations/mediawiki-config refs/changes/79/383179/1 diff --git a/wmf-config/CommonSettings.php b/wmf-config/CommonSettings.php index 0dc18ce..881bb64 100644 --- a/wmf-config/CommonSettings.php +++ b/wmf-config/CommonSettings.php @@ -652,11 +652,17 @@ // The Print logo for Vector should use the same wordmark as Minerva // This avoids duplicate config entries (T169732) -$wgVectorPrintLogo = [ - 'width' => $wgMinervaCustomLogos['copyright-width'], - 'height' => $wgMinervaCustomLogos['copyright-height'], - 'url' => $wgMinervaCustomLogos['copyright'], -]; +if ( + isset( $wgMinervaCustomLogos['copyright'] ) && + isset( $wgMinervaCustomLogos['copyright-height'] ) && + isset( $wgMinervaCustomLogos['copyright-width'] ) +) { + $wgVectorPrintLogo = [ + 'width' => $wgMinervaCustomLogos['copyright-width'], + 'height' => $wgMinervaCustomLogos['copyright-height'], + 'url' => $wgMinervaCustomLogos['copyright'], + ]; +} // Grants and rights // Note these have to be visible on all wikis, not just the ones the -- To view, visit https://gerrit.wikimedia.org/r/383179 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I979210617b594da60a883371c6d7073341add81f Gerrit-PatchSet: 1 Gerrit-Project: operations/mediawiki-config Gerrit-Branch: master Gerrit-Owner: Jdlrobson <jrob...@wikimedia.org> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits