From 0c37a16e6013dd774df8644748bd94b38b85347c Mon Sep 17 00:00:00 2001
From: Roan Kattouw <roan.kattouw@gmail.com>
Date: Mon, 20 Aug 2018 18:54:39 -0700
Subject: [PATCH] ChangeTags: Fix double escaping of tag descriptions

truncateTagDescription() was escaping its return value for HTML, even
though it had just produced plain text, and all similar functions
return plain text as well.

Bug: T192509
Change-Id: Ieaa631ef19c235121af9d8e09f2674f54fa3c25a
---
 includes/changetags/ChangeTags.php | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/includes/changetags/ChangeTags.php b/includes/changetags/ChangeTags.php
index b5bf488acb..9a015f4503 100644
--- a/includes/changetags/ChangeTags.php
+++ b/includes/changetags/ChangeTags.php
@@ -201,9 +201,8 @@ class ChangeTags {
 		}
 
 		$taglessDesc = Sanitizer::stripAllTags( $originalDesc->parse() );
-		$escapedDesc = Sanitizer::escapeHtmlAllowEntities( $taglessDesc );
 
-		return $context->getLanguage()->truncateForVisual( $escapedDesc, $length );
+		return $context->getLanguage()->truncateForVisual( $taglessDesc, $length );
 	}
 
 	/**
-- 
2.20.1