Only caller is Kartographer extension:
https://gerrit.wikimedia.org/g/mediawiki/extensions/Kartographer/+/ccb3b46ad3974103fa4d3eacbf332ccc3e1badc0/includes/ApiSanitizeMapData.php#49
and it probably has no effect there. We shouldn't be recycling parser objects, just create a new one for your parse.
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| remove Parser::clearState() | mediawiki/extensions/Kartographer | master | +1 -2 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | None | T261329 Prepare Parsoid for MW | |||
| Open | None | T236809 Refactor Parser.php to allow alternate parser (Parsoid) | |||
| Open | None | T236812 Parser.php should be split into a base class and a parser implementation | |||
| Open | cscott | T236811 Parser creation should always use factory | |||
| Resolved | MSantos | T250445 Deprecate and remove Parser::clearState() |
Everything code search: https://codesearch.wmflabs.org/search/?q=clearState%5C%28&i=nope&files=&repos=
WMF deployed code search: https://codesearch.wmflabs.org/deployed/?q=clearState%5C%28&i=nope&files=&repos=
Please be aware that in the current parser implementation, some extensions use the onClearState hook provided by MediaWiki. Are you planning on removing that too?
If this is the case, I would love to take a look at this patch for the Variables extension and the discussion about the necessity of the use of this hook for the extension. If we find a solution which works there (possibly by adding a hook to the ParserOutput constructor), it will work for all these extensions, except maybe for TemplateStyles.
Change 775791 had a related patch set uploaded (by MSantos; author: MSantos):
[mediawiki/extensions/Kartographer@master] remove Parser::clearState()
Change 775791 merged by jenkins-bot:
[mediawiki/extensions/Kartographer@master] remove Parser::clearState()