Well there was no problem in the first place. Nice work Werdna
[lhc/web/wiklou.git] / includes / Linker.php
index 32b7cb4..42ad07b 100644 (file)
@@ -16,7 +16,6 @@
  * @package MediaWiki
  */
 class Linker {
-
        function Linker() {}
 
        /**
@@ -28,15 +27,23 @@ class Linker {
 
        /** @todo document */
        function getExternalLinkAttributes( $link, $text, $class='' ) {
+               $link = htmlspecialchars( $link );
+
+               $r = ($class != '') ? " class=\"$class\"" : " class=\"external\"";
+
+               $r .= " title=\"{$link}\"";
+               return $r;
+       }
+
+       function getInterwikiLinkAttributes( $link, $text, $class='' ) {
                global $wgContLang;
 
-               $same = ($link == $text);
                $link = urldecode( $link );
                $link = $wgContLang->checkTitleEncoding( $link );
                $link = preg_replace( '/[\\x00-\\x1f]/', ' ', $link );
                $link = htmlspecialchars( $link );
 
-               $r = ($class != '') ? " class='$class'" : " class='external'";
+               $r = ($class != '') ? " class=\"$class\"" : " class=\"external\"";
 
                $r .= " title=\"{$link}\"";
                return $r;
@@ -61,7 +68,9 @@ class Linker {
        }
 
        /**
-        * @param bool $broken
+        * @param $nt Title object.
+        * @param $text String: FIXME
+        * @param $broken Boolean: FIXME, default 'false'.
         */
        function getInternalLinkAttributesObj( &$nt, $text, $broken = false ) {
                if( $broken == 'stub' ) {
@@ -80,10 +89,10 @@ class Linker {
         * 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 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
+        * @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.
         */
@@ -105,10 +114,10 @@ class Linker {
         * 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 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
+        * @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.
         */
@@ -147,10 +156,10 @@ class Linker {
         * 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 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
+        * @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.
         */
@@ -169,12 +178,14 @@ class Linker {
         * 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 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
+        * @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;
@@ -183,42 +194,25 @@ class Linker {
 
                # Fail gracefully
                if ( ! is_object($nt) ) {
-                       # wfDebugDieBacktrace();
+                       # throw new MWException();
                        wfProfileOut( $fname );
                        return "<!-- ERROR -->{$prefix}{$text}{$trail}";
                }
 
-               $ns = $nt->getNamespace();
-               $dbkey = $nt->getDBkey();
                if ( $nt->isExternal() ) {
                        $u = $nt->getFullURL();
                        $link = $nt->getPrefixedURL();
                        if ( '' == $text ) { $text = $nt->getPrefixedText(); }
-                       $style = $this->getExternalLinkAttributes( $link, $text, 'extiw' );
+                       $style = $this->getInterwikiLinkAttributes( $link, $text, 'extiw' );
 
                        $inside = '';
                        if ( '' != $trail ) {
+                               $m = array();
                                if ( preg_match( '/^([a-z]+)(.*)$$/sD', $trail, $m ) ) {
                                        $inside = $m[1];
                                        $trail = $m[2];
                                }
                        }
-
-                       # Check for anchors, normalize the anchor
-
-                       $parts = explode( '#', $u, 2 );
-                       if ( count( $parts ) == 2 ) {
-                               $anchor = urlencode( Sanitizer::decodeCharReferences( str_replace(' ', '_', $parts[1] ) ) );
-                               $replacearray = array(
-                                       '%3A' => ':',
-                                       '%' => '.'
-                               );
-                               $u = $parts[0] . '#' .
-                                    str_replace( array_keys( $replacearray ),
-                                                array_values( $replacearray ),
-                                                $anchor );
-                       }
-
                        $t = "<a href=\"{$u}\"{$style}>{$text}{$inside}</a>";
 
                        wfProfileOut( $fname );
@@ -270,13 +264,13 @@ class Linker {
         * it doesn't have to do a database query. It's also valid for interwiki titles and special
         * pages.
         *
-        * @param object $nt Title of target page
-        * @param string $text Text to replace the title
-        * @param string $query Link target
-        * @param string $trail Text after link
-        * @param string $prefix Text before link text
-        * @param string $aprops Extra attributes to the a-element
-        * @param string $style Style to apply - if empty, use getInternalLinkAttributesObj instead
+        * @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( $nt, $text = '', $query = '', $trail = '', $prefix = '' , $aprops = '', $style = '' ) {
@@ -297,12 +291,7 @@ class Linker {
                                        $text = htmlspecialchars( $nt->getFragment() );
                                }
                        }
-                       $anchor = urlencode( Sanitizer::decodeCharReferences( str_replace( ' ', '_', $nt->getFragment() ) ) );
-                       $replacearray = array(
-                               '%3A' => ':',
-                               '%' => '.'
-                       );
-                       $u .= '#' . str_replace(array_keys($replacearray),array_values($replacearray),$anchor);
+                       $u .= $nt->getFragmentForURL();
                }
                if ( $text == '' ) {
                        $text = htmlspecialchars( $nt->getPrefixedText() );
@@ -322,17 +311,17 @@ class Linker {
        /**
         * Make a red link to the edit page of a given title.
         * 
-        * @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
+        * @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 makeBrokenLinkObj( $nt, $text = '', $query = '', $trail = '', $prefix = '' ) {
                # Fail gracefully
                if ( ! isset($nt) ) {
-                       # wfDebugDieBacktrace();
+                       # throw new MWException();
                        return "<!-- ERROR -->{$prefix}{$text}{$trail}";
                }
 
@@ -361,16 +350,14 @@ class Linker {
        /**
         * Make a brown link to a short article.
         * 
-        * @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
+        * @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 makeStubLinkObj( $nt, $text = '', $query = '', $trail = '', $prefix = '' ) {
-               $link = $nt->getPrefixedURL();
-
                $u = $nt->escapeLocalURL( $query );
 
                if ( '' == $text ) {
@@ -387,12 +374,12 @@ class Linker {
         * Generate either a normal exists-style link or a stub link, depending
         * on the given page size.
         *
-        * @param int $size
-        * @param Title $nt
-        * @param string $text
-        * @param string $query
-        * @param string $trail
-        * @param string $prefix
+        * @param $size Integer
+        * @param $nt Title object.
+        * @param $text String
+        * @param $query String
+        * @param $trail String
+        * @param $prefix String
         * @return string HTML of link
         */
        function makeSizeLinkObj( $size, $nt, $text = '', $query = '', $trail = '', $prefix = '' ) {
@@ -415,7 +402,7 @@ class Linker {
                        $text = htmlspecialchars( $nt->getPrefixedText() );
                }
                list( $inside, $trail ) = Linker::splitTrail( $trail );
-               return "<strong>{$prefix}{$text}{$inside}</strong>{$trail}";
+               return "<strong class=\"selflink\">{$prefix}{$text}{$inside}</strong>{$trail}";
        }
 
        /** @todo document */
@@ -445,11 +432,16 @@ class Linker {
 
        /** @todo document */
        function makeImageLinkObj( $nt, $label, $alt, $align = '', $width = false, $height = false, $framed = false,
-         $thumb = false, $manual_thumb = '' )
+         $thumb = false, $manual_thumb = '', $page = null )
        {
-               global $wgContLang, $wgUser, $wgThumbLimits;
+               global $wgContLang, $wgUser, $wgThumbLimits, $wgGenerateThumbnailOnParse;
 
                $img   = new Image( $nt );
+
+               if ( ! is_null( $page ) ) {
+                       $img->selectPage( $page );
+               }
+
                if ( !$img->allowInlineDisplay() && $img->exists() ) {
                        return $this->makeKnownLinkObj( $nt );
                }
@@ -457,7 +449,7 @@ class Linker {
                $url   = $img->getViewURL();
                $error = $prefix = $postfix = '';
 
-               wfDebug( "makeImageLinkObj: '$width'x'$height'\n" );
+               wfDebug( "makeImageLinkObj: '$width'x'$height', \"$label\"\n" );
 
                if ( 'center' == $align )
                {
@@ -501,7 +493,7 @@ class Linker {
                        if ( $height == false )
                                $height = -1;
                        if ( $manual_thumb == '') {
-                               $thumb = $img->getThumbnail( $width, $height );
+                               $thumb = $img->getThumbnail( $width, $height, $wgGenerateThumbnailOnParse );
                                if ( $thumb ) {
                                        // In most cases, $width = $thumb->width or $height = $thumb->height.
                                        // If not, we're scaling the image larger than it can be scaled,
@@ -519,6 +511,8 @@ class Linker {
                                        $url = $thumb->getUrl();
                                } else {
                                        $error = htmlspecialchars( $img->getLastError() );
+                                       // Do client-side scaling...
+                                       $height = intval( $img->getHeight() * $width / $img->getWidth() );
                                }
                        }
                } else {
@@ -552,8 +546,7 @@ class Linker {
         * $img is an Image object
         */
        function makeThumbLinkObj( $img, $label = '', $alt, $align = 'right', $boxwidth = 180, $boxheight=false, $framed=false , $manual_thumb = "" ) {
-               global $wgStylePath, $wgContLang;
-               $url  = $img->getViewURL();
+               global $wgStylePath, $wgContLang, $wgGenerateThumbnailOnParse;
                $thumbUrl = '';
                $error = '';
 
@@ -563,21 +556,21 @@ class Linker {
                        $height = $img->getHeight();
                }
                if ( 0 == $width || 0 == $height ) {
-                       $width = $height = 200;
+                       $width = $height = 180;
                }
                if ( $boxwidth == 0 ) {
-                       $boxwidth = 200;
+                       $boxwidth = 180;
                }
                if ( $framed ) {
                        // Use image dimensions, don't scale
                        $boxwidth  = $width;
                        $boxheight = $height;
-                       $thumbUrl  = $url;
+                       $thumbUrl  = $img->getViewURL();
                } else {
                        if ( $boxheight === false )
                                $boxheight = -1;
                        if ( '' == $manual_thumb ) {
-                               $thumb = $img->getThumbnail( $boxwidth, $boxheight );
+                               $thumb = $img->getThumbnail( $boxwidth, $boxheight, $wgGenerateThumbnailOnParse );
                                if ( $thumb ) {
                                        $thumbUrl = $thumb->getUrl();
                                        $boxwidth = $thumb->width;
@@ -592,15 +585,17 @@ class Linker {
                if ( $manual_thumb != '' ) # Use manually specified thumbnail
                {
                        $manual_title = Title::makeTitleSafe( NS_IMAGE, $manual_thumb ); #new Title ( $manual_thumb ) ;
-                       $manual_img = new Image( $manual_title );
-                       $thumbUrl = $manual_img->getViewURL();
-                       if ( $manual_img->exists() )
-                       {
-                               $width  = $manual_img->getWidth();
-                               $height = $manual_img->getHeight();
-                               $boxwidth = $width ;
-                               $boxheight = $height ;
-                               $oboxwidth = $boxwidth + 2 ;
+                       if( $manual_title ) {
+                               $manual_img = new Image( $manual_title );
+                               $thumbUrl = $manual_img->getViewURL();
+                               if ( $manual_img->exists() )
+                               {
+                                       $width  = $manual_img->getWidth();
+                                       $height = $manual_img->getHeight();
+                                       $boxwidth = $width ;
+                                       $boxheight = $height ;
+                                       $oboxwidth = $boxwidth + 2 ;
+                               }
                        }
                }
 
@@ -610,10 +605,14 @@ class Linker {
                $magnifyalign = $wgContLang->isRTL() ? 'left' : 'right';
                $textalign = $wgContLang->isRTL() ? ' style="text-align:right"' : '';
 
-               $s = "<div class=\"thumb t{$align}\"><div style=\"width:{$oboxwidth}px;\">";
+               $s = "<div class=\"thumb t{$align}\"><div class=\"thumbinner\" style=\"width:{$oboxwidth}px;\">";
                if( $thumbUrl == '' ) {
                        // Couldn't generate thumbnail? Scale the image client-side.
-                       $thumbUrl = $url;
+                       $thumbUrl = $img->getViewURL();
+                       if( $boxheight == -1 ) {
+                               // Approximate...
+                               $boxheight = intval( $height * $boxwidth / $width );
+                       }
                }
                if ( $error ) {
                        $s .= htmlspecialchars( $error );
@@ -625,17 +624,17 @@ class Linker {
                        $s .= '<a href="'.$u.'" class="internal" title="'.$alt.'">'.
                                '<img src="'.$thumbUrl.'" alt="'.$alt.'" ' .
                                'width="'.$boxwidth.'" height="'.$boxheight.'" ' .
-                               'longdesc="'.$u.'" /></a>';
+                               'longdesc="'.$u.'" class="thumbimage" /></a>';
                        if ( $framed ) {
                                $zoomicon="";
                        } else {
                                $zoomicon =  '<div class="magnify" style="float:'.$magnifyalign.'">'.
                                        '<a href="'.$u.'" class="internal" title="'.$more.'">'.
                                        '<img src="'.$wgStylePath.'/common/images/magnify-clip.png" ' .
-                                       'width="15" height="11" alt="'.$more.'" /></a></div>';
+                                       'width="15" height="11" alt="" /></a></div>';
                        }
                }
-               $s .= '  <div class="thumbcaption" '.$textalign.'>'.$zoomicon.$label."</div></div></div>";
+               $s .= '  <div class="thumbcaption"'.$textalign.'>'.$zoomicon.$label."</div></div></div>";
                return str_replace("\n", ' ', $s);
        }
 
@@ -645,7 +644,7 @@ class Linker {
        function makeBrokenImageLinkObj( $nt, $text = '', $query = '', $trail = '', $prefix = '' ) {
                # Fail gracefully
                if ( ! isset($nt) ) {
-                       # wfDebugDieBacktrace();
+                       # throw new MWException();
                        return "<!-- ERROR -->{$prefix}{$text}{$trail}";
                }
 
@@ -656,7 +655,7 @@ class Linker {
                if ( '' != $query ) {
                        $q .= "&$query";
                }
-               $uploadTitle = Title::makeTitle( NS_SPECIAL, 'Upload' );
+               $uploadTitle = SpecialPage::getTitleFor( 'Upload' );
                $url = $uploadTitle->escapeLocalURL( $q );
 
                if ( '' == $text ) {
@@ -679,13 +678,13 @@ class Linker {
        /**
         * Create a direct link to a given uploaded file.
         *
-        * @param Title  $title
-        * @param string $text   pre-sanitized HTML
-        * @param bool   $nourl  Mask absolute URLs, so the parser doesn't
+        * @param $title Title object.
+        * @param $text  String: pre-sanitized HTML
+        * @param $nourl Boolean: Mask absolute URLs, so the parser doesn't
         *                       linkify them (it is currently not context-aware)
         * @return string HTML
         *
-        * @access public
+        * @public
         * @todo Handle invalid or missing images better.
         */
        function makeMediaLinkObj( $title, $text = '' ) {
@@ -693,13 +692,12 @@ class Linker {
                        ### HOTFIX. Instead of breaking, return empty string.
                        return $text;
                } else {
-                       $name = $title->getDBKey();
                        $img  = new Image( $title );
                        if( $img->exists() ) {
                                $url  = $img->getURL();
                                $class = 'internal';
                        } else {
-                               $upload = Title::makeTitle( NS_SPECIAL, 'Upload' );
+                               $upload = SpecialPage::getTitleFor( 'Upload' );
                                $url = $upload->getLocalUrl( 'wpDestFile=' . urlencode( $img->getName() ) );
                                $class = 'new';
                        }
@@ -708,7 +706,7 @@ class Linker {
                                $text = $alt;
                        }
                        $u = htmlspecialchars( $url );
-                       return "<a href=\"{$u}\" class='$class' title=\"{$alt}\">{$text}</a>";
+                       return "<a href=\"{$u}\" class=\"$class\" title=\"{$alt}\">{$text}</a>";
                }
        }
 
@@ -723,10 +721,10 @@ class Linker {
        }
 
        /** @todo document */
-       function makeExternalLink( $url, $text, $escape = true, $linktype = '' ) {
+       function makeExternalLink( $url, $text, $escape = true, $linktype = '', $ns = null ) {
                $style = $this->getExternalLinkAttributes( $url, $text, 'external ' . $linktype );
-               global $wgNoFollowLinks;
-               if( $wgNoFollowLinks ) {
+               global $wgNoFollowLinks, $wgNoFollowNsExceptions;
+               if( $wgNoFollowLinks && !(isset($ns) && in_array($ns, $wgNoFollowNsExceptions)) ) {
                        $style .= ' rel="nofollow"';
                }
                $url = htmlspecialchars( $url );
@@ -738,17 +736,17 @@ class Linker {
 
        /**
         * Make user link (or user contributions for unregistered users)
-        * @param int $userId
-        * @param string $userText
+        * @param $userId   Integer: user id in database.
+        * @param $userText String: user name in database
         * @return string HTML fragment
-        * @access private
+        * @private
         */
        function userLink( $userId, $userText ) {
                $encName = htmlspecialchars( $userText );
                if( $userId == 0 ) {
-                       $contribsPage = Title::makeTitle( NS_SPECIAL, 'Contributions' );
+                       $contribsPage = SpecialPage::getTitleFor( 'Contributions', $userText );
                        return $this->makeKnownLinkObj( $contribsPage,
-                               $encName, 'target=' . urlencode( $userText ) );
+                               $encName);
                } else {
                        $userPage = Title::makeTitle( NS_USER, $userText );
                        return $this->makeLinkObj( $userPage, $encName );
@@ -756,10 +754,10 @@ class Linker {
        }
 
        /**
-        * @param int $userId
-        * @param string $userText
+        * @param $userId Integer: user id in database.
+        * @param $userText String: user name in database.
         * @return string HTML fragment with talk and/or block links
-        * @access private
+        * @private
         */
        function userToolLinks( $userId, $userText ) {
                global $wgUser, $wgDisableAnonTalk, $wgSysopUserBans;
@@ -771,9 +769,9 @@ class Linker {
                        $items[] = $this->userTalkLink( $userId, $userText );
                }
                if( $userId ) {
-                       $contribsPage = Title::makeTitle( NS_SPECIAL, 'Contributions' );
-                       $items[] = $this->makeKnownLinkObj( $contribsPage,
-                               wfMsgHtml( 'contribslink' ), 'target=' . urlencode( $userText ) );
+                       $contribsPage = SpecialPage::getTitleFor( 'Contributions', $userText );
+                       $items[] = $this->makeKnownLinkObj( $contribsPage ,
+                               wfMsgHtml( 'contribslink' ) );
                }
                if( $blockable && $wgUser->isAllowed( 'block' ) ) {
                        $items[] = $this->blockLink( $userId, $userText );
@@ -787,14 +785,14 @@ class Linker {
        }
 
        /**
-        * @param int $userId
-        * @param string $userText
+        * @param $userId Integer: user id in database.
+        * @param $userText String: user name in database.
         * @return string HTML fragment with user talk link
-        * @access private
+        * @private
         */
        function userTalkLink( $userId, $userText ) {
-               global $wgContLang;
-               $talkname = $wgContLang->getNsText( NS_TALK ); # use the shorter name
+               global $wgLang;
+               $talkname = $wgLang->getNsText( NS_TALK ); # use the shorter name
 
                $userTalkPage = Title::makeTitle( NS_USER_TALK, $userText );
                $userTalkLink = $this->makeLinkObj( $userTalkPage, $talkname );
@@ -802,30 +800,30 @@ class Linker {
        }
 
        /**
-        * @param int $userId
-        * @param string $userText
+        * @param $userId Integer: userid
+        * @param $userText String: user name in database.
         * @return string HTML fragment with block link
-        * @access private
+        * @private
         */
        function blockLink( $userId, $userText ) {
-               $blockPage = Title::makeTitle( NS_SPECIAL, 'Blockip' );
+               $blockPage = SpecialPage::getTitleFor( 'Blockip', $userText );
                $blockLink = $this->makeKnownLinkObj( $blockPage,
-                       wfMsgHtml( 'blocklink' ), 'ip=' . urlencode( $userText ) );
+                       wfMsgHtml( 'blocklink' ) );
                return $blockLink;
        }
        
        /**
         * Generate a user link if the current user is allowed to view it
-        * @param Revision $rev
+        * @param $rev Revision object.
         * @return string HTML
         */
        function revUserLink( $rev ) {
-               if( $rev->userCan( MW_REV_DELETED_USER ) ) {
+               if( $rev->userCan( Revision::DELETED_USER ) ) {
                        $link = $this->userLink( $rev->getRawUser(), $rev->getRawUserText() );
                } else {
                        $link = wfMsgHtml( 'rev-deleted-user' );
                }
-               if( $rev->isDeleted( MW_REV_DELETED_USER ) ) {
+               if( $rev->isDeleted( Revision::DELETED_USER ) ) {
                        return '<span class="history-deleted">' . $link . '</span>';
                }
                return $link;
@@ -833,18 +831,18 @@ class Linker {
 
        /**
         * Generate a user tool link cluster if the current user is allowed to view it
-        * @param Revision $rev
+        * @param $rev Revision object.
         * @return string HTML
         */
        function revUserTools( $rev ) {
-               if( $rev->userCan( MW_REV_DELETED_USER ) ) {
+               if( $rev->userCan( Revision::DELETED_USER ) ) {
                        $link = $this->userLink( $rev->getRawUser(), $rev->getRawUserText() ) .
                                ' ' .
                                $this->userToolLinks( $rev->getRawUser(), $rev->getRawUserText() );
                } else {
                        $link = wfMsgHtml( 'rev-deleted-user' );
                }
-               if( $rev->isDeleted( MW_REV_DELETED_USER ) ) {
+               if( $rev->isDeleted( Revision::DELETED_USER ) ) {
                        return '<span class="history-deleted">' . $link . '</span>';
                }
                return $link;
@@ -856,17 +854,18 @@ class Linker {
         * comments. It escapes any HTML in the comment, but adds some CSS to format
         * auto-generated comments (from section editing) and formats [[wikilinks]].
         *
-        * The $title parameter must be a title OBJECT. It is used to generate a
-        * direct link to the section in the autocomment.
         * @author Erik Moeller <moeller@scireview.de>
         *
         * Note: there's not always a title to pass to this function.
         * 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
         */
-       function formatComment($comment, $title = NULL) {
-               $fname = 'Linker::formatComment';
-               wfProfileIn( $fname );
+       function formatComment($comment, $title = NULL, $local = false) {
+               wfProfileIn( __METHOD__ );
 
                global $wgContLang;
                $comment = str_replace( "\n", " ", $comment );
@@ -876,6 +875,7 @@ class Linker {
                # some nasty regex.
                # We look for all comments, match any text before and after the comment,
                # add a separator where needed and format the comment itself with CSS
+               $match = array();
                while (preg_match('/(.*)\/\*\s*(.*?)\s*\*\/(.*)/', $comment,$match)) {
                        $pre=$match[1];
                        $auto=$match[2];
@@ -884,11 +884,20 @@ class Linker {
                        if( $title ) {
                                $section = $auto;
 
-                               # This is hackish but should work in most cases.
+                               # 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.
+                               $section = str_replace( '[[:', '', $section );
                                $section = str_replace( '[[', '', $section );
                                $section = str_replace( ']]', '', $section );
-                               $sectionTitle = wfClone( $title );
-                               $sectionTitle->mFragment = $section;
+                               if ( $local ) {
+                                       $sectionTitle = Title::newFromText( '#' . $section);
+                               } else {
+                                       $sectionTitle = wfClone( $title );
+                                       $sectionTitle->mFragment = $section;
+                               }
                                $link = $this->makeKnownLinkObj( $sectionTitle, wfMsg( 'sectionlink' ) );
                        }
                        $sep='-';
@@ -901,14 +910,16 @@ class Linker {
 
                # format regular and media links - all other wiki formatting
                # is ignored
-               $medians = $wgContLang->getNsText( NS_MEDIA ) . ':';
-               while(preg_match('/\[\[(.*?)(\|(.*?))*\]\](.*)$/',$comment,$match)) {
+               $medians = '(?:' . preg_quote( Namespace::getCanonicalName( NS_MEDIA ), '/' ) . '|';
+               $medians .= preg_quote( $wgContLang->getNsText( NS_MEDIA ), '/' ) . '):';
+               while(preg_match('/\[\[:?(.*?)(\|(.*?))*\]\](.*)$/',$comment,$match)) {
                        # Handle link renaming [[foo|text]] will show link as "text"
                        if( "" != $match[3] ) {
                                $text = $match[3];
                        } else {
                                $text = $match[1];
                        }
+                       $submatch = array();
                        if( preg_match( '/^' . $medians . '(.*)$/i', $match[1], $submatch ) ) {
                                # Media link; trail not supported.
                                $linkRegexp = '/\[\[(.*?)\]\]/';
@@ -921,13 +932,13 @@ class Linker {
                                        $trail = "";
                                }
                                $linkRegexp = '/\[\[(.*?)\]\]' . preg_quote( $trail, '/' ) . '/';
-                               if ($match[1][0] == ':')
+                               if (isset($match[1][0]) && $match[1][0] == ':')
                                        $match[1] = substr($match[1], 1);
                                $thelink = $this->makeLink( $match[1], $text, "", $trail );
                        }
-                       $comment = preg_replace( $linkRegexp, wfRegexReplacement( $thelink ), $comment, 1 );
+                       $comment = preg_replace( $linkRegexp, StringUtils::escapeRegexReplacement( $thelink ), $comment, 1 );
                }
-               wfProfileOut( $fname );
+               wfProfileOut( __METHOD__ );
                return $comment;
        }
 
@@ -936,37 +947,40 @@ class Linker {
         * it's non-empty, otherwise return empty string.
         *
         * @param string $comment
-        * @param Title $title
+        * @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
         *
         * @return string
         */
-       function commentBlock( $comment, $title = NULL ) {
+       function commentBlock( $comment, $title = NULL, $local = false ) {
                // '*' used to be the comment inserted by the software way back
                // in antiquity in case none was provided, here for backwards
                // compatability, acc. to brion -ævar
                if( $comment == '' || $comment == '*' ) {
                        return '';
                } else {
-                       $formatted = $this->formatComment( $comment, $title );
-                       return " <span class='comment'>($formatted)</span>";
+                       $formatted = $this->formatComment( $comment, $title, $local );
+                       return " <span class=\"comment\">($formatted)</span>";
                }
        }
        
        /**
         * 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
         * @return string HTML
         */
-       function revComment( $rev ) {
-               if( $rev->userCan( MW_REV_DELETED_COMMENT ) ) {
-                       $block = $this->commentBlock( $rev->getRawComment(), $rev->getTitle() );
+       function revComment( Revision $rev, $local = false ) {
+               if( $rev->userCan( Revision::DELETED_COMMENT ) ) {
+                       $block = $this->commentBlock( $rev->getRawComment(), $rev->getTitle(), $local );
                } else {
-                       $block = " <span class='comment'>" .
+                       $block = " <span class=\"comment\">" .
                                wfMsgHtml( 'rev-deleted-comment' ) . "</span>";
                }
-               if( $rev->isDeleted( MW_REV_DELETED_COMMENT ) ) {
-                       return " <span class='history-deleted'>$block</span>";
+               if( $rev->isDeleted( Revision::DELETED_COMMENT ) ) {
+                       return " <span class=\"history-deleted\">$block</span>";
                }
                return $block;
        }
@@ -985,7 +999,7 @@ 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=\"#" .
+               return "\n<li class=\"toclevel-$level\"><a href=\"#" .
                        $anchor . '"><span class="tocnumber">' .
                        $tocnumber . '</span> <span class="toctext">' .
                        $tocline . '</span></a>';
@@ -1004,7 +1018,9 @@ class Linker {
                   '<table id="toc" class="toc" summary="' . $title .'"><tr><td>'
                 . '<div id="toctitle"><h2>' . $title . "</h2></div>\n"
                 . $toc
-                . "</ul>\n</td></tr></table>\n"
+                # no trailing newline, script should not be wrapped in a
+                # paragraph
+                . "</ul>\n</td></tr></table>"
                 . '<script type="' . $wgJsMimeType . '">'
                 . ' if (window.showTocToggle) {'
                 . ' var tocShowText = "' . wfEscapeJsString( wfMsgForContent('showtoc') ) . '";'
@@ -1022,21 +1038,14 @@ class Linker {
                $editurl = '&section='.$section;
                $url = $this->makeKnownLinkObj( $title, wfMsg('editsection'), 'action=edit'.$editurl );
 
-               if( $wgContLang->isRTL() ) {
-                       $farside = 'left';
-                       $nearside = 'right';
-               } else {
-                       $farside = 'right';
-                       $nearside = 'left';
-               }
-               return "<div class=\"editsection\" style=\"float:$farside;margin-$nearside:5px;\">[".$url."]</div>";
+               return "<span class=\"editsection\">[".$url."]</span>";
 
        }
 
-       /** 
-        * @param Title $title
-        * @param integer $section
-        * @param string $hint Link title, or default if omitted or empty
+       /**
+        * @param $title Title object.
+        * @param $section Integer: section number.
+        * @param $hint Link String: title, or default if omitted or empty
         */
        function editSectionLink( $nt, $section, $hint='' ) {
                global $wgContLang;
@@ -1045,14 +1054,7 @@ class Linker {
                $hint = ( $hint=='' ) ? '' : ' title="' . wfMsgHtml( 'editsectionhint', htmlspecialchars( $hint ) ) . '"';
                $url = $this->makeKnownLinkObj( $nt, wfMsg('editsection'), 'action=edit'.$editurl, '', '', '',  $hint );
 
-               if( $wgContLang->isRTL() ) {
-                       $farside = 'left';
-                       $nearside = 'right';
-               } else {
-                       $farside = 'right';
-                       $nearside = 'left';
-               }
-               return "<div class=\"editsection\" style=\"float:$farside;margin-$nearside:5px;\">[".$url."]</div>";
+               return "<span class=\"editsection\">[".$url."]</span>";
        }
 
        /**
@@ -1061,7 +1063,7 @@ class Linker {
         *
         * @static
         */
-       function splitTrail( $trail ) {
+       static function splitTrail( $trail ) {
                static $regex = false;
                if ( $regex === false ) {
                        global $wgContLang;
@@ -1069,6 +1071,7 @@ class Linker {
                }
                $inside = '';
                if ( '' != $trail ) {
+                       $m = array();
                        if ( preg_match( $regex, $trail, $m ) ) {
                                $inside = $m[1];
                                $trail = $m[2];
@@ -1077,5 +1080,112 @@ class Linker {
                return array( $inside, $trail );
        }
 
+       /**
+        * Generate a rollback link for a given revision.  Currently it's the
+        * caller's responsibility to ensure that the revision is the top one. If
+        * it's not, of course, the user will get an error message.
+        *
+        * If the calling page is called with the parameter &bot=1, all rollback
+        * links also get that parameter. It causes the edit itself and the rollback
+        * to be marked as "bot" edits. Bot edits are hidden by default from recent
+        * changes, so this allows sysops to combat a busy vandal without bothering
+        * other users.
+        *
+        * @param Revision $rev
+        */
+       function generateRollback( $rev ) {
+               global $wgUser, $wgRequest;
+               $title = $rev->getTitle();
+
+               $extraRollback = $wgRequest->getBool( 'bot' ) ? '&bot=1' : '';
+               $extraRollback .= '&token=' . urlencode(
+                       $wgUser->editToken( array( $title->getPrefixedText(), $rev->getUserText() ) ) );
+               return '<span class="mw-rollback-link">['. $this->makeKnownLinkObj( $title,
+                       wfMsg('rollbacklink'),
+                       'action=rollback&from=' . urlencode( $rev->getUserText() ) . $extraRollback ) .']</span>';
+       }
+
+       /**
+        * Returns HTML for the "templates used on this page" list.
+        *
+        * @param array $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
+        */
+       public function formatTemplates( $templates, $preview = false, $section = false) {
+               global $wgUser;
+               wfProfileIn( __METHOD__ );
+
+               $sk =& $wgUser->getSkin();
+
+               $outText = '';
+               if ( count( $templates ) > 0 ) {
+                       # Do a batch existence check
+                       $batch = new LinkBatch;
+                       foreach( $templates as $title ) {
+                               $batch->addObj( $title );
+                       }
+                       $batch->execute();
+
+                       # Construct the HTML
+                       $outText = '<div class="mw-templatesUsedExplanation">';
+                       if ( $preview ) {
+                               $outText .= wfMsgExt( 'templatesusedpreview', array( 'parse' ) );
+                       } elseif ( $section ) {
+                               $outText .= wfMsgExt( 'templatesusedsection', array( 'parse' ) );
+                       } else {
+                               $outText .= wfMsgExt( 'templatesused', array( 'parse' ) );
+                       }
+                       $outText .= '</div><ul>';
+
+                       foreach ( $templates as $titleObj ) {
+                               $r = $titleObj->getRestrictions( 'edit' );
+                               if ( in_array( 'sysop', $r ) ) { 
+                                       $protected = wfMsgExt( 'template-protected', array( 'parseinline' ) );
+                               } elseif ( in_array( 'autoconfirmed', $r ) ) {
+                                       $protected = wfMsgExt( 'template-semiprotected', array( 'parseinline' ) );
+                               } else {
+                                       $protected = '';
+                               }
+                               $outText .= '<li>' . $sk->makeLinkObj( $titleObj ) . ' ' . $protected . '</li>';
+                       }
+                       $outText .= '</ul>';
+               }
+               wfProfileOut( __METHOD__  );
+               return $outText;
+       }
+       
+       /**
+        * Format a size in bytes for output, using an appropriate
+        * unit (B, KB, MB or GB) according to the magnitude in question
+        *
+        * @param $size Size to format
+        * @return string
+        */
+       public function formatSize( $size ) {
+               global $wgLang;
+               if( $size > 1024 ) {
+                       $size = $size / 1024;
+                       if( $size > 1024 ) {
+                               $size = $size / 1024;
+                               if( $size > 1024 ) {
+                                       $size = $size / 1024;
+                                       $msg = 'size-gigabytes';
+                               } else {
+                                       $msg = 'size-megabytes';
+                               }
+                       } else {
+                               $msg = 'size-kilobytes';
+                       }
+               } else {
+                       $msg = 'size-bytes';
+               }
+               $size = round( $size, 0 );
+               return wfMsgHtml( $msg, $wgLang->formatNum( $size ) );
+       }
+       
 }
+
 ?>