remove trailing white space
[lhc/web/wiklou.git] / includes / Skin.php
index 873be41..5d39bc3 100644 (file)
@@ -1,33 +1,66 @@
 <?php
 
-require_once( "Feed.php" );
-require_once( "Image.php" );
+/**
+ *
+ */
+
+/**
+ * This is not a valid entry point, perform no further processing unless MEDIAWIKI is defined
+ */
+if( defined( "MEDIAWIKI" ) ) {
 
 # See skin.doc
 
-# These are the INTERNAL names, which get mapped
-# directly to class names.  For display purposes, the
-# Language class has internationalized names
+require_once( 'Feed.php' );  // should not be called if the actual page isn't feed enabled
+require_once( 'Image.php' );
+
+# These are the INTERNAL names, which get mapped directly to class names and
+# file names in ./skins/. For display purposes, the Language class has
+# internationalized names
 #
-/* private */ $wgValidSkinNames = array(
-       'standard' => "Standard",
-       'nostalgia' => "Nostalgia",
-       'cologneblue' => "CologneBlue"
+/*
+$wgValidSkinNames = array(
+       'standard'      => 'Standard',
+       'nostalgia'     => 'Nostalgia',
+       'cologneblue'   => 'CologneBlue'
 );
 if( $wgUsePHPTal ) {
-    #$wgValidSkinNames[] = "PHPTal";
-    #$wgValidSkinNames['davinci'] = "DaVinci";
-    #$wgValidSkinNames['mono'] = "Mono";
-    $wgValidSkinNames['monobook'] = "MonoBook";
-    #$wgValidSkinNames['monobookminimal'] = "MonoBookMinimal";
+       #$wgValidSkinNames[] = 'PHPTal';
+       #$wgValidSkinNames['davinci'] = 'DaVinci';
+       #$wgValidSkinNames['mono'] = 'Mono';
+       #$wgValidSkinNames['monobookminimal'] = 'MonoBookMinimal';
+       $wgValidSkinNames['monobook'] = 'MonoBook';
+       $wgValidSkinNames['myskin'] = 'MySkin';
+       $wgValidSkinNames['chick'] = 'Chick';
+}
+*/
+
+# Get a list of all skins available in /skins/
+# Build using the regular expression '^(.*).php$'
+# Array keys are all lower case, array value keep the case used by filename
+#
+
+$skinDir = dir($IP.'/skins');
+
+# while code from www.php.net
+while (false !== ($file = $skinDir->read())) {
+       if(preg_match('/^(.*).php$/',$file, $matches)) {
+               $aSkin = $matches[1];
+       $wgValidSkinNames[strtolower($aSkin)] = $aSkin;
+       }
 }
+$skinDir->close();
+unset($matches);
 
-require_once( "RecentChange.php" );
+require_once( 'RecentChange.php' );
 
+/**
+ * @todo document
+ */
 class RCCacheEntry extends RecentChange
 {
        var $secureName, $link;
-       var $curlink , $lastlink , $usertalklink , $versionlink ;
+       var $curlink , $difflink, $lastlink , $usertalklink , $versionlink ;
        var $userlink, $timestamp, $watched;
 
        function newFromParent( $rc )
@@ -39,52 +72,70 @@ class RCCacheEntry extends RecentChange
        }
 } ;
 
-class Skin {
 
-       /* private */ var $lastdate, $lastline;
+/**
+ * The main skin class that provide methods and properties for all other skins
+ * including PHPTal skins.
+ * This base class is also the "Standard" skin.
+ */
+class Skin {
+       /**#@+
+        * @access private
+        */
+       var $lastdate, $lastline;
        var $linktrail ; # linktrail regexp
        var $rc_cache ; # Cache for Enhanced Recent Changes
        var $rcCacheIndex ; # Recent Changes Cache Counter for visibility toggle
        var $rcMoveIndex;
+       var $postParseLinkColour = true;
+       /**#@-*/
 
-       function Skin()
-       {
-               $this->linktrail = wfMsg("linktrail");
+       function Skin() {
+               global $wgUseOldExistenceCheck;
+               $postParseLinkColour = !$wgUseOldExistenceCheck;
+               $this->linktrail = wfMsg('linktrail');
        }
 
-       function getSkinNames()
-       {
+       function getSkinNames() {
                global $wgValidSkinNames;
                return $wgValidSkinNames;
        }
 
-       function getStylesheet()
-       {
-               return "wikistandard.css";
+       function getStylesheet() {
+               return 'wikistandard.css';
        }
 
-       function qbSetting()
-       {
+       function getSkinName() {
+               return 'standard';
+       }
+
+       /**
+        * Get/set accessor for delayed link colouring
+        */
+       function postParseLinkColour( $setting = NULL ) {
+               return wfSetVar( $this->postParseLinkColour, $setting );
+       }
+
+       function qbSetting() {
                global $wgOut, $wgUser;
 
                if ( $wgOut->isQuickbarSuppressed() ) { return 0; }
-               $q = $wgUser->getOption( "quickbar" );
-               if ( "" == $q ) { $q = 0; }
+               $q = $wgUser->getOption( 'quickbar' );
+               if ( '' == $q ) { $q = 0; }
                return $q;
        }
 
-       function initPage( &$out )
-       {
-               $fname = "Skin::initPage";
+       function initPage( &$out ) {
+               $fname = 'Skin::initPage';
                wfProfileIn( $fname );
-               
-               $out->addLink( array( "rel" => "shortcut icon", "href" => "/favicon.ico" ) );
-               
+
+               $out->addLink( array( 'rel' => 'shortcut icon', 'href' => '/favicon.ico' ) );
+
                $this->addMetadataLinks($out);
-           
+
                wfProfileOut( $fname );
        }
-       
+
        function addMetadataLinks( &$out ) {
                global $wgTitle, $wgEnableDublinCoreRdf, $wgEnableCreativeCommonsRdf, $wgRdfMimeType, $action;
                global $wgRightsPage, $wgRightsUrl;
@@ -95,16 +146,16 @@ class Skin {
                                $out->addMetadataLink( array(
                                        'title' => 'Creative Commons',
                                        'type' => 'application/rdf+xml',
-                                       'href' => $wgTitle->getLocalURL( "action=creativecommons") ) );
+                                       'href' => $wgTitle->getLocalURL( 'action=creativecommons') ) );
                        }
                        if( $wgEnableDublinCoreRdf ) {
                                $out->addMetadataLink( array(
                                        'title' => 'Dublin Core',
                                        'type' => 'application/rdf+xml',
-                                       'href' => $wgTitle->getLocalURL( "action=dublincore" ) ) );
+                                       'href' => $wgTitle->getLocalURL( 'action=dublincore' ) ) );
                        }
                }
-               $copyright = "";
+               $copyright = '';
                if( $wgRightsPage ) {
                        $copy = Title::newFromText( $wgRightsPage );
                        if( $copy ) {
@@ -116,15 +167,15 @@ class Skin {
                }
                if( $copyright ) {
                        $out->addLink( array(
-                               "rel" => "copyright",
-                               "href" => $copyright ) );
+                               'rel' => 'copyright',
+                               'href' => $copyright ) );
                }
        }
-    
+
        function outputPage( &$out ) {
                global $wgDebugComments;
-               
-               wfProfileIn( "Skin::outputPage" );
+
+               wfProfileIn( 'Skin::outputPage' );
                $this->initPage( $out );
                $out->out( $out->headElement() );
 
@@ -143,7 +194,7 @@ class Skin {
                $out->out( $out->mBodytext . "\n" );
 
                $out->out( $this->afterContent() );
-               
+
                wfProfileClose();
                $out->out( $out->reportTime() );
 
@@ -151,170 +202,196 @@ class Skin {
        }
 
        function getHeadScripts() {
-               global $wgStylePath;
+               global $wgStylePath, $wgUser, $wgLang, $wgAllowUserJs;
                $r = "<script type=\"text/javascript\" src=\"{$wgStylePath}/wikibits.js\"></script>\n";
+               if( $wgAllowUserJs && $wgUser->getID() != 0 ) { # logged in
+                       $userpage = $wgLang->getNsText( Namespace::getUser() ) . ":" . $wgUser->getName();
+                       $userjs = htmlspecialchars($this->makeUrl($userpage.'/'.$this->getSkinName().'.js', 'action=raw&ctype=text/javascript'));
+                       $r .= '<script type="text/javascript" src="'.$userjs."\"></script>\n";
+               }
                return $r;
        }
 
-       function getUserStyles()
-       {
-               global $wgOut, $wgStylePath, $wgLang;
+       # get the user/site-specific stylesheet, SkinPHPTal called from RawPage.php (settings are cached that way)
+       function getUserStylesheet() {
+               global $wgOut, $wgStylePath, $wgLang, $wgUser, $wgRequest, $wgTitle, $wgAllowUserCss;
                $sheet = $this->getStylesheet();
-               $s = "<style type='text/css'>\n";
-               $s .= "/*/*/\n"; # <-- Hide the styles from Netscape 4 without hiding them from IE/Mac
-               $s .= "@import url(\"$wgStylePath/$sheet\");\n";
-               if($wgLang->isRTL()) $s .= "@import url(\"$wgStylePath/common_rtl.css\");\n";
+               $action = $wgRequest->getText('action');
+               $s = "@import \"$wgStylePath/$sheet\";\n";
+               if($wgLang->isRTL()) $s .= "@import \"$wgStylePath/common_rtl.css\";\n";
+               if( $wgAllowUserCss && $wgUser->getID() != 0 ) { # logged in
+                       if($wgTitle->isCssSubpage() and $action == 'submit' and  $wgTitle->userCanEditCssJsSubpage()) {
+                               $s .= $wgRequest->getText('wpTextbox1');
+                       } else {
+                               $userpage = $wgLang->getNsText( Namespace::getUser() ) . ":" . $wgUser->getName();
+                               $s.= '@import "'.$this->makeUrl($userpage.'/'.$this->getSkinName().'.css', 'action=raw&ctype=text/css').'";'."\n";
+                       }
+               }
                $s .= $this->doGetUserStyles();
-               $s .= "/* */\n";
+               return $s."\n";
+       }
+
+       /**
+        * placeholder, returns generated js in monobook
+        */
+       function getUserJs() { return; }
+
+       /**
+        * Return html code that include User stylesheets
+        */
+       function getUserStyles() {
+               global $wgOut, $wgStylePath, $wgLang;
+               $s = "<style type='text/css'>\n";
+               $s .= "/*/*/ /*<![CDATA[*/\n"; # <-- Hide the styles from Netscape 4 without hiding them from IE/Mac
+               $s .= $this->getUserStylesheet();
+               $s .= "/*]]>*/ /* */\n";
                $s .= "</style>\n";
                return $s;
        }
 
-       function doGetUserStyles()
-       {
-               global $wgUser;
+       /**
+        * Some styles that are set by user through the user settings interface.
+        */
+       function doGetUserStyles() {
+               global $wgUser, $wgLang;
 
-               $s = "";
-               if ( 1 == $wgUser->getOption( "underline" ) ) {
+               $csspage = $wgLang->getNsText( NS_MEDIAWIKI ) . ':' . $this->getSkinName() . '.css';
+               $s = '@import "'.$this->makeUrl($csspage, 'action=raw&ctype=text/css')."\";\n";
+
+               if ( 1 == $wgUser->getOption( 'underline' ) ) {
                        # Don't override browser settings
                } else {
                        # CHECK MERGE @@@
                        # Force no underline
-                       $s .= "a { " .
-                         "text-decoration: none; }\n";
+                       $s .= "a { text-decoration: none; }\n";
                }
-               if ( 1 == $wgUser->getOption( "highlightbroken" ) ) {
+               if ( 1 == $wgUser->getOption( 'highlightbroken' ) ) {
                        $s .= "a.new, #quickbar a.new { color: #CC2200; }\n";
                }
-               if ( 1 == $wgUser->getOption( "justify" ) ) {
+               if ( 1 == $wgUser->getOption( 'justify' ) ) {
                        $s .= "#article { text-align: justify; }\n";
                }
                return $s;
        }
 
-       function getBodyOptions()
-       {
+       function getBodyOptions() {
                global $wgUser, $wgTitle, $wgNamespaceBackgrounds, $wgOut, $wgRequest;
-               
+
                extract( $wgRequest->getValues( 'oldid', 'redirect', 'diff' ) );
 
                if ( 0 != $wgTitle->getNamespace() ) {
-                       $a = array( "bgcolor" => "#ffffec" );
+                       $a = array( 'bgcolor' => '#ffffec' );
                }
-               else $a = array( "bgcolor" => "#FFFFFF" );
-               if($wgOut->isArticle() && $wgUser->getOption("editondblclick") && 
+               else $a = array( 'bgcolor' => '#FFFFFF' );
+               if($wgOut->isArticle() && $wgUser->getOption('editondblclick') &&
                  (!$wgTitle->isProtected() || $wgUser->isSysop()) ) {
-                       $t = wfMsg( "editthispage" );
-                       $oid = $red = "";
-                       if ( !empty($redirect) ) { 
-                               $red = "&redirect={$redirect}"; 
+                       $t = wfMsg( 'editthispage' );
+                       $oid = $red = '';
+                       if ( !empty($redirect) ) {
+                               $red = "&redirect={$redirect}";
                        }
                        if ( !empty($oldid) && ! isset( $diff ) ) {
                                $oid = "&oldid={$oldid}";
                        }
                        $s = $wgTitle->getFullURL( "action=edit{$oid}{$red}" );
-                       $s = "document.location = \"" .$s ."\";";
-                       $a += array ("ondblclick" => $s);
+                       $s = 'document.location = "' .$s .'";';
+                       $a += array ('ondblclick' => $s);
 
                }
                $a['onload'] = $wgOut->getOnloadHandler();
                return $a;
        }
 
-       function getExternalLinkAttributes( $link, $text )
-       {
+       function getExternalLinkAttributes( $link, $text, $class='' ) {
                global $wgUser, $wgOut, $wgLang;
 
                $link = urldecode( $link );
                $link = $wgLang->checkTitleEncoding( $link );
-               $link = str_replace( "_", " ", $link );
-               $link = wfEscapeHTML( $link );
+               $link = str_replace( '_', ' ', $link );
+               $link = htmlspecialchars( $link );
 
-               $r = " class='external'";
+               $r = ($class != '') ? " class='$class'" : " class='external'";
 
-               if ( 1 == $wgUser->getOption( "hover" ) ) {
+               if ( 1 == $wgUser->getOption( 'hover' ) ) {
                        $r .= " title=\"{$link}\"";
                }
                return $r;
        }
 
-       function getInternalLinkAttributes( $link, $text, $broken = false )
-       {
+       function getInternalLinkAttributes( $link, $text, $broken = false ) {
                global $wgUser, $wgOut;
 
                $link = urldecode( $link );
-               $link = str_replace( "_", " ", $link );
-               $link = wfEscapeHTML( $link );
+               $link = str_replace( '_', ' ', $link );
+               $link = htmlspecialchars( $link );
 
-               if ( $broken == "stub" ) { 
-                       $r = " class='stub'"; 
-               } else if ( $broken == "yes" ) { 
-                       $r = " class='new'"; 
-               } else { 
-                       $r = ""; 
+               if ( $broken == 'stub' ) {
+                       $r = ' class="stub"';
+               } else if ( $broken == 'yes' ) {
+                       $r = ' class="new"';
+               } else {
+                       $r = '';
                }
 
-               if ( 1 == $wgUser->getOption( "hover" ) ) {
+               if ( 1 == $wgUser->getOption( 'hover' ) ) {
                        $r .= " title=\"{$link}\"";
                }
                return $r;
        }
-       
-       function getInternalLinkAttributesObj( &$nt, $text, $broken = false )
-       {
+
+       /**
+        * @param bool $broken
+        */
+       function getInternalLinkAttributesObj( &$nt, $text, $broken = false ) {
                global $wgUser, $wgOut;
 
-               if ( $broken == "stub" ) { 
-                       $r = " class='stub'"; 
-               } else if ( $broken == "yes" ) { 
-                       $r = " class='new'"; 
-               } else { 
-                       $r = ""; 
+               if ( $broken == 'stub' ) {
+                       $r = ' class="stub"';
+               } else if ( $broken == 'yes' ) {
+                       $r = ' class="new"';
+               } else {
+                       $r = '';
                }
 
-               if ( 1 == $wgUser->getOption( "hover" ) ) {
+               if ( 1 == $wgUser->getOption( 'hover' ) ) {
                        $r .= ' title ="' . $nt->getEscapedText() . '"';
                }
                return $r;
-       }               
-       
-       function getLogo()
-       {
+       }
+
+       /**
+        * URL to the logo
+        */
+       function getLogo() {
                global $wgLogo;
                return $wgLogo;
        }
 
-       # This will be called immediately after the <body> tag.  Split into
-       # two functions to make it easier to subclass.
-       #
-       function beforeContent()
-       {
-               global $wgUser, $wgOut, $wgSiteNotice;
+       /**
+        * This will be called immediately after the <body> tag.  Split into
+        * two functions to make it easier to subclass.
+        */
+       function beforeContent() {
+               global $wgUser, $wgOut;
 
-               if( $wgSiteNotice ) {
-                       $note = "\n<div id='siteNotice'>$wgSiteNotice</div>\n";
-               } else {
-                       $note = "";
-               }
-               return $this->doBeforeContent() . $note;
+               return $this->doBeforeContent();
        }
 
-       function doBeforeContent()
-       {
-               global $wgUser, $wgOut, $wgTitle, $wgLang;
-               $fname = "Skin::doBeforeContent";
+       function doBeforeContent() {
+               global $wgUser, $wgOut, $wgTitle, $wgLang, $wgSiteNotice;
+               $fname = 'Skin::doBeforeContent';
                wfProfileIn( $fname );
 
-               $s = "";
+               $s = '';
                $qb = $this->qbSetting();
 
                if( $langlinks = $this->otherLanguages() ) {
                        $rows = 2;
-                       $borderhack = "";
+                       $borderhack = '';
                } else {
                        $rows = 1;
                        $langlinks = false;
-                       $borderhack = "class='top'";
+                       $borderhack = 'class="top"';
                }
 
                $s .= "\n<div id='content'>\n<div id='topbar'>\n" .
@@ -323,14 +400,14 @@ class Skin {
                $shove = ($qb != 0);
                $left = ($qb == 1 || $qb == 3);
                if($wgLang->isRTL()) $left = !$left;
-               
+
                if ( !$shove ) {
                        $s .= "<td class='top' align='left' valign='top' rowspan='{$rows}'>\n" .
-                         $this->logoText() . "</td>";
+                         $this->logoText() . '</td>';
                } elseif( $left ) {
                        $s .= $this->getQuickbarCompensator( $rows );
                }
-               $l = $wgLang->isRTL() ? "right" : "left";
+               $l = $wgLang->isRTL() ? 'right' : 'left';
                $s .= "<td {$borderhack} align='$l' valign='top'>\n";
 
                $s .= $this->topLinks() ;
@@ -351,67 +428,110 @@ class Skin {
                $s .= "</tr>\n</table>\n</div>\n";
                $s .= "\n<div id='article'>\n";
 
+               if( $wgSiteNotice ) {
+                       $s .= "\n<div id='siteNotice'>$wgSiteNotice</div>\n";
+               }
                $s .= $this->pageTitle();
                $s .= $this->pageSubtitle() ;
                $s .= $this->getCategories();
                wfProfileOut( $fname );
                return $s;
        }
-       
+
        function getCategoryLinks () {
                global $wgOut, $wgTitle, $wgUser, $wgParser;
-               global $wgUseCategoryMagic;
-               if( !$wgUseCategoryMagic ) return "" ;
-               if( count( $wgOut->mCategoryLinks ) == 0 ) return "";
-               if( !$wgOut->isArticle() ) return "";
-               
-               $t = implode ( " | " , $wgOut->mCategoryLinks ) ;
-               $s = $this->makeKnownLink( "Special:Categories",
-                       wfMsg( "categories" ), "article=" . urlencode( $wgTitle->getPrefixedDBkey() ) )
-                       . ": " . $t;
+               global $wgUseCategoryMagic, $wgUseCategoryBrowser, $wgLang;
+
+               if( !$wgUseCategoryMagic ) return '' ;
+               if( count( $wgOut->mCategoryLinks ) == 0 ) return '';
+
+               # Taken out so that they will be displayed in previews -- TS
+               #if( !$wgOut->isArticle() ) return '';
+
+               $t = implode ( ' | ' , $wgOut->mCategoryLinks ) ;
+               $s = $this->makeKnownLink( 'Special:Categories',
+                       wfMsg( 'categories' ), 'article=' . urlencode( $wgTitle->getPrefixedDBkey() ) )
+                       . ': ' . $t;
+
+               # optional 'dmoz-like' category browser. Will be shown under the list
+               # of categories an article belong to
+               if($wgUseCategoryBrowser) {
+                       $s .= '<br/><hr/>';
+
+                       # get a big array of the parents tree
+                       $parenttree = $wgTitle->getCategorieBrowser();
+
+                       # Render the array as a serie of links
+                       function walkThrough ($tree) {
+                               global $wgUser;
+                               $sk = $wgUser->getSkin();
+                               $return = '';
+                               foreach($tree as $element => $parent) {
+                                       if(empty($parent)) {
+                                               # element start a new list
+                                               $return .= '<br />';
+                                       } else {
+                                               # grab the others elements
+                                               $return .= walkThrough($parent);
+                                       }
+                                       # add our current element to the list
+                                       $eltitle = Title::NewFromText($element);
+                                       # FIXME : should be makeLink() [AV]
+                                       $return .= $sk->makeKnownLink($element, $eltitle->getText()).' &gt; ';
+                               }
+                               return $return;
+                       }
+
+                       $s .= walkThrough($parenttree);
+               }
+
                return $s;
        }
-       
+
        function getCategories() {
-               return "<p class='catlinks'>" . $this->getCategoryLinks() . "</p>";
+               $catlinks=$this->getCategoryLinks();
+               if(!empty($catlinks)) {
+                       return "<p class='catlinks'>{$catlinks}</p>";
+               }
        }
 
-       function getQuickbarCompensator( $rows = 1 )
-       {
+       function getQuickbarCompensator( $rows = 1 ) {
                return "<td width='152' rowspan='{$rows}'>&nbsp;</td>";
        }
 
        # This gets called immediately before the </body> tag.
        #
-       function afterContent()
-       {
+       function afterContent() {
                global $wgUser, $wgOut, $wgServer;
                global $wgTitle, $wgLang;
-               
+
                $printfooter = "<div class=\"printfooter\">\n" . $this->printFooter() . "</div>\n";
                return $printfooter . $this->doAfterContent();
        }
-       
-       function printFooter() {
+
+       function printSource() {
                global $wgTitle;
                $url = htmlspecialchars( $wgTitle->getFullURL() );
-               return "<p>" . wfMsg( "retrievedfrom", "<a href=\"$url\">$url</a>" ) .
+               return wfMsg( "retrievedfrom", "<a href=\"$url\">$url</a>" );
+       }
+
+       function printFooter() {
+               return "<p>" .  $this->printSource() .
                        "</p>\n\n<p>" . $this->pageStats() . "</p>\n";
        }
-       
-       function doAfterContent()
-       {
+
+       function doAfterContent() {
                global $wgUser, $wgOut, $wgLang;
-               $fname =  "Skin::doAfterContent";
+               $fname =  'Skin::doAfterContent';
                wfProfileIn( $fname );
-               wfProfileIn( "$fname-1" );
+               wfProfileIn( $fname.'-1' );
 
                $s = "\n</div><br style=\"clear:both\" />\n";
                $s .= "\n<div id='footer'>";
-               $s .= "<table border='0' cellspacing='0'><tr>";
-               
-               wfProfileOut( "$fname-1" );
-               wfProfileIn( "$fname-2" );
+               $s .= '<table border="0" cellspacing="0"><tr>';
+
+               wfProfileOut( $fname.'-1' );
+               wfProfileIn( $fname.'-2' );
 
                $qb = $this->qbSetting();
                $shove = ($qb != 0);
@@ -421,46 +541,45 @@ class Skin {
                if ( $shove && $left ) { # Left
                        $s .= $this->getQuickbarCompensator();
                }
-               wfProfileOut( "$fname-2" );
-               wfProfileIn( "$fname-3" );
-               $l = $wgLang->isRTL() ? "right" : "left";
+               wfProfileOut( $fname.'-2' );
+               wfProfileIn( $fname.'-3' );
+               $l = $wgLang->isRTL() ? 'right' : 'left';
                $s .= "<td class='bottom' align='$l' valign='top'>";
 
                $s .= $this->bottomLinks();
                $s .= "\n<br />" . $this->mainPageLink()
-                 . " | " . $this->aboutLink()
-                 . " | " . $this->specialLink( "recentchanges" )
-                 . " | " . $this->searchForm()
-                 . "<br /><span id='pagestats'>" . $this->pageStats() . "</span>";
+                 . ' | ' . $this->aboutLink()
+                 . ' | ' . $this->specialLink( 'recentchanges' )
+                 . ' | ' . $this->searchForm()
+                 . '<br /><span id="pagestats">' . $this->pageStats() . '</span>';
 
                $s .= "</td>";
                if ( $shove && !$left ) { # Right
                        $s .= $this->getQuickbarCompensator();
                }
                $s .= "</tr></table>\n</div>\n</div>\n";
-               
-               wfProfileOut( "$fname-3" );
-               wfProfileIn( "$fname-4" );
+
+               wfProfileOut( $fname.'-3' );
+               wfProfileIn( $fname.'-4' );
                if ( 0 != $qb ) { $s .= $this->quickBar(); }
-               wfProfileOut( "$fname-4" );
+               wfProfileOut( $fname.'-4' );
                wfProfileOut( $fname );
                return $s;
        }
 
-       function pageTitleLinks()
-       {
+       function pageTitleLinks() {
                global $wgOut, $wgTitle, $wgUser, $wgLang, $wgUseApproval, $wgRequest;
 
                extract( $wgRequest->getValues( 'oldid', 'diff' ) );
                $action = $wgRequest->getText( 'action' );
 
                $s = $this->printableLink();
-               if ( wfMsg ( "disclaimers" ) != "-" ) $s .= " | " . $this->makeKnownLink( wfMsg( "disclaimerpage" ), wfMsg( "disclaimers" ) ) ;
+               if ( wfMsg ( 'disclaimers' ) != '-' ) $s .= ' | ' . $this->makeKnownLink( wfMsg( 'disclaimerpage' ), wfMsg( 'disclaimers' ) ) ;
 
                if ( $wgOut->isArticleRelated() ) {
                        if ( $wgTitle->getNamespace() == Namespace::getImage() ) {
                                $name = $wgTitle->getDBkey();
-                               $link = wfEscapeHTML( Image::wfImageUrl( $name ) );
+                               $link = htmlspecialchars( Image::wfImageUrl( $name ) );
                                $style = $this->getInternalLinkAttributes( $link, $name );
                                $s .= " | <a href=\"{$link}\"{$style}>{$name}</a>";
                        }
@@ -468,56 +587,67 @@ class Skin {
                        if ( isset ( $wgUseApproval ) && $wgUseApproval )
                        {
                                $t = $wgTitle->getDBkey();
-                               $name = "Approve this article" ; 
+                               $name = 'Approve this article' ;
                                $link = "http://test.wikipedia.org/w/magnus/wiki.phtml?title={$t}&action=submit&doit=1" ;
-                               #wfEscapeHTML( wfImageUrl( $name ) );
+                               #htmlspecialchars( wfImageUrl( $name ) );
                                $style = $this->getExternalLinkAttributes( $link, $name );
                                $s .= " | <a href=\"{$link}\"{$style}>{$name}</a>" ;
-                       }               
+                       }
                }
-               if ( "history" == $action || isset( $diff ) || isset( $oldid ) ) {
-                       $s .= " | " . $this->makeKnownLink( $wgTitle->getPrefixedText(),
-                                       wfMsg( "currentrev" ) );
+               if ( 'history' == $action || isset( $diff ) || isset( $oldid ) ) {
+                       $s .= ' | ' . $this->makeKnownLink( $wgTitle->getPrefixedText(),
+                                       wfMsg( 'currentrev' ) );
                }
 
                if ( $wgUser->getNewtalk() ) {
-               # do not show "You have new messages" text when we are viewing our 
-               # own talk page 
+               # do not show "You have new messages" text when we are viewing our
+               # own talk page
 
                        if(!(strcmp($wgTitle->getText(),$wgUser->getName()) == 0 &&
                                                $wgTitle->getNamespace()==Namespace::getTalk(Namespace::getUser()))) {
                                $n =$wgUser->getName();
                                $tl = $this->makeKnownLink( $wgLang->getNsText(
                                                        Namespace::getTalk( Namespace::getUser() ) ) . ":{$n}",
-                                               wfMsg("newmessageslink") );
-                               $s.=" | <strong>". wfMsg( "newmessages", $tl ) . "</strong>";
+                                               wfMsg('newmessageslink') );
+                               $s.= ' | <strong>'. wfMsg( 'newmessages', $tl ) . '</strong>';
+                               # disable caching
+                               $wgOut->setSquidMaxage(0);
                        }
                }
-               if( $wgUser->isSysop() &&
-                               (($wgTitle->getArticleId() == 0) || ($action == "history")) &&
-                               ($n = $wgTitle->isDeleted() ) ) {
-                       $s .= " | " . wfMsg( "thisisdeleted",
-                                       $this->makeKnownLink(
-                                               $wgLang->SpecialPage( "Undelete/" . $wgTitle->getPrefixedDBkey() ),
-                                               wfMsg( "restorelink", $n ) ) );
+
+               $undelete = $this->getUndeleteLink();
+               if( !empty( $undelete ) ) {
+                       $s .= ' | '.$undelete;
                }
                return $s;
        }
 
-       function printableLink()
-       {
+       function getUndeleteLink() {
+               global $wgUser, $wgTitle, $wgLang, $action;
+               if( $wgUser->isSysop() &&
+                       (($wgTitle->getArticleId() == 0) || ($action == "history")) &&
+                       ($n = $wgTitle->isDeleted() ) ) {
+                       return wfMsg( 'thisisdeleted',
+                               $this->makeKnownLink(
+                                       $wgLang->SpecialPage( 'Undelete/' . $wgTitle->getPrefixedDBkey() ),
+                                       wfMsg( 'restorelink', $n ) ) );
+               }
+               return '';
+       }
+
+       function printableLink() {
                global $wgOut, $wgFeedClasses, $wgRequest;
 
                $baseurl = $_SERVER['REQUEST_URI'];
-               if( strpos( "?", $baseurl ) == false ) {
-                       $baseurl .= "?";
+               if( strpos( '?', $baseurl ) == false ) {
+                       $baseurl .= '?';
                } else {
-                       $baseurl .= "&";
+                       $baseurl .= '&';
                }
                $baseurl = htmlspecialchars( $baseurl );
-               $printurl = $wgRequest->escapeAppendQuery( "printable=yes" );
-               
-               $s = "<a href=\"$printurl\">" . wfMsg( "printableversion" ) . "</a>";
+               $printurl = $wgRequest->escapeAppendQuery( 'printable=yes' );
+
+               $s = "<a href=\"$printurl\">" . wfMsg( 'printableversion' ) . '</a>';
                if( $wgOut->isSyndicated() ) {
                        foreach( $wgFeedClasses as $format => $class ) {
                                $feedurl = $wgRequest->escapeAppendQuery( "feed=$format" );
@@ -527,23 +657,21 @@ class Skin {
                return $s;
        }
 
-       function pageTitle()
-       {
+       function pageTitle() {
                global $wgOut, $wgTitle, $wgUser;
 
-               $s = "<h1 class='pagetitle'>" . htmlspecialchars( $wgOut->getPageTitle() ) . "</h1>";
-               if($wgUser->getOption("editsectiononrightclick") && $wgTitle->userCanEdit()) { $s=$this->editSectionScript(0,$s);}
+               $s = '<h1 class="pagetitle">' . htmlspecialchars( $wgOut->getPageTitle() ) . '</h1>';
+               if($wgUser->getOption( 'editsectiononrightclick' ) && $wgTitle->userCanEdit()) { $s=$this->editSectionScript(0,$s);}
                return $s;
        }
 
-       function pageSubtitle()
-       {
+       function pageSubtitle() {
                global $wgOut;
 
                $sub = $wgOut->getSubtitle();
-               if ( "" == $sub ) {
+               if ( '' == $sub ) {
                        global $wgExtraSubtitle;
-                       $sub = wfMsg( "fromwikipedia" ) . $wgExtraSubtitle;
+                       $sub = wfMsg( 'fromwikipedia' ) . $wgExtraSubtitle;
                }
                $subpages = $this->subPageSubtitle();
                $sub .= !empty($subpages)?"</p><p class='subpages'>$subpages":'';
@@ -551,29 +679,28 @@ class Skin {
                return $s;
        }
 
-       function subPageSubtitle()
-       {
+       function subPageSubtitle() {
                global $wgOut,$wgTitle,$wgNamespacesWithSubpages;
                $subpages = '';
                if($wgOut->isArticle() && !empty($wgNamespacesWithSubpages[$wgTitle->getNamespace()])) {
                        $ptext=$wgTitle->getPrefixedText();
-                       if(preg_match("/\//",$ptext)) {
-                               $links=explode("/",$ptext);
-                               $c=0;
-                               $growinglink="";
+                       if(preg_match('/\//',$ptext)) {
+                               $links = explode('/',$ptext);
+                               $c = 0;
+                               $growinglink = '';
                                foreach($links as $link) {
                                        $c++;
                                        if ($c<count($links)) {
                                                $growinglink .= $link;
                                                $getlink = $this->makeLink( $growinglink, $link );
-                                               if(preg_match("/class='new'/i",$getlink)) { break; } # this is a hack, but it saves time
+                                               if(preg_match('/class="new"/i',$getlink)) { break; } # this is a hack, but it saves time
                                                if ($c>1) {
-                                                       $subpages .= " | ";
+                                                       $subpages .= ' | ';
                                                } else  {
-                                                       $subpages .="&lt; ";
+                                                       $subpages .= '&lt; ';
                                                }
                                                $subpages .= $getlink;
-                                               $growinglink.="/";
+                                               $growinglink .= '/';
                                        }
                                }
                        }
@@ -581,34 +708,33 @@ class Skin {
                return $subpages;
        }
 
-       function nameAndLogin()
-       {
+       function nameAndLogin() {
                global $wgUser, $wgTitle, $wgLang, $wgShowIPinHeader, $wgIP;
 
-               $li = $wgLang->specialPage( "Userlogin" );
-               $lo = $wgLang->specialPage( "Userlogout" );
+               $li = $wgLang->specialPage( 'Userlogin' );
+               $lo = $wgLang->specialPage( 'Userlogout' );
 
-               $s = "";
+               $s = '';
                if ( 0 == $wgUser->getID() ) {
-                       if( $wgShowIPinHeader && isset(  $_COOKIE[ini_get("session.name")] ) ) {
+                       if( $wgShowIPinHeader && isset(  $_COOKIE[ini_get('session.name')] ) ) {
                                $n = $wgIP;
 
                                $tl = $this->makeKnownLink( $wgLang->getNsText(
                                  Namespace::getTalk( Namespace::getUser() ) ) . ":{$n}",
                                  $wgLang->getNsText( Namespace::getTalk( 0 ) ) );
-                         
-                               $s .= $n .  " (".$tl.")";
+
+                               $s .= $n . ' ('.$tl.')';
                        } else {
-                               $s .= wfMsg("notloggedin");
+                               $s .= wfMsg('notloggedin');
                        }
-                       
+
                        $rt = $wgTitle->getPrefixedURL();
                        if ( 0 == strcasecmp( urlencode( $lo ), $rt ) ) {
-                               $q = "";
+                               $q = '';
                        } else { $q = "returnto={$rt}"; }
-                       
+
                        $s .= "\n<br />" . $this->makeKnownLink( $li,
-                         wfMsg( "login" ), $q );
+                         wfMsg( 'login' ), $q );
                } else {
                        $n = $wgUser->getName();
                        $rt = $wgTitle->getPrefixedURL();
@@ -616,51 +742,49 @@ class Skin {
                          Namespace::getTalk( Namespace::getUser() ) ) . ":{$n}",
                          $wgLang->getNsText( Namespace::getTalk( 0 ) ) );
 
-                       $tl = " ({$tl})"; 
-                       
+                       $tl = " ({$tl})";
+
                        $s .= $this->makeKnownLink( $wgLang->getNsText(
                          Namespace::getUser() ) . ":{$n}", $n ) . "{$tl}<br />" .
-                         $this->makeKnownLink( $lo, wfMsg( "logout" ),
-                         "returnto={$rt}" ) . " | " .
-                         $this->specialLink( "preferences" );
+                         $this->makeKnownLink( $lo, wfMsg( 'logout' ),
+                         "returnto={$rt}" ) . ' | ' .
+                         $this->specialLink( 'preferences' );
                }
-               $s .= " | " . $this->makeKnownLink( wfMsg( "helppage" ),
-                 wfMsg( "help" ) ); 
+               $s .= ' | ' . $this->makeKnownLink( wfMsg( 'helppage' ),
+                 wfMsg( 'help' ) );
 
                return $s;
        }
-       
+
        function getSearchLink() {
-               $searchPage =& Title::makeTitle( NS_SPECIAL, "Search" );
+               $searchPage =& Title::makeTitle( NS_SPECIAL, 'Search' );
                return $searchPage->getLocalURL();
        }
-       
+
        function escapeSearchLink() {
                return htmlspecialchars( $this->getSearchLink() );
        }
-       
-       function searchForm()
-       {
+
+       function searchForm() {
                global $wgRequest;
                $search = $wgRequest->getText( 'search' );
-               
-               $s = "<form name='search' class='inline' method='post' action=\""
+
+               $s = '<form name="search" class="inline" method="post" action="'
                  . $this->escapeSearchLink() . "\">\n"
-                 . "<input type='text' name=\"search\" size='19' value=\""
+                 . '<input type="text" name="search" size="19" value="'
                  . htmlspecialchars(substr($search,0,256)) . "\" />\n"
-                 . "<input type='submit' name=\"go\" value=\"" . wfMsg ("go") . "\" />&nbsp;"
-                 . "<input type='submit' name=\"fulltext\" value=\"" . wfMsg ("search") . "\" />\n</form>";
+                 . '<input type="submit" name="go" value="' . wfMsg ('go') . '" />&nbsp;'
+                 . '<input type="submit" name="fulltext" value="' . wfMsg ('search') . "\" />\n</form>";
 
                return $s;
        }
 
-       function topLinks()
-       {
+       function topLinks() {
                global $wgOut;
                $sep = " |\n";
 
                $s = $this->mainPageLink() . $sep
-                 . $this->specialLink( "recentchanges" );
+                 . $this->specialLink( 'recentchanges' );
 
                if ( $wgOut->isArticleRelated() ) {
                        $s .=  $sep . $this->editThisPage()
@@ -672,14 +796,13 @@ class Skin {
                return $s;
        }
 
-       function bottomLinks()
-       { 
+       function bottomLinks() {
                global $wgOut, $wgUser, $wgTitle;
                $sep = " |\n";
 
-               $s = "";
+               $s = '';
                if ( $wgOut->isArticleRelated() ) {
-                       $s .= "<strong>" . $this->editThisPage() . "</strong>";
+                       $s .= '<strong>' . $this->editThisPage() . '</strong>';
                        if ( 0 != $wgUser->getID() ) {
                                $s .= $sep . $this->watchThisPage();
                        }
@@ -690,11 +813,11 @@ class Skin {
 
                        if ( $wgTitle->getNamespace() == Namespace::getUser()
                            || $wgTitle->getNamespace() == Namespace::getTalk(Namespace::getUser()) )
-                           
+
                        {
                                $id=User::idFromName($wgTitle->getText());
                                $ip=User::isIP($wgTitle->getText());
-                               
+
                                if($id || $ip) { # both anons and non-anons have contri list
                                        $s .= $sep . $this->userContribsLink();
                                }
@@ -714,115 +837,47 @@ class Skin {
                return $s;
        }
 
-       function pageStats()
-       {
+       function pageStats() {
                global $wgOut, $wgLang, $wgArticle, $wgRequest;
-               global $wgDisableCounters;
+               global $wgDisableCounters, $wgMaxCredits, $wgShowCreditsIfMax;
 
                extract( $wgRequest->getValues( 'oldid', 'diff' ) );
-               if ( ! $wgOut->isArticle() ) { return ""; }
-               if ( isset( $oldid ) || isset( $diff ) ) { return ""; }
-               if ( 0 == $wgArticle->getID() ) { return ""; }
+               if ( ! $wgOut->isArticle() ) { return ''; }
+               if ( isset( $oldid ) || isset( $diff ) ) { return ''; }
+               if ( 0 == $wgArticle->getID() ) { return ''; }
 
-               $s = "";
+               $s = '';
                if ( !$wgDisableCounters ) {
                        $count = $wgLang->formatNum( $wgArticle->getCount() );
                        if ( $count ) {
-                               $s = wfMsg( "viewcount", $count );
+                               $s = wfMsg( 'viewcount', $count );
                        }
                }
 
-               $s .= " " . $this->getCredits();
-           
-               return $s . " " .  $this->getCopyright();
-       }
-
-        function getCredits() {
-              global $wgMaxCredits;
-              
-              $s = '';
-           
-              if (!isset($wgMaxCredits) || $wgMaxCredits == 0) {
-                       $s = $this->lastModified();
-              } else {
-                       $s = $this->getAuthorCredits();
-                       if ($wgMaxCredits > 1) {
-                           $s .= " " . $this->getContributorCredits();
-                       }
-              }
-           
-              return $s;
-       }
-
-        function getAuthorCredits() {
-               global $wgLang, $wgArticle;
-
-               $last_author = $wgArticle->getUser();
-           
-               if ($last_author == 0) {
-                   $author_credit = wfMsg("anonymous");
+               if (isset($wgMaxCredits) && $wgMaxCredits != 0) {
+                   require_once("Credits.php");
+                   $s .= ' ' . getCredits($wgArticle, $wgMaxCredits, $wgShowCreditsIfMax);
                } else {
-                   $real_name = User::whoIsReal($last_author);
-                   if (!empty($real_name)) {
-                       $author_credit = $real_name;
-                   } else {
-                       $author_credit = wfMsg("siteuser", User::whoIs($last_author));
-                   }
-               }
-           
-               $timestamp = $wgArticle->getTimestamp();
-               if ( $timestamp ) {
-                       $d = $wgLang->timeanddate( $wgArticle->getTimestamp(), true );
-               } else {
-                       $d = "";
+                   $s .= $this->lastModified();
                }
-               return wfMsg("lastmodifiedby", $d, $author_credit);
-       }
 
-        function getContributorCredits() {
-           
-               global $wgArticle, $wgMaxCredits, $wgLang;
+               return $s . ' ' .  $this->getCopyright();
+       }
 
-                # don't count last editor
+       function getCopyright() {
+               global $wgRightsPage, $wgRightsUrl, $wgRightsText, $wgRequest;
 
-               $contributors = $wgArticle->getContributors($wgMaxCredits - 1);
-           
-               $real_names = array();
-               $user_names = array();
 
-               # Sift for real versus user names
-               
-               foreach ($contributors as $user_id => $user_parts) {
-                   if ($user_id != 0) {
-                       if (!empty($user_parts[1])) {
-                           $real_names[$user_id] = $user_parts[1];
-                       } else {
-                           $user_names[$user_id] = $user_parts[0];
-                       }
-                   }
-               }
-           
-                $real = $wgLang->listToText(array_values($real_names));
-               $user = $wgLang->listToText(array_values($user_names));
+               $oldid = $wgRequest->getVal( 'oldid' );
+               $diff = $wgRequest->getVal( 'diff' );
 
-               if (!empty($user)) {
-                   $user = wfMsg("siteusers", $user);
-               }
-           
-               if ($contributors[0] && $contributors[0][0] > 0) {
-                   $anon = wfMsg("anonymous");
+               if ( !is_null( $oldid ) && is_null( $diff ) && wfMsg( 'history_copyright' ) !== '-' ) {
+                       $msg = 'history_copyright';
                } else {
-                   $anon = '';
+                       $msg = 'copyright';
                }
-           
-               $creds = $wgLang->listToText(array($real, $user, $anon));
-           
-               return wfMsg("othercontribs", $creds);
-       }
-    
-       function getCopyright() {
-               global $wgRightsPage, $wgRightsUrl, $wgRightsText;
-               $out = "";
+
+               $out = '';
                if( $wgRightsPage ) {
                        $link = $this->makeKnownLink( $wgRightsPage, $wgRightsText );
                } elseif( $wgRightsUrl ) {
@@ -831,68 +886,70 @@ class Skin {
                        # Give up now
                        return $out;
                }
-               $out .= wfMsg( "copyright", $link );
+               $out .= wfMsg( $msg, $link );
                return $out;
        }
-       
+
        function getCopyrightIcon() {
                global $wgRightsPage, $wgRightsUrl, $wgRightsText, $wgRightsIcon;
-               $out = "";
+               $out = '';
                if( $wgRightsIcon ) {
                        $icon = htmlspecialchars( $wgRightsIcon );
                        if( $wgRightsUrl ) {
                                $url = htmlspecialchars( $wgRightsUrl );
-                               $out .= "<a href=\"$url\">";
+                               $out .= '<a href="'.$url.'">';
                        }
                        $text = htmlspecialchars( $wgRightsText );
                        $out .= "<img src=\"$icon\" alt='$text' />";
                        if( $wgRightsUrl ) {
-                               $out .= "</a>";
+                               $out .= '</a>';
                        }
                }
                return $out;
        }
-       
+
        function getPoweredBy() {
                global $wgStylePath;
                $url = htmlspecialchars( "$wgStylePath/images/poweredby_mediawiki_88x31.png" );
-               $img = "<a href='http://www.mediawiki.org/'><img src='$url' alt='MediaWiki' /></a>";
+               $img = '<a href="http://www.mediawiki.org/"><img src="'.$url.'" alt="MediaWiki" /></a>';
                return $img;
        }
 
-       function lastModified()
-       {
+       function lastModified() {
                global $wgLang, $wgArticle;
-               
+
                $timestamp = $wgArticle->getTimestamp();
                if ( $timestamp ) {
                        $d = $wgLang->timeanddate( $wgArticle->getTimestamp(), true );
-                       $s = " " . wfMsg( "lastmodified", $d );
+                       $s = ' ' . wfMsg( 'lastmodified', $d );
                } else {
-                       $s = "";
+                       $s = '';
                }
                return $s;
        }
 
-       function logoText( $align = "" )
-       {
-               if ( "" != $align ) { $a = " align='{$align}'"; }
-               else { $a = ""; }
-               
-               $mp = wfMsg( "mainpage" );
+       function logoText( $align = '' ) {
+               if ( '' != $align ) { $a = " align='{$align}'"; }
+               else { $a = ''; }
+
+               $mp = wfMsg( 'mainpage' );
                $titleObj = Title::newFromText( $mp );
-               $s = "<a href=\"" . $titleObj->escapeLocalURL()
-                 . "\"><img{$a} src=\""
-                 . $this->getLogo() . "\" alt=\"" . "[{$mp}]\" /></a>";
+               if ( is_object( $titleObj ) ) {
+                       $url = $titleObj->escapeLocalURL();
+               } else {
+                       $url = '';
+               }
+
+               $logourl = $this->getLogo();
+               $s = "<a href='{$url}'><img{$a} src='{$logourl}' alt='[{$mp}]' /></a>";
                return $s;
        }
 
-       function quickBar()
-       {
+       function quickBar() {
                global $wgOut, $wgTitle, $wgUser, $wgRequest, $wgLang;
                global $wgDisableUploads, $wgRemoteUploads;
-           
-               $fname =  "Skin::quickBar";
+
+               $fname =  'Skin::quickBar';
                wfProfileIn( $fname );
 
                $action = $wgRequest->getText( 'action' );
@@ -904,69 +961,69 @@ class Skin {
 
                $sep = "\n<br />";
                $s .= $this->mainPageLink()
-                 . $sep . $this->specialLink( "recentchanges" )
-                 . $sep . $this->specialLink( "randompage" );
-               if ($wgUser->getID()) { 
-               $s.= $sep . $this->specialLink( "watchlist" ) ; 
-               $s .= $sep .$this->makeKnownLink( $wgLang->specialPage( "Contributions" ),
-                 wfMsg( "mycontris" ), "target=" . wfUrlencode($wgUser->getName() ) );         
-               
+                 . $sep . $this->specialLink( 'recentchanges' )
+                 . $sep . $this->specialLink( 'randompage' );
+               if ($wgUser->getID()) {
+               $s.= $sep . $this->specialLink( 'watchlist' ) ;
+               $s .= $sep .$this->makeKnownLink( $wgLang->specialPage( 'Contributions' ),
+                 wfMsg( 'mycontris' ), 'target=' . wfUrlencode($wgUser->getName() ) );
+
                }
                // only show watchlist link if logged in
-               if ( wfMsg ( "currentevents" ) != "-" ) $s .= $sep . $this->makeKnownLink( wfMsg( "currentevents" ), "" ) ;
+               if ( wfMsg ( 'currentevents' ) != '-' ) $s .= $sep . $this->makeKnownLink( wfMsg( 'currentevents' ), '' ) ;
                $s .= "\n<br /><hr class='sep' />";
                $articleExists = $wgTitle->getArticleId();
-               if ( $wgOut->isArticle() || $action =="edit" || $action =="history" || $wpPreview) {                            
+               if ( $wgOut->isArticle() || $action =='edit' || $action =='history' || $wpPreview) {
                        if($wgOut->isArticle()) {
-                               $s .= "<strong>" . $this->editThisPage() . "</strong>";
+                               $s .= '<strong>' . $this->editThisPage() . '</strong>';
                        } else { # backlink to the article in edit or history mode
                                if($articleExists){ # no backlink if no article
                                        switch($tns) {
                                                case 0:
-                                               $text = wfMsg("articlepage");
+                                               $text = wfMsg('articlepage');
                                                break;
                                                case 1:
-                                               $text = wfMsg("viewtalkpage");
+                                               $text = wfMsg('viewtalkpage');
                                                break;
                                                case 2:
-                                               $text = wfMsg("userpage");                              
+                                               $text = wfMsg('userpage');
                                                break;
                                                case 3:
-                                               $text = wfMsg("viewtalkpage");
-                                               break;  
-                                               case 4: 
-                                               $text = wfMsg("wikipediapage");
+                                               $text = wfMsg('viewtalkpage');
                                                break;
-                                               case 5:                         
-                                               $text = wfMsg("viewtalkpage");
+                                               case 4:
+                                               $text = wfMsg('wikipediapage');
+                                               break;
+                                               case 5:
+                                               $text = wfMsg('viewtalkpage');
                                                break;
                                                case 6:
-                                               $text = wfMsg("imagepage");
+                                               $text = wfMsg('imagepage');
                                                break;
                                                case 7:
-                                               $text = wfMsg("viewtalkpage");
+                                               $text = wfMsg('viewtalkpage');
                                                break;
                                                default:
-                                               $text= wfMsg("articlepage");
+                                               $text= wfMsg('articlepage');
                                        }
-                               
+
                                        $link = $wgTitle->getText();
                                        if ($nstext = $wgLang->getNsText($tns) ) { # add namespace if necessary
-                                               $link = $nstext . ":" . $link ;
-                                       }                       
+                                               $link = $nstext . ':' . $link ;
+                                       }
 
                                        $s .= $this->makeLink( $link, $text );
                                } elseif( $wgTitle->getNamespace() != Namespace::getSpecial() ) {
                                        # we just throw in a "New page" text to tell the user that he's in edit mode,
                                        # and to avoid messing with the separator that is prepended to the next item
-                                       $s .= "<strong>" . wfMsg("newpage") . "</strong>";
+                                       $s .= '<strong>' . wfMsg('newpage') . '</strong>';
                                }
-                       
+
                        }
-                       
 
-                       if( $tns%2 && $action!="edit" && !$wpPreview) {
-                               $s.="<br />".$this->makeKnownLink($wgTitle->getPrefixedText(),wfMsg("postcomment"),"action=edit&section=new");
+
+                       if( $tns%2 && $action!='edit' && !$wpPreview) {
+                               $s.= '<br />'.$this->makeKnownLink($wgTitle->getPrefixedText(),wfMsg('postcomment'),'action=edit&section=new');
                        }
 
                        /*
@@ -976,7 +1033,7 @@ class Skin {
                        unwatched. Therefore we do not show the "Watch this page" link in edit mode
                        */
                        if ( 0 != $wgUser->getID() && $articleExists) {
-                               if($action!="edit" && $action != "submit" )
+                               if($action!='edit' && $action != 'submit' )
                                {
                                        $s .= $sep . $this->watchThisPage();
                                }
@@ -988,78 +1045,78 @@ class Skin {
                                $sep . $this->protectThisPage();
                        }
                        $s .= $sep . $this->talkLink();
-                       if ($articleExists && $action !="history") {
+                       if ($articleExists && $action !='history') {
                                $s .= $sep . $this->historyLink();
                        }
                        $s.=$sep . $this->whatLinksHere();
-                       
+
                        if($wgOut->isArticleRelated()) {
                                $s .= $sep . $this->watchPageLinksLink();
                        }
 
-                       if ( Namespace::getUser() == $wgTitle->getNamespace() 
+                       if ( Namespace::getUser() == $wgTitle->getNamespace()
                        || $wgTitle->getNamespace() == Namespace::getTalk(Namespace::getUser())
                        ) {
-                       
+
                                $id=User::idFromName($wgTitle->getText());
                                $ip=User::isIP($wgTitle->getText());
-                               
+
                                if($id||$ip) {
                                        $s .= $sep . $this->userContribsLink();
                                }
                                if ( 0 != $wgUser->getID() ) {
                                        if($id) { # can only email real users
-                                               $s .= $sep . $this->emailUserLink(); 
+                                               $s .= $sep . $this->emailUserLink();
                                        }
                                }
                        }
                        $s .= "\n<br /><hr class='sep' />";
-               } 
-               
+               }
+
                if ( 0 != $wgUser->getID() && ( !$wgDisableUploads || $wgRemoteUploads ) ) {
-                       $s .= $this->specialLink( "upload" ) . $sep;
+                       $s .= $this->specialLink( 'upload' ) . $sep;
                }
-               $s .= $this->specialLink( "specialpages" )
+               $s .= $this->specialLink( 'specialpages' )
                  . $sep . $this->bugReportsLink();
-               
+
                global $wgSiteSupportPage;
                if( $wgSiteSupportPage ) {
                        $s .= "\n<br /><a href=\"" . htmlspecialchars( $wgSiteSupportPage ) .
-                         "\" class=\"internal\">" . wfMsg( "sitesupport" ) . "</a>";
+                         '" class="internal">' . wfMsg( 'sitesupport' ) . '</a>';
                }
-       
+
                $s .= "\n<br /></div>\n";
                wfProfileOut( $fname );
                return $s;
        }
 
-       function specialPagesList()
-       {
+       function specialPagesList() {
                global $wgUser, $wgOut, $wgLang, $wgServer, $wgRedirectScript;
+               require_once('SpecialPage.php');
                $a = array();
                $pages = SpecialPage::getPages();
-               
-               foreach ( $pages[""] as $name => $page ) {
+
+               foreach ( $pages[''] as $name => $page ) {
                        $a[$name] = $page->getDescription();
                }
                if ( $wgUser->isSysop() )
-               { 
-                       foreach ( $pages["sysop"] as $name => $page ) {
+               {
+                       foreach ( $pages['sysop'] as $name => $page ) {
                                $a[$name] = $page->getDescription();
                        }
                }
                if ( $wgUser->isDeveloper() )
-               { 
-                       foreach ( $pages["developer"] as $name => $page ) {
+               {
+                       foreach ( $pages['developer'] as $name => $page ) {
                                $a[$name] = $page->getDescription() ;
                        }
                }
-               $go = wfMsg( "go" );
-               $sp = wfMsg( "specialpages" );
-               $spp = $wgLang->specialPage( "Specialpages" );
+               $go = wfMsg( 'go' );
+               $sp = wfMsg( 'specialpages' );
+               $spp = $wgLang->specialPage( 'Specialpages' );
 
-               $s = "<form id=\"specialpages\" method=\"get\" class=\"inline\" " .
-                 "action=\"" . htmlspecialchars( "{$wgServer}{$wgRedirectScript}" ) . "\">\n";
+               $s = '<form id="specialpages" method="get" class="inline" ' .
+                 'action="' . htmlspecialchars( "{$wgServer}{$wgRedirectScript}" ) . "\">\n";
                $s .= "<select name=\"wpDropdown\">\n";
                $s .= "<option value=\"{$spp}\">{$sp}</option>\n";
 
@@ -1073,82 +1130,75 @@ class Skin {
                return $s;
        }
 
-       function mainPageLink()
-       {
-               $mp = wfMsg( "mainpage" );
+       function mainPageLink() {
+               $mp = wfMsg( 'mainpage' );
                $s = $this->makeKnownLink( $mp, $mp );
                return $s;
        }
 
-       function copyrightLink()
-       {
-               $s = $this->makeKnownLink( wfMsg( "copyrightpage" ),
-                 wfMsg( "copyrightpagename" ) );
+       function copyrightLink() {
+               $s = $this->makeKnownLink( wfMsg( 'copyrightpage' ),
+                 wfMsg( 'copyrightpagename' ) );
                return $s;
        }
 
-       function aboutLink()
-       {
-               $s = $this->makeKnownLink( wfMsg( "aboutpage" ),
-                 wfMsg( "aboutwikipedia" ) );
+       function aboutLink() {
+               $s = $this->makeKnownLink( wfMsg( 'aboutpage' ),
+                 wfMsg( 'aboutwikipedia' ) );
                return $s;
        }
 
 
-      function disclaimerLink()
-       {
-               $s = $this->makeKnownLink( wfMsg( "disclaimerpage" ),
-                 wfMsg( "disclaimers" ) );
+       function disclaimerLink() {
+               $s = $this->makeKnownLink( wfMsg( 'disclaimerpage' ),
+                 wfMsg( 'disclaimers' ) );
                return $s;
        }
 
-       function editThisPage()
-       {
+       function editThisPage() {
                global $wgOut, $wgTitle, $wgRequest;
-               
+
                $oldid = $wgRequest->getVal( 'oldid' );
                $diff = $wgRequest->getVal( 'diff' );
                $redirect = $wgRequest->getVal( 'redirect' );
-               
+
                if ( ! $wgOut->isArticleRelated() ) {
-                       $s = wfMsg( "protectedpage" );
+                       $s = wfMsg( 'protectedpage' );
                } else {
                        $n = $wgTitle->getPrefixedText();
                        if ( $wgTitle->userCanEdit() ) {
-                               $t = wfMsg( "editthispage" );
+                               $t = wfMsg( 'editthispage' );
                        } else {
                                #$t = wfMsg( "protectedpage" );
-                               $t = wfMsg( "viewsource" );
+                               $t = wfMsg( 'viewsource' );
                        }
-                       $oid = $red = "";
+                       $oid = $red = '';
 
                        if ( !is_null( $redirect ) ) { $red = "&redirect={$redirect}"; }
                        if ( $oldid && ! isset( $diff ) ) {
-                               $oid = "&oldid={$oldid}";
+                               $oid = '&oldid='.$oldid;
                        }
                        $s = $this->makeKnownLink( $n, $t, "action=edit{$oid}{$red}" );
                }
                return $s;
        }
 
-       function deleteThisPage()
-       {
+       function deleteThisPage() {
                global $wgUser, $wgOut, $wgTitle, $wgRequest;
 
                $diff = $wgRequest->getVal( 'diff' );
                if ( $wgTitle->getArticleId() && ( ! $diff ) && $wgUser->isSysop() ) {
                        $n = $wgTitle->getPrefixedText();
-                       $t = wfMsg( "deletethispage" );
+                       $t = wfMsg( 'deletethispage' );
 
-                       $s = $this->makeKnownLink( $n, $t, "action=delete" );
+                       $s = $this->makeKnownLink( $n, $t, 'action=delete' );
                } else {
-                       $s = "";
+                       $s = '';
                }
                return $s;
        }
 
-       function protectThisPage()
-       {
+       function protectThisPage() {
                global $wgUser, $wgOut, $wgTitle, $wgRequest;
 
                $diff = $wgRequest->getVal( 'diff' );
@@ -1156,157 +1206,147 @@ class Skin {
                        $n = $wgTitle->getPrefixedText();
 
                        if ( $wgTitle->isProtected() ) {
-                               $t = wfMsg( "unprotectthispage" );
-                               $q = "action=unprotect";
+                               $t = wfMsg( 'unprotectthispage' );
+                               $q = 'action=unprotect';
                        } else {
-                               $t = wfMsg( "protectthispage" );
-                               $q = "action=protect";
+                               $t = wfMsg( 'protectthispage' );
+                               $q = 'action=protect';
                        }
                        $s = $this->makeKnownLink( $n, $t, $q );
                } else {
-                       $s = "";
+                       $s = '';
                }
                return $s;
        }
 
-       function watchThisPage()
-       {
+       function watchThisPage() {
                global $wgUser, $wgOut, $wgTitle;
 
                if ( $wgOut->isArticleRelated() ) {
                        $n = $wgTitle->getPrefixedText();
 
                        if ( $wgTitle->userIsWatching() ) {
-                               $t = wfMsg( "unwatchthispage" );
-                               $q = "action=unwatch";
+                               $t = wfMsg( 'unwatchthispage' );
+                               $q = 'action=unwatch';
                        } else {
-                               $t = wfMsg( "watchthispage" );
-                               $q = "action=watch";
+                               $t = wfMsg( 'watchthispage' );
+                               $q = 'action=watch';
                        }
                        $s = $this->makeKnownLink( $n, $t, $q );
                } else {
-                       $s = wfMsg( "notanarticle" );
+                       $s = wfMsg( 'notanarticle' );
                }
                return $s;
        }
 
-       function moveThisPage()
-       {
+       function moveThisPage() {
                global $wgTitle, $wgLang;
 
                if ( $wgTitle->userCanEdit() ) {
-                       $s = $this->makeKnownLink( $wgLang->specialPage( "Movepage" ),
-                         wfMsg( "movethispage" ), "target=" . $wgTitle->getPrefixedURL() );
+                       $s = $this->makeKnownLink( $wgLang->specialPage( 'Movepage' ),
+                         wfMsg( 'movethispage' ), 'target=' . $wgTitle->getPrefixedURL() );
                } // no message if page is protected - would be redundant
                return $s;
        }
 
-       function historyLink()
-       {
+       function historyLink() {
                global $wgTitle;
 
                $s = $this->makeKnownLink( $wgTitle->getPrefixedText(),
-                 wfMsg( "history" ), "action=history" );
+                 wfMsg( 'history' ), 'action=history' );
                return $s;
        }
 
-       function whatLinksHere()
-       {
+       function whatLinksHere() {
                global $wgTitle, $wgLang;
 
-               $s = $this->makeKnownLink( $wgLang->specialPage( "Whatlinkshere" ),
-                 wfMsg( "whatlinkshere" ), "target=" . $wgTitle->getPrefixedURL() );
+               $s = $this->makeKnownLink( $wgLang->specialPage( 'Whatlinkshere' ),
+                 wfMsg( 'whatlinkshere' ), 'target=' . $wgTitle->getPrefixedURL() );
                return $s;
        }
 
-       function userContribsLink()
-       {
+       function userContribsLink() {
                global $wgTitle, $wgLang;
 
-               $s = $this->makeKnownLink( $wgLang->specialPage( "Contributions" ),
-                 wfMsg( "contributions" ), "target=" . $wgTitle->getPartialURL() );
+               $s = $this->makeKnownLink( $wgLang->specialPage( 'Contributions' ),
+                 wfMsg( 'contributions' ), 'target=' . $wgTitle->getPartialURL() );
                return $s;
        }
 
-       function emailUserLink()
-       {
+       function emailUserLink() {
                global $wgTitle, $wgLang;
 
-               $s = $this->makeKnownLink( $wgLang->specialPage( "Emailuser" ),
-                 wfMsg( "emailuser" ), "target=" . $wgTitle->getPartialURL() );
+               $s = $this->makeKnownLink( $wgLang->specialPage( 'Emailuser' ),
+                 wfMsg( 'emailuser' ), 'target=' . $wgTitle->getPartialURL() );
                return $s;
        }
 
-       function watchPageLinksLink()
-       {
+       function watchPageLinksLink() {
                global $wgOut, $wgTitle, $wgLang;
 
                if ( ! $wgOut->isArticleRelated() ) {
-                       $s = "(" . wfMsg( "notanarticle" ) . ")";
+                       $s = '(' . wfMsg( 'notanarticle' ) . ')';
                } else {
                        $s = $this->makeKnownLink( $wgLang->specialPage(
-                         "Recentchangeslinked" ), wfMsg( "recentchangeslinked" ),
-                         "target=" . $wgTitle->getPrefixedURL() );
+                         'Recentchangeslinked' ), wfMsg( 'recentchangeslinked' ),
+                         'target=' . $wgTitle->getPrefixedURL() );
                }
                return $s;
        }
 
-       function otherLanguages()
-       {
+       function otherLanguages() {
                global $wgOut, $wgLang, $wgTitle, $wgUseNewInterlanguage;
 
                $a = $wgOut->getLanguageLinks();
                if ( 0 == count( $a ) ) {
-                       if ( !$wgUseNewInterlanguage ) return "";
+                       if ( !$wgUseNewInterlanguage ) return '';
                        $ns = $wgLang->getNsIndex ( $wgTitle->getNamespace () ) ;
-                       if ( $ns != 0 AND $ns != 1 ) return "" ;
-                       $pn = "Intl" ;
-                       $x = "mode=addlink&xt=".$wgTitle->getDBkey() ;
+                       if ( $ns != 0 AND $ns != 1 ) return '' ;
+                       $pn = 'Intl' ;
+                       $x = 'mode=addlink&xt='.$wgTitle->getDBkey() ;
                        return $this->makeKnownLink( $wgLang->specialPage( $pn ),
-                                 wfMsg( "intl" ) , $x );
+                                 wfMsg( 'intl' ) , $x );
                        }
 
                if ( !$wgUseNewInterlanguage ) {
-                       $s = wfMsg( "otherlanguages" ) . ": ";
+                       $s = wfMsg( 'otherlanguages' ) . ': ';
                } else {
                        global $wgLanguageCode ;
-                       $x = "mode=zoom&xt=".$wgTitle->getDBkey() ;
-                       $x .= "&xl=".$wgLanguageCode ;
-                       $s =  $this->makeKnownLink( $wgLang->specialPage( "Intl" ),
-                                 wfMsg( "otherlanguages" ) , $x ) . ": " ;
+                       $x = 'mode=zoom&xt='.$wgTitle->getDBkey() ;
+                       $x .= '&xl='.$wgLanguageCode ;
+                       $s =  $this->makeKnownLink( $wgLang->specialPage( 'Intl' ),
+                                 wfMsg( 'otherlanguages' ) , $x ) . ': ' ;
                        }
 
-               $s = wfMsg( "otherlanguages" ) . ": ";
+               $s = wfMsg( 'otherlanguages' ) . ': ';
                $first = true;
-               if($wgLang->isRTL()) $s .= "<span dir='LTR'>";
+               if($wgLang->isRTL()) $s .= '<span dir="LTR">';
                foreach( $a as $l ) {
-                       if ( ! $first ) { $s .= " | "; }
+                       if ( ! $first ) { $s .= ' | '; }
                        $first = false;
 
                        $nt = Title::newFromText( $l );
                        $url = $nt->getFullURL();
                        $text = $wgLang->getLanguageName( $nt->getInterwiki() );
 
-                       if ( "" == $text ) { $text = $l; }
+                       if ( '' == $text ) { $text = $l; }
                        $style = $this->getExternalLinkAttributes( $l, $text );
                        $s .= "<a href=\"{$url}\"{$style}>{$text}</a>";
                }
-               if($wgLang->isRTL()) $s .= "</span>";
+               if($wgLang->isRTL()) $s .= '</span>';
                return $s;
        }
 
-       function bugReportsLink()
-       {
-               $s = $this->makeKnownLink( wfMsg( "bugreportspage" ),
-                 wfMsg( "bugreports" ) );
+       function bugReportsLink() {
+               $s = $this->makeKnownLink( wfMsg( 'bugreportspage' ),
+                 wfMsg( 'bugreports' ) );
                return $s;
        }
 
-       function dateLink()
-       {
+       function dateLink() {
                global $wgLinkCache;
-               $t1 = Title::newFromText( gmdate( "F j" ) );
-               $t2 = Title::newFromText( gmdate( "Y" ) );
+               $t1 = Title::newFromText( gmdate( 'F j' ) );
+               $t2 = Title::newFromText( gmdate( 'Y' ) );
 
                $wgLinkCache->suspend();
                $id = $t1->getArticleID();
@@ -1317,7 +1357,7 @@ class Skin {
                } else {
                        $s = $this->makeKnownLink( $t1->getText() );
                }
-               $s .= ", ";
+               $s .= ', ';
 
                $wgLinkCache->suspend();
                $id = $t2->getArticleID();
@@ -1331,41 +1371,40 @@ class Skin {
                return $s;
        }
 
-       function talkLink()
-       {
+       function talkLink() {
                global $wgLang, $wgTitle, $wgLinkCache;
 
                $tns = $wgTitle->getNamespace();
-               if ( -1 == $tns ) { return ""; }
+               if ( -1 == $tns ) { return ''; }
 
                $pn = $wgTitle->getText();
-               $tp = wfMsg( "talkpage" );              
+               $tp = wfMsg( 'talkpage' );
                if ( Namespace::isTalk( $tns ) ) {
                        $lns = Namespace::getSubject( $tns );
                        switch($tns) {
                                case 1:
-                               $text = wfMsg("articlepage");
+                               $text = wfMsg('articlepage');
                                break;
                                case 3:
-                               $text = wfMsg("userpage");
+                               $text = wfMsg('userpage');
                                break;
-                               case 5: 
-                               $text = wfMsg("wikipediapage");
+                               case 5:
+                               $text = wfMsg('wikipediapage');
                                break;
                                case 7:
-                               $text = wfMsg("imagepage");
+                               $text = wfMsg('imagepage');
                                break;
                                default:
-                               $text= wfMsg("articlepage");
+                               $text= wfMsg('articlepage');
                        }
                } else {
 
                        $lns = Namespace::getTalk( $tns );
-                       $text=$tp;                      
+                       $text=$tp;
                }
                $n = $wgLang->getNsText( $lns );
-               if ( "" == $n ) { $link = $pn; }
-               else { $link = "{$n}:{$pn}"; }
+               if ( '' == $n ) { $link = $pn; }
+               else { $link = $n.':'.$pn; }
 
                $wgLinkCache->suspend();
                $s = $this->makeLink( $link, $text );
@@ -1374,12 +1413,11 @@ class Skin {
                return $s;
        }
 
-       function commentLink()
-       {
+       function commentLink() {
                global $wgLang, $wgTitle, $wgLinkCache;
 
                $tns = $wgTitle->getNamespace();
-               if ( -1 == $tns ) { return ""; }
+               if ( -1 == $tns ) { return ''; }
 
                $lns = ( Namespace::isTalk( $tns ) ) ? $tns : Namespace::getTalk( $tns );
 
@@ -1388,83 +1426,92 @@ class Skin {
                $n = $wgLang->getNsText( $lns );
                $pn = $wgTitle->getText();
 
-               $link = "{$n}:{$pn}";
+               $link = $n.':'.$pn;
 
                $wgLinkCache->suspend();
-               $s = $this->makeKnownLink($link, wfMsg("postcomment"), "action=edit&section=new");
+               $s = $this->makeKnownLink($link, wfMsg('postcomment'), 'action=edit&section=new');
                $wgLinkCache->resume();
 
                return $s;
        }
 
-       # After all the page content is transformed into HTML, it makes
-       # a final pass through here for things like table backgrounds.
-       #
-       function transformContent( $text )
-       {
+       /**
+        * After all the page content is transformed into HTML, it makes
+        * a final pass through here for things like table backgrounds.
+        * @todo probably deprecated [AV]
+        */
+       function transformContent( $text ) {
                return $text;
        }
 
-       # Note: This function MUST call getArticleID() on the link,
-       # otherwise the cache won't get updated properly.  See LINKCACHE.DOC.
-       #
-       function makeLink( $title, $text = "", $query = "", $trail = "" ) {
-               wfProfileIn( "Skin::makeLink" );
+       /**
+        * Note: This function MUST call getArticleID() on the link,
+        * otherwise the cache won't get updated properly.  See LINKCACHE.DOC.
+        */
+       function makeLink( $title, $text = '', $query = '', $trail = '' ) {
+               wfProfileIn( 'Skin::makeLink' );
                $nt = Title::newFromText( $title );
                if ($nt) {
                        $result = $this->makeLinkObj( Title::newFromText( $title ), $text, $query, $trail );
                } else {
-                       wfDebug( "Invalid title passed to Skin::makeLink(): \"$title\"\n" );
+                       wfDebug( 'Invalid title passed to Skin::makeLink(): "'.$title."\"\n" );
                        $result = $text == "" ? $title : $text;
                }
 
-               wfProfileOut( "Skin::makeLink" );
+               wfProfileOut( 'Skin::makeLink' );
                return $result;
        }
 
-       function makeKnownLink( $title, $text = "", $query = "", $trail = "", $prefix = '',$aprops = '') {
+       function makeKnownLink( $title, $text = '', $query = '', $trail = '', $prefix = '',$aprops = '') {
                $nt = Title::newFromText( $title );
                if ($nt) {
                        return $this->makeKnownLinkObj( Title::newFromText( $title ), $text, $query, $trail, $prefix , $aprops );
                } else {
-                       wfDebug( "Invalid title passed to Skin::makeKnownLink(): \"$title\"\n" );
-                       return $text == "" ? $title : $text;
+                       wfDebug( 'Invalid title passed to Skin::makeKnownLink(): "'.$title."\"\n" );
+                       return $text == '' ? $title : $text;
                }
        }
 
-       function makeBrokenLink( $title, $text = "", $query = "", $trail = "" ) {
+       function makeBrokenLink( $title, $text = '', $query = '', $trail = '' ) {
                $nt = Title::newFromText( $title );
                if ($nt) {
                        return $this->makeBrokenLinkObj( Title::newFromText( $title ), $text, $query, $trail );
                } else {
-                       wfDebug( "Invalid title passed to Skin::makeBrokenLink(): \"$title\"\n" );
-                       return $text == "" ? $title : $text;
+                       wfDebug( 'Invalid title passed to Skin::makeBrokenLink(): "'.$title."\"\n" );
+                       return $text == '' ? $title : $text;
                }
        }
-       
-       function makeStubLink( $title, $text = "", $query = "", $trail = "" ) {
+
+       function makeStubLink( $title, $text = '', $query = '', $trail = '' ) {
                $nt = Title::newFromText( $title );
-               if ($nt) {              
+               if ($nt) {
                        return $this->makeStubLinkObj( Title::newFromText( $title ), $text, $query, $trail );
                } else {
-                       wfDebug( "Invalid title passed to Skin::makeStubLink(): \"$title\"\n" );
-                       return $text == "" ? $title : $text;
+                       wfDebug( 'Invalid title passed to Skin::makeStubLink(): "'.$title."\"\n" );
+                       return $text == '' ? $title : $text;
                }
        }
 
-       # Pass a title object, not a title string
-       function makeLinkObj( &$nt, $text= "", $query = "", $trail = "", $prefix = "" )
-       {
+       /**
+        * Pass a title object, not a title string
+        */
+       function makeLinkObj( &$nt, $text= '', $query = '', $trail = '', $prefix = '' ) {
                global $wgOut, $wgUser;
+               $fname = 'Skin::makeLinkObj';
+
+               # Fail gracefully
+               if ( ! isset($nt) )
+                       return "<!-- ERROR -->{$prefix}{$text}{$trail}";
+
                if ( $nt->isExternal() ) {
                        $u = $nt->getFullURL();
                        $link = $nt->getPrefixedURL();
-                       if ( "" == $text ) { $text = $nt->getPrefixedText(); }
-                       $style = $this->getExternalLinkAttributes( $link, $text );
+                       if ( '' == $text ) { $text = $nt->getPrefixedText(); }
+                       $style = $this->getExternalLinkAttributes( $link, $text, 'extiw' );
 
-                       $inside = "";
-                       if ( "" != $trail ) {
-                               if ( preg_match( "/^([a-z]+)(.*)$$/sD", $trail, $m ) ) {
+                       $inside = '';
+                       if ( '' != $trail ) {
+                               if ( preg_match( '/^([a-z]+)(.*)$$/sD', $trail, $m ) ) {
                                        $inside = $m[1];
                                        $trail = $m[2];
                                }
@@ -1476,65 +1523,91 @@ class Skin {
                                ( Namespace::getImage() == $nt->getNamespace() ) ) {
                        $retVal = $this->makeKnownLinkObj( $nt, $text, $query, $trail, $prefix );
                } else {
-                       $aid = $nt->getArticleID() ;
-                       if ( 0 == $aid ) {
-                               $retVal = $this->makeBrokenLinkObj( $nt, $text, $query, $trail, $prefix );
+                       if ( $this->postParseLinkColour() ) {
+                               $inside = '';
+                               if ( '' != $trail ) {
+                                       if ( preg_match( $this->linktrail, $trail, $m ) ) {
+                                               $inside = $m[1];
+                                               $trail = $m[2];
+                                       }
+                               }
+
+                               # Allows wiki to bypass using linkcache, see OutputPage::parseLinkHolders()
+                               $retVal = '<!--LINK ' . implode( ' ', array( $nt->getNamespace(), $nt->getDBkey(),
+                                       $query, $prefix . $text . $inside ) ) . "-->{$trail}";
                        } else {
-                               $threshold = $wgUser->getOption("stubthreshold") ;
-                               if ( $threshold > 0 ) {
-                                       $res = wfQuery ( "SELECT LENGTH(cur_text) AS x, cur_namespace, cur_is_redirect FROM cur WHERE cur_id='{$aid}'", DB_READ ) ;
-
-                                       if ( wfNumRows( $res ) > 0 ) {
-                                               $s = wfFetchObject( $res );
-                                               $size = $s->x;
-                                               if ( $s->cur_is_redirect OR $s->cur_namespace != 0 ) {
+                               # Work out link colour immediately
+                               $aid = $nt->getArticleID() ;
+                               if ( 0 == $aid ) {
+                                       $retVal = $this->makeBrokenLinkObj( $nt, $text, $query, $trail, $prefix );
+                               } else {
+                                       $threshold = $wgUser->getOption('stubthreshold') ;
+                                       if ( $threshold > 0 ) {
+                                               $dbr =& wfGetDB( DB_SLAVE );
+                                               $s = $dbr->selectRow( 'cur', array( 'LENGTH(cur_text) AS x', 'cur_namespace',
+                                                       'cur_is_redirect' ), array( 'cur_id' => $aid ), $fname ) ;
+                                               if ( $s !== false ) {
+                                                       $size = $s->x;
+                                                       if ( $s->cur_is_redirect OR $s->cur_namespace != 0 ) {
+                                                               $size = $threshold*2 ; # Really big
+                                                       }
+                                                       $dbr->freeResult( $res );
+                                               } else {
                                                        $size = $threshold*2 ; # Really big
                                                }
-                                               wfFreeResult( $res );
                                        } else {
-                                               $size = $threshold*2 ; # Really big
+                                               $size = 1 ;
+                                       }
+                                       if ( $size < $threshold ) {
+                                               $retVal = $this->makeStubLinkObj( $nt, $text, $query, $trail, $prefix );
+                                       } else {
+                                               $retVal = $this->makeKnownLinkObj( $nt, $text, $query, $trail, $prefix );
                                        }
-                               } else {
-                                       $size = 1 ;
-                               }
-                               if ( $size < $threshold ) {
-                                       $retVal = $this->makeStubLinkObj( $nt, $text, $query, $trail, $prefix );
-                               } else {
-                                       $retVal = $this->makeKnownLinkObj( $nt, $text, $query, $trail, $prefix );
                                }
                        }
                }
                return $retVal;
        }
 
-       # Pass a title object, not a title string
-       function makeKnownLinkObj( &$nt, $text = "", $query = "", $trail = "", $prefix = "" , $aprops = '')
-       {
-               global $wgOut, $wgTitle;
+       /**
+        * Pass a title object, not a title string
+        */
+       function makeKnownLinkObj( &$nt, $text = '', $query = '', $trail = '', $prefix = '' , $aprops = '' ) {
+               global $wgOut, $wgTitle, $wgInputEncoding;
 
-               $fname = "Skin::makeKnownLinkObj";
+               $fname = 'Skin::makeKnownLinkObj';
                wfProfileIn( $fname );
 
+               if ( !is_object( $nt ) ) {
+                       return $text;
+               }
                $link = $nt->getPrefixedURL();
+#              if ( '' != $section && substr($section,0,1) != "#" ) {
+#                      $section = ''
 
-               if ( "" == $link ) {
-                       $u = "";
-                       if ( "" == $text ) {
+               if ( '' == $link ) {
+                       $u = '';
+                       if ( '' == $text ) {
                                $text = htmlspecialchars( $nt->getFragment() );
                        }
                } else {
                        $u = $nt->escapeLocalURL( $query );
                }
-               if ( "" != $nt->getFragment() ) {
-                       $u .= "#" . htmlspecialchars( $nt->getFragment() );
+               if ( '' != $nt->getFragment() ) {
+                       $anchor = urlencode( do_html_entity_decode( str_replace(' ', '_', $nt->getFragment()), ENT_COMPAT, $wgInputEncoding ) );
+                       $replacearray = array(
+                               '%3A' => ':',
+                               '%' => '.'
+                       );
+                       $u .= '#' . str_replace(array_keys($replacearray),array_values($replacearray),$anchor);
                }
-               if ( "" == $text ) {
+               if ( '' == $text ) {
                        $text = htmlspecialchars( $nt->getPrefixedText() );
                }
                $style = $this->getInternalLinkAttributesObj( $nt, $text );
 
-               $inside = "";
-               if ( "" != $trail ) {
+               $inside = '';
+               if ( '' != $trail ) {
                        if ( preg_match( $this->linktrail, $trail, $m ) ) {
                                $inside = $m[1];
                                $trail = $m[2];
@@ -1545,34 +1618,39 @@ class Skin {
                return $r;
        }
 
-       # Pass a title object, not a title string
-       function makeBrokenLinkObj( &$nt, $text = "", $query = "", $trail = "", $prefix = "" )
-       {
+       /**
+        * Pass a title object, not a title string
+        */
+       function makeBrokenLinkObj( &$nt, $text = '', $query = '', $trail = '', $prefix = '' ) {
                global $wgOut, $wgUser;
 
-               $fname = "Skin::makeBrokenLinkObj";
+               # Fail gracefully
+               if ( ! isset($nt) )
+                       return "<!-- ERROR -->{$prefix}{$text}{$trail}";
+
+               $fname = 'Skin::makeBrokenLinkObj';
                wfProfileIn( $fname );
 
-               if ( "" == $query ) {
-                       $q = "action=edit";
+               if ( '' == $query ) {
+                       $q = 'action=edit';
                } else {
-                       $q = "action=edit&{$query}";
+                       $q = 'action=edit&'.$query;
                }
                $u = $nt->escapeLocalURL( $q );
 
-               if ( "" == $text ) {
+               if ( '' == $text ) {
                        $text = htmlspecialchars( $nt->getPrefixedText() );
                }
                $style = $this->getInternalLinkAttributesObj( $nt, $text, "yes" );
 
-               $inside = "";
-               if ( "" != $trail ) {
+               $inside = '';
+               if ( '' != $trail ) {
                        if ( preg_match( $this->linktrail, $trail, $m ) ) {
                                $inside = $m[1];
                                $trail = $m[2];
                        }
                }
-               if ( $wgUser->getOption( "highlightbroken" ) ) {
+               if ( $wgUser->getOption( 'highlightbroken' ) ) {
                        $s = "<a href=\"{$u}\"{$style}>{$prefix}{$text}{$inside}</a>{$trail}";
                } else {
                        $s = "{$prefix}{$text}{$inside}<a href=\"{$u}\"{$style}>?</a>{$trail}";
@@ -1581,44 +1659,44 @@ class Skin {
                wfProfileOut( $fname );
                return $s;
        }
-       
-       # Pass a title object, not a title string
-       function makeStubLinkObj( &$nt, $text = "", $query = "", $trail = "", $prefix = "" )
-       {
+
+       /**
+        * Pass a title object, not a title string
+        */
+       function makeStubLinkObj( &$nt, $text = '', $query = '', $trail = '', $prefix = '' ) {
                global $wgOut, $wgUser;
 
                $link = $nt->getPrefixedURL();
 
                $u = $nt->escapeLocalURL( $query );
 
-               if ( "" == $text ) {
+               if ( '' == $text ) {
                        $text = htmlspecialchars( $nt->getPrefixedText() );
                }
-               $style = $this->getInternalLinkAttributesObj( $nt, $text, "stub" );
+               $style = $this->getInternalLinkAttributesObj( $nt, $text, 'stub' );
 
-               $inside = "";
-               if ( "" != $trail ) {
+               $inside = '';
+               if ( '' != $trail ) {
                        if ( preg_match( $this->linktrail, $trail, $m ) ) {
                                $inside = $m[1];
                                $trail = $m[2];
                        }
                }
-               if ( $wgUser->getOption( "highlightbroken" ) ) {
+               if ( $wgUser->getOption( 'highlightbroken' ) ) {
                        $s = "<a href=\"{$u}\"{$style}>{$prefix}{$text}{$inside}</a>{$trail}";
                } else {
                        $s = "{$prefix}{$text}{$inside}<a href=\"{$u}\"{$style}>!</a>{$trail}";
                }
                return $s;
        }
-       
-       function makeSelfLinkObj( &$nt, $text = "", $query = "", $trail = "", $prefix = "" )
-       {
+
+       function makeSelfLinkObj( &$nt, $text = '', $query = '', $trail = '', $prefix = '' ) {
                $u = $nt->escapeLocalURL( $query );
-               if ( "" == $text ) {
+               if ( '' == $text ) {
                        $text = htmlspecialchars( $nt->getPrefixedText() );
                }
-               $inside = "";
-               if ( "" != $trail ) {
+               $inside = '';
+               if ( '' != $trail ) {
                        if ( preg_match( $this->linktrail, $trail, $m ) ) {
                                $inside = $m[1];
                                $trail = $m[2];
@@ -1630,107 +1708,116 @@ class Skin {
        /* these are used extensively in SkinPHPTal, but also some other places */
        /*static*/ function makeSpecialUrl( $name, $urlaction='' ) {
                $title = Title::makeTitle( NS_SPECIAL, $name );
-               $this->checkTitle($title, $name);       
+               $this->checkTitle($title, $name);
                return $title->getLocalURL( $urlaction );
        }
        /*static*/ function makeTalkUrl ( $name, $urlaction='' ) {
                $title = Title::newFromText( $name );
                $title = $title->getTalkPage();
-               $this->checkTitle($title, $name);       
+               $this->checkTitle($title, $name);
                return $title->getLocalURL( $urlaction );
        }
        /*static*/ function makeArticleUrl ( $name, $urlaction='' ) {
                $title = Title::newFromText( $name );
                $title= $title->getSubjectPage();
-               $this->checkTitle($title, $name);       
+               $this->checkTitle($title, $name);
                return $title->getLocalURL( $urlaction );
        }
        /*static*/ function makeI18nUrl ( $name, $urlaction='' ) {
                $title = Title::newFromText( wfMsg($name) );
-               $this->checkTitle($title, $name);       
+               $this->checkTitle($title, $name);
                return $title->getLocalURL( $urlaction );
        }
        /*static*/ function makeUrl ( $name, $urlaction='' ) {
                $title = Title::newFromText( $name );
-               $this->checkTitle($title, $name);       
-               return $title->getLocalURL( $urlaction ); 
+               $this->checkTitle($title, $name);
+               return $title->getLocalURL( $urlaction );
        }
-       
+       # this can be passed the NS number as defined in Language.php
+       /*static*/ function makeNSUrl( $name, $urlaction='', $namespace=0 ) {
+               $title = Title::makeTitleSafe( $namespace, $name );
+               $this->checkTitle($title, $name);
+               return $title->getLocalURL( $urlaction );
+       }
+
        /* these return an array with the 'href' and boolean 'exists' */
        /*static*/ function makeUrlDetails ( $name, $urlaction='' ) {
                $title = Title::newFromText( $name );
                $this->checkTitle($title, $name);
-               return array( 
+               return array(
                        'href' => $title->getLocalURL( $urlaction ),
                        'exists' => $title->getArticleID() != 0?true:false
-               ); 
+               );
        }
        /*static*/ function makeTalkUrlDetails ( $name, $urlaction='' ) {
                $title = Title::newFromText( $name );
                $title = $title->getTalkPage();
                $this->checkTitle($title, $name);
-               return array( 
+               return array(
                        'href' => $title->getLocalURL( $urlaction ),
                        'exists' => $title->getArticleID() != 0?true:false
-               ); 
+               );
        }
        /*static*/ function makeArticleUrlDetails ( $name, $urlaction='' ) {
                $title = Title::newFromText( $name );
                $title= $title->getSubjectPage();
                $this->checkTitle($title, $name);
-               return array( 
+               return array(
                        'href' => $title->getLocalURL( $urlaction ),
                        'exists' => $title->getArticleID() != 0?true:false
-               ); 
+               );
        }
        /*static*/ function makeI18nUrlDetails ( $name, $urlaction='' ) {
                $title = Title::newFromText( wfMsg($name) );
                $this->checkTitle($title, $name);
-               return array( 
+               return array(
                        'href' => $title->getLocalURL( $urlaction ),
                        'exists' => $title->getArticleID() != 0?true:false
-               ); 
+               );
        }
 
        # make sure we have some title to operate on
-       /*static*/ function checkTitle ( &$title, &$name ) { 
+       /*static*/ function checkTitle ( &$title, &$name ) {
                if(!is_object($title)) {
                        $title = Title::newFromText( $name );
                        if(!is_object($title)) {
-                               $title = Title::newFromText( '<error: link target missing>' );
+                               $title = Title::newFromText( '--error: link target missing--' );
                        }
                }
        }
 
-       function fnamePart( $url )
-       {
-               $basename = strrchr( $url, "/" );
-               if ( false === $basename ) { $basename = $url; }
-               else { $basename = substr( $basename, 1 ); }
-               return wfEscapeHTML( $basename );
+       function fnamePart( $url ) {
+               $basename = strrchr( $url, '/' );
+               if ( false === $basename ) {
+                       $basename = $url;
+               } else {
+                       $basename = substr( $basename, 1 );
+               }
+               return htmlspecialchars( $basename );
        }
 
-       function makeImage( $url, $alt = "" )
-       {
+       function makeImage( $url, $alt = '' ) {
                global $wgOut;
 
-               if ( "" == $alt ) { $alt = $this->fnamePart( $url ); }
-               $s = "<img src=\"{$url}\" alt=\"{$alt}\" />";
+               if ( '' == $alt ) {
+                       $alt = $this->fnamePart( $url );
+               }
+               $s = '<img src="'.$url.'" alt="'.$alt.'" />';
                return $s;
        }
-       
-       function makeImageLink( $name, $url, $alt = "" ) {
-               $nt = Title::makeTitle( Namespace::getImage(), $name );
+
+       function makeImageLink( $name, $url, $alt = '' ) {
+               $nt = Title::makeTitleSafe( NS_IMAGE, $name );
                return $this->makeImageLinkObj( $nt, $alt );
        }
 
-       function makeImageLinkObj( $nt, $alt = "" ) {
+       function makeImageLinkObj( $nt, $alt = '' ) {
                global $wgLang, $wgUseImageResize;
                $img   = Image::newFromTitle( $nt );
                $url   = $img->getURL();
 
-               $align = "";
-               $prefix = $postfix = "";
+               $align = '';
+               $prefix = $postfix = '';
 
                if ( $wgUseImageResize ) {
                        # Check if the alt text is of the form "options|alt text"
@@ -1742,9 +1829,9 @@ class Skin {
                        #  * ___px              scale to ___ pixels width, no aligning. e.g. use in taxobox
                        #  * center             center the image
                        #  * framed             Keep original image size, no magnify-button.
-       
-                       $part = explode( "|", $alt);
-       
+
+                       $part = explode( '|', $alt);
+
                        $mwThumb  =& MagicWord::get( MAG_IMG_THUMBNAIL );
                        $mwLeft   =& MagicWord::get( MAG_IMG_LEFT );
                        $mwRight  =& MagicWord::get( MAG_IMG_RIGHT );
@@ -1755,25 +1842,32 @@ class Skin {
                        $alt = $part[count($part)-1];
 
                        $height = $framed = $thumb = false;
+                       $manual_thumb = "" ;
 
                        foreach( $part as $key => $val ) {
+                               $val_parts = explode ( "=" , $val , 2 ) ;
+                               $left_part = array_shift ( $val_parts ) ;
                                if ( ! is_null( $mwThumb->matchVariableStartToEnd($val) ) ) {
                                        $thumb=true;
+                               } elseif ( count ( $val_parts ) == 1 && ! is_null( $mwThumb->matchVariableStartToEnd($left_part) ) ) {
+                                       # use manually specified thumbnail
+                                       $thumb=true;
+                                       $manual_thumb = array_shift ( $val_parts ) ;
                                } elseif ( ! is_null( $mwRight->matchVariableStartToEnd($val) ) ) {
                                        # remember to set an alignment, don't render immediately
-                                       $align = "right";
+                                       $align = 'right';
                                } elseif ( ! is_null( $mwLeft->matchVariableStartToEnd($val) ) ) {
                                        # remember to set an alignment, don't render immediately
-                                       $align = "left";
+                                       $align = 'left';
                                } elseif ( ! is_null( $mwCenter->matchVariableStartToEnd($val) ) ) {
                                        # remember to set an alignment, don't render immediately
-                                       $align = "center";
+                                       $align = 'center';
                                } elseif ( ! is_null( $mwNone->matchVariableStartToEnd($val) ) ) {
                                        # remember to set an alignment, don't render immediately
-                                       $align = "none";
+                                       $align = 'none';
                                } elseif ( ! is_null( $match = $mwWidth->matchVariableStartToEnd($val) ) ) {
                                        # $match is the image width in pixels
-                                       if ( preg_match( "/^([0-9]*)x([0-9]*)$/", $match, $m ) ) {
+                                       if ( preg_match( '/^([0-9]*)x([0-9]*)$/', $match, $m ) ) {
                                                $width = intval( $m[1] );
                                                $height = intval( $m[2] );
                                        } else {
@@ -1783,15 +1877,15 @@ class Skin {
                                        $framed=true;
                                }
                        }
-                       if ( "center" == $align )
+                       if ( 'center' == $align )
                        {
                                $prefix  = '<span style="text-align: center">';
                                $postfix = '</span>';
-                               $align   = "none";
+                               $align   = 'none';
                        }
-       
+
                        if ( $thumb || $framed ) {
-       
+
                                # Create a thumbnail. Alignment depends on language
                                # writing direction, # right aligned for left-to-right-
                                # languages ("Western languages"), left-aligned
@@ -1799,66 +1893,76 @@ class Skin {
                                #
                                # If  thumbnail width has not been provided, it is set
                                # here to 180 pixels
-                               if ( $align == "" ) {
-                                       $align = $wgLang->isRTL() ? "left" : "right";
+                               if ( $align == '' ) {
+                                       $align = $wgLang->isRTL() ? 'left' : 'right';
                                }
                                if ( ! isset($width) ) {
                                        $width = 180;
                                }
-                               return $prefix.$this->makeThumbLinkObj( $img, $alt, $align, $width, $height, $framed ).$postfix;
-       
+                               return $prefix.$this->makeThumbLinkObj( $img, $alt, $align, $width, $height, $framed, $manual_thumb ).$postfix;
+
                        } elseif ( isset($width) ) {
-                               
+
                                # Create a resized image, without the additional thumbnail
                                # features
 
-                               if (    ( ! $height === false ) 
+                               if (    ( ! $height === false )
                                     && ( $img->getHeight() * $width / $img->getWidth() > $height ) ) {
                                        print "height=$height<br>\nimg->getHeight() = ".$img->getHeight()."<br>\n";
-                                       print "rescaling by factor ". $height / $img->getHeight() . "<br>\n";
+                                       print 'rescaling by factor '. $height / $img->getHeight() . "<br>\n";
                                        $width = $img->getWidth() * $height / $img->getHeight();
                                }
-                               $url = $img->createThumb( $width );
+                               if ( '' == $manual_thumb ) $url = $img->createThumb( $width );
                        }
                } # endif $wgUseImageResize
-                       
+
                if ( empty( $alt ) ) {
                        $alt = preg_replace( '/\.(.+?)^/', '', $img->getName() );
                }
                $alt = htmlspecialchars( $alt );
 
                $u = $nt->escapeLocalURL();
-               if ( $url == "" )
+               if ( $url == '' )
                {
-                       $s = str_replace( "$1", $img->getName(), wfMsg("missingimage") );
+                       $s = wfMsg( 'missingimage', $img->getName() );
                        $s .= "<br>{$alt}<br>{$url}<br>\n";
                } else {
-                       $s = "<a href=\"{$u}\" class='image' title=\"{$alt}\">" .
-                               "<img src=\"{$url}\" alt=\"{$alt}\" /></a>";
+                       $s = '<a href="'.$u.'" class="image" title="'.$alt.'">' .
+                                '<img src="'.$url.'" alt="'.$alt.'" /></a>';
                }
-               if ( "" != $align ) {
+               if ( '' != $align ) {
                        $s = "<div class=\"float{$align}\"><span>{$s}</span></div>";
                }
                return str_replace("\n", ' ',$prefix.$s.$postfix);
        }
 
-
-       function makeThumbLinkObj( $img, $label = "", $align = "right", $boxwidth = 180, $boxheight=false, $framed=false ) {
+       /**
+        * Make HTML for a thumbnail including image, border and caption
+        * $img is an Image object
+        */
+       function makeThumbLinkObj( $img, $label = '', $align = 'right', $boxwidth = 180, $boxheight=false, $framed=false , $manual_thumb = "" ) {
                global $wgStylePath, $wgLang;
-               # $image = Title::makeTitle( Namespace::getImage(), $name );
+               # $image = Title::makeTitleSafe( NS_IMAGE, $name );
                $url  = $img->getURL();
-               
+
                #$label = htmlspecialchars( $label );
-               $alt = preg_replace( "/<[^>]*>/", "", $label);
+               $alt = preg_replace( '/<[^>]*>/', '', $label);
                $alt = htmlspecialchars( $alt );
-               
+
+               $width = $height = 0;
                if ( $img->exists() )
                {
                        $width  = $img->getWidth();
                        $height = $img->getHeight();
-               } else {
+               }
+               if ( 0 == $width || 0 == $height )
+               {
                        $width = $height = 200;
                }
+               if ( $boxwidth == 0 )
+               {
+                       $boxwidth = 200;
+               }
                if ( $framed )
                {
                        // Use image dimensions, don't scale
@@ -1875,18 +1979,33 @@ class Skin {
                        } else {
                                $boxheight = $h;
                        }
-                       $thumbUrl = $img->createThumb( $boxwidth );
+                       if ( '' == $manual_thumb ) $thumbUrl = $img->createThumb( $boxwidth );
+               }
+
+               if ( $manual_thumb != '' ) # Use manually specified thumbnail
+               {
+                       $manual_title = Title::makeTitleSafe( NS_IMAGE, $manual_thumb ); #new Title ( $manual_thumb ) ;
+                       $manual_img = Image::newFromTitle( $manual_title );
+                       $thumbUrl = $manual_img->getURL();
+                       if ( $manual_img->exists() )
+                       {
+                               $width  = $manual_img->getWidth();
+                               $height = $manual_img->getHeight();
+                               $boxwidth = $width ;
+                               $boxheight = $height ;
+                               $oboxwidth = $boxwidth + 2 ;
+                       }
                }
 
                $u = $img->getEscapeLocalURL();
 
-               $more = htmlspecialchars( wfMsg( "thumbnail-more" ) );
-               $magnifyalign = $wgLang->isRTL() ? "left" : "right";
-               $textalign = $wgLang->isRTL() ? ' style="text-align:right"' : "";
+               $more = htmlspecialchars( wfMsg( 'thumbnail-more' ) );
+               $magnifyalign = $wgLang->isRTL() ? 'left' : 'right';
+               $textalign = $wgLang->isRTL() ? ' style="text-align:right"' : '';
 
                $s = "<div class=\"thumb t{$align}\"><div style=\"width:{$oboxwidth}px;\">";
-               if ( $thumbUrl == "" ) {
-                       $s .= str_replace( "$1", $img->getName(), wfMsg("missingimage") );
+               if ( $thumbUrl == '' ) {
+                       $s .= wfMsg( 'missingimage', $img->getName() );
                        $zoomicon = '';
                } else {
                        $s .= '<a href="'.$u.'" class="internal" title="'.$alt.'">'.
@@ -1905,16 +2024,15 @@ class Skin {
                return str_replace("\n", ' ', $s);
        }
 
-       function makeMediaLink( $name, $url, $alt = "" ) {
-               $nt = Title::makeTitle( Namespace::getMedia(), $name );
+       function makeMediaLink( $name, $url, $alt = '' ) {
+               $nt = Title::makeTitleSafe( Namespace::getMedia(), $name );
                return $this->makeMediaLinkObj( $nt, $alt );
        }
 
-       function makeMediaLinkObj( $nt, $alt = "" )
-       {
+       function makeMediaLinkObj( $nt, $alt = '' ) {
                if ( ! isset( $nt ) )
                {
-                       ### HOTFIX. Instead of breaking, return empry string.
+                       ### HOTFIX. Instead of breaking, return empty string.
                        $s = $alt;
                } else {
                        $name = $nt->getDBKey();
@@ -1922,57 +2040,55 @@ class Skin {
                        if ( empty( $alt ) ) {
                                $alt = preg_replace( '/\.(.+?)^/', '', $name );
                        }
-       
+
                        $u = htmlspecialchars( $url );
                        $s = "<a href=\"{$u}\" class='internal' title=\"{$alt}\">{$alt}</a>";
                }
                return $s;
        }
 
-       function specialLink( $name, $key = "" )
-       {
+       function specialLink( $name, $key = '' ) {
                global $wgLang;
 
-               if ( "" == $key ) { $key = strtolower( $name ); }
+               if ( '' == $key ) { $key = strtolower( $name ); }
                $pn = $wgLang->ucfirst( $name );
                return $this->makeKnownLink( $wgLang->specialPage( $pn ),
                  wfMsg( $key ) );
        }
-       
+
        function makeExternalLink( $url, $text, $escape = true ) {
                $style = $this->getExternalLinkAttributes( $url, $text );
                $url = htmlspecialchars( $url );
                if( $escape ) {
                        $text = htmlspecialchars( $text );
                }
-               return "<a href=\"$url\"$style>$text</a>";
+               return '<a href="'.$url.'"'.$style.'>'.$text.'</a>';
        }
 
        # Called by history lists and recent changes
        #
 
        # Returns text for the start of the tabular part of RC
-       function beginRecentChangesList()
-       {
+       function beginRecentChangesList() {
                $this->rc_cache = array() ;
                $this->rcMoveIndex = 0;
                $this->rcCacheIndex = 0 ;
-               $this->lastdate = "";
+               $this->lastdate = '';
                $this->rclistOpen = false;
-               return "";
+               return '';
        }
 
-       function beginImageHistoryList()
-       {
-               $s = "\n<h2>" . wfMsg( "imghistory" ) . "</h2>\n" .
-                 "<p>" . wfMsg( "imghistlegend" ) . "</p>\n<ul class='special'>";
+       function beginImageHistoryList() {
+               $s = "\n<h2>" . wfMsg( 'imghistory' ) . "</h2>\n" .
+                 "<p>" . wfMsg( 'imghistlegend' ) . "</p>\n".'<ul class="special">';
                return $s;
        }
 
-       # Returns text for the end of RC
-       # If enhanced RC is in use, returns pretty much all the text
-       function endRecentChangesList()
-       {
+       /**
+        * Returns text for the end of RC
+        * If enhanced RC is in use, returns pretty much all the text
+        */
+       function endRecentChangesList() {
                $s = $this->recentChangesBlock() ;
                if( $this->rclistOpen ) {
                        $s .= "</ul>\n";
@@ -1980,78 +2096,81 @@ class Skin {
                return $s;
        }
 
-       # Enhanced RC ungrouped line
-       function recentChangesBlockLine ( $rcObj )
-       {
+       /**
+        * Enhanced RC ungrouped line
+        */
+       function recentChangesBlockLine ( $rcObj ) {
                global $wgStylePath, $wgLang ;
 
                # Get rc_xxxx variables
                extract( $rcObj->mAttribs ) ;
-               $curIdEq = "curid=$rc_cur_id";
+               $curIdEq = 'curid='.$rc_cur_id;
 
                # Spacer image
-               $r = "" ;
+               $r = '' ;
 
-               $r .= "<img src='{$wgStylePath}/images/Arr_.png' width='12' height='12' border='0' />" ;                $r .= "<tt>" ;
+               $r .= '<img src="'.$wgStylePath.'/images/Arr_.png" width="12" height="12" border="0" />' ;
+               $r .= '<tt>' ;
 
-               if ( $rc_type == RC_MOVE ) {
-                       $r .= "&nbsp;&nbsp;";
+               if ( $rc_type == RC_MOVE || $rc_type == RC_MOVE_OVER_REDIRECT ) {
+                       $r .= '&nbsp;&nbsp;';
                } else {
                        # M & N (minor & new)
-                       $M = wfMsg( "minoreditletter" );
-                       $N = wfMsg( "newpageletter" );
+                       $M = wfMsg( 'minoreditletter' );
+                       $N = wfMsg( 'newpageletter' );
 
                        if ( $rc_type == RC_NEW ) {
                                $r .= $N ;
                        } else {
-                               $r .= "&nbsp;" ;
+                               $r .= '&nbsp;' ;
                        }
                        if ( $rc_minor ) {
                                $r .= $M ;
                        } else {
-                               $r .= "&nbsp;" ;
+                               $r .= '&nbsp;' ;
                        }
                }
 
                # Timestamp
-               $r .= " ".$rcObj->timestamp." " ;
-               $r .= "</tt>" ;
+               $r .= ' '.$rcObj->timestamp.' ' ;
+               $r .= '</tt>' ;
 
                # Article link
                $link = $rcObj->link ;
-               if ( $rcObj->watched ) $link = "<strong>{$link}</strong>" ;
+               if ( $rcObj->watched ) $link = '<strong>'.$link.'</strong>' ;
                $r .= $link ;
 
-               # Cur
-               $r .= " (" ;
-               $r .= $rcObj->curlink ;
-               $r .= "; " ;
+               # Diff
+               $r .= ' (' ;
+               $r .= $rcObj->difflink ;
+               $r .= '; ' ;
 
                # Hist
-               $r .= $this->makeKnownLinkObj( $rcObj->getTitle(), wfMsg( "hist" ), "{$curIdEq}&action=history" );
+               $r .= $this->makeKnownLinkObj( $rcObj->getTitle(), wfMsg( 'hist' ), $curIdEq.'&action=history' );
 
                # User/talk
-               $r .= ") . . ".$rcObj->userlink ;
+               $r .= ') . . '.$rcObj->userlink ;
                $r .= $rcObj->usertalklink ;
 
                # Comment
-               if ( $rc_comment != "" && $rc_type != RC_MOVE ) {
-                       $rc_comment=$this->formatComment($rc_comment);
-                       $r .= $wgLang->emphasize( " (".$rc_comment.")" );
+                if ( $rc_comment != '' && $rc_type != RC_MOVE && $rc_type != RC_MOVE_OVER_REDIRECT ) {
+                       $rc_comment=$this->formatComment($rc_comment, $rcObj->getTitle());
+                       $r .= $wgLang->emphasize( ' ('.$rc_comment.')' );
                }
 
                $r .= "<br />\n" ;
                return $r ;
        }
 
-       # Enhanced RC group
-       function recentChangesBlockGroup ( $block )
-       {
+       /**
+        * Enhanced RC group
+        */
+       function recentChangesBlockGroup ( $block ) {
                global $wgStylePath, $wgLang ;
 
-               $r = "" ;
-               $M = wfMsg( "minoreditletter" );
-               $N = wfMsg( "newpageletter" );
+               $r = '' ;
+               $M = wfMsg( 'minoreditletter' );
+               $N = wfMsg( 'newpageletter' );
 
                # Collate list of users
                $isnew = false ;
@@ -2073,86 +2192,86 @@ class Skin {
                        if ( $count > 1 ) $text .= " ({$count}&times;)" ;
                        array_push ( $users , $text ) ;
                }
-               $users = " <font size='-1'>[".implode("; ",$users)."]</font>" ;
+               $users = ' <font size="-1">['.implode('; ',$users).']</font>' ;
 
                # Arrow
-               $rci = "RCI{$this->rcCacheIndex}" ;
-               $rcl = "RCL{$this->rcCacheIndex}" ;
-               $rcm = "RCM{$this->rcCacheIndex}" ;
-               $toggleLink = "javascript:toggleVisibility(\"{$rci}\",\"{$rcm}\",\"{$rcl}\")" ;
-               $arrowdir = $wgLang->isRTL() ? "l" : "r";
-               $tl  = "<span id='{$rcm}'><a href='$toggleLink'><img src='{$wgStylePath}/images/Arr_{$arrowdir}.png' width='12' height='12' /></a></span>" ;
-               $tl .= "<span id='{$rcl}' style='display:none'><a href='$toggleLink'><img src='{$wgStylePath}/images/Arr_d.png' width='12' height='12' /></a></span>" ;
+               $rci = 'RCI'.$this->rcCacheIndex ;
+               $rcl = 'RCL'.$this->rcCacheIndex ;
+               $rcm = 'RCM'.$this->rcCacheIndex ;
+               $toggleLink = "javascript:toggleVisibility('$rci','$rcm','$rcl')" ;
+               $arrowdir = $wgLang->isRTL() ? 'l' : 'r';
+               $tl  = '<span id="'.$rcm.'"><a href="'.$toggleLink.'"><img src="'.$wgStylePath.'/images/Arr_'.$arrowdir.'.png" width="12" height="12" /></a></span>' ;
+               $tl .= '<span id="'.$rcl.'" style="display:none"><a href="'.$toggleLink.'"><img src="'.$wgStylePath.'/images/Arr_d.png" width="12" height="12" /></a></span>' ;
                $r .= $tl ;
 
                # Main line
                # M/N
-               $r .= "<tt>" ;
+               $r .= '<tt>' ;
                if ( $isnew ) $r .= $N ;
-               else $r .= "&nbsp;" ;
-               $r .= "&nbsp;" ; # Minor
+               else $r .= '&nbsp;' ;
+               $r .= '&nbsp;' ; # Minor
 
                # Timestamp
-               $r .= " ".$block[0]->timestamp." " ;
-               $r .= "</tt>" ;
+               $r .= ' '.$block[0]->timestamp.' ' ;
+               $r .= '</tt>' ;
 
                # Article link
                $link = $block[0]->link ;
-               if ( $block[0]->watched ) $link = "<strong>{$link}</strong>" ;
+               if ( $block[0]->watched ) $link = '<strong>'.$link.'</strong>' ;
                $r .= $link ;
-               
-               $curIdEq = "curid=" . $block[0]->mAttribs['rc_cur_id'];
+
+               $curIdEq = 'curid=' . $block[0]->mAttribs['rc_cur_id'];
                if ( $block[0]->mAttribs['rc_type'] != RC_LOG ) {
                        # Changes
-                       $r .= " (".count($block)." " ;
-                       if ( $isnew ) $r .= wfMsg("changes");
-                       else $r .= $this->makeKnownLinkObj( $block[0]->getTitle() , wfMsg("changes") , 
-                               "{$curIdEq}&diff=0&oldid=".$oldid ) ;
-                       $r .= "; " ;
+                       $r .= ' ('.count($block).' ' ;
+                       if ( $isnew ) $r .= wfMsg('changes');
+                       else $r .= $this->makeKnownLinkObj( $block[0]->getTitle() , wfMsg('changes') ,
+                               $curIdEq.'&diff=0&oldid='.$oldid ) ;
+                       $r .= '; ' ;
 
                        # History
-                       $r .= $this->makeKnownLinkObj( $block[0]->getTitle(), wfMsg( "history" ), "{$curIdEq}&action=history" );
-                       $r .= ")" ;
+                       $r .= $this->makeKnownLinkObj( $block[0]->getTitle(), wfMsg( 'history' ), $curIdEq.'&action=history' );
+                       $r .= ')' ;
                }
 
                $r .= $users ;
                $r .= "<br />\n" ;
 
                # Sub-entries
-               $r .= "<div id='{$rci}' style='display:none'>" ;
+               $r .= '<div id="'.$rci.'" style="display:none">' ;
                foreach ( $block AS $rcObj ) {
                        # Get rc_xxxx variables
                        extract( $rcObj->mAttribs );
-                       
-                       $r .= "<img src='{$wgStylePath}/images/Arr_.png' width=12 height=12 />";
-                       $r .= "<tt>&nbsp; &nbsp; &nbsp; &nbsp;" ;
+
+                       $r .= '<img src="'.$wgStylePath.'/images/Arr_.png" width="12" height="12" />';
+                       $r .= '<tt>&nbsp; &nbsp; &nbsp; &nbsp;' ;
                        if ( $rc_new ) $r .= $N ;
-                       else $r .= "&nbsp;" ;
+                       else $r .= '&nbsp;' ;
                        if ( $rc_minor ) $r .= $M ;
-                       else $r .= "&nbsp;" ;
-                       $r .= "</tt>" ;
+                       else $r .= '&nbsp;' ;
+                       $r .= '</tt>' ;
 
-                       $o = "" ;
+                       $o = '' ;
                        if ( $rc_last_oldid != 0 ) {
-                               $o = "oldid=".$rc_last_oldid ;
+                               $o = 'oldid='.$rc_last_oldid ;
                        }
                        if ( $rc_type == RC_LOG ) {
                                $link = $rcObj->timestamp ;
                        } else {
                                $link = $this->makeKnownLinkObj( $rcObj->getTitle(), $rcObj->timestamp , "{$curIdEq}&$o" ) ;
                        }
-                       $link = "<tt>{$link}</tt>" ;
+                       $link = '<tt>'.$link.'</tt>' ;
 
                        $r .= $link ;
-                       $r .= " (" ;
+                       $r .= ' (' ;
                        $r .= $rcObj->curlink ;
-                       $r .= "; " ;
+                       $r .= '; ' ;
                        $r .= $rcObj->lastlink ;
-                       $r .= ") . . ".$rcObj->userlink ;
+                       $r .= ') . . '.$rcObj->userlink ;
                        $r .= $rcObj->usertalklink ;
-                       if ( $rc_comment != "" ) {
-                               $rc_comment=$this->formatComment($rc_comment);
-                               $r .= $wgLang->emphasize( " (".$rc_comment.")" ) ;
+                       if ( $rc_comment != '' ) {
+                               $rc_comment=$this->formatComment($rc_comment, $rcObj->getTitle());
+                               $r .= $wgLang->emphasize( ' ('.$rc_comment.')' ) ;
                        }
                        $r .= "<br />\n" ;
                }
@@ -2162,13 +2281,14 @@ class Skin {
                return $r ;
        }
 
-       # If enhanced RC is in use, this function takes the previously cached
-       # RC lines, arranges them, and outputs the HTML
-       function recentChangesBlock ()
-       {
+       /**
+        * If enhanced RC is in use, this function takes the previously cached
+        * RC lines, arranges them, and outputs the HTML
+        */
+       function recentChangesBlock () {
                global $wgStylePath ;
-               if ( count ( $this->rc_cache ) == 0 ) return "" ;
-               $blockOut = "";
+               if ( count ( $this->rc_cache ) == 0 ) return '' ;
+               $blockOut = '';
                foreach ( $this->rc_cache AS $secureName => $block ) {
                        if ( count ( $block ) < 2 ) {
                                $blockOut .= $this->recentChangesBlockLine ( array_shift ( $block ) ) ;
@@ -2177,15 +2297,16 @@ class Skin {
                        }
                }
 
-               return "<div>{$blockOut}</div>" ;
+               return '<div>'.$blockOut.'</div>' ;
        }
 
-       # Called in a loop over all displayed RC entries
-       # Either returns the line, or caches it for later use
-       function recentChangesLine( &$rc, $watched = false )
-       {
+       /**
+        * Called in a loop over all displayed RC entries
+        * Either returns the line, or caches it for later use
+        */
+       function recentChangesLine( &$rc, $watched = false ) {
                global $wgUser ;
-               $usenew = $wgUser->getOption( "usenewrc" );
+               $usenew = $wgUser->getOption( 'usenewrc' );
                if ( $usenew )
                        $line = $this->recentChangesLineNew ( $rc, $watched ) ;
                else
@@ -2193,76 +2314,91 @@ class Skin {
                return $line ;
        }
 
-       function recentChangesLineOld( &$rc, $watched = false )
-       {
-               global $wgTitle, $wgLang, $wgUser, $wgRCSeconds;
+       function recentChangesLineOld( &$rc, $watched = false ) {
+               global $wgTitle, $wgLang, $wgUser, $wgRCSeconds, $wgUseRCPatrol, $wgOnlySysopsCanPatrol;
 
                # Extract DB fields into local scope
                extract( $rc->mAttribs );
-               $curIdEq = "curid=" . $rc_cur_id;
+               $curIdEq = 'curid=' . $rc_cur_id;
 
                # Make date header if necessary
                $date = $wgLang->date( $rc_timestamp, true);
-               $s = "";
+               $s = '';
                if ( $date != $this->lastdate ) {
-                       if ( "" != $this->lastdate ) { $s .= "</ul>\n"; }
+                       if ( '' != $this->lastdate ) { $s .= "</ul>\n"; }
                        $s .= "<h4>{$date}</h4>\n<ul class='special'>";
                        $this->lastdate = $date;
                        $this->rclistOpen = true;
                }
-               $s .= "<li> ";
 
-               if ( $rc_type == RC_MOVE ) {
+               # If this edit has not yet been patrolled, make it stick out
+               $s .= ( ! $wgUseRCPatrol || $rc_patrolled ) ? '<li> ' : '<li class="not_patrolled"> ';
+
+               if ( $rc_type == RC_MOVE || $rc_type == RC_MOVE_OVER_REDIRECT ) {
                        # Diff
-                       $s .= "(" . wfMsg( "diff" ) . ") (";
+                       $s .= '(' . wfMsg( 'diff' ) . ') (';
                        # Hist
-                       $s .= $this->makeKnownLinkObj( $rc->getMovedToTitle(), wfMsg( "hist" ), "action=history" ) .
-                               ") . . ";
-                       
-                       # "[[x]] moved to [[y]]"
-
-                       $s .= wfMsg( "1movedto2", $this->makeKnownLinkObj( $rc->getTitle(), "", "redirect=no" ),
-                               $this->makeKnownLinkObj( $rc->getMovedToTitle(), "" ) );
+                       $s .= $this->makeKnownLinkObj( $rc->getMovedToTitle(), wfMsg( 'hist' ), 'action=history' ) .
+                               ') . . ';
 
+                       # "[[x]] moved to [[y]]"
+                       $msg = ( $rc_type == RC_MOVE ) ? '1movedto2' : '1movedto2_redir';
+                       $s .= wfMsg( $msg, $this->makeKnownLinkObj( $rc->getTitle(), '', 'redirect=no' ),
+                               $this->makeKnownLinkObj( $rc->getMovedToTitle(), '' ) );
+               } elseif( $rc_namespace == NS_SPECIAL && preg_match( '!^Log/(.*)$!', $rc_title, $matches ) ) {
+                       # Log updates, etc
+                       $logtype = $matches[1];
+                       $logname = LogPage::logName( $logtype );
+                       $s .= '(' . $this->makeKnownLinkObj( $rc->getTitle(), $logname ) . ')';
                } else {
                        # Diff link
                        if ( $rc_type == RC_NEW || $rc_type == RC_LOG ) {
-                               $diffLink = wfMsg( "diff" );
+                               $diffLink = wfMsg( 'diff' );
                        } else {
-                               $diffLink = $this->makeKnownLinkObj( $rc->getTitle(), wfMsg( "diff" ),
-                                 "{$curIdEq}&diff={$rc_this_oldid}&oldid={$rc_last_oldid}"  ,'' ,'' , ' tabindex="'.$rc->counter.'"');
+                               if ( $wgUseRCPatrol && $rc_patrolled == 0 && $wgUser->getID() != 0 &&
+                                    ( $wgUser->isSysop() || !$wgOnlySysopsCanPatrol ) )
+                                       $rcidparam = "&rcid={$rc_id}";
+                               else
+                                       $rcidparam = "";
+                               $diffLink = $this->makeKnownLinkObj( $rc->getTitle(), wfMsg( 'diff' ),
+                                 "{$curIdEq}&diff={$rc_this_oldid}&oldid={$rc_last_oldid}{$rcidparam}",
+                                 '', '', ' tabindex="'.$rc->counter.'"');
                        }
-                       $s .= "($diffLink) (";
+                       $s .= '('.$diffLink.') (';
 
                        # History link
-                       $s .= $this->makeKnownLinkObj( $rc->getTitle(), wfMsg( "hist" ), "{$curIdEq}&action=history" );
-                       $s .= ") . . ";
+                       $s .= $this->makeKnownLinkObj( $rc->getTitle(), wfMsg( 'hist' ), $curIdEq.'&action=history' );
+                       $s .= ') . . ';
 
                        # M and N (minor and new)
-                       $M = wfMsg( "minoreditletter" );
-                       $N = wfMsg( "newpageletter" );
-                       if ( $rc_minor ) { $s .= " <strong>{$M}</strong>"; }
-                       if ( $rc_type == RC_NEW ) { $s .= "<strong>{$N}</strong>"; }
+                       if ( $rc_minor ) { $s .= ' <span class="minor">'.wfMsg( "minoreditletter" ).'</span>'; }
+                       if ( $rc_type == RC_NEW ) { $s .= '<span class="newpage">'.wfMsg( "newpageletter" ).'</span>'; }
 
                        # Article link
-                       $articleLink = $this->makeKnownLinkObj( $rc->getTitle(), "" );
+                       # If it's a new article, there is no diff link, but if it hasn't been
+                       # patrolled yet, we need to give users a way to do so
+                       if ( $wgUseRCPatrol && $rc_type == RC_NEW && $rc_patrolled == 0 &&
+                            $wgUser->getID() != 0 && ( $wgUser->isSysop() || !$wgOnlySysopsCanPatrol ) )
+                               $articleLink = $this->makeKnownLinkObj( $rc->getTitle(), '', "rcid={$rc_id}" );
+                       else
+                               $articleLink = $this->makeKnownLinkObj( $rc->getTitle(), '' );
 
                        if ( $watched ) {
-                               $articleLink = "<strong>{$articleLink}</strong>";
+                               $articleLink = '<strong>'.$articleLink.'</strong>';
                        }
-                       $s .= " $articleLink";
+                       $s .= ' '.$articleLink;
 
                }
 
                # Timestamp
-               $s .= "; " . $wgLang->time( $rc_timestamp, true, $wgRCSeconds ) . " . . ";
+               $s .= '; ' . $wgLang->time( $rc_timestamp, true, $wgRCSeconds ) . ' . . ';
 
                # User link (or contributions for unregistered users)
                if ( 0 == $rc_user ) {
-                       $userLink = $this->makeKnownLink( $wgLang->specialPage( "Contributions" ),
-                       $rc_user_text, "target=" . $rc_user_text );
+                       $userLink = $this->makeKnownLink( $wgLang->specialPage( 'Contributions' ),
+                       $rc_user_text, 'target=' . $rc_user_text );
                } else {
-                       $userLink = $this->makeLink( $wgLang->getNsText( NS_USER ) . ":{$rc_user_text}", $rc_user_text );
+                       $userLink = $this->makeLink( $wgLang->getNsText( NS_USER ) . ':'.$rc_user_text, $rc_user_text );
                }
                $s .= $userLink;
 
@@ -2270,37 +2406,35 @@ class Skin {
                $talkname=$wgLang->getNsText(NS_TALK); # use the shorter name
                global $wgDisableAnonTalk;
                if( 0 == $rc_user && $wgDisableAnonTalk ) {
-                       $userTalkLink = "";
+                       $userTalkLink = '';
                } else {
                        $utns=$wgLang->getNsText(NS_USER_TALK);
-                       $userTalkLink= $this->makeLink($utns . ":{$rc_user_text}", $talkname );
+                       $userTalkLink= $this->makeLink($utns . ':'.$rc_user_text, $talkname );
                }
                # Block link
-               $blockLink="";
+               $blockLink='';
                if ( ( 0 == $rc_user ) && $wgUser->isSysop() ) {
                        $blockLink = $this->makeKnownLink( $wgLang->specialPage(
-                         "Blockip" ), wfMsg( "blocklink" ), "ip={$rc_user_text}" );
+                         'Blockip' ), wfMsg( 'blocklink' ), 'ip='.$rc_user_text );
 
                }
                if($blockLink) {
-                       if($userTalkLink) $userTalkLink .= " | ";
+                       if($userTalkLink) $userTalkLink .= ' | ';
                        $userTalkLink .= $blockLink;
                }
-               if($userTalkLink) $s.=" ({$userTalkLink})";
+               if($userTalkLink) $s.=' ('.$userTalkLink.')';
 
                # Add comment
-               if ( "" != $rc_comment && "*" != $rc_comment && $rc_type != RC_MOVE ) {
-                       $rc_comment=$this->formatComment($rc_comment);
-                       $s .= $wgLang->emphasize(" (" . $rc_comment . ")");
+               if ( '' != $rc_comment && '*' != $rc_comment && $rc_type != RC_MOVE && $rc_type != RC_MOVE_OVER_REDIRECT ) {
+                       $rc_comment=$this->formatComment($rc_comment,$rc->getTitle());
+                       $s .= $wgLang->emphasize(' (' . $rc_comment . ')');
                }
                $s .= "</li>\n";
 
                return $s;
        }
 
-#      function recentChangesLineNew( $ts, $u, $ut, $ns, $ttl, $c, $isminor, $isnew, $watched = false, $oldid = 0 , $diffid = 0 )
-       function recentChangesLineNew( &$baseRC, $watched = false )
-       {
+       function recentChangesLineNew( &$baseRC, $watched = false ) {
                global $wgTitle, $wgLang, $wgUser, $wgRCSeconds;
 
                # Create a specialised object
@@ -2308,11 +2442,11 @@ class Skin {
 
                # Extract fields from DB into the function scope (rc_xxxx variables)
                extract( $rc->mAttribs );
-               $curIdEq = "curid=" . $rc_cur_id;
+               $curIdEq = 'curid=' . $rc_cur_id;
 
                # If it's a new day, add the headline and flush the cache
                $date = $wgLang->date( $rc_timestamp, true);
-               $ret = "" ;
+               $ret = '';
                if ( $date != $this->lastdate ) {
                        # Process current cache
                        $ret = $this->recentChangesBlock () ;
@@ -2320,78 +2454,87 @@ class Skin {
                        $ret .= "<h4>{$date}</h4>\n";
                        $this->lastdate = $date;
                }
-               
+
                # Make article link
-               if ( $rc_type == RC_MOVE ) {
-                       $clink = $this->makeKnownLinkObj( $rc->getTitle(), "", "redirect=no" );
-                       $clink .= " " . wfMsg("movedto") . " ";
-                       $clink .= $this->makeKnownLinkObj( $rc->getMovedToTitle(), "" );
+               if ( $rc_type == RC_MOVE || $rc_type == RC_MOVE_OVER_REDIRECT ) {
+                       $msg = ( $rc_type == RC_MOVE ) ? "1movedto2" : "1movedto2_redir";
+                       $clink = wfMsg( $msg, $this->makeKnownLinkObj( $rc->getTitle(), '', 'redirect=no' ),
+                         $this->makeKnownLinkObj( $rc->getMovedToTitle(), '' ) );
+               } elseif( $rc_namespace == NS_SPECIAL && preg_match( '!^Log/(.*)$!', $rc_title, $matches ) ) {
+                       # Log updates, etc
+                       $logtype = $matches[1];
+                       $logname = LogPage::logName( $logtype );
+                       $clink = '(' . $this->makeKnownLinkObj( $rc->getTitle(), $logname ) . ')';
                } else {
-                       $clink = $this->makeKnownLinkObj( $rc->getTitle(), "" ) ;
+                       $clink = $this->makeKnownLinkObj( $rc->getTitle(), '' ) ;
                }
-               
+
                $time = $wgLang->time( $rc_timestamp, true, $wgRCSeconds );
                $rc->watched = $watched ;
                $rc->link = $clink ;
                $rc->timestamp = $time;
-               
-               # Make "cur" link
-               if ( ( $rc_type == RC_NEW && $rc_this_oldid == 0 ) || $rc_type == RC_LOG || $rc_type == RC_MOVE) {
-                       $curLink = wfMsg( "cur" );
+
+               # Make "cur" and "diff" links
+               if ( ( $rc_type == RC_NEW && $rc_this_oldid == 0 ) || $rc_type == RC_LOG || $rc_type == RC_MOVE || $rc_type == RC_MOVE_OVER_REDIRECT ) {
+                       $curLink = wfMsg( 'cur' );
+                       $diffLink = wfMsg( 'diff' );
                } else {
-                       $curLink = $this->makeKnownLinkObj( $rc->getTitle(), wfMsg( "cur" ),
-                         "{$curIdEq}&diff=0&oldid={$rc_this_oldid}" ,'' ,'' , ' tabindex="'.$baseRC->counter.'"' );
+                       $query = $curIdEq.'&diff=0&oldid='.$rc_this_oldid;
+                       $aprops = ' tabindex="'.$baseRC->counter.'"';
+                       $curLink = $this->makeKnownLinkObj( $rc->getTitle(), wfMsg( 'cur' ), $query, '' ,'' , $aprops );
+                       $diffLink = $this->makeKnownLinkObj( $rc->getTitle(), wfMsg( 'diff'), $query, '' ,'' , $aprops );
                }
 
                # Make "last" link
                $titleObj = $rc->getTitle();
-               if ( $rc_last_oldid == 0 || $rc_type == RC_LOG || $rc_type == RC_MOVE ) {
-                       $lastLink = wfMsg( "last" );
+               if ( $rc_last_oldid == 0 || $rc_type == RC_LOG || $rc_type == RC_MOVE || $rc_type == RC_MOVE_OVER_REDIRECT ) {
+                       $lastLink = wfMsg( 'last' );
                } else {
-                       $lastLink = $this->makeKnownLinkObj( $rc->getTitle(), wfMsg( "last" ),
-                         "{$curIdEq}&diff={$rc_this_oldid}&oldid={$rc_last_oldid}" );
+                       $lastLink = $this->makeKnownLinkObj( $rc->getTitle(), wfMsg( 'last' ),
+                         $curIdEq.'&diff='.$rc_this_oldid.'&oldid='.$rc_last_oldid );
                }
 
                # Make user link (or user contributions for unregistered users)
-               if ( 0 == $rc_user ) {
-                       $userLink = $this->makeKnownLink( $wgLang->specialPage( "Contributions" ),
-                       $rc_user_text, "target=" . $rc_user_text );
-               } else { 
+               if ( $rc_user == 0 ) {
+                       $userLink = $this->makeKnownLink( $wgLang->specialPage( 'Contributions' ),
+                       $rc_user_text, 'target=' . $rc_user_text );
+               } else {
                        $userLink = $this->makeLink( $wgLang->getNsText(
-                         Namespace::getUser() ) . ":{$rc_user_text}", $rc_user_text ); 
+                         Namespace::getUser() ) . ':'.$rc_user_text, $rc_user_text );
                }
 
-               $rc->userlink = $userLink ;
-               $rc->lastlink = $lastLink ;
-               $rc->curlink = $curLink ;
+               $rc->userlink = $userLink;
+               $rc->lastlink = $lastLink;
+               $rc->curlink  = $curLink;
+               $rc->difflink = $diffLink;
 
-               # Make user talk link           
+               # Make user talk link
                $utns=$wgLang->getNsText(NS_USER_TALK);
                $talkname=$wgLang->getNsText(NS_TALK); # use the shorter name
-               $userTalkLink= $this->makeLink($utns . ":{$rc_user_text}", $talkname ); 
-               
+               $userTalkLink= $this->makeLink($utns . ':'.$rc_user_text, $talkname );
+
                global $wgDisableAnonTalk;
                if ( ( 0 == $rc_user ) && $wgUser->isSysop() ) {
                        $blockLink = $this->makeKnownLink( $wgLang->specialPage(
-                         "Blockip" ), wfMsg( "blocklink" ), "ip={$rc_user_text}" );
+                         'Blockip' ), wfMsg( 'blocklink' ), 'ip='.$rc_user_text );
                        if( $wgDisableAnonTalk )
-                               $rc->usertalklink = " ({$blockLink})";
+                               $rc->usertalklink = ' ('.$blockLink.')';
                        else
-                               $rc->usertalklink = " ({$userTalkLink} | {$blockLink})";
+                               $rc->usertalklink = ' ('.$userTalkLink.' | '.$blockLink.')';
                } else {
                        if( $wgDisableAnonTalk && ($rc_user == 0) )
-                               $rc->usertalklink = "";
+                               $rc->usertalklink = '';
                        else
-                               $rc->usertalklink = " ({$userTalkLink})";
+                               $rc->usertalklink = ' ('.$userTalkLink.')';
                }
 
                # Put accumulated information into the cache, for later display
                # Page moves go on their own line
                $title = $rc->getTitle();
                $secureName = $title->getPrefixedDBkey();
-               if ( $rc_type == RC_MOVE ) {
+               if ( $rc_type == RC_MOVE || $rc_type == RC_MOVE_OVER_REDIRECT ) {
                        # Use an @ character to prevent collision with page names
-                       $this->rc_cache["@@" . ($this->rcMoveIndex++)] = array($rc);
+                       $this->rc_cache['@@' . ($this->rcMoveIndex++)] = array($rc);
                } else {
                        if ( !isset ( $this->rc_cache[$secureName] ) ) $this->rc_cache[$secureName] = array() ;
                        array_push ( $this->rc_cache[$secureName] , $rc ) ;
@@ -2399,95 +2542,119 @@ class Skin {
                return $ret;
        }
 
-       function endImageHistoryList()
-       {
+       function endImageHistoryList() {
                $s = "</ul>\n";
                return $s;
        }
 
-       /* This function is called by all recent changes variants, by the page history,
-          and by the user contributions list. It is responsible for formatting edit
-          comments. It escapes any HTML in the comment, but adds some CSS to format
-          auto-generated comments (from section editing) and formats [[wikilinks]].
-          Main author: Erik Möller (moeller@scireview.de)
-       */
-       function formatComment($comment)
-       {
+       /**
+        * This function is called by all recent changes variants, by the page history,
+        * and by the user contributions list. It is responsible for formatting edit
+        * 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
+        */
+       function formatComment($comment, $title = NULL) {
                global $wgLang;
-               $comment=wfEscapeHTML($comment);
+               $comment = htmlspecialchars( $comment );
 
                # The pattern for autogen comments is / * foo * /, which makes for
                # 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
-               while (preg_match("/(.*)\/\*\s*(.*?)\s*\*\/(.*)/", $comment,$match)) {
+               while (preg_match('/(.*)\/\*\s*(.*?)\s*\*\/(.*)/', $comment,$match)) {
                        $pre=$match[1];
                        $auto=$match[2];
                        $post=$match[3];
-                       $sep="-";
-                       if($pre) { $auto="$sep ".$auto; }
-                       if($post) { $auto.=" $sep"; }
-                       $auto="<span class=\"autocomment\">".$auto."</span>";
+                       $link='';
+                       if($title) {
+                               $section=$auto;
+
+                               # This is hackish but should work in most cases.
+                               $section=str_replace('[[','',$section);
+                               $section=str_replace(']]','',$section);
+                               $title->mFragment=$section;
+                               $link=$this->makeKnownLinkObj($title,wfMsg('sectionlink'));
+                       }
+                       $sep='-';
+                       $auto=$link.$auto;
+                       if($pre) { $auto = $sep.' '.$auto; }
+                       if($post) { $auto .= ' '.$sep; }
+                       $auto='<span class="autocomment">'.$auto.'</span>';
                        $comment=$pre.$auto.$post;
                }
 
                # format regular and media links - all other wiki formatting
                # is ignored
-               while(preg_match("/\[\[(.*?)(\|(.*?))*\]\]/",$comment,$match)) {
-
-                       $medians = $wgLang->getNsText(Namespace::getMedia());
-                       $func="makeLink";
-                       if(preg_match("/^".$medians."/i",$match[1])) {
-                               $func="makeMediaLink";
+               $medians = $wgLang->getNsText(Namespace::getMedia()).':';
+               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];
                        }
-                       if(isset($match[3]) ) {
-                               $comment=
-                               preg_replace("/\[\[(.*?)\]\]/",
-                               $this->$func($match[1],$match[3]),$comment,1);
+                       if( preg_match( '/^' . $medians . '(.*)$/i', $match[1], $submatch ) ) {
+                               # Media link; trail not supported.
+                               $linkRegexp = '/\[\[(.*?)\]\]/';
+                               $thelink = $this->makeMediaLink( $submatch[1], "", $text );
                        } else {
-                               $comment=
-                               preg_replace("/\[\[(.*?)\]\]/",
-                               $this->$func($match[1],$match[1]),$comment,1);
+                               # Other kind of link
+                               if( preg_match( wfMsg( "linktrail" ), $match[4], $submatch ) ) {
+                                       $trail = $submatch[1];
+                               } else {
+                                       $trail = "";
+                               }
+                               $linkRegexp = '/\[\[(.*?)\]\]' . preg_quote( $trail, '/' ) . '/';
+                               if ($match[1][0] == ':')
+                                       $match[1] = substr($match[1], 1);
+                               $thelink = $this->makeLink( $match[1], $text, "", $trail );
                        }
+                       $comment = preg_replace( $linkRegexp, $thelink, $comment, 1 );
                }
-
                return $comment;
-
        }
 
-       function imageHistoryLine( $iscur, $timestamp, $img, $user, $usertext, $size, $description )
-       {
+       function imageHistoryLine( $iscur, $timestamp, $img, $user, $usertext, $size, $description ) {
                global $wgUser, $wgLang, $wgTitle;
 
                $datetime = $wgLang->timeanddate( $timestamp, true );
-               $del = wfMsg( "deleteimg" );
-               $cur = wfMsg( "cur" );
+               $del = wfMsg( 'deleteimg' );
+               $delall = wfMsg( 'deleteimgcompletely' );
+               $cur = wfMsg( 'cur' );
 
                if ( $iscur ) {
                        $url = Image::wfImageUrl( $img );
                        $rlink = $cur;
                        if ( $wgUser->isSysop() ) {
-                               $link = $wgTitle->escapeLocalURL( "image=" . $wgTitle->getPartialURL() .
-                                 "&action=delete" );
-                               $style = $this->getInternalLinkAttributes( $link, $del );
+                               $link = $wgTitle->escapeLocalURL( 'image=' . $wgTitle->getPartialURL() .
+                                 '&action=delete' );
+                               $style = $this->getInternalLinkAttributes( $link, $delall );
 
-                               $dlink = "<a href=\"{$link}\"{$style}>{$del}</a>";
+                               $dlink = '<a href="'.$link.'"'.$style.'>'.$delall.'</a>';
                        } else {
                                $dlink = $del;
                        }
                } else {
-                       $url = wfEscapeHTML( wfImageArchiveUrl( $img ) );
-                       if( $wgUser->getID() != 0 ) {
+                       $url = htmlspecialchars( wfImageArchiveUrl( $img ) );
+                       if( $wgUser->getID() != 0 && $wgTitle->userCanEdit() ) {
                                $rlink = $this->makeKnownLink( $wgTitle->getPrefixedText(),
-                                          wfMsg( "revertimg" ), "action=revert&oldimage=" .
+                                          wfMsg( 'revertimg' ), 'action=revert&oldimage=' .
                                           urlencode( $img ) );
                                $dlink = $this->makeKnownLink( $wgTitle->getPrefixedText(),
-                                          $del, "action=delete&oldimage=" . urlencode( $img ) );
+                                          $del, 'action=delete&oldimage=' . urlencode( $img ) );
                        } else {
                                # Having live active links for non-logged in users
                                # means that bots and spiders crawling our site can
                                # inadvertently change content. Baaaad idea.
-                               $rlink = wfMsg( "revertimg" );
+                               $rlink = wfMsg( 'revertimg' );
                                $dlink = $del;
                        }
                }
@@ -2495,37 +2662,39 @@ class Skin {
                        $userlink = $usertext;
                } else {
                        $userlink = $this->makeLink( $wgLang->getNsText( Namespace::getUser() ) .
-                                      ":{$usertext}", $usertext );
+                                      ':'.$usertext, $usertext );
                }
-               $nbytes = wfMsg( "nbytes", $size );
+               $nbytes = wfMsg( 'nbytes', $size );
                $style = $this->getInternalLinkAttributes( $url, $datetime );
 
                $s = "<li> ({$dlink}) ({$rlink}) <a href=\"{$url}\"{$style}>{$datetime}</a>"
                  . " . . {$userlink} ({$nbytes})";
 
-               if ( "" != $description && "*" != $description ) {
+               if ( '' != $description && '*' != $description ) {
                        $sk=$wgUser->getSkin();
-                       $s .= $wgLang->emphasize(" (" . $sk->formatComment($description) . ")");
+                       $s .= $wgLang->emphasize(' (' . $sk->formatComment($description,$wgTitle) . ')');
                }
                $s .= "</li>\n";
                return $s;
        }
 
        function tocIndent($level) {
-               return str_repeat( "<div class='tocindent'>\n", $level>0 ? $level : 0 );
+               return str_repeat( '<div class="tocindent">'."\n", $level>0 ? $level : 0 );
        }
 
        function tocUnindent($level) {
                return str_repeat( "</div>\n", $level>0 ? $level : 0 );
        }
 
-       # parameter level defines if we are on an indentation level
+       /**
+        * parameter level defines if we are on an indentation level
+        */
        function tocLine( $anchor, $tocline, $level ) {
-               $link = "<a href=\"#$anchor\">$tocline</a><br />";
+               $link = '<a href="#'.$anchor.'">'.$tocline.'</a><br />';
                if($level) {
-                       return "$link\n";
+                       return $link."\n";
                } else {
-                       return "<div class='tocline'>$link</div>\n";
+                       return '<div class="tocline">'.$link."</div>\n";
                }
 
        }
@@ -2533,159 +2702,167 @@ class Skin {
        function tocTable($toc) {
                # note to CSS fanatics: putting this in a div does not work -- div won't auto-expand
                # try min-width & co when somebody gets a chance
-               $hideline = " <script type='text/javascript'>showTocToggle(\"" . addslashes( wfMsg("showtoc") ) . "\",\"" . addslashes( wfMsg("hidetoc") ) . "\")</script>";
+               $hideline = ' <script type="text/javascript">showTocToggle("' . addslashes( wfMsg('showtoc') ) . '","' . addslashes( wfMsg('hidetoc') ) . '")</script>';
                return
-               "<table border=\"0\" id=\"toc\"><tr><td align=\"center\">\n".
-               "<b>".wfMsg("toc")."</b>" .
+               '<table border="0" id="toc"><tr id="toctitle"><td align="center">'."\n".
+               '<b>'.wfMsg('toc').'</b>' .
                $hideline .
-               "</td></tr><tr id='tocinside'><td>\n".
+               '</td></tr><tr id="tocinside"><td>'."\n".
                $toc."</td></tr></table>\n";
        }
 
-       # These two do not check for permissions: check $wgTitle->userCanEdit before calling them
+       /**
+        * These two do not check for permissions: check $wgTitle->userCanEdit
+        * before calling them
+        */
        function editSectionScript( $section, $head ) {
                global $wgTitle, $wgRequest;
-               if( $wgRequest->getInt( "oldid" ) && ( $wgRequest->getVal( "diff" ) != "0" ) ) {
+               if( $wgRequest->getInt( 'oldid' ) && ( $wgRequest->getVal( 'diff' ) != '0' ) ) {
                        return $head;
                }
-               $url = $wgTitle->escapeLocalURL( "action=edit&section=$section" );
-               return "<span oncontextmenu='document.location=\"$url\";return false;'>{$head}</span>";
+               $url = $wgTitle->escapeLocalURL( 'action=edit&section='.$section );
+               return '<span oncontextmenu=\'document.location="'.$url.'";return false;\'>'.$head.'</span>';
        }
 
        function editSectionLink( $section ) {
                global $wgRequest;
                global $wgTitle, $wgUser, $wgLang;
 
-               if( $wgRequest->getInt( "oldid" ) && ( $wgRequest->getVal( "diff" ) != "0" ) ) {
+               if( $wgRequest->getInt( 'oldid' ) && ( $wgRequest->getVal( 'diff' ) != '0' ) ) {
                        # Section edit links would be out of sync on an old page.
                        # But, if we're diffing to the current page, they'll be
                        # correct.
-                       return "";
+                       return '';
                }
 
-               $editurl = "&section={$section}";
-               $url = $this->makeKnownLink($wgTitle->getPrefixedText(),wfMsg("editsection"),"action=edit".$editurl);
+               $editurl = '&section='.$section;
+               $url = $this->makeKnownLink($wgTitle->getPrefixedText(),wfMsg('editsection'),'action=edit'.$editurl);
 
                if( $wgLang->isRTL() ) {
-                       $farside = "left";
-                       $nearside = "right";
+                       $farside = 'left';
+                       $nearside = 'right';
                } else {
-                       $farside = "right";
-                       $nearside = "left";
+                       $farside = 'right';
+                       $nearside = 'left';
                }
                return "<div class=\"editsection\" style=\"float:$farside;margin-$nearside:5px;\">[".$url."]</div>";
 
        }
 
-       // This function is called by EditPage.php and shows a bulletin board style
-       // toolbar for common editing functions. It can be disabled in the user preferences.
-       // The necsesary JavaScript code can be found in style/wikibits.js.
+       /**
+        * This function is called by EditPage.php and shows a bulletin board style
+        * toolbar for common editing functions. It can be disabled in the user
+        * preferences.
+        * The necsesary JavaScript code can be found in style/wikibits.js.
+        */
        function getEditToolbar() {
                global $wgStylePath, $wgLang, $wgMimeType;
 
-               // toolarray an array of arrays which each include the filename of
-               // the button image (without path), the opening tag, the closing tag,
-               // and optionally a sample text that is inserted between the two when no
-               // selection is highlighted.
-               // The tip text is shown when the user moves the mouse over the button.
-
-               // Already here are accesskeys (key), which are not used yet until someone
-               // can figure out a way to make them work in IE. However, we should make
-               // sure these keys are not defined on the edit page.
+               /**
+                * toolarray an array of arrays which each include the filename of
+                * the button image (without path), the opening tag, the closing tag,
+                * and optionally a sample text that is inserted between the two when no
+                * selection is highlighted.
+                * The tip text is shown when the user moves the mouse over the button.
+                *
+                * Already here are accesskeys (key), which are not used yet until someone
+                * can figure out a way to make them work in IE. However, we should make
+                * sure these keys are not defined on the edit page.
+                */
                $toolarray=array(
-                       array(  "image"=>"button_bold.png",
-                               "open"=>"\'\'\'",
-                               "close"=>"\'\'\'",
-                               "sample"=>wfMsg("bold_sample"),
-                               "tip"=>wfMsg("bold_tip"),
-                               "key"=>"B"
+                       array(  'image'=>'button_bold.png',
+                                       'open'  =>      "\'\'\'",
+                                       'close' =>      "\'\'\'",
+                                       'sample'=>      wfMsg('bold_sample'),
+                                       'tip'   =>      wfMsg('bold_tip'),
+                                       'key'   =>      'B'
                                ),
-                       array(  "image"=>"button_italic.png",
-                               "open"=>"\'\'",
-                               "close"=>"\'\'",
-                               "sample"=>wfMsg("italic_sample"),
-                               "tip"=>wfMsg("italic_tip"),
-                               "key"=>"I"
+                       array(  'image'=>'button_italic.png',
+                                       'open'  =>      "\'\'",
+                                       'close' =>      "\'\'",
+                                       'sample'=>      wfMsg('italic_sample'),
+                                       'tip'   =>      wfMsg('italic_tip'),
+                                       'key'   =>      'I'
                                ),
-                       array(  "image"=>"button_link.png",
-                               "open"=>"[[",
-                               "close"=>"]]",
-                               "sample"=>wfMsg("link_sample"),
-                               "tip"=>wfMsg("link_tip"),
-                               "key"=>"L"
+                       array(  'image'=>'button_link.png',
+                                       'open'  =>      '[[',
+                                       'close' =>      ']]',
+                                       'sample'=>      wfMsg('link_sample'),
+                                       'tip'   =>      wfMsg('link_tip'),
+                                       'key'   =>      'L'
                                ),
-                       array(  "image"=>"button_extlink.png",
-                               "open"=>"[",
-                               "close"=>"]",
-                               "sample"=>wfMsg("extlink_sample"),
-                               "tip"=>wfMsg("extlink_tip"),
-                               "key"=>"X"
+                       array(  'image'=>'button_extlink.png',
+                                       'open'  =>      '[',
+                                       'close' =>      ']',
+                                       'sample'=>      wfMsg('extlink_sample'),
+                                       'tip'   =>      wfMsg('extlink_tip'),
+                                       'key'   =>      'X'
                                ),
-                       array(  "image"=>"button_headline.png",
-                               "open"=>"\\n== ",
-                               "close"=>" ==\\n",
-                               "sample"=>wfMsg("headline_sample"),
-                               "tip"=>wfMsg("headline_tip"),
-                               "key"=>"H"
+                       array(  'image'=>'button_headline.png',
+                                       'open'  =>      "\\n== ",
+                                       'close' =>      " ==\\n",
+                                       'sample'=>      wfMsg('headline_sample'),
+                                       'tip'   =>      wfMsg('headline_tip'),
+                                       'key'   =>      'H'
                                ),
-                       array(  "image"=>"button_image.png",
-                               "open"=>"[[".$wgLang->getNsText(NS_IMAGE).":",
-                               "close"=>"]]",
-                               "sample"=>wfMsg("image_sample"),
-                               "tip"=>wfMsg("image_tip"),
-                               "key"=>"D"
+                       array(  'image'=>'button_image.png',
+                                       'open'  =>      '[['.$wgLang->getNsText(NS_IMAGE).":",
+                                       'close' =>      ']]',
+                                       'sample'=>      wfMsg('image_sample'),
+                                       'tip'   =>      wfMsg('image_tip'),
+                                       'key'   =>      'D'
                                ),
-                       array(  "image"=>"button_media.png",
-                               "open"=>"[[".$wgLang->getNsText(NS_MEDIA).":",
-                               "close"=>"]]",
-                               "sample"=>wfMsg("media_sample"),
-                               "tip"=>wfMsg("media_tip"),
-                               "key"=>"M"
+                       array(  'image' =>      'button_media.png',
+                                       'open'  =>      '[['.$wgLang->getNsText(NS_MEDIA).':',
+                                       'close' =>      ']]',
+                                       'sample'=>      wfMsg('media_sample'),
+                                       'tip'   =>      wfMsg('media_tip'),
+                                       'key'   =>      'M'
                                ),
-                       array(  "image"=>"button_math.png",
-                               "open"=>"\\<math\\>",
-                               "close"=>"\\</math\\>",
-                               "sample"=>wfMsg("math_sample"),
-                               "tip"=>wfMsg("math_tip"),
-                               "key"=>"C"
+                       array(  'image' =>      'button_math.png',
+                                       'open'  =>      "\\<math\\>",
+                                       'close' =>      "\\</math\\>",
+                                       'sample'=>      wfMsg('math_sample'),
+                                       'tip'   =>      wfMsg('math_tip'),
+                                       'key'   =>      'C'
                                ),
-                       array(  "image"=>"button_nowiki.png",
-                               "open"=>"\\<nowiki\\>",
-                               "close"=>"\\</nowiki\\>",
-                               "sample"=>wfMsg("nowiki_sample"),
-                               "tip"=>wfMsg("nowiki_tip"),
-                               "key"=>"N"
+                       array(  'image' =>      'button_nowiki.png',
+                                       'open'  =>      "\\<nowiki\\>",
+                                       'close' =>      "\\</nowiki\\>",
+                                       'sample'=>      wfMsg('nowiki_sample'),
+                                       'tip'   =>      wfMsg('nowiki_tip'),
+                                       'key'   =>      'N'
                                ),
-                       array(  "image"=>"button_sig.png",
-                               "open"=>"--~~~~",
-                               "close"=>"",
-                               "sample"=>"",
-                               "tip"=>wfMsg("sig_tip"),
-                               "key"=>"Y"
+                       array(  'image' =>      'button_sig.png',
+                                       'open'  =>      '--~~~~',
+                                       'close' =>      '',
+                                       'sample'=>      '',
+                                       'tip'   =>      wfMsg('sig_tip'),
+                                       'key'   =>      'Y'
                                ),
-                       array(  "image"=>"button_hr.png",
-                               "open"=>"\\n----\\n",
-                               "close"=>"",
-                               "sample"=>"",
-                               "tip"=>wfMsg("hr_tip"),
-                               "key"=>"R"
+                       array(  'image' =>      'button_hr.png',
+                                       'open'  =>      "\\n----\\n",
+                                       'close' =>      '',
+                                       'sample'=>      '',
+                                       'tip'   =>      wfMsg('hr_tip'),
+                                       'key'   =>      'R'
                                )
                );
                $toolbar ="<script type='text/javascript'>\n/*<![CDATA[*/\n";
-               
+
                $toolbar.="document.writeln(\"<div id='toolbar'>\");\n";
                foreach($toolarray as $tool) {
 
-                       $image=$wgStylePath."/images/".$tool["image"];
-                       $open=$tool["open"];
-                       $close=$tool["close"];
-                       $sample = addslashes( $tool["sample"] );
+                       $image=$wgStylePath.'/images/'.$tool['image'];
+                       $open=$tool['open'];
+                       $close=$tool['close'];
+                       $sample = addslashes( $tool['sample'] );
 
                        // Note that we use the tip both for the ALT tag and the TITLE tag of the image.
                        // Older browsers show a "speedtip" type message only for ALT.
                        // Ideally these should be different, realistically they
                        // probably don't need to be.
-                       $tip = addslashes( $tool["tip"] );
+                       $tip = addslashes( $tool['tip'] );
 
                        #$key = $tool["key"];
 
@@ -2694,20 +2871,18 @@ class Skin {
 
                $toolbar.="addInfobox('" . addslashes( wfMsg( "infobox" ) ) . "','" . addslashes(wfMsg("infobox_alert")) . "');\n";
                $toolbar.="document.writeln(\"</div>\");\n";
-               
+
                $toolbar.="/*]]>*/\n</script>";
                return $toolbar;
        }
 
+       /**
+        * @access public
+        */
+       function suppressUrlExpansion() {
+               return false;
+       }
 }
 
-require_once( "SkinStandard.php" );
-require_once( "SkinNostalgia.php" );
-require_once( "SkinCologneBlue.php" );
-
-if( $wgUsePHPTal ) {
-       require_once( "SkinPHPTal.php" );
 }
-
-
 ?>