Page MenuHomePhabricator

ExtensionRegistry::readFromQueue() should not autoload classes
Open, LowPublic

Description

In the initial RfC, there was an idea to allow extensions to have custom processors, but that never panned out due to lack of implementation, and lack of need. To support this theoretical functionality, we had to initialize the autoloader early, during read time. But since we no longer need to support that, we should treat autoloading like everything else, and move it back to the normal global extraction.

When doing this, we need to carefully test that the installer still works.

Event Timeline

Change 439769 had a related patch set uploaded (by Legoktm; owner: Legoktm):
[mediawiki/core@master] [WIP] registration: Stop special casing autoloading

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

Vvjjkkii renamed this task from ExtensionRegistry::readFromQueue() should not autoload classes to jybaaaaaaa.Jul 1 2018, 1:06 AM
Vvjjkkii triaged this task as High priority.
Vvjjkkii updated the task description. (Show Details)
Vvjjkkii removed subscribers: gerritbot, Aklapper.
CommunityTechBot renamed this task from jybaaaaaaa to ExtensionRegistry::readFromQueue() should not autoload classes.Jul 2 2018, 3:48 AM
CommunityTechBot raised the priority of this task from High to Needs Triage.
CommunityTechBot updated the task description. (Show Details)
CommunityTechBot added subscribers: gerritbot, Aklapper.

Not actively working on this, but it does need to get fixed.