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 <o...@wikimedia.org>
Gerrit-Reviewer: AndyRussG <andrew.green...@gmail.com>
Gerrit-Reviewer: Awight <awi...@wikimedia.org>
Gerrit-Reviewer: Cdentinger <cdentin...@wikimedia.org>
Gerrit-Reviewer: Ejegg <eeggles...@wikimedia.org>
Gerrit-Reviewer: Krinkle <krinklem...@gmail.com>
Gerrit-Reviewer: Ssmith <ssm...@wikimedia.org>
Gerrit-Reviewer: XenoRyet <dkozlow...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to