Page MenuHomePhabricator

Split ExtVariables into multiple classes
Open, MediumPublic12 Story Points

Description

As ExtVariables becomes more complex, it seems useful to split in mutliple classes, for example for Hooks, the actual parser functions and so on. rECIT and rEPFN could be helpful as an example how to do this. This might especially be helpful for more advanced open feature requests.

While doing this, we could also do some breaking changes to get more standard file and function names. If some feature requests like T191575 are implemented on the way, it should justify bumping the version number to 3.0.

Details

Related Gerrit Patches:
mediawiki/extensions/Variables : masterMoving hooks to their own class

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 21 2018, 4:02 PM
Aklapper renamed this task from Split ExtVariables in mutltiple classes to Split ExtVariables in multiple classes.Aug 21 2018, 5:01 PM
MGChecker triaged this task as Low priority.Aug 24 2018, 1:32 AM

To make a start, we could move the hook functions to a seperate file. After the MW 1.32 branching point, we could split the parser functions and the actual Variables store as well.

MGChecker raised the priority of this task from Low to Medium.Sep 19 2018, 12:56 AM

Change 461262 had a related patch set uploaded (by MGChecker; owner: MGChecker):
[mediawiki/extensions/Variables@master] Moving hooks to their own class

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

MGChecker set the point value for this task to 12.
MGChecker renamed this task from Split ExtVariables in multiple classes to Split ExtVariables into multiple classes.Oct 27 2018, 9:27 AM

Change 461262 merged by jenkins-bot:
[mediawiki/extensions/Variables@master] Moving hooks to their own class

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