Page MenuHomePhabricator

Paste Check: Improve handling of quoted content
Open, Needs TriagePublic

Description

As discussed in T426362, our ignoreQuotedContent check just looks at the first character of the flagged range to see if it is inside a quote.

However if I were to paste

Smith said "Lorem ipsum dolor sit amet, consectetur adipiscing elit. Quisque sit amet justo ligula." to Jones

that would trigger paste check, even though only a trivial amount of text is not in the quotation.

Using the approach outlined in T426362 we could strip quoted text and see if the amount of non-quoted text exceeds the threshold instead.

In this case we would analyse

Smith said to Jones

and see there is not enough text to trigger paste check.

Event Timeline

Change #1289354 had a related patch set uploaded (by Esanders; author: Esanders):

[mediawiki/extensions/VisualEditor@master] PasteCheck: Exclude quoted content when calculating length

https://gerrit.wikimedia.org/r/1289354