Remove <a> tag hook for now, pending resolution of implementation issues as discussed...

diff --git a/includes/parser/Parser.php b/includes/parser/Parser.php
index 1805b15..67ec760 100644 (file)
--- a/includes/parser/Parser.php
+++ b/includes/parser/Parser.php
@@ -129,7 +129,7 @@ class Parser
                $this->mFunctionHooks = array();
                $this->mFunctionTagHooks = array();
                $this->mFunctionSynonyms = array( 0 => array(), 1 => array() );
-               $this->mDefaultStripList = $this->mStripList = array( 'nowiki', 'gallery', 'a' );
+               $this->mDefaultStripList = $this->mStripList = array( 'nowiki', 'gallery' );
                $this->mUrlProtocols = wfUrlProtocols();
                $this->mExtLinkBracketedRegex = '/\[(\b(' . wfUrlProtocols() . ')'.
                        '[^][<>"\\x00-\\x20\\x7F]+) *([^\]\\x0a\\x0d]*?)\]/S';
@@ -3308,9 +3308,6 @@ class Parser
                                case 'gallery':
                                        $output = $this->renderImageGallery( $content, $attributes );
                                        break;
-                               case 'a':
-                                       $output = $this->renderHyperlink( $content, $attributes, $frame );
-                                       break;
                                case 'math':
                                        if ( $this->mOptions->getUseTeX() ) {
                                                $output = $wgContLang->armourMath(
@@ -4357,35 +4354,6 @@ class Parser
                        '</pre>';
        }
 
-       /**
-       * Tag hook handler for 'a'. Renders a HTML &lt;a&gt; tag, allowing most attributes, filtering href against
-       * allowed protocols and spam blacklist.
-       **/
-       function renderHyperlink( $text, $params, $frame = false ) {
-               foreach ( $params as $name => $value ) {
-                       $params[ $name ] = $this->replaceVariables( $value, $frame );
-               }
-
-               $whitelist = Sanitizer::attributeWhitelist( 'a' );
-               $params = Sanitizer::validateAttributes( $params, $whitelist );
-
-               $content = $this->recursiveTagParse( trim( $text ), $frame );
-
-               if ( isset( $params[ 'href' ] ) ) {
-                       $href = $params[ 'href' ];
-                       $this->mOutput->addExternalLink( $href );
-                       unset( $params[ 'href' ] );
-               } else {
-                       # Non-link <a> tag
-                       return Xml::openElement( 'a', $params ) . $content . Xml::closeElement( 'a' );
-               }
-
-               $sk = $this->mOptions->getSkin();
-               $html = $sk->makeExternalLink( $href, $content, false, '', $params );
-
-               return $html;
-       }
-
        /**
         * Renders an image gallery from a text with one line per image.
         * text labels may be given by using |-style alternative text. E.g.