From ab05b123666fb3e67b2b9690568d5a358969b507 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Bartosz=20Dziewo=C5=84ski?= Date: Mon, 29 May 2017 15:22:03 +0200 Subject: [PATCH] ImagePage: Do not parse message 'imgmultipageprev' This message is used as alt text for an image thumbnail, and so it can't contain HTML tags anyway. This matches the 'imgmultipagenext' message now. (This inconsistency has been present ever since the feature was introduced in 2006: 7f7e79022bfd421e49084b089e74c7a16b77bc03.) Due to using OutputPage::parse() instead of Message::parse(), an incorrect `
` wrapper was also being added. Bug: T166508 Change-Id: I092887cc91e2c800685d4e55177aed814bb3fce9 --- includes/page/ImagePage.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/includes/page/ImagePage.php b/includes/page/ImagePage.php index f3c3ed8521..6a751ac706 100644 --- a/includes/page/ImagePage.php +++ b/includes/page/ImagePage.php @@ -434,7 +434,7 @@ class ImagePage extends Article { $count = $this->displayImg->pageCount(); if ( $page > 1 ) { - $label = $out->parse( $this->getContext()->msg( 'imgmultipageprev' )->text(), false ); + $label = $this->getContext()->msg( 'imgmultipageprev' )->text(); // on the client side, this link is generated in ajaxifyPageNavigation() // in the mediawiki.page.image.pagination module $link = Linker::linkKnown( -- 2.20.1