From: Siebrand Mazeland Date: Wed, 17 Sep 2008 09:41:51 +0000 (+0000) Subject: (bug 7509) Don't hardcode separation strings, make them configurable X-Git-Tag: 1.31.0-rc.0~45248 X-Git-Url: http://git.cyclocoop.org/%24image?a=commitdiff_plain;h=0057d58e099cd6441ca7d31f43989b23919d1a42;p=lhc%2Fweb%2Fwiklou.git (bug 7509) Don't hardcode separation strings, make them configurable * Introduced message 'pipe-separator' to replace many occurrence of " | ". No changes made in skin "Nostalgia". --- diff --git a/includes/GlobalFunctions.php b/includes/GlobalFunctions.php index 44d35120da..c43a133a81 100644 --- a/includes/GlobalFunctions.php +++ b/includes/GlobalFunctions.php @@ -926,10 +926,12 @@ function wfViewPrevNext( $offset, $limit, $link, $query = '', $atend = false ) { } else { $nlink = '{$next}"; } - $nums = wfNumLink( $offset, 20, $title, $query ) . ' | ' . - wfNumLink( $offset, 50, $title, $query ) . ' | ' . - wfNumLink( $offset, 100, $title, $query ) . ' | ' . - wfNumLink( $offset, 250, $title, $query ) . ' | ' . + + $separator = wfMsgExt( 'pipe-separator' , 'escapenoentities' ); + $nums = wfNumLink( $offset, 20, $title, $query ) . $separator . + wfNumLink( $offset, 50, $title, $query ) . $separator . + wfNumLink( $offset, 100, $title, $query ) . $separator . + wfNumLink( $offset, 250, $title, $query ) . $separator . wfNumLink( $offset, 500, $title, $query ); return wfMsg( 'viewprevnext', $plink, $nlink, $nums ); diff --git a/includes/Linker.php b/includes/Linker.php index 54f5633dad..cd34d33f33 100644 --- a/includes/Linker.php +++ b/includes/Linker.php @@ -1062,7 +1062,7 @@ class Linker { } if( $items ) { - return ' (' . implode( ' | ', $items ) . ')'; + return ' (' . implode( wfMsgExt( 'pipe-separator' , 'escapenoentities' ), $items ) . ')'; } else { return ''; } diff --git a/includes/PageHistory.php b/includes/PageHistory.php index 4ba67a289b..79e5464d5a 100644 --- a/includes/PageHistory.php +++ b/includes/PageHistory.php @@ -379,7 +379,7 @@ class PageHistory { } if( $tools ) { - $s .= ' (' . implode( ' | ', $tools ) . ')'; + $s .= ' (' . implode( wfMsgExt( 'pipe-separator' , 'escapenoentities' ), $tools ) . ')'; } wfRunHooks( 'PageHistoryLineEnding', array( $this, &$row , &$s ) ); diff --git a/includes/Pager.php b/includes/Pager.php index 3125610754..251082589a 100644 --- a/includes/Pager.php +++ b/includes/Pager.php @@ -513,7 +513,8 @@ abstract class AlphabeticPager extends IndexPager { $pagingLinks = $this->getPagingLinks( $linkTexts ); $limitLinks = $this->getLimitLinks(); - $limits = implode( ' | ', $limitLinks ); + $separator = wfMsgExt( 'pipe-separator' , 'escapenoentities' ); + $limits = implode( $separator, $limitLinks ); $this->mNavigationBar = "({$pagingLinks['first']} | {$pagingLinks['last']}) " . @@ -532,7 +533,7 @@ abstract class AlphabeticPager extends IndexPager { if( $first ) { $first = false; } else { - $extra .= ' | '; + $extra .= $separator; } if( $order == $this->mOrderType ) { @@ -591,11 +592,12 @@ abstract class ReverseChronologicalPager extends IndexPager { 'last' => wfMsgHtml( 'histfirst' ) ); + $separator = wfMsgExt( 'pipe-separator' , 'escapenoentities' ); $pagingLinks = $this->getPagingLinks( $linkTexts ); $limitLinks = $this->getLimitLinks(); - $limits = implode( ' | ', $limitLinks ); + $limits = implode( $separator, $limitLinks ); - $this->mNavigationBar = "({$pagingLinks['first']} | {$pagingLinks['last']}) " . + $this->mNavigationBar = "({$pagingLinks['first']}" . $separator . "{$pagingLinks['last']}) " . wfMsgHtml("viewprevnext", $pagingLinks['prev'], $pagingLinks['next'], $limits); return $this->mNavigationBar; } diff --git a/includes/Skin.php b/includes/Skin.php index f6ba839b3d..f725489fab 100644 --- a/includes/Skin.php +++ b/includes/Skin.php @@ -913,14 +913,15 @@ END; $diff = $wgRequest->getVal( 'diff' ); $action = $wgRequest->getText( 'action' ); + $separator = wfMsgExt( 'pipe-separator' , 'escapenoentities' ); $s = $this->printableLink(); $disclaimer = $this->disclaimerLink(); # may be empty if( $disclaimer ) { - $s .= ' | ' . $disclaimer; + $s .= $separator . $disclaimer; } $privacy = $this->privacyLink(); # may be empty too if( $privacy ) { - $s .= ' | ' . $privacy; + $s .= $separator . $privacy; } if ( $wgOut->isArticleRelated() ) { @@ -930,12 +931,12 @@ END; if( $image ) { $link = htmlspecialchars( $image->getURL() ); $style = $this->getInternalLinkAttributes( $link, $name ); - $s .= " | {$name}"; + $s .= $separator . "{$name}"; } } } if ( 'history' == $action || isset( $diff ) || isset( $oldid ) ) { - $s .= ' | ' . $this->makeKnownLinkObj( $wgTitle, + $s .= $separator . $this->makeKnownLinkObj( $wgTitle, wfMsg( 'currentrev' ) ); } @@ -945,7 +946,7 @@ END; if( !$wgTitle->equals( $wgUser->getTalkPage() ) ) { $tl = $this->makeKnownLinkObj( $wgUser->getTalkPage(), wfMsgHtml( 'newmessageslink' ), 'redirect=no' ); $dl = $this->makeKnownLinkObj( $wgUser->getTalkPage(), wfMsgHtml( 'newmessagesdifflink' ), 'diff=cur' ); - $s.= ' | '. wfMsg( 'youhavenewmessages', $tl, $dl ) . ''; + $s.= $separator . ''. wfMsg( 'youhavenewmessages', $tl, $dl ) . ''; # disable caching $wgOut->setSquidMaxage(0); $wgOut->enableClientCache(false); @@ -954,7 +955,7 @@ END; $undelete = $this->getUndeleteLink(); if( !empty( $undelete ) ) { - $s .= ' | '.$undelete; + $s .= $separator . $undelete; } return $s; } @@ -987,7 +988,7 @@ END; if( $wgOut->isSyndicated() ) { foreach( $wgFeedClasses as $format => $class ) { $feedurl = $wgRequest->escapeAppendQuery( "feed=$format" ); - $s .= " | {$format}"; + $s .= wfMsgExt( 'pipe-separator' , 'escapenoentities' ) . "{$format}"; } } return $s; @@ -1035,7 +1036,7 @@ END; $getlink = $this->makeKnownLinkObj( $linkObj, htmlspecialchars( $display ) ); $c++; if ($c>1) { - $subpages .= ' | '; + $subpages .= wfMsgExt( 'pipe-separator' , 'escapenoentities' ); } else { $subpages .= '< '; } @@ -1102,9 +1103,9 @@ END; SpecialPage::getTitleFor( 'Userlogout' ), wfMsg( 'logout' ), array(), array( 'returnto' => $returnTo ) ); - $ret .= ' | ' . $this->specialLink( 'preferences' ); + $ret .= wfMsgExt( 'pipe-separator' , 'escapenoentities' ) . $this->specialLink( 'preferences' ); } - $ret .= ' | ' . $this->link( + $ret .= wfMsgExt( 'pipe-separator' , 'escapenoentities' ) . $this->link( Title::newFromText( wfMsgForContent( 'helppage' ) ), wfMsg( 'help' ) ); @@ -1170,7 +1171,7 @@ END; $s = ''; wfRunHooks( 'SkinTemplateTabs', array( $this, &$tabs ) ); foreach( $tabs as $tab ) { - $s .= ' | ' . Xml::element( 'a', + $s .= wfMsgExt( 'pipe-separator' , 'escapenoentities' ) . Xml::element( 'a', array( 'href' => $tab['href'] ), $tab['text'] ); } @@ -1191,7 +1192,7 @@ END; $varname = $wgContLang->getVariantname( $code ); if( $varname == 'disable' ) continue; - $s .= ' | ' . htmlspecialchars( $varname ) . ''; + $s .= wfMsgExt( 'pipe-separator' , 'escapenoentities' ) . '' . htmlspecialchars( $varname ) . ''; } } return $s; @@ -1622,7 +1623,7 @@ END; $first = true; if($wgContLang->isRTL()) $s .= ''; foreach( $a as $l ) { - if ( ! $first ) { $s .= ' | '; } + if ( ! $first ) { $s .= wfMsgExt( 'pipe-separator' , 'escapenoentities' ); } $first = false; $nt = Title::newFromText( $l ); diff --git a/includes/WatchlistEditor.php b/includes/WatchlistEditor.php index fcfdb78294..faaa8b93ed 100644 --- a/includes/WatchlistEditor.php +++ b/includes/WatchlistEditor.php @@ -391,7 +391,7 @@ class WatchlistEditor { } return '
  • ' . Xml::check( 'titles[]', false, array( 'value' => $title->getPrefixedText() ) ) - . $link . ' (' . implode( ' | ', $tools ) . ')' . '
    • '; + . $link . ' (' . implode( wfMsgExt( 'pipe-separator' , 'escapenoentities' ), $tools ) . ')' . ''; } /** @@ -457,6 +457,6 @@ class WatchlistEditor { foreach( $modes as $mode => $subpage ) { $tools[] = $skin->makeKnownLinkObj( SpecialPage::getTitleFor( 'Watchlist', $subpage ), wfMsgHtml( "watchlisttools-{$mode}" ) ); } - return implode( ' | ', $tools ); + return implode( wfMsgExt( 'pipe-separator' , 'escapenoentities' ), $tools ); } } diff --git a/includes/specials/SpecialAllmessages.php b/includes/specials/SpecialAllmessages.php index c2a8de4ef0..c4bc313fcf 100644 --- a/includes/specials/SpecialAllmessages.php +++ b/includes/specials/SpecialAllmessages.php @@ -45,7 +45,8 @@ function wfSpecialAllmessages() { $wgOut->addScriptFile( 'allmessages.js' ); if ( $ot == 'php' ) { $navText .= wfAllMessagesMakePhp( $messages ); - $wgOut->addHTML( 'PHP | HTML | ' . + $wgOut->addHTML( 'PHP | HTML' . + wfMsgExt( 'pipe-separator' , 'escapenoentities' ) . 'XML' . '
    ' . htmlspecialchars( $navText ) . '
    ' ); } else if ( $ot == 'xml' ) { @@ -53,8 +54,9 @@ function wfSpecialAllmessages() { header( 'Content-type: text/xml' ); echo wfAllMessagesMakeXml( $messages ); } else { - $wgOut->addHTML( 'PHP | ' . - 'HTML | XML' ); + $wgOut->addHTML( 'PHP' . + wfMsgExt( 'pipe-separator' , 'escapenoentities' ) . + 'HTML' . wfMsgExt( 'pipe-separator' , 'escapenoentities' ) . 'XML' ); $wgOut->addWikiText( $navText ); $wgOut->addHTML( wfAllMessagesMakeHTMLText( $messages ) ); } diff --git a/includes/specials/SpecialAllpages.php b/includes/specials/SpecialAllpages.php index f27c12005f..afe6738a59 100644 --- a/includes/specials/SpecialAllpages.php +++ b/includes/specials/SpecialAllpages.php @@ -328,6 +328,8 @@ class SpecialAllpages extends IncludableSpecialPage { } } + $separator = wfMsgExt( 'pipe-separator' , 'escapenoentities' ); + if ( $this->including() ) { $out2 = ''; } else { @@ -382,7 +384,7 @@ class SpecialAllpages extends IncludableSpecialPage { . ( $namespace ? '&namespace=' . $namespace : '' ); $prevLink = $sk->makeKnownLinkObj( $self, wfMsgHTML( 'prevpage', htmlspecialchars( $pt ) ), $q ); - $out2 .= ' | ' . $prevLink; + $out2 .= $separator . $prevLink; } if( $n == $this->maxPerPage && $s = $res->fetchObject() ) { @@ -392,7 +394,7 @@ class SpecialAllpages extends IncludableSpecialPage { . ( $namespace ? '&namespace=' . $namespace : '' ); $nextLink = $sk->makeKnownLinkObj( $self, wfMsgHtml( 'nextpage', htmlspecialchars( $t->getText() ) ), $q ); - $out2 .= ' | ' . $nextLink; + $out2 .= $separator . $nextLink; } $out2 .= "
    "; } @@ -404,7 +406,7 @@ class SpecialAllpages extends IncludableSpecialPage { $wgOut->addHTML( $prevLink ); } if( isset( $prevLink ) && isset( $nextLink ) ) { - $wgOut->addHTML( ' | ' ); + $wgOut->addHTML( $separator ); } if( isset( $nextLink ) ) { $wgOut->addHTML( $nextLink ); diff --git a/includes/specials/SpecialBlockip.php b/includes/specials/SpecialBlockip.php index a03d564a33..01afac1b86 100644 --- a/includes/specials/SpecialBlockip.php +++ b/includes/specials/SpecialBlockip.php @@ -457,7 +457,7 @@ class IPBlockForm { $links[] = $skin->makeLink ( 'MediaWiki:Ipbreason-dropdown', wfMsgHtml( 'ipb-edit-dropdown' ) ); $links[] = $this->getUnblockLink( $skin ); $links[] = $this->getBlockListLink( $skin ); - return '

    ' . implode( ' | ', $links ) . '

    '; + return '

    ' . implode( wfMsgExt( 'pipe-separator' , 'escapenoentities' ), $links ) . '

    '; } /** diff --git a/includes/specials/SpecialContributions.php b/includes/specials/SpecialContributions.php index 694690dc2a..f1ce6fd9eb 100644 --- a/includes/specials/SpecialContributions.php +++ b/includes/specials/SpecialContributions.php @@ -338,7 +338,7 @@ function contributionsSub( $nt, $id ) { wfRunHooks( 'ContributionsToolLinks', array( $id, $nt, &$tools ) ); - $links = implode( ' | ', $tools ); + $links = implode( wfMsgExt( 'pipe-separator' , 'escapenoentities' ), $tools ); } // Old message 'contribsub' had one parameter, but that doesn't work for diff --git a/includes/specials/SpecialNewpages.php b/includes/specials/SpecialNewpages.php index 8612246083..e65ca96ab7 100644 --- a/includes/specials/SpecialNewpages.php +++ b/includes/specials/SpecialNewpages.php @@ -148,7 +148,7 @@ class SpecialNewpages extends SpecialPage { $links[$key] = wfMsgHtml( $msg, $link ); } - return implode( ' | ', $links ); + return implode( wfMsgExt( 'pipe-separator' , 'escapenoentities' ), $links ); } protected function form() { diff --git a/includes/specials/SpecialPreferences.php b/includes/specials/SpecialPreferences.php index 03f94ab82f..352311309f 100644 --- a/includes/specials/SpecialPreferences.php +++ b/includes/specials/SpecialPreferences.php @@ -634,7 +634,7 @@ class PreferencesForm { $this->tableRow( wfMsgExt( 'prefs-memberingroups', array( 'parseinline' ), count( $userEffectiveGroupsArray ) ), implode( wfMsg( 'comma-separator' ), $userEffectiveGroupsArray ) . - '
    (' . implode( ' | ', $toolLinks ) . ')' + '
    (' . implode( wfMsgExt( 'pipe-separator' , 'escapenoentities' ), $toolLinks ) . ')' ) . $this->tableRow( diff --git a/includes/specials/SpecialPrefixindex.php b/includes/specials/SpecialPrefixindex.php index a360d509b2..c2168c56db 100644 --- a/includes/specials/SpecialPrefixindex.php +++ b/includes/specials/SpecialPrefixindex.php @@ -175,7 +175,7 @@ class SpecialPrefixindex extends SpecialAllpages { wfMsg ( 'allpages' ) ); if( isset( $res ) && $res && ( $n == $this->maxPerPage ) && ( $s = $res->fetchObject() ) ) { $namespaceparam = $namespace ? "&namespace=$namespace" : ""; - $out2 .= " | " . $sk->makeKnownLinkObj( + $out2 .= wfMsgExt( 'pipe-separator' , 'escapenoentities' ) . $sk->makeKnownLinkObj( $self, wfMsgHtml( 'nextpage', htmlspecialchars( $s->page_title ) ), "from=" . wfUrlEncode( $s->page_title ) . diff --git a/includes/specials/SpecialRecentchanges.php b/includes/specials/SpecialRecentchanges.php index 0fe134580f..7eaed2659c 100644 --- a/includes/specials/SpecialRecentchanges.php +++ b/includes/specials/SpecialRecentchanges.php @@ -618,14 +618,16 @@ class SpecialRecentChanges extends SpecialPage { $cl[] = $this->makeOptionsLink( $wgLang->formatNum( $value ), array( 'limit' => $value ), $nondefaults, $value == $options['limit'] ) ; } - $cl = implode( ' | ', $cl ); + + $separator = wfMsgExt( 'pipe-separator' , 'escapenoentities' ); + $cl = implode( $separator, $cl ); // day links, reset 'from' to none foreach( $wgRCLinkDays as $value ) { $dl[] = $this->makeOptionsLink( $wgLang->formatNum( $value ), array( 'days' => $value, 'from' => '' ), $nondefaults, $value == $options['days'] ) ; } - $dl = implode( ' | ', $dl ); + $dl = implode( $separator, $dl ); // show/hide links @@ -650,7 +652,7 @@ class SpecialRecentChanges extends SpecialPage { if( $wgUser->useRCPatrol() ) $links[] = wfMsgHtml( 'rcshowhidepatr', $patrLink ); $links[] = wfMsgHtml( 'rcshowhidemine', $myselfLink ); - $hl = implode( ' | ', $links ); + $hl = implode( $separator, $links ); // show from this onward link $now = $wgLang->timeanddate( wfTimestampNow(), true ); diff --git a/includes/specials/SpecialUserlogin.php b/includes/specials/SpecialUserlogin.php index b6908c94f2..97e7fd4a34 100644 --- a/includes/specials/SpecialUserlogin.php +++ b/includes/specials/SpecialUserlogin.php @@ -970,7 +970,7 @@ class LoginForm { $links[] = $this->makeLanguageSelectorLink( $parts[0], $parts[1] ); } } - return count( $links ) > 0 ? wfMsgHtml( 'loginlanguagelabel', implode( ' | ', $links ) ) : ''; + return count( $links ) > 0 ? wfMsgHtml( 'loginlanguagelabel', implode( wfMsgExt( 'pipe-separator' , 'escapenoentities' ), $links ) ) : ''; } else { return ''; } diff --git a/includes/specials/SpecialWatchlist.php b/includes/specials/SpecialWatchlist.php index ccdc722977..f12ab973df 100644 --- a/includes/specials/SpecialWatchlist.php +++ b/includes/specials/SpecialWatchlist.php @@ -271,7 +271,7 @@ function wfSpecialWatchlist( $par ) { # Namespace filter and put the whole form together. $form .= $wlInfo; $form .= $cutofflinks; - $form .= implode( ' | ', $links ); + $form .= implode( wfMsgExt( 'pipe-separator' , 'escapenoentities' ), $links ); $form .= Xml::openElement( 'form', array( 'method' => 'post', 'action' => $thisTitle->getLocalUrl() ) ); $form .= '

    '; $form .= Xml::label( wfMsg( 'namespace' ), 'namespace' ) . ' '; @@ -382,10 +382,11 @@ function wlCutoffLinks( $days, $page = 'Watchlist', $options = array() ) { foreach( $days as $d ) { $days[$i++] = wlDaysLink( $d, $page, $options ); } + $separator = wfMsgExt( 'pipe-separator' , 'escapenoentities' ); return wfMsgExt('wlshowlast', array('parseinline', 'replaceafter'), - implode(' | ', $hours), - implode(' | ', $days), + implode( $separator, $hours ), + implode( $separator, $days ), wlDaysLink( 0, $page, $options ) ); } diff --git a/includes/specials/SpecialWhatlinkshere.php b/includes/specials/SpecialWhatlinkshere.php index ade28f19a1..d3fa691dec 100644 --- a/includes/specials/SpecialWhatlinkshere.php +++ b/includes/specials/SpecialWhatlinkshere.php @@ -340,7 +340,7 @@ class WhatLinksHerePage { $limitLinks[] = $this->makeSelfLink( $prettyLimit, wfArrayToCGI( $overrides, $changed ) ); } - $nums = implode ( ' | ', $limitLinks ); + $nums = implode ( wfMsgExt( 'pipe-separator' , 'escapenoentities' ), $limitLinks ); return wfMsgHtml( 'viewprevnext', $prev, $next, $nums ); } @@ -403,6 +403,6 @@ class WhatLinksHerePage { $overrides = array( $type => !$chosen ); $links[] = $this->makeSelfLink( $msg, wfArrayToCGI( $overrides, $changed ) ); } - return Xml::fieldset( wfMsg( 'whatlinkshere-filters' ), implode( ' | ', $links ) ); + return Xml::fieldset( wfMsg( 'whatlinkshere-filters' ), implode( wfMsgExt( 'pipe-separator' , 'escapenoentities' ), $links ) ); } } diff --git a/languages/messages/MessagesEn.php b/languages/messages/MessagesEn.php index 27d80ffad9..97c3976506 100644 --- a/languages/messages/MessagesEn.php +++ b/languages/messages/MessagesEn.php @@ -3416,6 +3416,7 @@ $1', 'comma-separator' => ', ', # only translate this message to other languages if you have to change it 'colon-separator' => ': ', # only translate this message to other languages if you have to change it 'autocomment-prefix' => '- ', # only translate this message to other languages if you have to change it +'pipe-separator' => ' | ', # only translate this message to other languages if you have to change it # Multipage image navigation 'imgmultipageprev' => '← previous page', diff --git a/maintenance/language/messageTypes.inc b/maintenance/language/messageTypes.inc index 98d3d8f66d..ed67aab13c 100644 --- a/maintenance/language/messageTypes.inc +++ b/maintenance/language/messageTypes.inc @@ -314,6 +314,7 @@ $wgOptionalMessages = array( 'comma-separator', 'colon-separator', 'autocomment-prefix', + 'pipe-separator', 'listgrouprights-right-display', 'timezone-utc', 'whatlinkshere-backlink', diff --git a/maintenance/language/messages.inc b/maintenance/language/messages.inc index 3ba87df017..2c28b448d9 100644 --- a/maintenance/language/messages.inc +++ b/maintenance/language/messages.inc @@ -2549,6 +2549,7 @@ $wgMessageStructure = array( 'comma-separator', 'colon-separator', 'autocomment-prefix', + 'pipe-separator', ), 'imgmulti' => array( 'imgmultipageprev', diff --git a/skins/CologneBlue.php b/skins/CologneBlue.php index 797396223e..d2a681c2c5 100644 --- a/skins/CologneBlue.php +++ b/skins/CologneBlue.php @@ -79,9 +79,10 @@ class SkinCologneBlue extends Skin { } $s .= ""; + $separator = wfMsgExt( 'pipe-separator' , 'escapenoentities' ); $s .= $this->bottomLinks(); - $s .= "\n
    " . $this->makeKnownLinkObj( Title::newMainPage() ) . " | " - . $this->aboutLink() . " | " + $s .= "\n
    " . $this->makeKnownLinkObj( Title::newMainPage() ) . $separator + . $this->aboutLink() . $separator . $this->searchForm( wfMsg( "qbfind" ) ); $s .= "\n
    " . $this->pageStats(); @@ -132,22 +133,19 @@ class SkinCologneBlue extends Skin { $q = "returnto={$rt}"; } + $separator = wfMsgExt( 'pipe-separator' , 'escapenoentities' ); $s = "" . - $this->mainPageLink() - . " | " . - $this->makeKnownLink( wfMsgForContent( "aboutpage" ), wfMsg( "about" ) ) - . " | " . - $this->makeKnownLink( wfMsgForContent( "helppage" ), wfMsg( "help" ) ) - . " | " . - $this->makeKnownLink( wfMsgForContent( "faqpage" ), wfMsg("faq") ) - . " | " . + $this->mainPageLink(). $separator . + $this->makeKnownLink( wfMsgForContent( "aboutpage" ), wfMsg( "about" ) ) . $separator . + $this->makeKnownLink( wfMsgForContent( "helppage" ), wfMsg( "help" ) ) . $separator . + $this->makeKnownLink( wfMsgForContent( "faqpage" ), wfMsg("faq") ) . $separator . $this->specialLink( "specialpages" ); /* show links to different language variants */ $s .= $this->variantLinks(); $s .= $this->extensionTabLinks(); - $s .= " | "; + $s .= $separator; if ( $wgUser->isLoggedIn() ) { $s .= $this->makeKnownLink( $lo, wfMsg( "logout" ), $q ); } else { diff --git a/skins/Standard.php b/skins/Standard.php index 9b65de718e..f8703900e6 100644 --- a/skins/Standard.php +++ b/skins/Standard.php @@ -107,11 +107,13 @@ class SkinStandard extends Skin { $l = $wgContLang->isRTL() ? 'right' : 'left'; $s .= ""; + $separator = wfMsgExt( 'pipe-separator' , 'escapenoentities' ); + $s .= $this->bottomLinks(); $s .= "\n
    " . $this->mainPageLink() - . ' | ' . $this->aboutLink() - . ' | ' . $this->specialLink( 'recentchanges' ) - . ' | ' . $this->searchForm() + . $separator . $this->aboutLink() + . $separator . $this->specialLink( 'recentchanges' ) + . $separator . $this->searchForm() . '
    ' . $this->pageStats() . ''; $s .= "";