remove trailing white space
[lhc/web/wiklou.git] / includes / Skin.php
index 16750d2..5d39bc3 100644 (file)
@@ -1,4 +1,12 @@
 <?php
+
+/**
+ *
+ */
+
+/**
+ * This is not a valid entry point, perform no further processing unless MEDIAWIKI is defined
+ */
 if( defined( "MEDIAWIKI" ) ) {
 
 # See skin.doc
@@ -6,26 +14,49 @@ if( defined( "MEDIAWIKI" ) ) {
 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.  For display purposes, the
-# Language class has internationalized names
+# 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['myskin'] = 'MySkin';
-    #$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' );
 
+/**
+ * @todo document
+ */
 class RCCacheEntry extends RecentChange
 {
        var $secureName, $link;
@@ -41,41 +72,51 @@ 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()
-       {
+       function Skin() {
+               global $wgUseOldExistenceCheck;
+               $postParseLinkColour = !$wgUseOldExistenceCheck;
                $this->linktrail = wfMsg('linktrail');
        }
 
-       function getSkinNames()
-       {
+       function getSkinNames() {
                global $wgValidSkinNames;
                return $wgValidSkinNames;
        }
 
-       function getStylesheet()
-       {
+       function getStylesheet() {
                return 'wikistandard.css';
        }
+
        function getSkinName() {
-               return "standard";
+               return 'standard';
        }
 
-       # Get/set accessor for delayed link colouring
+       /**
+        * Get/set accessor for delayed link colouring
+        */
        function postParseLinkColour( $setting = NULL ) {
                return wfSetVar( $this->postParseLinkColour, $setting );
        }
 
-       function qbSetting()
-       {
+       function qbSetting() {
                global $wgOut, $wgUser;
 
                if ( $wgOut->isQuickbarSuppressed() ) { return 0; }
@@ -84,8 +125,7 @@ class Skin {
                return $q;
        }
 
-       function initPage( &$out )
-       {
+       function initPage( &$out ) {
                $fname = 'Skin::initPage';
                wfProfileIn( $fname );
 
@@ -190,27 +230,32 @@ class Skin {
                $s .= $this->doGetUserStyles();
                return $s."\n";
        }
-       # placeholder, returns generated js in monobook
-       function getUserJs() {
-               return;
-       }
 
-       function getUserStyles()
-       {
+       /**
+        * 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 .= "/*/*/\n"; # <-- Hide the styles from Netscape 4 without hiding them from IE/Mac
+               $s .= "/*/*/ /*<![CDATA[*/\n"; # <-- Hide the styles from Netscape 4 without hiding them from IE/Mac
                $s .= $this->getUserStylesheet();
-               $s .= "/* */\n";
+               $s .= "/*]]>*/ /* */\n";
                $s .= "</style>\n";
                return $s;
        }
 
-       function doGetUserStyles()
-       {
+       /**
+        * Some styles that are set by user through the user settings interface.
+        */
+       function doGetUserStyles() {
                global $wgUser, $wgLang;
 
-               $csspage = $wgLang->getNsText( NS_MEDIAWIKI ) . ":" . $this->getSkinName() . ".css";
+               $csspage = $wgLang->getNsText( NS_MEDIAWIKI ) . ':' . $this->getSkinName() . '.css';
                $s = '@import "'.$this->makeUrl($csspage, 'action=raw&ctype=text/css')."\";\n";
 
                if ( 1 == $wgUser->getOption( 'underline' ) ) {
@@ -229,8 +274,7 @@ class Skin {
                return $s;
        }
 
-       function getBodyOptions()
-       {
+       function getBodyOptions() {
                global $wgUser, $wgTitle, $wgNamespaceBackgrounds, $wgOut, $wgRequest;
 
                extract( $wgRequest->getValues( 'oldid', 'redirect', 'diff' ) );
@@ -258,14 +302,13 @@ class Skin {
                return $a;
        }
 
-       function getExternalLinkAttributes( $link, $text, $class='' )
-       {
+       function getExternalLinkAttributes( $link, $text, $class='' ) {
                global $wgUser, $wgOut, $wgLang;
 
                $link = urldecode( $link );
                $link = $wgLang->checkTitleEncoding( $link );
                $link = str_replace( '_', ' ', $link );
-               $link = wfEscapeHTML( $link );
+               $link = htmlspecialchars( $link );
 
                $r = ($class != '') ? " class='$class'" : " class='external'";
 
@@ -275,13 +318,12 @@ class Skin {
                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 = htmlspecialchars( $link );
 
                if ( $broken == 'stub' ) {
                        $r = ' class="stub"';
@@ -297,8 +339,10 @@ class Skin {
                return $r;
        }
 
-       function getInternalLinkAttributesObj( &$nt, $text, $broken = false )
-       {
+       /**
+        * @param bool $broken
+        */
+       function getInternalLinkAttributesObj( &$nt, $text, $broken = false ) {
                global $wgUser, $wgOut;
 
                if ( $broken == 'stub' ) {
@@ -315,24 +359,25 @@ class Skin {
                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()
-       {
+       /**
+        * This will be called immediately after the <body> tag.  Split into
+        * two functions to make it easier to subclass.
+        */
+       function beforeContent() {
                global $wgUser, $wgOut;
 
                return $this->doBeforeContent();
        }
 
-       function doBeforeContent()
-       {
+       function doBeforeContent() {
                global $wgUser, $wgOut, $wgTitle, $wgLang, $wgSiteNotice;
                $fname = 'Skin::doBeforeContent';
                wfProfileIn( $fname );
@@ -408,10 +453,36 @@ class Skin {
                        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/>';
-                       $catstack = array();
-                       $s.= $wgTitle->getAllParentCategories($catstack);
+
+                       # 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;
@@ -424,15 +495,13 @@ class Skin {
                }
        }
 
-       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;
 
@@ -451,8 +520,7 @@ class Skin {
                        "</p>\n\n<p>" . $this->pageStats() . "</p>\n";
        }
 
-       function doAfterContent()
-       {
+       function doAfterContent() {
                global $wgUser, $wgOut, $wgLang;
                $fname =  'Skin::doAfterContent';
                wfProfileIn( $fname );
@@ -499,8 +567,7 @@ class Skin {
                return $s;
        }
 
-       function pageTitleLinks()
-       {
+       function pageTitleLinks() {
                global $wgOut, $wgTitle, $wgUser, $wgLang, $wgUseApproval, $wgRequest;
 
                extract( $wgRequest->getValues( 'oldid', 'diff' ) );
@@ -512,7 +579,7 @@ class Skin {
                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>";
                        }
@@ -522,7 +589,7 @@ class Skin {
                                $t = $wgTitle->getDBkey();
                                $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>" ;
                        }
@@ -543,6 +610,8 @@ class Skin {
                                                        Namespace::getTalk( Namespace::getUser() ) ) . ":{$n}",
                                                wfMsg('newmessageslink') );
                                $s.= ' | <strong>'. wfMsg( 'newmessages', $tl ) . '</strong>';
+                               # disable caching
+                               $wgOut->setSquidMaxage(0);
                        }
                }
 
@@ -566,8 +635,7 @@ class Skin {
                return '';
        }
 
-       function printableLink()
-       {
+       function printableLink() {
                global $wgOut, $wgFeedClasses, $wgRequest;
 
                $baseurl = $_SERVER['REQUEST_URI'];
@@ -589,8 +657,7 @@ class Skin {
                return $s;
        }
 
-       function pageTitle()
-       {
+       function pageTitle() {
                global $wgOut, $wgTitle, $wgUser;
 
                $s = '<h1 class="pagetitle">' . htmlspecialchars( $wgOut->getPageTitle() ) . '</h1>';
@@ -598,8 +665,7 @@ class Skin {
                return $s;
        }
 
-       function pageSubtitle()
-       {
+       function pageSubtitle() {
                global $wgOut;
 
                $sub = $wgOut->getSubtitle();
@@ -613,8 +679,7 @@ class Skin {
                return $s;
        }
 
-       function subPageSubtitle()
-       {
+       function subPageSubtitle() {
                global $wgOut,$wgTitle,$wgNamespacesWithSubpages;
                $subpages = '';
                if($wgOut->isArticle() && !empty($wgNamespacesWithSubpages[$wgTitle->getNamespace()])) {
@@ -643,8 +708,7 @@ class Skin {
                return $subpages;
        }
 
-       function nameAndLogin()
-       {
+       function nameAndLogin() {
                global $wgUser, $wgTitle, $wgLang, $wgShowIPinHeader, $wgIP;
 
                $li = $wgLang->specialPage( 'Userlogin' );
@@ -701,8 +765,7 @@ class Skin {
                return htmlspecialchars( $this->getSearchLink() );
        }
 
-       function searchForm()
-       {
+       function searchForm() {
                global $wgRequest;
                $search = $wgRequest->getText( 'search' );
 
@@ -716,8 +779,7 @@ class Skin {
                return $s;
        }
 
-       function topLinks()
-       {
+       function topLinks() {
                global $wgOut;
                $sep = " |\n";
 
@@ -734,8 +796,7 @@ class Skin {
                return $s;
        }
 
-       function bottomLinks()
-       {
+       function bottomLinks() {
                global $wgOut, $wgUser, $wgTitle;
                $sep = " |\n";
 
@@ -776,8 +837,7 @@ class Skin {
                return $s;
        }
 
-       function pageStats()
-       {
+       function pageStats() {
                global $wgOut, $wgLang, $wgArticle, $wgRequest;
                global $wgDisableCounters, $wgMaxCredits, $wgShowCreditsIfMax;
 
@@ -855,8 +915,7 @@ class Skin {
                return $img;
        }
 
-       function lastModified()
-       {
+       function lastModified() {
                global $wgLang, $wgArticle;
 
                $timestamp = $wgArticle->getTimestamp();
@@ -869,21 +928,24 @@ class Skin {
                return $s;
        }
 
-       function logoText( $align = '' )
-       {
-               if ( '' != $align ) { $a = ' align="'.$align.'"'; }
+       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;
 
@@ -1028,8 +1090,7 @@ class Skin {
                return $s;
        }
 
-       function specialPagesList()
-       {
+       function specialPagesList() {
                global $wgUser, $wgOut, $wgLang, $wgServer, $wgRedirectScript;
                require_once('SpecialPage.php');
                $a = array();
@@ -1069,37 +1130,32 @@ class Skin {
                return $s;
        }
 
-       function mainPageLink()
-       {
+       function mainPageLink() {
                $mp = wfMsg( 'mainpage' );
                $s = $this->makeKnownLink( $mp, $mp );
                return $s;
        }
 
-       function copyrightLink()
-       {
+       function copyrightLink() {
                $s = $this->makeKnownLink( wfMsg( 'copyrightpage' ),
                  wfMsg( 'copyrightpagename' ) );
                return $s;
        }
 
-       function aboutLink()
-       {
+       function aboutLink() {
                $s = $this->makeKnownLink( wfMsg( 'aboutpage' ),
                  wfMsg( 'aboutwikipedia' ) );
                return $s;
        }
 
 
-      function disclaimerLink()
-       {
+       function disclaimerLink() {
                $s = $this->makeKnownLink( wfMsg( 'disclaimerpage' ),
                  wfMsg( 'disclaimers' ) );
                return $s;
        }
 
-       function editThisPage()
-       {
+       function editThisPage() {
                global $wgOut, $wgTitle, $wgRequest;
 
                $oldid = $wgRequest->getVal( 'oldid' );
@@ -1120,15 +1176,14 @@ class Skin {
 
                        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' );
@@ -1143,8 +1198,7 @@ class Skin {
                return $s;
        }
 
-       function protectThisPage()
-       {
+       function protectThisPage() {
                global $wgUser, $wgOut, $wgTitle, $wgRequest;
 
                $diff = $wgRequest->getVal( 'diff' );
@@ -1165,8 +1219,7 @@ class Skin {
                return $s;
        }
 
-       function watchThisPage()
-       {
+       function watchThisPage() {
                global $wgUser, $wgOut, $wgTitle;
 
                if ( $wgOut->isArticleRelated() ) {
@@ -1186,8 +1239,7 @@ class Skin {
                return $s;
        }
 
-       function moveThisPage()
-       {
+       function moveThisPage() {
                global $wgTitle, $wgLang;
 
                if ( $wgTitle->userCanEdit() ) {
@@ -1197,8 +1249,7 @@ class Skin {
                return $s;
        }
 
-       function historyLink()
-       {
+       function historyLink() {
                global $wgTitle;
 
                $s = $this->makeKnownLink( $wgTitle->getPrefixedText(),
@@ -1206,8 +1257,7 @@ class Skin {
                return $s;
        }
 
-       function whatLinksHere()
-       {
+       function whatLinksHere() {
                global $wgTitle, $wgLang;
 
                $s = $this->makeKnownLink( $wgLang->specialPage( 'Whatlinkshere' ),
@@ -1215,8 +1265,7 @@ class Skin {
                return $s;
        }
 
-       function userContribsLink()
-       {
+       function userContribsLink() {
                global $wgTitle, $wgLang;
 
                $s = $this->makeKnownLink( $wgLang->specialPage( 'Contributions' ),
@@ -1224,8 +1273,7 @@ class Skin {
                return $s;
        }
 
-       function emailUserLink()
-       {
+       function emailUserLink() {
                global $wgTitle, $wgLang;
 
                $s = $this->makeKnownLink( $wgLang->specialPage( 'Emailuser' ),
@@ -1233,8 +1281,7 @@ class Skin {
                return $s;
        }
 
-       function watchPageLinksLink()
-       {
+       function watchPageLinksLink() {
                global $wgOut, $wgTitle, $wgLang;
 
                if ( ! $wgOut->isArticleRelated() ) {
@@ -1247,8 +1294,7 @@ class Skin {
                return $s;
        }
 
-       function otherLanguages()
-       {
+       function otherLanguages() {
                global $wgOut, $wgLang, $wgTitle, $wgUseNewInterlanguage;
 
                $a = $wgOut->getLanguageLinks();
@@ -1291,15 +1337,13 @@ class Skin {
                return $s;
        }
 
-       function bugReportsLink()
-       {
+       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' ) );
@@ -1327,8 +1371,7 @@ class Skin {
                return $s;
        }
 
-       function talkLink()
-       {
+       function talkLink() {
                global $wgLang, $wgTitle, $wgLinkCache;
 
                $tns = $wgTitle->getNamespace();
@@ -1370,8 +1413,7 @@ class Skin {
                return $s;
        }
 
-       function commentLink()
-       {
+       function commentLink() {
                global $wgLang, $wgTitle, $wgLinkCache;
 
                $tns = $wgTitle->getNamespace();
@@ -1393,17 +1435,19 @@ class Skin {
                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.
-       #
+       /**
+        * 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 );
@@ -1448,9 +1492,10 @@ class Skin {
                }
        }
 
-       # 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';
 
@@ -1524,9 +1569,10 @@ class Skin {
                return $retVal;
        }
 
-       # Pass a title object, not a title string
-       function makeKnownLinkObj( &$nt, $text = '', $query = '', $trail = '', $prefix = '' , $aprops = '')
-       {
+       /**
+        * Pass a title object, not a title string
+        */
+       function makeKnownLinkObj( &$nt, $text = '', $query = '', $trail = '', $prefix = '' , $aprops = '' ) {
                global $wgOut, $wgTitle, $wgInputEncoding;
 
                $fname = 'Skin::makeKnownLinkObj';
@@ -1536,6 +1582,8 @@ class Skin {
                        return $text;
                }
                $link = $nt->getPrefixedURL();
+#              if ( '' != $section && substr($section,0,1) != "#" ) {
+#                      $section = ''
 
                if ( '' == $link ) {
                        $u = '';
@@ -1570,11 +1618,16 @@ 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;
 
+               # Fail gracefully
+               if ( ! isset($nt) )
+                       return "<!-- ERROR -->{$prefix}{$text}{$trail}";
+
                $fname = 'Skin::makeBrokenLinkObj';
                wfProfileIn( $fname );
 
@@ -1607,9 +1660,10 @@ class Skin {
                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();
@@ -1636,8 +1690,7 @@ class Skin {
                return $s;
        }
 
-       function makeSelfLinkObj( &$nt, $text = '', $query = '', $trail = '', $prefix = '' )
-       {
+       function makeSelfLinkObj( &$nt, $text = '', $query = '', $trail = '', $prefix = '' ) {
                $u = $nt->escapeLocalURL( $query );
                if ( '' == $text ) {
                        $text = htmlspecialchars( $nt->getPrefixedText() );
@@ -1682,7 +1735,7 @@ class Skin {
        }
        # this can be passed the NS number as defined in Language.php
        /*static*/ function makeNSUrl( $name, $urlaction='', $namespace=0 ) {
-               $title = Title::makeTitle( $namespace, $name );
+               $title = Title::makeTitleSafe( $namespace, $name );
                $this->checkTitle($title, $name);
                return $title->getLocalURL( $urlaction );
        }
@@ -1733,25 +1786,28 @@ class Skin {
                }
        }
 
-       function fnamePart( $url )
-       {
+       function fnamePart( $url ) {
                $basename = strrchr( $url, '/' );
-               if ( false === $basename ) { $basename = $url; }
-               else { $basename = substr( $basename, 1 ); }
-               return wfEscapeHTML( $basename );
+               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 ); }
+               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 );
+               $nt = Title::makeTitleSafe( NS_IMAGE, $name );
                return $this->makeImageLinkObj( $nt, $alt );
        }
 
@@ -1880,24 +1936,33 @@ class Skin {
                return str_replace("\n", ' ',$prefix.$s.$postfix);
        }
 
-       # Make HTML for a thumbnail including image, border and caption
-       # $img is an Image object
+       /**
+        * 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 = 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
@@ -1919,7 +1984,7 @@ class Skin {
 
                if ( $manual_thumb != '' ) # Use manually specified thumbnail
                {
-                       $manual_title = Title::makeTitle( Namespace::getImage(), $manual_thumb ); #new Title ( $manual_thumb ) ;
+                       $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() )
@@ -1959,13 +2024,12 @@ 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 empty string.
@@ -1983,8 +2047,7 @@ class Skin {
                return $s;
        }
 
-       function specialLink( $name, $key = "" )
-       {
+       function specialLink( $name, $key = '' ) {
                global $wgLang;
 
                if ( '' == $key ) { $key = strtolower( $name ); }
@@ -2006,8 +2069,7 @@ class Skin {
        #
 
        # 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 ;
@@ -2016,17 +2078,17 @@ class Skin {
                return '';
        }
 
-       function beginImageHistoryList()
-       {
+       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";
@@ -2034,9 +2096,10 @@ 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
@@ -2091,7 +2154,7 @@ class Skin {
 
                # Comment
                 if ( $rc_comment != '' && $rc_type != RC_MOVE && $rc_type != RC_MOVE_OVER_REDIRECT ) {
-                       $rc_comment=$this->formatComment($rc_comment);
+                       $rc_comment=$this->formatComment($rc_comment, $rcObj->getTitle());
                        $r .= $wgLang->emphasize( ' ('.$rc_comment.')' );
                }
 
@@ -2099,9 +2162,10 @@ class Skin {
                return $r ;
        }
 
-       # Enhanced RC group
-       function recentChangesBlockGroup ( $block )
-       {
+       /**
+        * Enhanced RC group
+        */
+       function recentChangesBlockGroup ( $block ) {
                global $wgStylePath, $wgLang ;
 
                $r = '' ;
@@ -2206,7 +2270,7 @@ class Skin {
                        $r .= ') . . '.$rcObj->userlink ;
                        $r .= $rcObj->usertalklink ;
                        if ( $rc_comment != '' ) {
-                               $rc_comment=$this->formatComment($rc_comment);
+                               $rc_comment=$this->formatComment($rc_comment, $rcObj->getTitle());
                                $r .= $wgLang->emphasize( ' ('.$rc_comment.')' ) ;
                        }
                        $r .= "<br />\n" ;
@@ -2217,10 +2281,11 @@ 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 = '';
@@ -2235,10 +2300,11 @@ class Skin {
                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' );
                if ( $usenew )
@@ -2248,9 +2314,8 @@ 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 );
@@ -2267,7 +2332,7 @@ class Skin {
                }
 
                # If this edit has not yet been patrolled, make it stick out
-               $s .= ( $rc_patrolled ) ? '<li> ' : '<li class="not_patrolled"> ';
+               $s .= ( ! $wgUseRCPatrol || $rc_patrolled ) ? '<li> ' : '<li class="not_patrolled"> ';
 
                if ( $rc_type == RC_MOVE || $rc_type == RC_MOVE_OVER_REDIRECT ) {
                        # Diff
@@ -2280,12 +2345,17 @@ class Skin {
                        $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' );
                        } else {
-                               if ( $rc_patrolled == 0 && $wgUser->getID() != 0 &&
+                               if ( $wgUseRCPatrol && $rc_patrolled == 0 && $wgUser->getID() != 0 &&
                                     ( $wgUser->isSysop() || !$wgOnlySysopsCanPatrol ) )
                                        $rcidparam = "&rcid={$rc_id}";
                                else
@@ -2301,16 +2371,14 @@ class Skin {
                        $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
                        # 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 ( $rc_type == RC_NEW && $rc_patrolled == 0 && $wgUser->getID() != 0 &&
-                            ( $wgUser->isSysop() || !$wgOnlySysopsCanPatrol ) )
+                       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(), '' );
@@ -2358,7 +2426,7 @@ class Skin {
 
                # Add comment
                if ( '' != $rc_comment && '*' != $rc_comment && $rc_type != RC_MOVE && $rc_type != RC_MOVE_OVER_REDIRECT ) {
-                       $rc_comment=$this->formatComment($rc_comment);
+                       $rc_comment=$this->formatComment($rc_comment,$rc->getTitle());
                        $s .= $wgLang->emphasize(' (' . $rc_comment . ')');
                }
                $s .= "</li>\n";
@@ -2366,8 +2434,7 @@ class Skin {
                return $s;
        }
 
-       function recentChangesLineNew( &$baseRC, $watched = false )
-       {
+       function recentChangesLineNew( &$baseRC, $watched = false ) {
                global $wgTitle, $wgLang, $wgUser, $wgRCSeconds;
 
                # Create a specialised object
@@ -2393,6 +2460,11 @@ class Skin {
                        $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(), '' ) ;
                }
@@ -2470,22 +2542,28 @@ 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.
@@ -2495,7 +2573,18 @@ class Skin {
                        $pre=$match[1];
                        $auto=$match[2];
                        $post=$match[3];
+                       $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>';
@@ -2524,6 +2613,8 @@ class Skin {
                                        $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 );
@@ -2531,8 +2622,7 @@ class Skin {
                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 );
@@ -2553,7 +2643,7 @@ class Skin {
                                $dlink = $del;
                        }
                } else {
-                       $url = wfEscapeHTML( wfImageArchiveUrl( $img ) );
+                       $url = htmlspecialchars( wfImageArchiveUrl( $img ) );
                        if( $wgUser->getID() != 0 && $wgTitle->userCanEdit() ) {
                                $rlink = $this->makeKnownLink( $wgTitle->getPrefixedText(),
                                           wfMsg( 'revertimg' ), 'action=revert&oldimage=' .
@@ -2582,7 +2672,7 @@ class Skin {
 
                if ( '' != $description && '*' != $description ) {
                        $sk=$wgUser->getSkin();
-                       $s .= $wgLang->emphasize(' (' . $sk->formatComment($description) . ')');
+                       $s .= $wgLang->emphasize(' (' . $sk->formatComment($description,$wgTitle) . ')');
                }
                $s .= "</li>\n";
                return $s;
@@ -2596,7 +2686,9 @@ class Skin {
                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 />';
                if($level) {
@@ -2619,7 +2711,10 @@ class Skin {
                $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' ) ) {
@@ -2654,98 +2749,103 @@ class Skin {
 
        }
 
-       // 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'
+                                       '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";
@@ -2776,6 +2876,12 @@ class Skin {
                return $toolbar;
        }
 
+       /**
+        * @access public
+        */
+       function suppressUrlExpansion() {
+               return false;
+       }
 }
 
 }