Use Sanitizer::stripAllTags instead of Flow\Parsoid\Utils::htmlToPlaintext
Open, MediumPublic
Actions

Assigned To

None

Authored By

	matthiasmullie
	Apr 15 2015, 2:27 PM

Description

Flow\Parsoid\Utils::htmlToPlaintext does what Sanitizer::stripAllTags already does & we should probably re-use that instead of inventing the wheel.

Flow\Parsoid\Utils::htmlToPlaintext also truncates. We can either keep htmlToPlaintext around & make it call Sanitizer::stripAllTags (instead of stripping ourselves) & Language::truncate, or get rid of htmlToPlaintext altogether & call both of those wherever we're currently calling htmlToPlaintext

Event Timeline

matthiasmullie created this task.Apr 15 2015, 2:27 PM

matthiasmullie raised the priority of this task from to Needs Triage.

matthiasmullie updated the task description. (Show Details)

matthiasmullie added a project: StructuredDiscussions.

matthiasmullie subscribed.

Restricted Application added a project: Collaboration-Team-Triage. · View Herald TranscriptApr 15 2015, 2:27 PM

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

EBernhardson triaged this task as Medium priority.Apr 17 2015, 5:53 PM

EBernhardson added a project: good first task.

EBernhardson set Security to None.

@matthiasmullie Is it about replacing Utils::htmlToPlaintext to Sanitizer::stripAllTags in Flow?

Kizule subscribed.Feb 1 2018, 3:14 PM

Aklapper edited projects, added Growth-Team; removed Collaboration-Team-Triage.Oct 12 2019, 4:08 AM

MBinder_WMF added a project: Growth-Team-Filtering.Apr 15 2021, 7:03 PM

Use Sanitizer::stripAllTags instead of Flow\Parsoid\Utils::htmlToPlaintextOpen, MediumPublicActions

Description

Event Timeline

Use Sanitizer::stripAllTags instead of Flow\Parsoid\Utils::htmlToPlaintext
Open, MediumPublic
Actions