Improve test coverage for OutputPage

[lhc/web/wiklou.git] / includes / OutputPage.php
diff --git a/includes/OutputPage.php b/includes/OutputPage.php

index 5965cbe..a1c5919 100644 (file)
--- a/includes/OutputPage.php
+++ b/includes/OutputPage.php
@@ -55,9 +55,8 @@ class OutputPage extends ContextSource {
         protected $mCanonicalUrl = false;
  
         /**
-        * @var string Should be private - has getter and setter. Contains
-        *   the HTML title */
-       public $mPagetitle = '';
+        * @var string The contents of <h1> */
+       private $mPageTitle = '';
  
         /**
          * @var string Contains all of the "<body>" content. Should be private we
@@ -404,18 +403,6 @@ class OutputPage extends ContextSource {
                 return $this->mLinktags;
         }
  
-       /**
-        * Add a new \<link\> with "rel" attribute set to "meta"
-        *
-        * @param array $linkarr Associative array mapping attribute names to their
-        *                 values, both keys and values will be escaped, and the
-        *                 "rel" attribute will be automatically added
-        */
-       function addMetadataLink( array $linkarr ) {
-               $linkarr['rel'] = $this->getMetadataAttribute();
-               $this->addLink( $linkarr );
-       }
-
         /**
          * Set the URL to be used for the <link rel=canonical>. This should be used
          * in preference to addLink(), to avoid duplicate link tags.
@@ -436,22 +423,6 @@ class OutputPage extends ContextSource {
                 return $this->mCanonicalUrl;
         }
  
-       /**
-        * Get the value of the "rel" attribute for metadata links
-        *
-        * @return string
-        */
-       public function getMetadataAttribute() {
-               # note: buggy CC software only reads first "meta" link
-               static $haveMeta = false;
-               if ( $haveMeta ) {
-                       return 'alternate meta';
-               } else {
-                       $haveMeta = true;
-                       return 'meta';
-               }
-       }
-
         /**
          * Add raw HTML to the list of scripts (including \<script\> tag, etc.)
          * Internal use only. Use OutputPage::addModules() or OutputPage::addJsConfigVars()
@@ -809,7 +780,7 @@ class OutputPage extends ContextSource {
                 # this breaks strtotime().
                 $clientHeader = preg_replace( '/;.*$/', '', $clientHeader );
  
-               Wikimedia\suppressWarnings(); // E_STRICT system time bitching
+               Wikimedia\suppressWarnings(); // E_STRICT system time warnings
                 $clientHeaderTime = strtotime( $clientHeader );
                 Wikimedia\restoreWarnings();
                 if ( !$clientHeaderTime ) {
@@ -992,7 +963,7 @@ class OutputPage extends ContextSource {
                 # change "<script>foo&bar</script>" to "&lt;script&gt;foo&amp;bar&lt;/script&gt;"
                 # but leave "<i>foobar</i>" alone
                 $nameWithTags = Sanitizer::normalizeCharReferences( Sanitizer::removeHTMLtags( $name ) );
-               $this->mPagetitle = $nameWithTags;
+               $this->mPageTitle = $nameWithTags;
  
                 # change "<i>foo&amp;bar</i>" to "foo&bar"
                 $this->setHTMLTitle(
@@ -1007,7 +978,7 @@ class OutputPage extends ContextSource {
          * @return string
          */
         public function getPageTitle() {
-               return $this->mPagetitle;
+               return $this->mPageTitle;
         }
  
         /**