0001-thumb.php-Set-proper-output-formats-for-messages-goi.patch
Legoktm (Legoktm)
Actions

Authored By

	Legoktm
	Dec 4 2014, 10:09 PM

Size

2 KB

Referenced Files

None

Subscribers

None

0001-thumb.php-Set-proper-output-formats-for-messages-goi.patch
View Options

	From 938b077e72da809ceb27965519bc5f877a142a3d Mon Sep 17 00:00:00 2001
	From: Kunal Mehta <legoktm@gmail.com>
	Date: Thu, 4 Dec 2014 14:06:55 -0800
	Subject: [PATCH] thumb.php: Set proper output formats for messages going into
	HTML

	* Use ->parse() instead of ->text() for wikitext messages that were
	being treated as HTML
	* Explicitly specify ->parse() if no output format was set
	* Document that wfThumbError() takes HTML

	Bug: T76686
	Change-Id: Id6e7548b2e081cfda7803772ed0395a15feb1f84
	---
	thumb.php \| 12 ++++++------
	1 file changed, 6 insertions(+), 6 deletions(-)

	diff --git a/thumb.php b/thumb.php
	index 3d8612d..6c935c1 100644
	--- a/thumb.php
	+++ b/thumb.php
	@@ -135,12 +135,12 @@ function wfStreamThumb( array $params ) {
	// Format is <timestamp>!<name>
	$bits = explode( '!', $fileName, 2 );
	if ( count( $bits ) != 2 ) {
	- wfThumbError( 404, wfMessage( 'badtitletext' )->text() );
	+ wfThumbError( 404, wfMessage( 'badtitletext' )->parse() );
	return;
	}
	$title = Title::makeTitleSafe( NS_FILE, $bits[1] );
	if ( !$title ) {
	- wfThumbError( 404, wfMessage( 'badtitletext' )->text() );
	+ wfThumbError( 404, wfMessage( 'badtitletext' )->parse() );
	return;
	}
	$img = RepoGroup::singleton()->getLocalRepo()->newFromArchiveName( $title, $fileName );
	@@ -150,7 +150,7 @@ function wfStreamThumb( array $params ) {

	// Check the source file title
	if ( !$img ) {
	- wfThumbError( 404, wfMessage( 'badtitletext' )->text() );
	+ wfThumbError( 404, wfMessage( 'badtitletext' )->parse() );
	return;
	}

	@@ -310,10 +310,10 @@ function wfStreamThumb( array $params ) {

	$user = RequestContext::getMain()->getUser();
	if ( !wfThumbIsStandard( $img, $params ) && $user->pingLimiter( 'renderfile-nonstandard' ) ) {
	- wfThumbError( 500, wfMessage( 'actionthrottledtext' ) );
	+ wfThumbError( 500, wfMessage( 'actionthrottledtext' )->parse() );
	return;
	} elseif ( $user->pingLimiter( 'renderfile' ) ) {
	- wfThumbError( 500, wfMessage( 'actionthrottledtext' ) );
	+ wfThumbError( 500, wfMessage( 'actionthrottledtext' )->parse() );
	return;
	}

	@@ -598,7 +598,7 @@ function wfExtractThumbParams( $file, $params ) {
	* Output a thumbnail generation error message
	*
	* @param int $status
	- * @param string $msg
	+ * @param string $msg HTML
	* @return void
	*/
	function wfThumbError( $status, $msg ) {
	--
	1.9.3

File Metadata

Mime Type: text/x-diff
Storage Engine: blob
Storage Format: Raw Data
Storage Handle: 17815
Default Alt Text: 0001-thumb.php-Set-proper-output-formats-for-messages-goi.patch (2 KB)

0001-thumb.php-Set-proper-output-formats-for-messages-goi.patchLegoktm (Legoktm)Actions

0001-thumb.php-Set-proper-output-formats-for-messages-goi.patchView Options

File Metadata

Event Timeline

0001-thumb.php-Set-proper-output-formats-for-messages-goi.patch
Legoktm (Legoktm)
Actions

0001-thumb.php-Set-proper-output-formats-for-messages-goi.patch
View Options