Remove $wgColorErrors. Apparently unused at least since r17991 at 2006-11-29
[lhc/web/wiklou.git] / includes / Linker.php
index 7e78f1e..7531b88 100644 (file)
@@ -21,7 +21,7 @@ class Linker {
         * Get the appropriate HTML attributes to add to the "a" element of an ex-
         * ternal link, as created by [wikisyntax].
         *
-        * @param string $class  The contents of the class attribute; if an empty
+        * @param $class String: the contents of the class attribute; if an empty
         *   string is passed, which is the default value, defaults to 'external'.
         */
        function getExternalLinkAttributes( $class = 'external' ) {
@@ -32,10 +32,10 @@ class Linker {
         * Get the appropriate HTML attributes to add to the "a" element of an in-
         * terwiki link.
         *
-        * @param string $title  The title text for the link, URL-encoded (???) but
+        * @param $title String: the title text for the link, URL-encoded (???) but
         *   not HTML-escaped
-        * @param string $unused Unused
-        * @param string $class  The contents of the class attribute; if an empty
+        * @param $unused String: unused
+        * @param $class String: the contents of the class attribute; if an empty
         *   string is passed, which is the default value, defaults to 'external'.
         */
        function getInterwikiLinkAttributes( $title, $unused = null, $class = 'external' ) {
@@ -54,10 +54,10 @@ class Linker {
         * Get the appropriate HTML attributes to add to the "a" element of an in-
         * ternal link.
         *
-        * @param string $title  The title text for the link, URL-encoded (???) but
+        * @param $title String: the title text for the link, URL-encoded (???) but
         *   not HTML-escaped
-        * @param string $unused Unused
-        * @param string $class  The contents of the class attribute, default none
+        * @param $unused String: unused
+        * @param $class String: the contents of the class attribute, default none
         */
        function getInternalLinkAttributes( $title, $unused = null, $class='' ) {
                $title = urldecode( $title );
@@ -69,10 +69,10 @@ class Linker {
         * Get the appropriate HTML attributes to add to the "a" element of an in-
         * ternal link, given the Title object for the page we want to link to.
         *
-        * @param Title  $nt     The Title object
-        * @param string $unused Unused
-        * @param string $class  The contents of the class attribute, default none
-        * @param mixed  $title  Optional (unescaped) string to use in the title
+        * @param $nt The Title object
+        * @param $unused String: unused
+        * @param $class String: the contents of the class attribute, default none
+        * @param $title Mixed: optional (unescaped) string to use in the title
         *   attribute; if false, default to the name of the page we're linking to
         */
        function getInternalLinkAttributesObj( $nt, $unused = null, $class = '', $title = false ) {
@@ -101,9 +101,9 @@ class Linker {
        /**
         * Return the CSS colour of a known link
         *
-        * @param Title $t
-        * @param integer $threshold user defined threshold
-        * @return string CSS class
+        * @param $t Title object
+        * @param $threshold Integer: user defined threshold
+        * @return String: CSS class
         */
        function getLinkColour( $t, $threshold ) {
                $colour = '';
@@ -112,7 +112,7 @@ class Linker {
                        $colour = 'mw-redirect';
                } elseif ( $threshold > 0 &&
                           $t->exists() && $t->getLength() < $threshold &&
-                          MWNamespace::isContent( $t->getNamespace() ) ) {
+                          $t->isContentPage() ) {
                        # Page is a stub
                        $colour = 'stub';
                }
@@ -266,14 +266,10 @@ class Linker {
                                $classes[] = 'extiw';
                        }
 
-                       # Note that redirects never count as stubs here.
-                       if ( !in_array( 'broken', $options ) && $target->isRedirect() ) {
-                               $classes[] = 'mw-redirect';
-                       } elseif( $target->isContentPage() ) {
-                               # Check for stub.
-                               $threshold = $wgUser->getOption( 'stubthreshold' );
-                               if( $threshold > 0 and $target->exists() and $target->getLength() < $threshold ) {
-                                       $classes[] = 'stub';
+                       if ( !in_array( 'broken', $options ) ) { # Avoid useless calls to LinkCache (see r50387)
+                               $colour = $this->getLinkColour( $target, $wgUser->getStubThreshold() );
+                               if ( $colour !== '' ) {
+                                       $classes[] = $colour; # mw-redirect or stub
                                }
                        }
                        if( $classes != array() ) {
@@ -335,10 +331,13 @@ class Linker {
         * @param $trail String
         * @param $prefix String
         * @return string HTML of link
+        * @deprecated
         */
        function makeSizeLinkObj( $size, $nt, $text = '', $query = '', $trail = '', $prefix = '' ) {
                global $wgUser;
-               $threshold = intval( $wgUser->getOption( 'stubthreshold' ) );
+               wfDeprecated( __METHOD__ );
+               
+               $threshold = $wgUser->getStubThreshold();
                $colour = ( $size < $threshold ) ? 'stub' : '';
                // FIXME: replace deprecated makeColouredLinkObj by link()
                return $this->makeColouredLinkObj( $nt, $colour, $text, $query, $trail, $prefix );
@@ -407,10 +406,9 @@ class Linker {
         * Given parameters derived from [[Image:Foo|options...]], generate the
         * HTML that that syntax inserts in the page.
         *
-        * @param Title $title Title object
-        * @param File $file File object, or false if it doesn't exist
-        *
-        * @param array $frameParams Associative array of parameters external to the media handler.
+        * @param $title Title object
+        * @param $file File object, or false if it doesn't exist
+        * @param $frameParams Array: associative array of parameters external to the media handler.
         *     Boolean parameters are indicated by presence or absence, the value is arbitrary and
         *     will often be false.
         *          thumbnail       If present, downscale and frame
@@ -429,13 +427,13 @@ class Linker {
         *          link-title      Title object to link to
         *          no-link         Boolean, suppress description link
         *
-        * @param array $handlerParams Associative array of media handler parameters, to be passed
+        * @param $handlerParams Array: associative array of media handler parameters, to be passed
         *       to transform(). Typical keys are "width" and "page".
-        * @param string $time, timestamp of the file, set as false for current
-        * @param string $query, query params for desc url
-        * @return string HTML for an image, with links, wrappers, etc.
+        * @param $time String: timestamp of the file, set as false for current
+        * @param $query String: query params for desc url
+        * @return String: HTML for an image, with links, wrappers, etc.
         */
-       function makeImageLink2( Title $title, $file, $frameParams = array(), $handlerParams = array(), $time = false, $query = "" ) {
+       function makeImageLink2( Title $title, $file, $frameParams = array(), $handlerParams = array(), $time = false, $query = "", $widthOption = null ) {
                $res = null;
                if( !wfRunHooks( 'ImageBeforeProduceHTML', array( &$this, &$title,
                &$file, &$frameParams, &$handlerParams, &$time, &$res ) ) ) {
@@ -468,11 +466,9 @@ class Linker {
                if ( $file && !isset( $hp['width'] ) ) {
                        $hp['width'] = $file->getWidth( $page );
 
-                       if( isset( $fp['thumbnail'] ) || isset( $fp['framed'] ) || isset( $fp['frameless'] ) ) {
-                               $wopt = $wgUser->getOption( 'thumbsize' );
-
-                               if( !isset( $wgThumbLimits[$wopt] ) ) {
-                                        $wopt = User::getDefaultOption( 'thumbsize' );
+                       if( isset( $fp['thumbnail'] ) || isset( $fp['framed'] ) || isset( $fp['frameless'] ) || !$hp['width'] ) {
+                               if( !isset( $widthOption ) || !isset( $wgThumbLimits[$widthOption] ) ) {
+                                        $widthOption = User::getDefaultOption( 'thumbsize' );
                                }
 
                                // Reduce width for upright images when parameter 'upright' is used
@@ -482,8 +478,8 @@ class Linker {
                                // Use width which is smaller: real image width or user preference width
                                // For caching health: If width scaled down due to upright parameter, round to full __0 pixel to avoid the creation of a lot of odd thumbs
                                $prefWidth = isset( $fp['upright'] ) ?
-                                       round( $wgThumbLimits[$wopt] * $fp['upright'], -1 ) :
-                                       $wgThumbLimits[$wopt];
+                                       round( $wgThumbLimits[$widthOption] * $fp['upright'], -1 ) :
+                                       $wgThumbLimits[$widthOption];
                                if ( $hp['width'] <= 0 || $prefWidth < $hp['width'] ) {
                                        if( !isset( $hp['height'] ) ) {
                                                $hp['width'] = $prefWidth;
@@ -530,16 +526,7 @@ class Linker {
                                'title' => $fp['title'],
                                'valign' => isset( $fp['valign'] ) ? $fp['valign'] : false ,
                                'img-class' => isset( $fp['border'] ) ? 'thumbborder' : false );
-                       if ( !empty( $fp['link-url'] ) ) {
-                               $params['custom-url-link'] = $fp['link-url'];
-                       } elseif ( !empty( $fp['link-title'] ) ) {
-                               $params['custom-title-link'] = $fp['link-title'];
-                       } elseif ( !empty( $fp['no-link'] ) ) {
-                               // No link
-                       } else {
-                               $params['desc-link'] = true;
-                               $params['desc-query'] = $query;
-                       }
+                       $params = $this->getImageLinkMTOParams( $fp, $query ) + $params;
 
                        $s = $thumb->toHtml( $params );
                }
@@ -549,10 +536,37 @@ class Linker {
                return str_replace("\n", ' ',$prefix.$s.$postfix);
        }
 
+       /**
+        * Get the link parameters for MediaTransformOutput::toHtml() from given 
+        * frame parameters supplied by the Parser.
+        * @param $frameParams The frame parameters
+        * @param $query An optional query string to add to description page links
+        */
+       function getImageLinkMTOParams( $frameParams, $query = '' ) {
+               $mtoParams = array();
+               if ( isset( $frameParams['link-url'] ) && $frameParams['link-url'] !== '' ) {
+                       $mtoParams['custom-url-link'] = $frameParams['link-url'];
+               } elseif ( isset( $frameParams['link-title'] ) && $frameParams['link-title'] !== '' ) {
+                       $mtoParams['custom-title-link'] = $frameParams['link-title'];
+               } elseif ( !empty( $frameParams['no-link'] ) ) {
+                       // No link
+               } else {
+                       $mtoParams['desc-link'] = true;
+                       $mtoParams['desc-query'] = $query;
+               }
+               return $mtoParams;
+       }
+
        /**
         * Make HTML for a thumbnail including image, border and caption
-        * @param Title $title
-        * @param File $file File object or false if it doesn't exist
+        * @param $title Title object
+        * @param $file File object or false if it doesn't exist
+        * @param $label String
+        * @param $alt String
+        * @param $align String
+        * @param $params Array
+        * @param $framed Boolean
+        * @param $manualthumb String
         */
        function makeThumbLinkObj( Title $title, $file, $label = '', $alt, $align = 'right', $params = array(), $framed=false , $manualthumb = "" ) {
                $frameParams = array(
@@ -566,7 +580,7 @@ class Linker {
        }
 
        function makeThumbLink2( Title $title, $file, $frameParams = array(), $handlerParams = array(), $time = false, $query = "" ) {
-               global $wgStylePath, $wgContLang;
+               global $wgStylePath;
                $exists = $file && $file->exists();
 
                # Shortcuts
@@ -594,14 +608,14 @@ class Linker {
                                if( $manual_title ) {
                                        $manual_img = wfFindFile( $manual_title );
                                        if ( $manual_img ) {
-                                               $thumb = $manual_img->getUnscaledThumb();
+                                               $thumb = $manual_img->getUnscaledThumb( $hp );
                                        } else {
                                                $exists = false;
                                        }
                                }
                        } elseif ( isset( $fp['framed'] ) ) {
                                // Use image dimensions, don't scale
-                               $thumb = $file->getUnscaledThumb( $page );
+                               $thumb = $file->getUnscaledThumb( $hp );
                        } else {
                                # Do not present an image bigger than the source, for bitmap-style images
                                # This is a hack to maintain compatibility with arbitrary pre-1.10 behaviour
@@ -627,52 +641,51 @@ class Linker {
                        $url = wfAppendQuery( $url, 'page=' . urlencode( $page ) );
                }
 
-               $more = htmlspecialchars( wfMsg( 'thumbnail-more' ) );
-
                $s = "<div class=\"thumb t{$fp['align']}\"><div class=\"thumbinner\" style=\"width:{$outerWidth}px;\">";
                if( !$exists ) {
                        $s .= $this->makeBrokenImageLinkObj( $title, $fp['title'], '', '', '', $time==true );
-                       $zoomicon = '';
+                       $zoomIcon = '';
                } elseif ( !$thumb ) {
                        $s .= htmlspecialchars( wfMsg( 'thumbnail_error', '' ) );
-                       $zoomicon = '';
+                       $zoomIcon = '';
                } else {
-                       $s .= $thumb->toHtml( array(
+                       $params = array(
                                'alt' => $fp['alt'],
                                'title' => $fp['title'],
-                               'img-class' => 'thumbimage',
-                               'desc-link' => true,
-                               'desc-query' => $query ) );
+                               'img-class' => 'thumbimage' );
+                       $params = $this->getImageLinkMTOParams( $fp, $query ) + $params;
+                       $s .= $thumb->toHtml( $params );
                        if ( isset( $fp['framed'] ) ) {
-                               $zoomicon="";
+                               $zoomIcon = "";
                        } else {
-                               $zoomicon =  '<div class="magnify">'.
-                                       '<a href="'.$url.'" class="internal" title="'.$more.'">'.
-                                       '<img src="'.$wgStylePath.'/common/images/magnify-clip.png" ' .
-                                       'width="15" height="11" alt="" /></a></div>';
+                               $zoomIcon =  '<div class="magnify">'.
+                                       '<a href="' . htmlspecialchars( $url ) . '" class="internal" ' .
+                                               'title="' . htmlspecialchars( wfMsg( 'thumbnail-more' ) ) . '">'.
+                                       '<img src="' . htmlspecialchars( $wgStylePath ) . '/common/images/magnify-clip.png" ' .
+                                               'width="15" height="11" alt="" /></a></div>';
                        }
                }
-               $s .= '  <div class="thumbcaption">'.$zoomicon.$fp['caption']."</div></div></div>";
+               $s .= '  <div class="thumbcaption">' . $zoomIcon . $fp['caption'] . "</div></div></div>";
                return str_replace("\n", ' ', $s);
        }
 
        /**
         * Make a "broken" link to an image
         *
-        * @param Title $title Image title
-        * @param string $text Link label
-        * @param string $query Query string
-        * @param string $trail Link trail
-        * @param string $prefix Link prefix
-        * @param bool $time, a file of a certain timestamp was requested
-        * @return string
+        * @param $title Title object
+        * @param $text String: link label
+        * @param $query String: query string
+        * @param $trail String: link trail
+        * @param $prefix String: link prefix
+        * @param $time Boolean: a file of a certain timestamp was requested
+        * @return String
         */
        public function makeBrokenImageLinkObj( $title, $text = '', $query = '', $trail = '', $prefix = '', $time = false ) {
-               global $wgEnableUploads, $wgUploadNavigationUrl;
+               global $wgEnableUploads, $wgUploadMissingFileUrl;
                if( $title instanceof Title ) {
                        wfProfileIn( __METHOD__ );
                        $currentExists = $time ? ( wfFindFile( $title ) != false ) : false;
-                       if( ( $wgUploadNavigationUrl || $wgEnableUploads ) && !$currentExists ) {
+                       if( ( $wgUploadMissingFileUrl || $wgEnableUploads ) && !$currentExists ) {
                                if( $text == '' )
                                        $text = htmlspecialchars( $title->getPrefixedText() );
 
@@ -688,11 +701,9 @@ class Linker {
                                list( $inside, $trail ) = self::splitTrail( $trail );
 
                                wfProfileOut( __METHOD__ );
-                               return Html::element( 'a', array(
-                                       'href' => $href,
-                                       'class' => 'new',
-                                       'title' => $title->getPrefixedText()
-                               ), $prefix . $text . $inside ) . $trail;
+                               return '<a href="' . htmlspecialchars( $href ) . '" class="new" title="' . 
+                                                               htmlspecialchars( $title->getPrefixedText(), ENT_QUOTES ) . '">' .
+                                                               htmlspecialchars( $prefix . $text . $inside, ENT_NOQUOTES ) . '</a>' . $trail;
                        } else {
                                wfProfileOut( __METHOD__ );
                                return $this->makeKnownLinkObj( $title, $text, $query, $trail, $prefix );
@@ -705,18 +716,18 @@ class Linker {
        /**
         * Get the URL to upload a certain file
         * 
-        * @param $destFile Title Title of the file to upload
-        * @param $query string Urlencoded query string to prepend
-        * @return string Urlencoded URL
+        * @param $destFile Title object of the file to upload
+        * @param $query String: urlencoded query string to prepend
+        * @return String: urlencoded URL
         */
        protected function getUploadUrl( $destFile, $query = '' ) {
-               global $wgUploadNavigationUrl;
+               global $wgUploadMissingFileUrl;
                $q = 'wpDestFile=' . $destFile->getPartialUrl();
                if( $query != '' )
                        $q .= '&' . $query;
 
-               if( $wgUploadNavigationUrl ) {
-                       return wfAppendQuery( $wgUploadNavigationUrl, $q );
+               if( $wgUploadMissingFileUrl ) {
+                       return wfAppendQuery( $wgUploadMissingFileUrl, $q );
                } else {
                        $upload = SpecialPage::getTitleFor( 'Upload' );
                        return $upload->getLocalUrl( $q );
@@ -729,12 +740,11 @@ class Linker {
         * @param $title Title object.
         * @param $text String: pre-sanitized HTML
         * @param $time string: time image was created
-        * @return string HTML
+        * @return String: HTML
         *
-        * @public
         * @todo Handle invalid or missing images better.
         */
-       function makeMediaLinkObj( $title, $text = '', $time = false ) {
+       public function makeMediaLinkObj( $title, $text = '', $time = false ) {
                if( is_null( $title ) ) {
                        ### HOTFIX. Instead of breaking, return empty string.
                        return $text;
@@ -747,7 +757,7 @@ class Linker {
                                $url = $this->getUploadUrl( $title );
                                $class = 'new';
                        }
-                       $alt = htmlspecialchars( $title->getText() );
+                       $alt = htmlspecialchars( $title->getText(),  ENT_QUOTES );
                        if( $text == '' ) {
                                $text = $alt;
                        }
@@ -772,11 +782,11 @@ class Linker {
 
        /**
         * Make an external link
-        * @param String $url URL to link to
-        * @param String $text text of link
-        * @param boolean $escape Do we escape the link text?
-        * @param String $linktype Type of external link. Gets added to the classes
-        * @param array $attribs Array of extra attributes to <a>
+        * @param $url String: URL to link to
+        * @param $text String: text of link
+        * @param $escape Boolean: do we escape the link text?
+        * @param $linktype String: type of external link. Gets added to the classes
+        * @param $attribs Array of extra attributes to <a>
         *
         * @todo FIXME: This is a really crappy implementation. $linktype and
         * 'external' are mashed into the class attrib for the link (which is made
@@ -811,7 +821,7 @@ class Linker {
         * Make user link (or user contributions for unregistered users)
         * @param $userId   Integer: user id in database.
         * @param $userText String: user name in database
-        * @return string HTML fragment
+        * @return String: HTML fragment
         * @private
         */
        function userLink( $userId, $userText ) {
@@ -826,12 +836,13 @@ class Linker {
        /**
         * Generate standard user tool links (talk, contributions, block link, etc.)
         *
-        * @param int $userId User identifier
-        * @param string $userText User name or IP address
-        * @param bool $redContribsWhenNoEdits Should the contributions link be red if the user has no edits?
-        * @param int $flags Customisation flags (e.g. self::TOOL_LINKS_NOBLOCK)
-        * @param int $edits, user edit count (optional, for performance)
-        * @return string
+        * @param $userId Integer: user identifier
+        * @param $userText String: user name or IP address
+        * @param $redContribsWhenNoEdits Boolean: should the contributions link be
+        *        red if the user has no edits?
+        * @param $flags Integer: customisation flags (e.g. self::TOOL_LINKS_NOBLOCK)
+        * @param $edits Integer: user edit count (optional, for performance)
+        * @return String: HTML fragment
         */
        public function userToolLinks( $userId, $userText, $redContribsWhenNoEdits = false, $flags = 0, $edits=null ) {
                global $wgUser, $wgDisableAnonTalk, $wgSysopUserBans, $wgLang;
@@ -868,9 +879,9 @@ class Linker {
 
        /**
         * Alias for userToolLinks( $userId, $userText, true );
-        * @param int $userId User identifier
-        * @param string $userText User name or IP address
-        * @param int $edits, user edit count (optional, for performance)
+        * @param $userId Integer: user identifier
+        * @param $userText String: user name or IP address
+        * @param $edits Integer: user edit count (optional, for performance)
         */
        public function userToolLinksRedContribs( $userId, $userText, $edits=null ) {
                return $this->userToolLinks( $userId, $userText, true, 0, $edits );
@@ -880,7 +891,7 @@ class Linker {
        /**
         * @param $userId Integer: user id in database.
         * @param $userText String: user name in database.
-        * @return string HTML fragment with user talk link
+        * @return String: HTML fragment with user talk link
         * @private
         */
        function userTalkLink( $userId, $userText ) {
@@ -892,7 +903,7 @@ class Linker {
        /**
         * @param $userId Integer: userid
         * @param $userText String: user name in database.
-        * @return string HTML fragment with block link
+        * @return String: HTML fragment with block link
         * @private
         */
        function blockLink( $userId, $userText ) {
@@ -904,8 +915,8 @@ class Linker {
        /**
         * Generate a user link if the current user is allowed to view it
         * @param $rev Revision object.
-        * @param $isPublic, bool, show only if all users can see it
-        * @return string HTML
+        * @param $isPublic Boolean: show only if all users can see it
+        * @return String: HTML fragment
         */
        function revUserLink( $rev, $isPublic = false ) {
                if( $rev->isDeleted( Revision::DELETED_USER ) && $isPublic ) {
@@ -925,7 +936,7 @@ class Linker {
        /**
         * Generate a user tool link cluster if the current user is allowed to view it
         * @param $rev Revision object.
-        * @param $isPublic, bool, show only if all users can see it
+        * @param $isPublic Boolean: show only if all users can see it
         * @return string HTML
         */
        function revUserTools( $rev, $isPublic = false ) {
@@ -957,9 +968,9 @@ class Linker {
         * Since you can't set a default parameter for a reference, I've turned it
         * temporarily to a value pass. Should be adjusted further. --brion
         *
-        * @param string $comment
-        * @param mixed $title Title object (to generate link to the section in autocomment) or null
-        * @param bool $local Whether section links should refer to local page
+        * @param $comment String
+        * @param $title Mixed: Title object (to generate link to the section in autocomment) or null
+        * @param $local Boolean: whether section links should refer to local page
         */
        function formatComment($comment, $title = null, $local = false) {
                wfProfileIn( __METHOD__ );
@@ -970,7 +981,7 @@ class Linker {
                $comment = Sanitizer::escapeHtmlAllowEntities( $comment );
 
                # Render autocomments and make links:
-               $comment = $this->formatAutoComments( $comment, $title, $local );
+               $comment = $this->formatAutocomments( $comment, $title, $local );
                $comment = $this->formatLinksInComment( $comment, $title, $local );
 
                wfProfileOut( __METHOD__ );
@@ -984,11 +995,10 @@ class Linker {
         * add a separator where needed and format the comment itself with CSS
         * Called by Linker::formatComment.
         *
-        * @param string $comment Comment text
-        * @param object $title An optional title object used to links to sections
-        * @return string $comment formatted comment
-        *
-        * @todo Document the $local parameter.
+        * @param $comment String: comment text
+        * @param $title An optional title object used to links to sections
+        * @param $local Boolean: whether section links should refer to local page
+        * @return String: formatted comment
         */
        private function formatAutocomments( $comment, $title = null, $local = false ) {
                // Bah!
@@ -1007,21 +1017,20 @@ class Linker {
                $title = $this->autocommentTitle;
                $local = $this->autocommentLocal;
 
-               $pre=$match[1];
-               $auto=$match[2];
-               $post=$match[3];
-               $link='';
-               if( $title ) {
+               $pre = $match[1];
+               $auto = $match[2];
+               $post = $match[3];
+               $link = '';
+               if ( $title ) {
                        $section = $auto;
 
-                       # Generate a valid anchor name from the section title.
-                       # Hackish, but should generally work - we strip wiki
-                       # syntax, including the magic [[: that is used to
-                       # "link rather than show" in case of images and
-                       # interlanguage links.
+                       # Remove links that a user may have manually put in the autosummary
+                       # This could be improved by copying as much of Parser::stripSectionName as desired.
                        $section = str_replace( '[[:', '', $section );
                        $section = str_replace( '[[', '', $section );
                        $section = str_replace( ']]', '', $section );
+
+                       $section = Sanitizer::normalizeSectionNameWhitespace( $section ); # bug 22784
                        if ( $local ) {
                                $sectionTitle = Title::newFromText( '#' . $section );
                        } else {
@@ -1055,8 +1064,10 @@ class Linker {
         * is ignored
         *
         * @todo Fixme: doesn't handle sub-links as in image thumb texts like the main parser
-        * @param string $comment Text to format links in
-        * @return string
+        * @param $comment String: text to format links in
+        * @param $title An optional title object used to links to sections
+        * @param $local Boolean: whether section links should refer to local page
+        * @return String
         */
        public function formatLinksInComment( $comment, $title = null, $local = false ) {
                $this->commentContextTitle = $title;
@@ -1114,7 +1125,9 @@ class Linker {
 
                        $target = Title::newFromText( $linkTarget );
                        if( $target ) {
-                               if( $target->getText() == '' && !$this->commentLocal && $this->commentContextTitle ) {
+                               if( $target->getText() == '' && $target->getInterwiki() === ''
+                                       && !$this->commentLocal && $this->commentContextTitle )
+                               {
                                        $newTarget = clone( $this->commentContextTitle );
                                        $newTarget->setFragment( '#' . $target->getFragment() );
                                        $target = $newTarget;
@@ -1209,9 +1222,9 @@ class Linker {
         * Wrap a comment in standard punctuation and formatting if
         * it's non-empty, otherwise return empty string.
         *
-        * @param string $comment
-        * @param mixed $title Title object (to generate link to section in autocomment) or null
-        * @param bool $local Whether section links should refer to local page
+        * @param $comment String
+        * @param $title Mixed: Title object (to generate link to section in autocomment) or null
+        * @param $local Boolean: whether section links should refer to local page
         *
         * @return string
         */
@@ -1231,10 +1244,10 @@ class Linker {
         * Wrap and format the given revision's comment block, if the current
         * user is allowed to view it.
         *
-        * @param Revision $rev
-        * @param bool $local Whether section links should refer to local page
-        * @param $isPublic, show only if all users can see it
-        * @return string HTML
+        * @param $rev Revision object
+        * @param $local Boolean: whether section links should refer to local page
+        * @param $isPublic Boolean: show only if all users can see it
+        * @return String: HTML fragment
         */
        function revComment( Revision $rev, $local = false, $isPublic = false ) {
                if( $rev->getRawComment() == "" ) return "";
@@ -1302,8 +1315,9 @@ class Linker {
 
        /**
         * Wraps the TOC in a table and provides the hide/collapse javascript.
-        * @param string $toc html of the Table Of Contents
-        * @return string Full html of the TOC
+        *
+        * @param $toc String: html of the Table Of Contents
+        * @return String: full html of the TOC
         */
        function tocList($toc) {
                $title = wfMsgHtml('toc') ;
@@ -1326,8 +1340,9 @@ class Linker {
        /**
         * Generate a table of contents from a section tree
         * Currently unused.
+        *
         * @param $tree Return value of ParserOutput::getSections()
-        * @return string HTML
+        * @return String: HTML fragment
         */
        public function generateTOC( $tree ) {
                $toc = '';
@@ -1405,16 +1420,17 @@ class Linker {
        /**
         * Create a headline for content
         *
-        * @param int    $level   The level of the headline (1-6)
-        * @param string $attribs Any attributes for the headline, starting with a space and ending with '>'
-        *                        This *must* be at least '>' for no attribs
-        * @param string $anchor  The anchor to give the headline (the bit after the #)
-        * @param string $text    The text of the header
-        * @param string $link    HTML to add for the section edit link
-        * @param mixed  $legacyAnchor A second, optional anchor to give for
+        * @param $level Integer: the level of the headline (1-6)
+        * @param $attribs String: any attributes for the headline, starting with
+        *                 a space and ending with '>'
+        *                 This *must* be at least '>' for no attribs
+        * @param $anchor String: the anchor to give the headline (the bit after the #)
+        * @param $text String: the text of the header
+        * @param $link String: HTML to add for the section edit link
+        * @param $legacyAnchor Mixed: a second, optional anchor to give for
         *   backward compatibility (false to omit)
         *
-        * @return string HTML headline
+        * @return String: HTML headline
         */
        public function makeHeadline( $level, $attribs, $anchor, $text, $link, $legacyAnchor = false ) {
                $ret = "<h$level$attribs"
@@ -1430,8 +1446,6 @@ class Linker {
        /**
         * Split a link trail, return the "inside" portion and the remainder of the trail
         * as a two-element array
-        *
-        * @static
         */
        static function splitTrail( $trail ) {
                static $regex = false;
@@ -1440,7 +1454,7 @@ class Linker {
                        $regex = $wgContLang->linkTrail();
                }
                $inside = '';
-               if ( $trail != '' ) {
+               if ( $trail !== '' ) {
                        $m = array();
                        if ( preg_match( $regex, $trail, $m ) ) {
                                $inside = $m[1];
@@ -1461,7 +1475,7 @@ class Linker {
         * changes, so this allows sysops to combat a busy vandal without bothering
         * other users.
         *
-        * @param Revision $rev
+        * @param $rev Revision object
         */
        function generateRollback( $rev ) {
                return '<span class="mw-rollback-link">['
@@ -1472,8 +1486,8 @@ class Linker {
        /**
         * Build a raw rollback link, useful for collections of "tool" links
         *
-        * @param Revision $rev
-        * @return string
+        * @param $rev Revision object
+        * @return String: HTML fragment
         */
        public function buildRollbackLink( $rev ) {
                global $wgRequest, $wgUser;
@@ -1496,11 +1510,11 @@ class Linker {
        /**
         * Returns HTML for the "templates used on this page" list.
         *
-        * @param array $templates Array of templates from Article::getUsedTemplate
+        * @param $templates Array of templates from Article::getUsedTemplate
         * or similar
-        * @param bool $preview Whether this is for a preview
-        * @param bool $section Whether this is for a section edit
-        * @return string HTML output
+        * @param $preview Boolean: whether this is for a preview
+        * @param $section Boolean: whether this is for a section edit
+        * @return String: HTML output
         */
        public function formatTemplates( $templates, $preview = false, $section = false ) {
                wfProfileIn( __METHOD__ );
@@ -1561,9 +1575,9 @@ class Linker {
        /**
         * Returns HTML for the "hidden categories on this page" list.
         *
-        * @param array $hiddencats Array of hidden categories from Article::getHiddenCategories
+        * @param $hiddencats Array of hidden categories from Article::getHiddenCategories
         * or similar
-        * @return string HTML output
+        * @return String: HTML output
         */
        public function formatHiddenCategories( $hiddencats ) {
                global $wgLang;
@@ -1590,7 +1604,7 @@ class Linker {
         * unit (B, KB, MB or GB) according to the magnitude in question
         *
         * @param $size Size to format
-        * @return string
+        * @return String
         */
        public function formatSize( $size ) {
                global $wgLang;
@@ -1603,22 +1617,25 @@ class Linker {
         * isn't always, because sometimes the accesskey needs to go on a different
         * element than the id, for reverse-compatibility, etc.)
         *
-        * @param string $name    Id of the element, minus prefixes.
-        * @param mixed  $options null or the string 'withaccess' to add an access-
+        * @param $name String: id of the element, minus prefixes.
+        * @param $options Mixed: null or the string 'withaccess' to add an access-
         *   key hint
-        * @return string Contents of the title attribute (which you must HTML-
+        * @return String: contents of the title attribute (which you must HTML-
         *   escape), or false for no title attribute
         */
        public function titleAttrib( $name, $options = null ) {
                wfProfileIn( __METHOD__ );
 
-               $tooltip = wfMsg( "tooltip-$name" );
-               # Compatibility: formerly some tooltips had [alt-.] hardcoded
-               $tooltip = preg_replace( "/ ?\[alt-.\]$/", '', $tooltip );
-
-               # Message equal to '-' means suppress it.
-               if ( wfEmptyMsg( "tooltip-$name", $tooltip ) || $tooltip == '-' ) {
+               if ( wfEmptyMsg( "tooltip-$name" ) ) {
                        $tooltip = false;
+               } else {
+                       $tooltip = wfMsg( "tooltip-$name" );
+                       # Compatibility: formerly some tooltips had [alt-.] hardcoded
+                       $tooltip = preg_replace( "/ ?\[alt-.\]$/", '', $tooltip );
+                       # Message equal to '-' means suppress it.
+                       if (  $tooltip == '-' ) {
+                               $tooltip = false;
+                       }
                }
 
                if ( $options == 'withaccess' ) {
@@ -1642,35 +1659,37 @@ class Linker {
         * the id but isn't always, because sometimes the accesskey needs to go on
         * a different element than the id, for reverse-compatibility, etc.)
         *
-        * @param string $name    Id of the element, minus prefixes.
-        * @return string Contents of the accesskey attribute (which you must HTML-
+        * @param $name String: id of the element, minus prefixes.
+        * @return String: contents of the accesskey attribute (which you must HTML-
         *   escape), or false for no accesskey attribute
         */
        public function accesskey( $name ) {
                wfProfileIn( __METHOD__ );
 
-               $accesskey = wfMsg( "accesskey-$name" );
-
-               # FIXME: Per standard MW behavior, a value of '-' means to suppress the
-               # attribute, but this is broken for accesskey: that might be a useful
-               # value.
-               if( $accesskey != '' && $accesskey != '-' && !wfEmptyMsg( "accesskey-$name", $accesskey ) ) {
-                       wfProfileOut( __METHOD__ );
-                       return $accesskey;
+               if ( wfEmptyMsg( "accesskey-$name" ) ) {
+                       $accesskey = false;
+               } else {
+                       $accesskey = wfMsg( "accesskey-$name" );
+                       if ( $accesskey === '' || $accesskey === '-' ) {
+                               # FIXME: Per standard MW behavior, a value of '-' means to suppress the
+                               # attribute, but this is broken for accesskey: that might be a useful
+                               # value.
+                               $accesskey = false;
+                       }
                }
 
                wfProfileOut( __METHOD__ );
-               return false;
+               return $accesskey;
        }
 
        /**
         * Creates a (show/hide) link for deleting revisions/log entries
         *
-        * @param array $query  Query parameters to be passed to link()
-        * @param bool $restricted  Set to true to use a <strong> instead of a <span>
-        * @param bool $delete Set to true to use (show/hide) rather than (show)
+        * @param $query Array: query parameters to be passed to link()
+        * @param $restricted Boolean: set to true to use a <strong> instead of a <span>
+        * @param $delete Boolean: set to true to use (show/hide) rather than (show)
         *
-        * @return string HTML <a> link to Special:Revisiondelete, wrapped in a
+        * @return String: HTML <a> link to Special:Revisiondelete, wrapped in a
         * span to allow for customization of appearance with CSS
         */
        public function revDeleteLink( $query = array(), $restricted = false, $delete = true ) {
@@ -1684,7 +1703,7 @@ class Linker {
        /**
         * Creates a dead (show/hide) link for deleting revisions/log entries
         *
-        * @param bool $delete Set to true to use (show/hide) rather than (show)
+        * @param $delete Boolean: set to true to use (show/hide) rather than (show)
         *
         * @return string HTML text wrapped in a span to allow for customization
         * of appearance with CSS
@@ -1744,6 +1763,8 @@ class Linker {
         * @param $trail String: optional trail. Alphabetic characters at the start of this string will
         *                      be included in the link text. Other characters will be appended after
         *                      the end of the link.
+        * @param $prefix String: Optional prefix
+        * @param $aprops String: extra attributes to the a-element
         */
        function makeKnownLink( $title, $text = '', $query = '', $trail = '', $prefix = '',$aprops = '') {
                $nt = Title::newFromText( $title );
@@ -1761,12 +1782,12 @@ class Linker {
         * This function is a shortcut to makeBrokenLinkObj(Title::newFromText($title),...). Do not call
         * it if you already have a title object handy. See makeBrokenLinkObj for further documentation.
         *
-        * @param string $title The text of the title
-        * @param string $text Link text
-        * @param string $query Optional query part
-        * @param string $trail Optional trail. Alphabetic characters at the start of this string will
-        *                      be included in the link text. Other characters will be appended after
-        *                      the end of the link.
+        * @param $title String: The text of the title
+        * @param $text String: Link text
+        * @param $query String: Optional query part
+        * @param $trail String: Optional trail. Alphabetic characters at the start of this string will
+        *               be included in the link text. Other characters will be appended after
+        *               the end of the link.
         */
        function makeBrokenLink( $title, $text = '', $query = '', $trail = '' ) {
                $nt = Title::newFromText( $title );
@@ -1840,7 +1861,7 @@ class Linker {
         * it doesn't have to do a database query. It's also valid for interwiki titles and special
         * pages.
         *
-        * @param $nt Title object of target page
+        * @param $title  Title object of target page
         * @param $text   String: text to replace the title
         * @param $query  String: link target
         * @param $trail  String: text after link
@@ -1874,12 +1895,13 @@ class Linker {
         *
         * Make a red link to the edit page of a given title.
         *
-        * @param $nt Title object of the target page
+        * @param $title Title object of the target page
         * @param $text  String: Link text
         * @param $query String: Optional query part
         * @param $trail String: Optional trail. Alphabetic characters at the start of this string will
         *                      be included in the link text. Other characters will be appended after
         *                      the end of the link.
+        * @param $prefix String: Optional prefix
         */
        function makeBrokenLinkObj( $title, $text = '', $query = '', $trail = '', $prefix = '' ) {
                wfProfileIn( __METHOD__ );
@@ -1908,9 +1930,10 @@ class Linker {
         * @param $trail String: optional trail. Alphabetic characters at the start of this string will
         *                      be included in the link text. Other characters will be appended after
         *                      the end of the link.
+        * @param $prefix String: Optional prefix
         */
        function makeStubLinkObj( $nt, $text = '', $query = '', $trail = '', $prefix = '' ) {
-               wfDeprecated( __METHOD__ );
+               //wfDeprecated( __METHOD__ );
                return $this->makeColouredLinkObj( $nt, 'stub', $text, $query, $trail, $prefix );
        }
 
@@ -1926,8 +1949,10 @@ class Linker {
         * @param $trail  String:  optional trail. Alphabetic characters at the start of this string will
         *                      be included in the link text. Other characters will be appended after
         *                      the end of the link.
+        * @param $prefix String: Optional prefix
         */
        function makeColouredLinkObj( $nt, $colour, $text = '', $query = '', $trail = '', $prefix = '' ) {
+               //wfDeprecated( __METHOD__ );
                if($colour != ''){
                        $style = $this->getInternalLinkAttributesObj( $nt, $text, $colour );
                } else $style = '';
@@ -1944,17 +1969,17 @@ class Linker {
         * Creates the HTML source for images
         * @deprecated use makeImageLink2
         *
-        * @param object $title
-        * @param string $label label text
-        * @param string $alt alt text
-        * @param string $align horizontal alignment: none, left, center, right)
-        * @param array $handlerParams Parameters to be passed to the media handler
-        * @param boolean $framed shows image in original size in a frame
-        * @param boolean $thumb shows image as thumbnail in a frame
-        * @param string $manualthumb image name for the manual thumbnail
-        * @param string $valign vertical alignment: baseline, sub, super, top, text-top, middle, bottom, text-bottom
-        * @param string $time, timestamp of the file, set as false for current
-        * @return string
+        * @param $title Title object
+        * @param $label String: label text
+        * @param $alt String: alt text
+        * @param $align String: horizontal alignment: none, left, center, right)
+        * @param $handlerParams Array: parameters to be passed to the media handler
+        * @param $framed Boolean: shows image in original size in a frame
+        * @param $thumb Boolean: shows image as thumbnail in a frame
+        * @param $manualthumb String: image name for the manual thumbnail
+        * @param $valign String: vertical alignment: baseline, sub, super, top, text-top, middle, bottom, text-bottom
+        * @param $time String: timestamp of the file, set as false for current
+        * @return String
         */
        function makeImageLinkObj( $title, $label, $alt, $align = '', $handlerParams = array(), $framed = false,
          $thumb = false, $manualthumb = '', $valign = '', $time = false )
@@ -2016,7 +2041,7 @@ class Linker {
        }
 
        /**
-        * Returns the attributes for the tooltip and access key
+        * Returns the attributes for the tooltip and access key.
         */
        public function tooltipAndAccesskeyAttribs( $name ) {
                global $wgEnableTooltipsAndAccesskeys;