Page MenuHomePhabricator

Eliminate circular dependencies
Closed, ResolvedPublic

Description

Parsoid has circular dependencies in a couple places.
Investigate getting rid of these by suitable refactoring.

Event Timeline

ssastry moved this task from Backlog to Post-Port Work on the Parsoid-PHP board.

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

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

Change 472731 abandoned by Subramanya Sastry:
Eliminate circular dependency between parser.defines & TokenUtils

Reason:
An alternative patch fixes this with other things.

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

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

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

Change 472747 merged by jenkins-bot:
[mediawiki/services/parsoid@master] Split Token types into separate classes in lib/tokens

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

ssastry claimed this task.

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

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

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

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

Change 473946 merged by jenkins-bot:
[mediawiki/services/parsoid@master] Create a FromHTML class as the entry point for serializeDOM

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

Change 470906 merged by jenkins-bot:
[mediawiki/services/parsoid@master] Split utils/DOMUtils.js into separate functional units

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