Fix double escaping issues. Related to r51559 and friends.
authorSiebrand Mazeland <siebrand@users.mediawiki.org>
Mon, 15 Jun 2009 12:32:59 +0000 (12:32 +0000)
committerSiebrand Mazeland <siebrand@users.mediawiki.org>
Mon, 15 Jun 2009 12:32:59 +0000 (12:32 +0000)
16 files changed:
includes/EditPage.php
includes/ImagePage.php
includes/LogEventsList.php
includes/OutputPage.php
includes/PageHistory.php
includes/SkinTemplate.php
includes/specials/SpecialAllpages.php
includes/specials/SpecialConfirmemail.php
includes/specials/SpecialContributions.php
includes/specials/SpecialDeletedContributions.php
includes/specials/SpecialProtectedpages.php
includes/specials/SpecialRevisiondelete.php
includes/specials/SpecialUnusedtemplates.php
includes/specials/SpecialUpload.php
includes/specials/SpecialWatchlist.php
includes/specials/SpecialWhatlinkshere.php

index 0df1da6..020af5e 100644 (file)
@@ -1875,7 +1875,7 @@ END
                        $loginTitle,
                        wfMsgHtml( 'loginreqlink' ),
                        array(),
-                       array( 'returnto' => $wgTitle->getPrefixedUrl() ),
+                       array( 'returnto' => $wgTitle->getPrefixedText() ),
                        array( 'known', 'noclasses' )
                );
 
index 60d7b3b..3d815c6 100644 (file)
@@ -921,7 +921,7 @@ class ImageHistoryList {
                                $wgLang->timeAndDate( $timestamp, true ),
                                array(),
                                array(
-                                       'target' => $wgTitle->getPrefixedUrl(),
+                                       'target' => $wgTitle->getPrefixedText(),
                                        'file' => $img,
                                        'token' => $wgUser->editToken( $img )
                                ),
index e89e427..c1dbe43 100644 (file)
@@ -367,7 +367,7 @@ class LogEventsList {
                                        $this->message['revdel-restore'],
                                        array(),
                                        array(
-                                               'target' => $title->getPrefixedUrl(),
+                                               'target' => $title->getPrefixedText(),
                                                'type' => $key,
                                                'ids' => $query
                                        ),
@@ -389,7 +389,7 @@ class LogEventsList {
                                        $this->message['revdel-restore'], 
                                        array(),
                                        array(
-                                               'target' => $title->getPrefixedUrl(),
+                                               'target' => $title->getPrefixedText(),
                                                'type' => 'logging',
                                                'ids' => $query
                                        ),
index d390eaa..623411b 100644 (file)
@@ -1258,7 +1258,7 @@ class OutputPage {
                        $loginTitle,
                        wfMsgHtml( 'loginreqlink' ),
                        array(),
-                       array( 'returnto' => $this->getTitle()->getPrefixedUrl() ),
+                       array( 'returnto' => $this->getTitle()->getPrefixedText() ),
                        array( 'known', 'noclasses' )
                );
                $this->addHTML( wfMsgWikiHtml( 'loginreqpagetext', $loginLink ) );
index 47bd5bb..279f97b 100644 (file)
@@ -94,7 +94,7 @@ class PageHistory {
                        $logPage,
                        wfMsgHtml( 'viewpagelogs' ),
                        array(),
-                       array( 'page' => $this->mTitle->getPrefixedUrl() ),
+                       array( 'page' => $this->mTitle->getPrefixedText() ),
                        array( 'known', 'noclasses' )
                );
                $wgOut->setSubtitle( $logLink );
index 8100611..0604c87 100644 (file)
@@ -938,8 +938,13 @@ class SkinTemplate extends Skin {
 
                        if( $id ) {
                                $logPage = SpecialPage::getTitleFor( 'Log' );
-                               $nav_urls['log'] = array( 'href' => $logPage->getLocalUrl( 'user='
-                                       . $this->mTitle->getPartialUrl() ) );
+                               $nav_urls['log'] = array(
+                                       'href' => $logPage->getLocalUrl(
+                                               array(
+                                                       'user' => $this->mTitle->getText()
+                                               )
+                                       )
+                               );
                        } else {
                                $nav_urls['log'] = false;
                        }
index b3d2d51..1868935 100644 (file)
@@ -384,7 +384,7 @@ class SpecialAllpages extends IncludableSpecialPage {
 
                        # Do we put a previous link ?
                        if( isset( $prevTitle ) &&  $pt = $prevTitle->getText() ) {
-                               $query = array( 'from' => $prevTitle->getPartialUrl() );
+                               $query = array( 'from' => $prevTitle->getText() );
 
                                if( $namespace )
                                        $query['namespace'] = $namespace;
@@ -401,7 +401,7 @@ class SpecialAllpages extends IncludableSpecialPage {
                        if( $n == $this->maxPerPage && $s = $res->fetchObject() ) {
                                # $s is the first link of the next chunk
                                $t = Title::MakeTitle($namespace, $s->page_title);
-                               $query = array( 'from' => $t->getPartialUrl() );
+                               $query = array( 'from' => $t->getText() );
 
                                if( $namespace )
                                        $query['namespace'] = $namespace;
index 7090ee4..dd3cece 100644 (file)
@@ -39,7 +39,7 @@ class EmailConfirmation extends UnlistedSpecialPage {
                                        $title,
                                        wfMsgHtml( 'loginreqlink' ),
                                        array(),
-                                       array( 'returnto' => $this->getTitle()->getPrefixedUrl() )
+                                       array( 'returnto' => $this->getTitle()->getPrefixedText() )
                                );
                                $wgOut->addHTML( wfMsgWikiHtml( 'confirmemail_needlogin', $llink ) );
                        }
index 63ae3c8..a61a9ea 100644 (file)
@@ -168,7 +168,7 @@ class SpecialContributions extends SpecialPage {
                                        array(),
                                        array(
                                                'type' => 'block',
-                                               'page' => $nt->getPrefixedUrl()
+                                               'page' => $nt->getPrefixedText()
                                        )
                                );
                        }
@@ -177,7 +177,7 @@ class SpecialContributions extends SpecialPage {
                                SpecialPage::getTitleFor( 'Log' ),
                                wfMsg( 'sp-contributions-logs' ),
                                array(),
-                               array( 'user' => $nt->getPartialUrl() )
+                               array( 'user' => $nt->getText() )
                        );
 
                        # Add link to deleted user contributions for priviledged users
index 7f25681..1fe7947 100644 (file)
@@ -132,7 +132,7 @@ class DeletedContribsPager extends IndexPager {
                        array(),
                        array(
                                'type' => 'delete',
-                               'page' => $page->getPrefixedUrl()
+                               'page' => $page->getPrefixedText()
                        )
                );
 
@@ -146,7 +146,7 @@ class DeletedContribsPager extends IndexPager {
                        htmlspecialchars( $page->getPrefixedText() ),
                        array(),
                        array(
-                               'target' => $page->getPrefixedUrl(),
+                               'target' => $page->getPrefixedText(),
                                'timestamp' => $rev->getTimestamp()
                        )
                );
@@ -156,7 +156,7 @@ class DeletedContribsPager extends IndexPager {
                        $this->messages['diff'],
                        array(),
                        array(
-                               'target' => $page->getPrefixedUrl(),
+                               'target' => $page->getPrefixedText(),
                                'timestamp' => $rev->getTimestamp(),
                                'diff' => 'prev'
                        )
@@ -173,7 +173,7 @@ class DeletedContribsPager extends IndexPager {
                                $d,
                                array(),
                                array(
-                                       'target' => $page->getPrefixedUrl(),
+                                       'target' => $page->getPrefixedText(),
                                        'timestamp' => $rev->getTimestamp()
                                )
                        );
@@ -347,7 +347,7 @@ class DeletedContributionsPage extends SpecialPage {
                                        array(),
                                        array(
                                                'type' => 'block',
-                                               'page' => $nt->getPrefixedUrl()
+                                               'page' => $nt->getPrefixedText()
                                        )
                                );
                        }
@@ -356,7 +356,7 @@ class DeletedContributionsPage extends SpecialPage {
                                SpecialPage::getTitleFor( 'Log' ),
                                wfMsgHtml( 'sp-contributions-logs' ),
                                array(),
-                               array( 'user' => $nt->getPartialUrl() )
+                               array( 'user' => $nt->getText() )
                        );
                        # Link to undeleted contributions
                        $tools[] = $sk->linkKnown(
index 01b895f..46602f6 100644 (file)
@@ -109,7 +109,7 @@ class ProtectedPagesForm {
                                array(),
                                array(
                                        'type' => 'protect',
-                                       'page' => $title->getPrefixedUrl()
+                                       'page' => $title->getPrefixedText()
                                )
                        ) . ')';
                }
index f8697a2..cc4c8a4 100644 (file)
@@ -190,7 +190,7 @@ class SpecialRevisionDelete extends UnlistedSpecialPage {
                                $logtitle,
                                wfMsgHtml( 'viewpagelogs' ),
                                array(),
-                               array( 'page' => $this->targetObj->getPrefixedUrl() )
+                               array( 'page' => $this->targetObj->getPrefixedText() )
                        );
                        # Give a link to the page history
                        $links[] = $this->skin->linkKnown(
@@ -1582,8 +1582,12 @@ class RevDel_LogItem extends RevDel_Item {
                $title = Title::makeTitle( $this->row->log_namespace, $this->row->log_title );
 
                $logtitle = SpecialPage::getTitleFor( 'Log' );
-               $loglink = $this->special->skin->link( $logtitle, wfMsgHtml( 'log' ), array(),
-                       array( 'page' => $title->getPrefixedUrl() ) );
+               $loglink = $this->special->skin->link(
+                       $logtitle,
+                       wfMsgHtml( 'log' ),
+                       array(),
+                       array( 'page' => $title->getPrefixedText() )
+               );
                // Action text
                if( !$this->canView() ) {
                        $action = '<span class="history-deleted">' . wfMsgHtml('rev-deleted-event') . '</span>';
index 5aaf363..6ddbab3 100644 (file)
@@ -43,7 +43,7 @@ class UnusedtemplatesPage extends QueryPage {
                        SpecialPage::getTitleFor( 'Whatlinkshere' ),
                        wfMsgHtml( 'unusedtemplateswlh' ),
                        array(),
-                       array( 'target' => $title->getPrefixedUrl() )
+                       array( 'target' => $title->getPrefixedText() )
                );
                return wfSpecialList( $pageLink, $wlhLink );
        }
index 6afc06e..bb4bae1 100644 (file)
@@ -727,7 +727,7 @@ class UploadForm {
                                array(),
                                array(
                                        'type' => 'delete',
-                                       'page' => $file->getTitle()->getPrefixedUrl()
+                                       'page' => $file->getTitle()->getPrefixedText()
                                )
                        );
                        $warning .= '<li>' . wfMsgWikiHtml( 'filewasdeleted', $llink ) . '</li>';
index 885c814..8703d1b 100644 (file)
@@ -25,7 +25,7 @@ function wfSpecialWatchlist( $par ) {
                        SpecialPage::getTitleFor( 'Userlogin' ), 
                        wfMsgHtml( 'loginreqlink' ),
                        array(),
-                       array( 'returnto' => $specialTitle->getPrefixedUrl() )
+                       array( 'returnto' => $specialTitle->getPrefixedText() )
                );
                $wgOut->addHTML( wfMsgWikiHtml( 'watchlistanontext', $llink ) );
                return;
index 5e21c73..255e76b 100644 (file)
@@ -310,7 +310,7 @@ class SpecialWhatLinksHere extends SpecialPage {
                        $title,
                        $text,
                        array(),
-                       array( 'target' => $targetText )
+                       array( 'target' => $target->getPrefixedText() )
                );
        }