Split Utils and DOMUtils into smaller chunks based on functionality
Closed, ResolvedPublic

Description

Utils and DOMUtils have, over time, come to encapsulate a lot of useful functionality, but where they have become large files by grouping unrelated helpers in one place. It would be useful to separate out groups of helper utilities based on functionality they support.

ssastry created this task.Oct 30 2018, 9:27 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptOct 30 2018, 9:27 PM
ssastry triaged this task as Normal priority.

There are 100 helpers in Util.js and ~200 helpers in DOMUtils.js which is far too many!

Change 470708 had a related patch set uploaded (by Subramanya Sastry; owner: Subramanya Sastry):
[mediawiki/services/parsoid@master] Extract ScriptUtils from Utils

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

Change 470708 merged by jenkins-bot:
[mediawiki/services/parsoid@master] Extract ScriptUtils from Utils

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

Change 470725 had a related patch set uploaded (by Subramanya Sastry; owner: Subramanya Sastry):
[mediawiki/services/parsoid@master] Extracted token-related helpers from Util to TokenUtils

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

Change 470725 merged by jenkins-bot:
[mediawiki/services/parsoid@master] Extracted token-related helpers from Util to TokenUtils

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

Change 470742 had a related patch set uploaded (by Subramanya Sastry; owner: Subramanya Sastry):
[mediawiki/services/parsoid@master] Move parser functions helper to ParserFunctions.js

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

Change 470743 had a related patch set uploaded (by Subramanya Sastry; owner: Subramanya Sastry):
[mediawiki/services/parsoid@master] Extract parsing pipeline related utils from Util.js

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

Change 470750 had a related patch set uploaded (by Subramanya Sastry; owner: Subramanya Sastry):
[mediawiki/services/parsoid@master] Move pipeline helpers from DOMUtils.js to PipelineUtils.js

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

Change 470742 merged by jenkins-bot:
[mediawiki/services/parsoid@master] Move sole use util functions closer to use site

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

Change 470743 merged by jenkins-bot:
[mediawiki/services/parsoid@master] Extract parsing pipeline related utils from Util.js

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

Change 470750 merged by jenkins-bot:
[mediawiki/services/parsoid@master] Move pipeline helpers from DOMUtils.js to PipelineUtils.js

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

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 470906 merged by jenkins-bot:
[mediawiki/services/parsoid@master] Split utils/DOMUtils.js into separate functional units

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

ssastry closed this task as Resolved.Mon, Dec 3, 9:33 PM