Parsoid has circular dependencies in a couple places.
Investigate getting rid of these by suitable refactoring.
Description
Details
Related Objects
- Mentioned In
- T208360: Split Utils and DOMUtils into smaller chunks based on functionality
T210647: Paragraph wrapper introduces <p></p> in output HTML -- investigate and kill them where they are a result of edge case diffs between PHP parser and Parsoid code - Mentioned Here
- rGPARa6058e32ed64: Fixing about ids is only necessary when coming from cache
T208360: Split Utils and DOMUtils into smaller chunks based on functionality
T210647: Paragraph wrapper introduces <p></p> in output HTML -- investigate and kill them where they are a result of edge case diffs between PHP parser and Parsoid code
Event Timeline
Change 472731 had a related patch set uploaded (by Subramanya Sastry; owner: Subramanya Sastry):
[mediawiki/services/parsoid@master] Eliminate circular dependency between parser.defines & TokenUtils
Change 472731 abandoned by Subramanya Sastry:
Eliminate circular dependency between parser.defines & TokenUtils
Reason:
An alternative patch fixes this with other things.
Change 472747 had a related patch set uploaded (by Subramanya Sastry; owner: Subramanya Sastry):
[mediawiki/services/parsoid@master] Split Token types into separate classes in lib/types
Change 472747 merged by jenkins-bot:
[mediawiki/services/parsoid@master] Split Token types into separate classes in lib/tokens
Change 473946 had a related patch set uploaded (by Subramanya Sastry; owner: Subramanya Sastry):
[mediawiki/services/parsoid@master] Create a Html2Wt class as the entry point for serializeDOM
Change 470906 had a related patch set uploaded (by Subramanya Sastry; owner: Subramanya Sastry):
[mediawiki/services/parsoid@master] Split utils/DOMUtils.js into separate functional units
Change 473946 merged by jenkins-bot:
[mediawiki/services/parsoid@master] Create a FromHTML class as the entry point for serializeDOM
Change 470906 merged by jenkins-bot:
[mediawiki/services/parsoid@master] Split utils/DOMUtils.js into separate functional units
Mentioned in SAL (#wikimedia-operations) [2018-12-05T21:39:08Z] <arlolra> Updated Parsoid to a6058e3 (T210647, T208360, T205333)