jenkins-bot has submitted this change and it was merged.
Change subject: Namespace the dns-prefetch head element item
......................................................................
Namespace the dns-prefetch head element item
Fix-up for Iae39a92912 (from 2012). The $name element for
OutputPage::addHeadItem() is meant to allow extensions to override head
elements of which there can be only one instance -- e.g., apple-touch-icon. But
there can be multiple <link rel=dns-prefetch> tags, and we don't want
extensions to be clobbering each other's. So use a name that is scoped to
CentralNotice, instead of the generic "dns-prefetch".
Change-Id: I2fb7f3b9f3763f2395cd3120eda5d7e2d2aaebe0
---
M CentralNotice.hooks.php
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Krinkle: Looks good to me, approved
jenkins-bot: Verified
diff --git a/CentralNotice.hooks.php b/CentralNotice.hooks.php
index 21c0efc..a8a6e33 100644
--- a/CentralNotice.hooks.php
+++ b/CentralNotice.hooks.php
@@ -217,7 +217,7 @@
// Insert DNS prefetch for banner loading
if ( $wgCentralHost && $wgCentralHost !== $wgServer ) {
- $out->addHeadItem( 'dns-prefetch', '<link rel="dns-prefetch"
href="' . htmlspecialchars( $wgCentralHost ) . '" />' );
+ $out->addHeadItem( 'cn-dns-prefetch', '<link rel="dns-prefetch"
href="' . htmlspecialchars( $wgCentralHost ) . '" />' );
}
// Insert the banner controller
// TODO Change this to startUp once it's determined that a rollback is
not
--
To view, visit https://gerrit.wikimedia.org/r/293040
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I2fb7f3b9f3763f2395cd3120eda5d7e2d2aaebe0
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/CentralNotice
Gerrit-Branch: master
Gerrit-Owner: Ori.livneh <[email protected]>
Gerrit-Reviewer: AndyRussG <[email protected]>
Gerrit-Reviewer: Awight <[email protected]>
Gerrit-Reviewer: Cdentinger <[email protected]>
Gerrit-Reviewer: Ejegg <[email protected]>
Gerrit-Reviewer: Krinkle <[email protected]>
Gerrit-Reviewer: Ssmith <[email protected]>
Gerrit-Reviewer: XenoRyet <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
