Page MenuHomePhabricator

Move SimpleSecurity classes to separate files
Closed, ResolvedPublic

Description

SimpleSecurity_body.php contains multiple PHP classes

LBFactorySimpleSecurity and LoadBalancerSimpleSecurity should be moved to separate files

Event Timeline

Reedy renamed this task from Move SimpleSecurity classes to seperate files to Move SimpleSecurity classes to separate files.Dec 14 2016, 10:48 PM
Reedy updated the task description. (Show Details)

Looking at https://www.mediawiki.org/wiki/Extension:SimpleSecurity I'm wondering a little bit if that's the best use of time (though I agree that it's a very valid task). :)

Looking at https://www.mediawiki.org/wiki/Extension:SimpleSecurity I'm wondering a little bit if that's the best use of time (though I agree that it's a very valid task). :)

Yeah, it's a hard one.

We've many of these unmaintained extensions... Makes you wonder if we should just be blanking the repos out instead

Change 329135 had a related patch set uploaded (by MtDu):
Move a few SimpleSecurity classes to separate files

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

Reedy assigned this task to MtDu.

Change 329135 merged by jenkins-bot:
Move a few SimpleSecurity classes to separate files

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