From 7d5be9e428b58631ef82fcd03fe967877daabff9 Mon Sep 17 00:00:00 2001 From: Brion Vibber Date: Sun, 19 Jul 2009 17:07:03 +0000 Subject: [PATCH] * (bug 19761) Removed autogenerated tag with link data. Keyword set was not useful, and is ignored by modern search engines anway. --- RELEASE-NOTES | 2 ++ includes/OutputPage.php | 37 ------------------------------------- 2 files changed, 2 insertions(+), 37 deletions(-) diff --git a/RELEASE-NOTES b/RELEASE-NOTES index bf625edf0e..60f6bfa5ae 100644 --- a/RELEASE-NOTES +++ b/RELEASE-NOTES @@ -290,6 +290,8 @@ this. Was used when mwEmbed was going to be an extension. * (bug 19814) interwiki links from file links ([[File:Foo.jpg|link=de:Test]]) are no longer recorded in the pagelinks table * (bug 19784) date option "ISO 8601" produced illegal id +* (bug 19761) Removed autogenerated tag with link data. + Keyword set was not useful, and is ignored by modern search engines anway. == API changes in 1.16 == diff --git a/includes/OutputPage.php b/includes/OutputPage.php index 614b812912..8d80657886 100644 --- a/includes/OutputPage.php +++ b/includes/OutputPage.php @@ -749,7 +749,6 @@ class OutputPage { $this->setIndexPolicy( $parserOutput->getIndexPolicy() ); } - $this->addKeywords( $parserOutput ); $this->mParseWarnings = $parserOutput->getWarnings(); if ( $parserOutput->getCacheTime() == -1 ) { $this->enableClientCache( false ); @@ -1681,42 +1680,6 @@ class OutputPage { $this->addReturnTo( $titleObj, $returntoquery ); } - /** - * This function takes the title (first item of mGoodLinks), categories, - * existing and broken links for the page - * and uses the first 10 of them for META keywords - * - * @param ParserOutput &$parserOutput - */ - private function addKeywords( &$parserOutput ) { - global $wgContLang; - // Get an array of keywords if there are more than one - // variant of the site language - $text = $wgContLang->autoConvertToAllVariants( $this->getTitle()->getPrefixedText()); - // array_values: We needn't to merge variant's code name - // into $this->mKeywords; - // array_unique: We should insert a keyword just for once - if( is_array( $text )) - $text = array_unique( array_values( $text )); - $this->addKeyword( $text ); - $count = 1; - $links2d =& $parserOutput->getLinks(); - if ( !is_array( $links2d ) ) { - return; - } - foreach ( $links2d as $dbkeys ) { - foreach( $dbkeys as $dbkey => $unused ) { - $dbkey = $wgContLang->autoConvertToAllVariants( $dbkey ); - if( is_array( $dbkey )) - $dbkey = array_unique( array_values( $dbkey )); - $this->addKeyword( $dbkey ); - if ( ++$count > 10 ) { - break 2; - } - } - } - } - /** * @return string The doctype, opening , and head element. */ -- 2.20.1