Eliminate circular dependencies
Closed, ResolvedPublic

Description

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

ssastry created this task.Sep 24 2018, 8:26 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptSep 24 2018, 8:26 PM
ssastry triaged this task as Low priority.Sep 24 2018, 8:27 PM
ssastry moved this task from Backlog to Parsoid Code Refactoring 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 closed this task as Resolved.Tue, Nov 13, 10:57 PM
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