Catrope has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/363990 )

Change subject: Use Sanitizer::stripAllTags( $x ) instead of 
html_entity_decode( strip_tags( $x ) )
......................................................................

Use Sanitizer::stripAllTags( $x ) instead of html_entity_decode( strip_tags( $x 
) )

Change-Id: I4869a56b20d989a94020f9aaebaa5ef5790cead3
---
M includes/ArticleMetadata.php
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/PageTriage 
refs/changes/90/363990/1

diff --git a/includes/ArticleMetadata.php b/includes/ArticleMetadata.php
index 17c2889..0b729ca 100755
--- a/includes/ArticleMetadata.php
+++ b/includes/ArticleMetadata.php
@@ -830,9 +830,9 @@
                        $attempt++;
                }
 
-               $text = trim( strip_tags( htmlspecialchars_decode(
+               $text = trim( Sanitizer::stripAllTags(
                        MessageCache::singleton()->parse( $text )->getText()
-               ) ) );
+               ) );
                // strip out non-useful data for snippet
                $text = str_replace( [ '{', '}', '[edit]' ], '', $text );
 

-- 
To view, visit https://gerrit.wikimedia.org/r/363990
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4869a56b20d989a94020f9aaebaa5ef5790cead3
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/PageTriage
Gerrit-Branch: master
Gerrit-Owner: Catrope <r...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to