See parent task T317255: PHP's getElementsByTagName is slow prior to PHP 8.3 for justification and suggested replacements.
Description
Description
Details
Details
Subject | Repo | Branch | Lines +/- | |
---|---|---|---|---|
Use Zest's getElementsByTagName() | mediawiki/extensions/Translate | master | +2 -1 |
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Open | None | T317255 PHP's getElementsByTagName is slow prior to PHP 8.3 | |||
Resolved | Wangombe | T317523 Migrate Translate extension away from use of PHPDOM's getElementsByTagName |
Event Timeline
Comment Actions
The only usage that I see of this is in XliffFFS::writeReal. This can be replace with \Wikimedia\Parsoid\Utils\DOMCompat::getElementsByTagName. Note that there are no tests for this class so we will have to be careful on how we test this.
Comment Actions
Change 859051 had a related patch set uploaded (by Wangombe; author: Wangombe):
[mediawiki/extensions/Translate@master] Use Zest's getElementsByTagName()
Comment Actions
Change 859051 merged by jenkins-bot:
[mediawiki/extensions/Translate@master] Use Zest's getElementsByTagName()
Comment Actions
I've tested this locally. There is no easy way to test this on production hence marking this as done.