Page MenuHomePhabricator

Reduce RelatedArticles to two ResourceLoader modules
Closed, ResolvedPublic

Description

Several changes are proposed to reduce the number of ResourceLoader modules in RelatedArticles. Historically most of these modules exist to support testing, implemented long before the packageFiles construct.

  • Merge the ext.relatedArticles.readMore and ext.relatedArticles.cards modules. Both modules are always loaded together. This was initially flagged by @phuedx during another maintenance change.
  • Merge ext.relatedArticles.readMore.gateway module into ext.relatedArticles.readMore.bootstrap
  • Drop the unnecessary OO dependency on the RelatedArticlesGateway

Event Timeline

Change 781076 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/extensions/RelatedArticles@master] [Maintenance] Reduce related articles to two modules

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

Change 781084 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/extensions/RelatedArticles@master] Limit RelatedArticles feature to ES6 browsers

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

Change 781112 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/extensions/RelatedArticles@master] [organization] organize file by module

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

Change 781076 merged by jenkins-bot:

[mediawiki/extensions/RelatedArticles@master] [Maintenance] Reduce related articles to two modules

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

Change 781112 merged by jenkins-bot:

[mediawiki/extensions/RelatedArticles@master] [organization] organize file by module

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

Remove A/C as I created T306355 for ES6.