Page MenuHomePhabricator

Convert CentralNotice to use extension registration
Closed, ResolvedPublic4 Estimated Story Points

Description

Unsuccessful automatic migration:
CentralNotice: https://gerrit.wikimedia.org/r/#/c/186899/

Conditional initialization is an issue. We're considering using the callback mechanism to continue doing the conditional stuff programmatically.

This task is important because it should improve overall site performance.

Event Timeline

awight raised the priority of this task from to Medium.
awight updated the task description. (Show Details)
awight renamed this task from Convert CheckUser to use extension registration to Convert CentralNotice to use extension registration.Aug 11 2015, 7:07 PM
awight set Security to None.

We have a problem here, that extension setup is controlled by conditionals, and different hooks are registered on infrastructure vs subscribing wikis. See efCentralNoticeSetup.

awight added a subscriber: Paladox.

@Paladox gave us the answer to conditional initialization, that for now we can write a callback function for that junk. Once that's working, we can consider something fancier like sub-extension configuration.

@awright I am not sure what your asking. But yes you have to use a callback currently.

But anyways it dosent need a callback it already uses the ExtensionFunctions config which does it.

Change 186899 had a related patch set uploaded (by Awight):
[WIP] Add extension.json, empty php entry point

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

@DStrine @atgo @AndyRussG
I'd like to pull this one in before the code freeze, a lot of work went into the patch and a rebase in January would be too difficult. There's also a decent chance that we'll be ambushed by a surprise requirement to become compatible with MediaWiki extension registration, doing it ahead of time avoids that risk.

Change 186899 had a related patch set uploaded (by Paladox):
[WIP] Add extension.json, empty php entry point

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

awight updated the task description. (Show Details)
awight changed the point value for this task from 1 to 2.

Here are some notes I'm using for reviewing and smoke testing.

Finished tweaking and smoke testing! Some else should do review the additional changes I made, though, I think. :) Thx!!!!

Change 186899 merged by jenkins-bot:
Migrate to extension registration

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

Following deploy, a single error was discovered on production, leading to the patch below, also now deployed!

https://gerrit.wikimedia.org/r/#/c/304410/

AndyRussG changed the point value for this task from 2 to 4.