Add 'X-Robots-Tag: noindex' header in action=render pages

Bug: 63891
Change-Id: I297a6269fcab999610bc01755462938c70eb64a1

diff --git a/RELEASE-NOTES-1.24 b/RELEASE-NOTES-1.24
index cd77b0e..0389b7c 100644 (file)
--- a/RELEASE-NOTES-1.24
+++ b/RELEASE-NOTES-1.24
@@ -122,6 +122,7 @@ changes to languages because of Bugzilla reports.
   $useHeadElement = false; is no longer supported and will not cause old keys
   like "headlinks", "skinnameclass", etc. to be defined.
 * The deprecated 'SpecialVersionExtensionTypes' hook was removed.
+* (bug 63891) Add 'X-Robots-Tag: noindex' header in action=render pages.
 
 ==== Renamed classes ====
 * CLDRPluralRuleConverter_Expression to CLDRPluralRuleConverterExpression

diff --git a/includes/Article.php b/includes/Article.php
index bcdd0cf..4cf7d3e 100644 (file)
--- a/includes/Article.php
+++ b/includes/Article.php
@@ -1494,6 +1494,7 @@ class Article implements Page {
         * Handle action=render
         */
        public function render() {
+               $this->getContext()->getRequest()->response()->header( 'X-Robots-Tag: noindex' );
                $this->getContext()->getOutput()->setArticleBodyOnly( true );
                $this->getContext()->getOutput()->enableSectionEditLinks( false );
                $this->view();

diff --git a/includes/actions/RenderAction.php b/includes/actions/RenderAction.php
index c9b3f8f..16e407f 100644 (file)
--- a/includes/actions/RenderAction.php
+++ b/includes/actions/RenderAction.php
@@ -41,7 +41,6 @@ class RenderAction extends FormlessAction {
        }
 
        public function show() {
-
                $this->page->render();
        }
 }