Page MenuHomePhabricator

Replace cite-tracking-category-cite-diffing-error category by an entry in the ParserLimit report
Closed, ResolvedPublic

Description

After consideration and discussion, suppressing errors in visual-diff based on categories is a bad idea - mostly because said categories can be suppressed by wikis, for whom it is actually not useful.
For this task, we remove cite-tracking-category-cite-diffing-error (but leave the localization messages for the time being for cache reasons), and replace it by an entry in the ParserLimit report.

Event Timeline

Restricted Application edited projects, added: Parsoid; removed: Patch-For-Review. · View Herald TranscriptJan 7 2025, 5:28 PM
Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change #1108804 had a related patch set uploaded (by Isabelle Hurbain-Palatin; author: Isabelle Hurbain-Palatin):

[mediawiki/extensions/Cite@master] Replace cite-tracking-category-cite-diffing-error by LimitReport entry

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

We keep the entries in the localization file for now until the parser cache has had a chance to expire existing entries.

Probably shouldn't close out this task until we go back and remove that in a few weeks.

Change #1108804 merged by jenkins-bot:

[mediawiki/extensions/Cite@master] Remove cite-tracking-category-cite-diffing-error

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

Moved to "Block" - will unblock in a few weeks by virtue of caches expiring.

Change #1248860 had a related patch set uploaded (by Isabelle Hurbain-Palatin; author: Isabelle Hurbain-Palatin):

[mediawiki/extensions/Cite@master] Remove cite-diffing-error tracking category

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

Change #1248860 merged by jenkins-bot:

[mediawiki/extensions/Cite@master] Remove cite-diffing-error tracking category

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