Page MenuHomePhabricator

Most or all of SpecialGettingStarted::isAllowedArticle should be moved to CategoryRoulette
Closed, ResolvedPublic

Description

The logic for which articles can be picked should be moved to one place, CategoryRoulette.


Version: master
Severity: enhancement

Details

Reference
bz55598

Event Timeline

bzimport raised the priority of this task from to High.
bzimport set Reference to bz55598.

swalling wrote:

Not really a bug, but marking it as a high priority enhancement, since it's confusing even to you and I to have this in two places. Seems like it blocks turning this in to an API?

Change 97691 had a related patch set uploaded by Mattflaschen:
New API for selecting a task; related refactoring and minor fm

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

I actually broke it into a separate class, PageFilter. This is then used as a dependency for the API (see bug 55773 and Icaf84e75d95bb06d108137221aa13e99003355e1).

Change 97691 merged by jenkins-bot:
New API for selecting a task; related refactoring and minor fm

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