Add returntoquery= parameter to Special:Userlogin which adds a query string to the...
[lhc/web/wiklou.git] / includes / Linker.php
index ad41cb0..93ac43d 100644 (file)
@@ -17,25 +17,15 @@ class Linker {
 
        function __construct() {}
 
-       /**
-        * @deprecated
-        */
-       function postParseLinkColour( $s = null ) {
-               wfDeprecated( __METHOD__ );
-               return null;
-       }
-
        /**
         * Get the appropriate HTML attributes to add to the "a" element of an ex-
         * ternal link, as created by [wikisyntax].
         *
-        * @param string $title  The (unescaped) title text for the link
-        * @param string $unused Unused
         * @param string $class  The contents of the class attribute; if an empty
         *   string is passed, which is the default value, defaults to 'external'.
         */
-       function getExternalLinkAttributes( $title, $unused = null, $class='' ) {
-               return $this->getLinkAttributesInternal( $title, $class, 'external' );
+       function getExternalLinkAttributes( $class = 'external' ) {
+               return $this->getLinkAttributesInternal( '', $class );
        }
 
        /**
@@ -48,7 +38,7 @@ class Linker {
         * @param string $class  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='' ) {
+       function getInterwikiLinkAttributes( $title, $unused = null, $class = 'external' ) {
                global $wgContLang;
 
                # FIXME: We have a whole bunch of handling here that doesn't happen in
@@ -57,7 +47,7 @@ class Linker {
                $title = $wgContLang->checkTitleEncoding( $title );
                $title = preg_replace( '/[\\x00-\\x1f]/', ' ', $title );
 
-               return $this->getLinkAttributesInternal( $title, $class, 'external' );
+               return $this->getLinkAttributesInternal( $title, $class );
        }
 
        /**
@@ -95,20 +85,16 @@ class Linker {
        /**
         * Common code for getLinkAttributesX functions
         */
-       private function getLinkAttributesInternal( $title, $class, $classDefault = false ) {
+       private function getLinkAttributesInternal( $title, $class ) {
                $title = htmlspecialchars( $title );
-               if( $class === '' and $classDefault !== false ) {
-                       # FIXME: Parameter defaults the hard way!  We should just have
-                       # $class = 'external' or whatever as the default in the externally-
-                       # exposed functions, not $class = ''.
-                       $class = $classDefault;
-               }
                $class = htmlspecialchars( $class );
                $r = '';
-               if( $class !== '' ) {
+               if ( $class != '' ) {
                        $r .= " class=\"$class\"";
                }
-               $r .= " title=\"$title\"";
+               if ( $title != '') {
+                       $r .= " title=\"$title\"";
+               }
                return $r;
        }
 
@@ -190,15 +176,7 @@ class Linker {
                # If we don't know whether the page exists, let's find out.
                wfProfileIn( __METHOD__ . '-checkPageExistence' );
                if( !in_array( 'known', $options ) and !in_array( 'broken', $options ) ) {
-                       if( $target->getNamespace() == NS_SPECIAL ) {
-                               if( SpecialPage::exists( $target->getDbKey() ) ) {
-                                       $options []= 'known';
-                               } else {
-                                       $options []= 'broken';
-                               }
-                       } elseif( $target->isAlwaysKnown() or
-                       ($target->getPrefixedText() == '' and $target->getFragment() != '')
-                       or $target->exists() ) {
+                       if( $target->isKnown() ) {
                                $options []= 'known';
                        } else {
                                $options []= 'broken';
@@ -227,17 +205,21 @@ class Linker {
                }
 
                $ret = null;
-               if( wfRunHooks( 'LinkEnd', array( $this, $target, $options, &$text,
-               &$attribs, &$ret ) ) ) {
-                       $ret = Xml::openElement( 'a', $attribs )
-                               . $text
-                               . Xml::closeElement( 'a' );
+               if( wfRunHooks( 'LinkEnd', array( $this, $target, $options, &$text, &$attribs, &$ret ) ) ) {
+                       $ret = Xml::openElement( 'a', $attribs ) . $text . Xml::closeElement( 'a' );
                }
 
                wfProfileOut( __METHOD__ );
                return $ret;
        }
 
+       /**
+        * Identical to link(), except $options defaults to 'known'.
+        */
+       public function linkKnown( $target, $text = null, $customAttribs = array(), $query = array(), $options = 'known' ) {
+               return $this->link( $target, $text, $customAttribs, $query, $options );
+       }
+
        private function linkUrl( $target, $query, $options ) {
                wfProfileIn( __METHOD__ );
                # We don't want to include fragments for broken links, because they
@@ -279,7 +261,7 @@ class Linker {
                        }
 
                        # Note that redirects never count as stubs here.
-                       if ( $target->isRedirect() ) {
+                       if ( !in_array( 'broken', $options ) && $target->isRedirect() ) {
                                $classes[] = 'mw-redirect';
                        } elseif( $target->isContentPage() ) {
                                # Check for stub.
@@ -295,7 +277,10 @@ class Linker {
                }
 
                # Get a default title attribute.
-               if( in_array( 'known', $options ) ) {
+               if( $target->getPrefixedText() == '' ) {
+                       # A link like [[#Foo]].  This used to mean an empty title
+                       # attribute, but that's silly.  Just don't output a title.
+               } elseif( in_array( 'known', $options ) ) {
                        $defaults['title'] = $target->getPrefixedText();
                } else {
                        $defaults['title'] = wfMsg( 'red-link-title', $target->getPrefixedText() );
@@ -330,236 +315,6 @@ class Linker {
                return htmlspecialchars( $target->getPrefixedText() );
        }
 
-       /**
-        * @deprecated Use link()
-        *
-        * This function is a shortcut to makeLinkObj(Title::newFromText($title),...). Do not call
-        * it if you already have a title object handy. See makeLinkObj for further documentation.
-        *
-        * @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 makeLink( $title, $text = '', $query = '', $trail = '' ) {
-               wfProfileIn( __METHOD__ );
-               $nt = Title::newFromText( $title );
-               if ( $nt instanceof Title ) {
-                       $result = $this->makeLinkObj( $nt, $text, $query, $trail );
-               } else {
-                       wfDebug( 'Invalid title passed to Linker::makeLink(): "'.$title."\"\n" );
-                       $result = $text == "" ? $title : $text;
-               }
-
-               wfProfileOut( __METHOD__ );
-               return $result;
-       }
-
-       /**
-        * @deprecated Use link()
-        *
-        * This function is a shortcut to makeKnownLinkObj(Title::newFromText($title),...). Do not call
-        * it if you already have a title object handy. See makeKnownLinkObj for further documentation.
-        *
-        * @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 makeKnownLink( $title, $text = '', $query = '', $trail = '', $prefix = '',$aprops = '') {
-               $nt = Title::newFromText( $title );
-               if ( $nt instanceof Title ) {
-                       return $this->makeKnownLinkObj( $nt, $text, $query, $trail, $prefix , $aprops );
-               } else {
-                       wfDebug( 'Invalid title passed to Linker::makeKnownLink(): "'.$title."\"\n" );
-                       return $text == '' ? $title : $text;
-               }
-       }
-
-       /**
-        * @deprecated Use link()
-        *
-        * 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.
-        */
-       function makeBrokenLink( $title, $text = '', $query = '', $trail = '' ) {
-               $nt = Title::newFromText( $title );
-               if ( $nt instanceof Title ) {
-                       return $this->makeBrokenLinkObj( $nt, $text, $query, $trail );
-               } else {
-                       wfDebug( 'Invalid title passed to Linker::makeBrokenLink(): "'.$title."\"\n" );
-                       return $text == '' ? $title : $text;
-               }
-       }
-
-       /**
-        * @deprecated Use link()
-        *
-        * This function is a shortcut to makeStubLinkObj(Title::newFromText($title),...). Do not call
-        * it if you already have a title object handy. See makeStubLinkObj for further documentation.
-        *
-        * @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 makeStubLink( $title, $text = '', $query = '', $trail = '' ) {
-               wfDeprecated( __METHOD__ );
-               $nt = Title::newFromText( $title );
-               if ( $nt instanceof Title ) {
-                       return $this->makeStubLinkObj( $nt, $text, $query, $trail );
-               } else {
-                       wfDebug( 'Invalid title passed to Linker::makeStubLink(): "'.$title."\"\n" );
-                       return $text == '' ? $title : $text;
-               }
-       }
-
-       /**
-        * @deprecated Use link()
-        *
-        * Make a link for a title which may or may not be in the database. If you need to
-        * call this lots of times, pre-fill the link cache with a LinkBatch, otherwise each
-        * call to this will result in a DB query.
-        *
-        * @param $nt     Title: the title object to make the link from, e.g. from
-        *                      Title::newFromText.
-        * @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. As trail, only before instead of after.
-        */
-       function makeLinkObj( $nt, $text= '', $query = '', $trail = '', $prefix = '' ) {
-               global $wgUser;
-               wfProfileIn( __METHOD__ );
-
-               $query = wfCgiToArray( $query );
-               list( $inside, $trail ) = Linker::splitTrail( $trail );
-               if( $text === '' ) {
-                       $text = $this->linkText( $nt );
-               }
-
-               $ret = $this->link( $nt, "$prefix$text$inside", array(), $query ) . $trail;
-
-               wfProfileOut( __METHOD__ );
-               return $ret;
-       }
-
-       /**
-        * @deprecated Use link()
-        *
-        * Make a link for a title which definitely exists. This is faster than makeLinkObj because
-        * 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 $text   String: text to replace the title
-        * @param $query  String: link target
-        * @param $trail  String: text after link
-        * @param $prefix String: text before link text
-        * @param $aprops String: extra attributes to the a-element
-        * @param $style  String: style to apply - if empty, use getInternalLinkAttributesObj instead
-        * @return the a-element
-        */
-       function makeKnownLinkObj( $title, $text = '', $query = '', $trail = '', $prefix = '' , $aprops = '', $style = '' ) {
-               wfProfileIn( __METHOD__ );
-
-               if ( $text == '' ) {
-                       $text = $this->linkText( $title );
-               }
-               $attribs = Sanitizer::mergeAttributes(
-                       Sanitizer::decodeTagAttributes( $aprops ),
-                       Sanitizer::decodeTagAttributes( $style )
-               );
-               $query = wfCgiToArray( $query );
-               list( $inside, $trail ) = Linker::splitTrail( $trail );
-
-               $ret = $this->link( $title, "$prefix$text$inside", $attribs, $query,
-                       array( 'known', 'noclasses' ) ) . $trail;
-
-               wfProfileOut( __METHOD__ );
-               return $ret;
-       }
-
-       /**
-        * @deprecated Use link()
-        *
-        * Make a red link to the edit page of a given title.
-        *
-        * @param $nt 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.
-        */
-       function makeBrokenLinkObj( $title, $text = '', $query = '', $trail = '', $prefix = '' ) {
-               wfProfileIn( __METHOD__ );
-
-               list( $inside, $trail ) = Linker::splitTrail( $trail );
-               if( $text === '' ) {
-                       $text = $this->linkText( $title );
-               }
-               $nt = $this->normaliseSpecialPage( $title );
-
-               $ret = $this->link( $title, "$prefix$text$inside", array(),
-                       wfCgiToArray( $query ), 'broken' ) . $trail;
-
-               wfProfileOut( __METHOD__ );
-               return $ret;
-       }
-
-       /**
-        * @deprecated Use link()
-        *
-        * Make a brown link to a short article.
-        *
-        * @param $nt 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.
-        */
-       function makeStubLinkObj( $nt, $text = '', $query = '', $trail = '', $prefix = '' ) {
-               wfDeprecated( __METHOD__ );
-               return $this->makeColouredLinkObj( $nt, 'stub', $text, $query, $trail, $prefix );
-       }
-
-       /**
-        * @deprecated Use link()
-        *
-        * Make a coloured link.
-        *
-        * @param $nt Title object of the target page
-        * @param $colour Integer: colour of the link
-        * @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 makeColouredLinkObj( $nt, $colour, $text = '', $query = '', $trail = '', $prefix = '' ) {
-               if($colour != ''){
-                       $style = $this->getInternalLinkAttributesObj( $nt, $text, $colour );
-               } else $style = '';
-               return $this->makeKnownLinkObj( $nt, $text, $query, $trail, $prefix, '', $style );
-       }
-
        /**
         * Generate either a normal exists-style link or a stub link, depending
         * on the given page size.
@@ -576,6 +331,7 @@ class Linker {
                global $wgUser;
                $threshold = intval( $wgUser->getOption( 'stubthreshold' ) );
                $colour = ( $size < $threshold ) ? 'stub' : '';
+               // FIXME: replace deprecated makeColouredLinkObj by link()
                return $this->makeColouredLinkObj( $nt, $colour, $text, $query, $trail, $prefix );
        }
 
@@ -615,12 +371,6 @@ class Linker {
                return $basename;
        }
 
-       /** Obsolete alias */
-       function makeImage( $url, $alt = '' ) {
-               wfDeprecated( __METHOD__ );
-               return $this->makeExternalImage( $url, $alt );
-       }
-
        /** @todo document */
        function makeExternalImage( $url, $alt = '' ) {
                if ( '' == $alt ) {
@@ -629,7 +379,7 @@ class Linker {
                $img = '';
                $success = wfRunHooks('LinkerMakeExternalImage', array( &$url, &$alt, &$img ) );
                if(!$success) {
-                       wfDebug("Hook LinkerMakeExternalImage changed the output of external image with url {$url} and alt text {$alt} to {$img}", true);
+                       wfDebug("Hook LinkerMakeExternalImage changed the output of external image with url {$url} and alt text {$alt} to {$img}\n", true);
                        return $img;
                }
                return Xml::element( 'img',
@@ -638,45 +388,6 @@ class Linker {
                                'alt' => $alt ) );
        }
 
-       /**
-        * 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
-        */
-       function makeImageLinkObj( $title, $label, $alt, $align = '', $handlerParams = array(), $framed = false,
-         $thumb = false, $manualthumb = '', $valign = '', $time = false )
-       {
-               $frameParams = array( 'alt' => $alt, 'caption' => $label );
-               if ( $align ) {
-                       $frameParams['align'] = $align;
-               }
-               if ( $framed ) {
-                       $frameParams['framed'] = true;
-               }
-               if ( $thumb ) {
-                       $frameParams['thumbnail'] = true;
-               }
-               if ( $manualthumb ) {
-                       $frameParams['manualthumb'] = $manualthumb;
-               }
-               if ( $valign ) {
-                       $frameParams['valign'] = $valign;
-               }
-               $file = wfFindFile( $title, $time );
-               return $this->makeImageLink2( $title, $file, $frameParams, $handlerParams, $time );
-       }
-
        /**
         * Given parameters derived from [[Image:Foo|options...]], generate the
         * HTML that that syntax inserts in the page.
@@ -730,8 +441,7 @@ class Linker {
                $page = isset( $hp['page'] ) ? $hp['page'] : false;
                if ( !isset( $fp['align'] ) ) $fp['align'] = '';
                if ( !isset( $fp['alt'] ) ) $fp['alt'] = '';
-               # Backward compatibility, title used to always be equal to alt text
-               if ( !isset( $fp['title'] ) ) $fp['title'] = $fp['alt'];
+               if ( !isset( $fp['title'] ) ) $fp['title'] = '';
 
                $prefix = $postfix = '';
 
@@ -849,8 +559,7 @@ class Linker {
                $page = isset( $hp['page'] ) ? $hp['page'] : false;
                if ( !isset( $fp['align'] ) ) $fp['align'] = 'right';
                if ( !isset( $fp['alt'] ) ) $fp['alt'] = '';
-               # Backward compatibility, title used to always be equal to alt text
-               if ( !isset( $fp['title'] ) ) $fp['title'] = $fp['alt'];
+               if ( !isset( $fp['title'] ) ) $fp['title'] = '';
                if ( !isset( $fp['caption'] ) ) $fp['caption'] = '';
 
                if ( empty( $hp['width'] ) ) {
@@ -864,7 +573,7 @@ class Linker {
                } else {
                        if ( isset( $fp['manualthumb'] ) ) {
                                # Use manually specified thumbnail
-                               $manual_title = Title::makeTitleSafe( NS_IMAGE, $fp['manualthumb'] );
+                               $manual_title = Title::makeTitleSafe( NS_FILE, $fp['manualthumb'] );
                                if( $manual_title ) {
                                        $manual_img = wfFindFile( $manual_title );
                                        if ( $manual_img ) {
@@ -893,10 +602,13 @@ class Linker {
                        }
                }
 
-               if( $page ) {
-                       $query = $query ? '&page=' . urlencode( $page ) : 'page=' . urlencode( $page );
-               }
+               # ThumbnailImage::toHtml() already adds page= onto the end of DjVu URLs
+               # So we don't need to pass it here in $query. However, the URL for the
+               # zoom icon still needs it, so we make a unique query for it. See bug 14771
                $url = $title->getLocalURL( $query );
+               if( $page ) { 
+                       $url = wfAppendQuery( $url, 'page=' . urlencode( $page ) );
+               }
 
                $more = htmlspecialchars( wfMsg( 'thumbnail-more' ) );
 
@@ -968,12 +680,6 @@ class Linker {
                }
        }
 
-       /** @deprecated use Linker::makeMediaLinkObj() */
-       function makeMediaLink( $name, $unused = '', $text = '', $time = false ) {
-               $nt = Title::makeTitleSafe( NS_IMAGE, $name );
-               return $this->makeMediaLinkObj( $nt, $text, $time );
-       }
-
        /**
         * Create a direct link to a given uploaded file.
         *
@@ -1018,22 +724,37 @@ class Linker {
                  wfMsg( $key ) );
        }
 
-       /** @todo document */
+       /**
+        * 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>
+        * 
+        * @TODO! @FIXME! This is a really crappy implementation. $linktype and 
+        * 'external' are mashed into the class attrib for the link (which is made
+        * into a string). Then, if we've got additional params in $attribs, we 
+        * add to it. People using this might want to change the classes (or other
+        * default link attributes), but passing $attribsText is just messy. Would 
+        * make a lot more sense to make put the classes into $attribs, let the 
+        * hook play with them, *then* expand it all at once. 
+        */
        function makeExternalLink( $url, $text, $escape = true, $linktype = '', $attribs = array() ) {
-               $attribsText = $this->getExternalLinkAttributes( $url, $text, 'external ' . $linktype );
-               if ( $attribs ) {
-                       $attribsText .= Xml::expandAttributes( $attribs );
-               }
+               $attribsText = $this->getExternalLinkAttributes( 'external ' . $linktype );
                $url = htmlspecialchars( $url );
                if( $escape ) {
                        $text = htmlspecialchars( $text );
                }
                $link = '';
-               $success = wfRunHooks('LinkerMakeExternalLink', array( &$url, &$text, &$link ) );
+               $success = wfRunHooks('LinkerMakeExternalLink', array( &$url, &$text, &$link, &$attribs, $linktype ) );
                if(!$success) {
-                       wfDebug("Hook LinkerMakeExternalLink changed the output of link with url {$url} and text {$text} to {$link}", true);
+                       wfDebug("Hook LinkerMakeExternalLink changed the output of link with url {$url} and text {$text} to {$link}\n", true);
                        return $link;
                }
+               if ( $attribs ) {
+                       $attribsText .= Xml::expandAttributes( $attribs );
+               }
                return '<a href="'.$url.'"'.$attribsText.'>'.$text.'</a>';
        }
 
@@ -1050,7 +771,7 @@ class Linker {
                } else {
                        $page = Title::makeTitle( NS_USER, $userText );
                }
-               return $this->link( $page, htmlspecialchars( $userText ) );
+               return $this->link( $page, htmlspecialchars( $userText ), array( 'class' => 'mw-userlink' ) );
        }
 
        /**
@@ -1064,7 +785,7 @@ class Linker {
         * @return string
         */
        public function userToolLinks( $userId, $userText, $redContribsWhenNoEdits = false, $flags = 0, $edits=null ) {
-               global $wgUser, $wgDisableAnonTalk, $wgSysopUserBans;
+               global $wgUser, $wgDisableAnonTalk, $wgSysopUserBans, $wgLang;
                $talkable = !( $wgDisableAnonTalk && 0 == $userId );
                $blockable = ( $wgSysopUserBans || 0 == $userId ) && !$flags & self::TOOL_LINKS_NOBLOCK;
 
@@ -1090,7 +811,7 @@ class Linker {
                }
 
                if( $items ) {
-                       return ' (' . implode( ' | ', $items ) . ')';
+                       return ' <span class="mw-usertoollinks">(' . $wgLang->pipeList( $items ) . ')</span>';
                } else {
                        return '';
                }
@@ -1260,7 +981,7 @@ class Linker {
                        }
                        if ( $sectionTitle ) {
                                $link = $this->link( $sectionTitle,
-                                       wfMsgForContent( 'sectionlink' ), array(), array(),
+                                       htmlspecialchars( wfMsgForContent( 'sectionlink' ) ), array(), array(),
                                        'noclasses' );
                        } else {
                                $link = '';
@@ -1318,7 +1039,8 @@ class Linker {
                if( preg_match( '/^' . $medians . '(.*)$/i', $match[1], $submatch ) ) {
                        # Media link; trail not supported.
                        $linkRegexp = '/\[\[(.*?)\]\]/';
-                       $thelink = $this->makeMediaLink( $submatch[1], "", $text );
+                       $title = Title::makeTitleSafe( NS_FILE, $submatch[1] );
+                       $thelink = $this->makeMediaLinkObj( $title, $text );
                } else {
                        # Other kind of link
                        if( preg_match( $wgContLang->linkTrail(), $match[4], $submatch ) ) {
@@ -1329,7 +1051,11 @@ class Linker {
                        $linkRegexp = '/\[\[(.*?)\]\]' . preg_quote( $trail, '/' ) . '/';
                        if (isset($match[1][0]) && $match[1][0] == ':')
                                $match[1] = substr($match[1], 1);
-                       $thelink = $this->makeLink( $match[1], $text, "", $trail );
+                       list( $inside, $trail ) = Linker::splitTrail( $trail );
+                       $thelink = $this->link(
+                               Title::newFromText( $match[1] ),
+                               $text . $inside
+                       ) . $trail;
                }
                $comment = preg_replace( $linkRegexp, StringUtils::escapeRegexReplacement( $thelink ), $comment, 1 );
 
@@ -1368,6 +1094,7 @@ class Linker {
         * @return string HTML
         */
        function revComment( Revision $rev, $local = false, $isPublic = false ) {
+               if( $rev->getRawComment() == "" ) return "";
                if( $rev->isDeleted( Revision::DELETED_COMMENT ) && $isPublic ) {
                        $block = " <span class=\"comment\">" . wfMsgHtml( 'rev-deleted-comment' ) . "</span>";
                } else if( $rev->userCan( Revision::DELETED_COMMENT ) ) {
@@ -1407,8 +1134,11 @@ class Linker {
        /**
         * parameter level defines if we are on an indentation level
         */
-       function tocLine( $anchor, $tocline, $tocnumber, $level ) {
-               return "\n<li class=\"toclevel-$level\"><a href=\"#" .
+       function tocLine( $anchor, $tocline, $tocnumber, $level, $sectionIndex = false ) {
+               $classes = "toclevel-$level";
+               if ( $sectionIndex !== false )
+                       $classes .= " tocsection-$sectionIndex";
+               return "\n<li class=\"$classes\"><a href=\"#" .
                        $anchor . '"><span class="tocnumber">' .
                        $tocnumber . '</span> <span class="toctext">' .
                        $tocline . '</span></a>';
@@ -1432,39 +1162,37 @@ class Linker {
                 . "</ul>\n</td></tr></table>"
                 . '<script type="' . $wgJsMimeType . '">'
                 . ' if (window.showTocToggle) {'
-                . ' var tocShowText = "' . wfEscapeJsString( wfMsg('showtoc') ) . '";'
-                . ' var tocHideText = "' . wfEscapeJsString( wfMsg('hidetoc') ) . '";'
+                . ' var tocShowText = "' . Xml::escapeJsString( wfMsg('showtoc') ) . '";'
+                . ' var tocHideText = "' . Xml::escapeJsString( wfMsg('hidetoc') ) . '";'
                 . ' showTocToggle();'
                 . ' } '
                 . "</script>\n";
        }
-
-       /**
-        * Used to generate section edit links that point to "other" pages
-        * (sections that are really part of included pages).
-        *
-        * @param $title Title string.
-        * @param $section Integer: section number.
-        */
-       public function editSectionLinkForOther( $title, $section ) {
-               wfDeprecated( __METHOD__ );
-               $title = Title::newFromText( $title );
-               return $this->doEditSectionLink( $title, $section );
-       }
-
+       
        /**
-        * @param $nt Title object.
-        * @param $section Integer: section number.
-        * @param $hint Link String: title, or default if omitted or empty
+        * Generate a table of contents from a section tree
+        * @param $tree Return value of ParserOutput::getSections()
+        * @return string HTML
         */
-       public function editSectionLink( Title $nt, $section, $hint = '' ) {
-               wfDeprecated( __METHOD__ );
-               if( $hint === '' ) {
-                       # No way to pass an actual empty $hint here!  The new interface al-
-                       # lows this, so we have to do this for compatibility.
-                       $hint = null;
-               }
-               return $this->doEditSectionLink( $nt, $section, $hint );
+       public function generateTOC( $tree ) {
+               $toc = '';
+               $lastLevel = 0;
+               foreach ( $tree as $section ) {
+                       if ( $section['toclevel'] > $lastLevel )
+                               $toc .= $this->tocIndent();
+                       else if ( $section['toclevel'] < $lastLevel )
+                               $toc .= $this->tocUnindent(
+                                       $lastLevel - $section['toclevel'] );
+                       else
+                               $toc .= $this->tocLineEnd();
+                       
+                       $toc .= $this->tocLine( $section['anchor'],
+                               $section['line'], $section['number'],
+                               $section['toclevel'], $section['index'] );
+                       $lastLevel = $section['toclevel'];
+               }
+               $toc .= $this->tocLineEnd();
+               return $this->tocList( $toc );
        }
 
        /**
@@ -1526,11 +1254,20 @@ class Linker {
         * @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
+        *   backward compatibility (false to omit)
         *
         * @return string HTML headline
         */
-       public function makeHeadline( $level, $attribs, $anchor, $text, $link ) {
-               return "<a name=\"$anchor\"></a><h$level$attribs$link <span class=\"mw-headline\">$text</span></h$level>";
+       public function makeHeadline( $level, $attribs, $anchor, $text, $link, $legacyAnchor = false ) {
+               $ret = "<h$level id=\"$anchor\"$attribs"
+                       . $link
+                       . " <span class=\"mw-headline\">$text</span>"
+                       . "</h$level>";
+               if ( $legacyAnchor !== false ) {
+                       $ret = "<a id=\"$legacyAnchor\"></a>$ret";
+               }
+               return $ret;
        }
 
        /**
@@ -1590,6 +1327,7 @@ class Linker {
                );
                if( $wgRequest->getBool( 'bot' ) ) {
                        $query['bot'] = '1';
+                       $query['hidediff'] = '1'; // bug 15999
                }
                $query['token'] = $wgUser->editToken( array( $title->getPrefixedText(),
                        $rev->getUserText() ) );
@@ -1607,12 +1345,9 @@ class Linker {
         * @param bool $section Whether this is for a section edit
         * @return string HTML output
         */
-       public function formatTemplates( $templates, $preview = false, $section = false) {
-               global $wgUser;
+       public function formatTemplates( $templates, $preview = false, $section = false ) {
                wfProfileIn( __METHOD__ );
 
-               $sk = $wgUser->getSkin();
-
                $outText = '';
                if ( count( $templates ) > 0 ) {
                        # Do a batch existence check
@@ -1631,7 +1366,7 @@ class Linker {
                        } else {
                                $outText .= wfMsgExt( 'templatesused', array( 'parse' ) );
                        }
-                       $outText .= '</div><ul>';
+                       $outText .= "</div><ul>\n";
 
                        usort( $templates, array( 'Title', 'compare' ) );
                        foreach ( $templates as $titleObj ) {
@@ -1644,11 +1379,21 @@ class Linker {
                                        $protected = '';
                                }
                                if( $titleObj->quickUserCan( 'edit' ) ) {
-                                       $editLink = $sk->makeLinkObj( $titleObj, wfMsg('editlink'), 'action=edit' );
+                                       $editLink = $this->link(
+                                               $titleObj,
+                                               wfMsg( 'editlink' ),
+                                               array(),
+                                               array( 'action' => 'edit' )
+                                       );
                                } else {
-                                       $editLink = $sk->makeLinkObj( $titleObj, wfMsg('viewsourcelink'), 'action=edit' );
+                                       $editLink = $this->link(
+                                               $titleObj,
+                                               wfMsg( 'viewsourcelink' ),
+                                               array(),
+                                               array( 'action' => 'edit' )
+                                       );
                                }
-                               $outText .= '<li>' . $sk->link( $titleObj ) . ' (' . $editLink . ') ' . $protected . '</li>';
+                               $outText .= '<li>' . $this->link( $titleObj ) . ' (' . $editLink . ') ' . $protected . '</li>';
                        }
                        $outText .= '</ul>';
                }
@@ -1663,21 +1408,19 @@ class Linker {
         * or similar
         * @return string HTML output
         */
-       public function formatHiddenCategories( $hiddencats) {
-               global $wgUser, $wgLang;
+       public function formatHiddenCategories( $hiddencats ) {
+               global $wgLang;
                wfProfileIn( __METHOD__ );
 
-               $sk = $wgUser->getSkin();
-
                $outText = '';
                if ( count( $hiddencats ) > 0 ) {
                        # Construct the HTML
                        $outText = '<div class="mw-hiddenCategoriesExplanation">';
                        $outText .= wfMsgExt( 'hiddencategories', array( 'parse' ), $wgLang->formatnum( count( $hiddencats ) ) );
-                       $outText .= '</div><ul>';
+                       $outText .= "</div><ul>\n";
 
                        foreach ( $hiddencats as $titleObj ) {
-                               $outText .= '<li>' . $sk->link( $titleObj, null, array(), array(), 'known' ) . '</li>'; # If it's hidden, it must exist - no need to check with a LinkBatch
+                               $outText .= '<li>' . $this->link( $titleObj, null, array(), array(), 'known' ) . "</li>\n"; # If it's hidden, it must exist - no need to check with a LinkBatch
                        }
                        $outText .= '</ul>';
                }
@@ -1699,71 +1442,439 @@ class Linker {
 
        /**
         * Given the id of an interface element, constructs the appropriate title
-        * and accesskey attributes from the system messages.  (Note, this is usu-
-        * ally the id but isn't always, because sometimes the accesskey needs to
-        * go on a different element than the id, for reverse-compatibility, etc.)
+        * attribute from the system messages.  (Note, this is usually 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 title and accesskey attributes, ready to drop in an
-        *   element (e.g., ' title="This does something [x]" accesskey="x"').
+        * @param string $name    Id of the element, minus prefixes.
+        * @param mixed  $options null or the string 'withaccess' to add an access-
+        *   key hint
+        * @return string Contents of the title attribute (which you must HTML-
+        *   escape), or false for no title attribute
         */
-       public function tooltipAndAccesskey( $name ) {
+       public function titleAttrib( $name, $options = null ) {
                wfProfileIn( __METHOD__ );
-               $attribs = array();
 
                $tooltip = wfMsg( "tooltip-$name" );
-               if( !wfEmptyMsg( "tooltip-$name", $tooltip ) && $tooltip != '-' ) {
-                       // Compatibility: formerly some tooltips had [alt-.] hardcoded
-                       $tooltip = preg_replace( "/ ?\[alt-.\]$/", '', $tooltip );
-                       $attribs['title'] = $tooltip;
+               # Compatibility: formerly some tooltips had [alt-.] hardcoded
+               $tooltip = preg_replace( "/ ?\[alt-.\]$/", '', $tooltip );
+
+               # Message equal to '-' means suppress it.
+               if ( wfEmptyMsg( "tooltip-$name", $tooltip ) || $tooltip == '-' ) {
+                       $tooltip = false;
                }
 
-               $accesskey = wfMsg( "accesskey-$name" );
-               if( $accesskey != '' && $accesskey != '-' &&
-               !wfEmptyMsg( "accesskey-$name", $accesskey ) ) {
-                       if( isset( $attribs['title'] ) ) {
-                               $attribs['title'] .= " [$accesskey]";
+               if ( $options == 'withaccess' ) {
+                       $accesskey = $this->accesskey( $name );
+                       if( $accesskey !== false ) {
+                               if ( $tooltip === false || $tooltip === '' ) {
+                                       $tooltip = "[$accesskey]";
+                               } else {
+                                       $tooltip .= " [$accesskey]";
+                               }
                        }
-                       $attribs['accesskey'] = $accesskey;
                }
 
-               $ret = Xml::expandAttributes( $attribs );
                wfProfileOut( __METHOD__ );
-               return $ret;
+               return $tooltip;
        }
 
        /**
-        * Given the id of an interface element, constructs the appropriate title
-        * attribute from the system messages.  (Note, this is usually the id but
-        * isn't always, because sometimes the accesskey needs to go on a different
-        * element than the id, for reverse-compatibility, etc.)
+        * Given the id of an interface element, constructs the appropriate
+        * accesskey attribute from the system messages.  (Note, this is usually
+        * 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.
-        * @param mixed  $options null or the string 'withaccess' to add an access-
-        *   key hint
-        * @return string title attribute, ready to drop in an element
-        * (e.g., ' title="This does something"').
+        * @return string Contents of the accesskey attribute (which you must HTML-
+        *   escape), or false for no accesskey attribute
         */
-       public function tooltip( $name, $options = null ) {
+       public function accesskey( $name ) {
                wfProfileIn( __METHOD__ );
 
-               $attribs = array();
+               $accesskey = wfMsg( "accesskey-$name" );
 
-               $tooltip = wfMsg( "tooltip-$name" );
-               if( !wfEmptyMsg( "tooltip-$name", $tooltip ) && $tooltip != '-' ) {
-                       $attribs['title'] = $tooltip;
+               # 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( isset( $attribs['title'] ) && $options == 'withaccess' ) {
-                       $accesskey = wfMsg( "accesskey-$name" );
-                       if( $accesskey != '' && $accesskey != '-' &&
-                       !wfEmptyMsg( "accesskey-$name", $accesskey ) ) {
-                               $attribs['title'] .= " [$accesskey]";
-                       }
+               wfProfileOut( __METHOD__ );
+               return false;
+       }
+       
+       /**
+        * 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>
+        *
+        * @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 ) {
+               $sp = SpecialPage::getTitleFor( 'Revisiondelete' );
+               $text = wfMsgHtml( 'rev-delundel' );
+               $tag = $restricted ? 'strong' : 'span';
+               $link = $this->link( $sp, $text, array(), $query, array( 'known', 'noclasses' ) );
+               return Xml::tags( $tag, array( 'class' => 'mw-revdelundel-link' ), "($link)" );
+       }
+
+       /* Deprecated methods */
+
+       /**
+        * @deprecated
+        */
+       function postParseLinkColour( $s = null ) {
+               wfDeprecated( __METHOD__ );
+               return null;
+       }
+
+
+       /**
+        * @deprecated Use link()
+        *
+        * This function is a shortcut to makeLinkObj(Title::newFromText($title),...). Do not call
+        * it if you already have a title object handy. See makeLinkObj for further documentation.
+        *
+        * @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 makeLink( $title, $text = '', $query = '', $trail = '' ) {
+               wfProfileIn( __METHOD__ );
+               $nt = Title::newFromText( $title );
+               if ( $nt instanceof Title ) {
+                       $result = $this->makeLinkObj( $nt, $text, $query, $trail );
+               } else {
+                       wfDebug( 'Invalid title passed to Linker::makeLink(): "'.$title."\"\n" );
+                       $result = $text == "" ? $title : $text;
+               }
+
+               wfProfileOut( __METHOD__ );
+               return $result;
+       }
+
+       /**
+        * @deprecated Use link()
+        *
+        * This function is a shortcut to makeKnownLinkObj(Title::newFromText($title),...). Do not call
+        * it if you already have a title object handy. See makeKnownLinkObj for further documentation.
+        *
+        * @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 makeKnownLink( $title, $text = '', $query = '', $trail = '', $prefix = '',$aprops = '') {
+               $nt = Title::newFromText( $title );
+               if ( $nt instanceof Title ) {
+                       return $this->makeKnownLinkObj( $nt, $text, $query, $trail, $prefix , $aprops );
+               } else {
+                       wfDebug( 'Invalid title passed to Linker::makeKnownLink(): "'.$title."\"\n" );
+                       return $text == '' ? $title : $text;
+               }
+       }
+
+       /**
+        * @deprecated Use link()
+        *
+        * 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.
+        */
+       function makeBrokenLink( $title, $text = '', $query = '', $trail = '' ) {
+               $nt = Title::newFromText( $title );
+               if ( $nt instanceof Title ) {
+                       return $this->makeBrokenLinkObj( $nt, $text, $query, $trail );
+               } else {
+                       wfDebug( 'Invalid title passed to Linker::makeBrokenLink(): "'.$title."\"\n" );
+                       return $text == '' ? $title : $text;
+               }
+       }
+
+       /**
+        * @deprecated Use link()
+        *
+        * This function is a shortcut to makeStubLinkObj(Title::newFromText($title),...). Do not call
+        * it if you already have a title object handy. See makeStubLinkObj for further documentation.
+        *
+        * @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 makeStubLink( $title, $text = '', $query = '', $trail = '' ) {
+               wfDeprecated( __METHOD__ );
+               $nt = Title::newFromText( $title );
+               if ( $nt instanceof Title ) {
+                       return $this->makeStubLinkObj( $nt, $text, $query, $trail );
+               } else {
+                       wfDebug( 'Invalid title passed to Linker::makeStubLink(): "'.$title."\"\n" );
+                       return $text == '' ? $title : $text;
+               }
+       }
+
+       /**
+        * @deprecated Use link()
+        *
+        * Make a link for a title which may or may not be in the database. If you need to
+        * call this lots of times, pre-fill the link cache with a LinkBatch, otherwise each
+        * call to this will result in a DB query.
+        *
+        * @param $nt     Title: the title object to make the link from, e.g. from
+        *                      Title::newFromText.
+        * @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. As trail, only before instead of after.
+        */
+       function makeLinkObj( $nt, $text= '', $query = '', $trail = '', $prefix = '' ) {
+               wfProfileIn( __METHOD__ );
+
+               $query = wfCgiToArray( $query );
+               list( $inside, $trail ) = Linker::splitTrail( $trail );
+               if( $text === '' ) {
+                       $text = $this->linkText( $nt );
                }
 
-               $ret = Xml::expandAttributes( $attribs );
+               $ret = $this->link( $nt, "$prefix$text$inside", array(), $query ) . $trail;
+
                wfProfileOut( __METHOD__ );
                return $ret;
        }
+
+       /**
+        * @deprecated Use link()
+        *
+        * Make a link for a title which definitely exists. This is faster than makeLinkObj because
+        * 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 $text   String: text to replace the title
+        * @param $query  String: link target
+        * @param $trail  String: text after link
+        * @param $prefix String: text before link text
+        * @param $aprops String: extra attributes to the a-element
+        * @param $style  String: style to apply - if empty, use getInternalLinkAttributesObj instead
+        * @return the a-element
+        */
+       function makeKnownLinkObj( $title, $text = '', $query = '', $trail = '', $prefix = '' , $aprops = '', $style = '' ) {
+               wfProfileIn( __METHOD__ );
+
+               if ( $text == '' ) {
+                       $text = $this->linkText( $title );
+               }
+               $attribs = Sanitizer::mergeAttributes(
+                       Sanitizer::decodeTagAttributes( $aprops ),
+                       Sanitizer::decodeTagAttributes( $style )
+               );
+               $query = wfCgiToArray( $query );
+               list( $inside, $trail ) = Linker::splitTrail( $trail );
+
+               $ret = $this->link( $title, "$prefix$text$inside", $attribs, $query,
+                       array( 'known', 'noclasses' ) ) . $trail;
+
+               wfProfileOut( __METHOD__ );
+               return $ret;
+       }
+
+       /**
+        * @deprecated Use link()
+        *
+        * Make a red link to the edit page of a given title.
+        *
+        * @param $nt 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.
+        */
+       function makeBrokenLinkObj( $title, $text = '', $query = '', $trail = '', $prefix = '' ) {
+               wfProfileIn( __METHOD__ );
+
+               list( $inside, $trail ) = Linker::splitTrail( $trail );
+               if( $text === '' ) {
+                       $text = $this->linkText( $title );
+               }
+               $nt = $this->normaliseSpecialPage( $title );
+
+               $ret = $this->link( $title, "$prefix$text$inside", array(),
+                       wfCgiToArray( $query ), 'broken' ) . $trail;
+
+               wfProfileOut( __METHOD__ );
+               return $ret;
+       }
+
+       /**
+        * @deprecated Use link()
+        *
+        * Make a brown link to a short article.
+        *
+        * @param $nt 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.
+        */
+       function makeStubLinkObj( $nt, $text = '', $query = '', $trail = '', $prefix = '' ) {
+               wfDeprecated( __METHOD__ );
+               return $this->makeColouredLinkObj( $nt, 'stub', $text, $query, $trail, $prefix );
+       }
+
+       /**
+        * @deprecated Use link()
+        *
+        * Make a coloured link.
+        *
+        * @param $nt Title object of the target page
+        * @param $colour Integer: colour of the link
+        * @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 makeColouredLinkObj( $nt, $colour, $text = '', $query = '', $trail = '', $prefix = '' ) {
+               if($colour != ''){
+                       $style = $this->getInternalLinkAttributesObj( $nt, $text, $colour );
+               } else $style = '';
+               return $this->makeKnownLinkObj( $nt, $text, $query, $trail, $prefix, '', $style );
+       }
+
+       /** Obsolete alias */
+       function makeImage( $url, $alt = '' ) {
+               wfDeprecated( __METHOD__ );
+               return $this->makeExternalImage( $url, $alt );
+       }
+
+       /**
+        * 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
+        */
+       function makeImageLinkObj( $title, $label, $alt, $align = '', $handlerParams = array(), $framed = false,
+         $thumb = false, $manualthumb = '', $valign = '', $time = false )
+       {
+               $frameParams = array( 'alt' => $alt, 'caption' => $label );
+               if ( $align ) {
+                       $frameParams['align'] = $align;
+               }
+               if ( $framed ) {
+                       $frameParams['framed'] = true;
+               }
+               if ( $thumb ) {
+                       $frameParams['thumbnail'] = true;
+               }
+               if ( $manualthumb ) {
+                       $frameParams['manualthumb'] = $manualthumb;
+               }
+               if ( $valign ) {
+                       $frameParams['valign'] = $valign;
+               }
+               $file = wfFindFile( $title, $time );
+               return $this->makeImageLink2( $title, $file, $frameParams, $handlerParams, $time );
+       }
+
+       /** @deprecated use Linker::makeMediaLinkObj() */
+       function makeMediaLink( $name, $unused = '', $text = '', $time = false ) {
+               $nt = Title::makeTitleSafe( NS_FILE, $name );
+               return $this->makeMediaLinkObj( $nt, $text, $time );
+       }
+
+       /**
+        * Used to generate section edit links that point to "other" pages
+        * (sections that are really part of included pages).
+        *
+        * @deprecated use Linker::doEditSectionLink()
+        * @param $title Title string.
+        * @param $section Integer: section number.
+        */
+       public function editSectionLinkForOther( $title, $section ) {
+               wfDeprecated( __METHOD__ );
+               $title = Title::newFromText( $title );
+               return $this->doEditSectionLink( $title, $section );
+       }
+
+       /**
+        * @deprecated use Linker::doEditSectionLink()
+        * @param $nt Title object.
+        * @param $section Integer: section number.
+        * @param $hint Link String: title, or default if omitted or empty
+        */
+       public function editSectionLink( Title $nt, $section, $hint = '' ) {
+               wfDeprecated( __METHOD__ );
+               if( $hint === '' ) {
+                       # No way to pass an actual empty $hint here!  The new interface al-
+                       # lows this, so we have to do this for compatibility.
+                       $hint = null;
+               }
+               return $this->doEditSectionLink( $nt, $section, $hint );
+       }
+
+       /**
+        * @deprecated Returns raw bits of HTML, use titleAttrib() and accesskey()
+        */
+       public function tooltipAndAccesskey( $name ) {
+               # FIXME: If Sanitizer::expandAttributes() treated "false" as "output
+               # no attribute" instead of "output '' as value for attribute", this
+               # would be three lines.
+               $attribs = array(
+                       'title' => $this->titleAttrib( $name, 'withaccess' ),
+                       'accesskey' => $this->accesskey( $name )
+               );
+               if ( $attribs['title'] === false ) {
+                       unset( $attribs['title'] );
+               }
+               if ( $attribs['accesskey'] === false ) {
+                       unset( $attribs['accesskey'] );
+               }
+               return Xml::expandAttributes( $attribs );
+       }
+
+       /** @deprecated Returns raw bits of HTML, use titleAttrib() */
+       public function tooltip( $name, $options = null ) {
+               # FIXME: If Sanitizer::expandAttributes() treated "false" as "output
+               # no attribute" instead of "output '' as value for attribute", this
+               # would be two lines.
+               $tooltip = $this->titleAttrib( $name, $options );
+               if ( $tooltip === false ) {
+                       return '';
+               }
+               return Xml::expandAttributes( array(
+                       'title' => $this->titleAttrib( $name, $options )
+               ) );
+       }
 }