(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 44d3512..c43a133 100644 (file)
--- a/includes/GlobalFunctions.php
+++ b/includes/GlobalFunctions.php
@@ -926,10 +926,12 @@ function wfViewPrevNext( $offset, $limit, $link, $query = '', $atend = false ) {
        } else {
                $nlink = '<a href="' . $title->escapeLocalUrl( $q ) . "\" class=\"mw-nextlink\">{$next}</a>";
        }
-       $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 54f5633..cd34d33 100644 (file)
--- 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 4ba67a2..79e5464 100644 (file)
--- 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 3125610..2510825 100644 (file)
--- 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 f6ba839..f725489 100644 (file)
--- 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 .= " | <a href=\"{$link}\"{$style}>{$name}</a>";
+                                       $s .= $separator . "<a href=\"{$link}\"{$style}>{$name}</a>";
                                }
                        }
                }
                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.= ' | <strong>'. wfMsg( 'youhavenewmessages', $tl, $dl ) . '</strong>';
+                               $s.= $separator . '<strong>'. wfMsg( 'youhavenewmessages', $tl, $dl ) . '</strong>';
                                # 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 .= " | <a href=\"$feedurl\">{$format}</a>";
+                               $s .= wfMsgExt( 'pipe-separator' , 'escapenoentities' ) . "<a href=\"$feedurl\">{$format}</a>";
                        }
                }
                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 .= '&lt; ';
                                                }
@@ -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 .= ' | <a href="' . $wgTitle->escapeLocalUrl( 'variant=' . $code ) . '">' . htmlspecialchars( $varname ) . '</a>';
+                               $s .= wfMsgExt( 'pipe-separator' , 'escapenoentities' ) . '<a href="' . $wgTitle->escapeLocalUrl( 'variant=' . $code ) . '">' . htmlspecialchars( $varname ) . '</a>';
                        }
                }
                return $s;
@@ -1622,7 +1623,7 @@ END;
                $first = true;
                if($wgContLang->isRTL()) $s .= '<span dir="LTR">';
                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 fcfdb78..faaa8b9 100644 (file)
--- a/includes/WatchlistEditor.php
+++ b/includes/WatchlistEditor.php
@@ -391,7 +391,7 @@ class WatchlistEditor {
                }
                return '<li>'
                        . Xml::check( 'titles[]', false, array( 'value' => $title->getPrefixedText() ) )
-                       . $link . ' (' . implode( ' | ', $tools ) . ')' . '</li>';
+                       . $link . ' (' . implode( wfMsgExt( 'pipe-separator' , 'escapenoentities' ), $tools ) . ')' . '</li>';
                }
 
        /**
@@ -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 c2a8de4..c4bc313 100644 (file)
--- 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 | <a href="' . $wgTitle->escapeLocalUrl( 'ot=html' ) . '">HTML</a> | ' .
+               $wgOut->addHTML( 'PHP | <a href="' . $wgTitle->escapeLocalUrl( 'ot=html' ) . '">HTML</a>' .
+                       wfMsgExt( 'pipe-separator' , 'escapenoentities' ) .
                        '<a href="' . $wgTitle->escapeLocalUrl( 'ot=xml' ) . '">XML</a>' .
                        '<pre>' . htmlspecialchars( $navText ) . '</pre>' );
        } else if ( $ot == 'xml' ) {
@@ -53,8 +54,9 @@ function wfSpecialAllmessages() {
                header( 'Content-type: text/xml' );
                echo wfAllMessagesMakeXml( $messages );
        } else {
-               $wgOut->addHTML( '<a href="' . $wgTitle->escapeLocalUrl( 'ot=php' ) . '">PHP</a> | ' .
-                       'HTML |  <a href="' . $wgTitle->escapeLocalUrl( 'ot=xml' ) . '">XML</a>' );
+               $wgOut->addHTML( '<a href="' . $wgTitle->escapeLocalUrl( 'ot=php' ) . '">PHP</a>' .
+                       wfMsgExt( 'pipe-separator' , 'escapenoentities' ) .
+                       'HTML' . wfMsgExt( 'pipe-separator' , 'escapenoentities' ) . '<a href="' . $wgTitle->escapeLocalUrl( 'ot=xml' ) . '">XML</a>' );
                $wgOut->addWikiText( $navText );
                $wgOut->addHTML( wfAllMessagesMakeHTMLText( $messages ) );
        }

diff --git a/includes/specials/SpecialAllpages.php b/includes/specials/SpecialAllpages.php
index f27c120..afe6738 100644 (file)
--- 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 .= "</td></tr></table><hr />";
                }
@@ -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 a03d564..01afac1 100644 (file)
--- 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 '<p class="mw-ipb-conveniencelinks">' . implode( ' | ', $links ) . '</p>';
+               return '<p class="mw-ipb-conveniencelinks">' . implode( wfMsgExt( 'pipe-separator' , 'escapenoentities' ), $links ) . '</p>';
        }
 
        /**

diff --git a/includes/specials/SpecialContributions.php b/includes/specials/SpecialContributions.php
index 694690d..f1ce6fd 100644 (file)
--- 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 8612246..e65ca96 100644 (file)
--- 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 03f94ab..3523113 100644 (file)
--- 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 ) .
-                               '<br />(' . implode( ' | ', $toolLinks ) . ')'
+                               '<br />(' . implode( wfMsgExt( 'pipe-separator' , 'escapenoentities' ), $toolLinks ) . ')'
                        ) .
 
                        $this->tableRow(

diff --git a/includes/specials/SpecialPrefixindex.php b/includes/specials/SpecialPrefixindex.php
index a360d50..c2168c5 100644 (file)
--- 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 0fe1345..7eaed26 100644 (file)
--- 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 b6908c9..97e7fd4 100644 (file)
--- 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 ccdc722..f12ab97 100644 (file)
--- 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 .= '<p>';
        $form .= Xml::label( wfMsg( 'namespace' ), 'namespace' ) . '&nbsp;';
@@ -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 ade28f1..d3fa691 100644 (file)
--- 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 27d80ff..97c3976 100644 (file)
--- 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 98d3d8f..ed67aab 100644 (file)
--- 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 3ba87df..2c28b44 100644 (file)
--- 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 7973962..d2a681c 100644 (file)
--- a/skins/CologneBlue.php
+++ b/skins/CologneBlue.php
@@ -79,9 +79,10 @@ class SkinCologneBlue extends Skin {
                }
                $s .= "<td class='bottom' align='center' valign='top'>";
 
+               $separator = wfMsgExt( 'pipe-separator' , 'escapenoentities' );
                $s .= $this->bottomLinks();
-               $s .= "\n<br />" . $this->makeKnownLinkObj( Title::newMainPage() ) . " | "
-                 . $this->aboutLink() . " | "
+               $s .= "\n<br />" . $this->makeKnownLinkObj( Title::newMainPage() ) . $separator
+                 . $this->aboutLink() . $separator
                  . $this->searchForm( wfMsg( "qbfind" ) );
 
                $s .= "\n<br />" . $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 9b65de7..f870390 100644 (file)
--- a/skins/Standard.php
+++ b/skins/Standard.php
@@ -107,11 +107,13 @@ class SkinStandard extends Skin {
                $l = $wgContLang->isRTL() ? 'right' : 'left';
                $s .= "<td class='bottom' align='$l' valign='top'>";
 
+               $separator = wfMsgExt( 'pipe-separator' , 'escapenoentities' );
+
                $s .= $this->bottomLinks();
                $s .= "\n<br />" . $this->mainPageLink()
-                 . ' | ' . $this->aboutLink()
-                 . ' | ' . $this->specialLink( 'recentchanges' )
-                 . ' | ' . $this->searchForm()
+                 . $separator . $this->aboutLink()
+                 . $separator . $this->specialLink( 'recentchanges' )
+                 . $separator . $this->searchForm()
                  . '<br /><span id="pagestats">' . $this->pageStats() . '</span>';
 
                $s .= "</td>";