Page MenuHomePhabricator

Create a generic container object that stores HTML and ResourceLoader modules
Open, LowPublic

Description

In TablePager::getBodyOutput(), a ParserOutput object is currently used because it is a container for HTML and ResourceLoader modules. But it also has a bunch of other methods which aren't appropriate to use here.

So I think we need a generic container object that has getters and setters for:

  • HTML text
  • addModuleStyles (CSS-only modules)
  • addModuleScripts (JS-only modules)
  • addModules (combined)
  • addModuleMessages

This should be easy to do, except I can't come up with a name for it.


Version: unspecified
Severity: normal

Details

Reference
bz71605

Event Timeline

bzimport raised the priority of this task from to Low.Nov 22 2014, 4:00 AM
bzimport set Reference to bz71605.
bzimport added a subscriber: Unknown Object (MLST).
Legoktm created this task.Oct 3 2014, 10:20 AM
Krinkle added a subscriber: Krinkle.

Hello I'm all new to MediaWiki development and FOSS projects. And I would like to try work on this bug as my first fix.

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJan 23 2016, 12:27 AM
Krinkle set Security to None.
Krinkle removed a subscriber: wikibugs-l-list.
TheDJ added a subscriber: TheDJ.Oct 26 2016, 10:55 AM

Note: I shared the same feeling back when I was trying to mess with T60478: Improve interface for MediaHandlers to add JavaScript