Remove redundant entries from wgNamespacesAliases
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	MarcoAurelio
	Mar 27 2016, 3:37 PM

Description

I think I see most entries there that are not needed or redundant since they match with the defaults. Things such as:

'+abwiki' => array( 'Wikipedia' => NS_PROJECT, ),

are IMHO redundant as the defaults should do it:

	// defaults to aid when things are switched
	'+wikibooks' => array( 'Wikibooks' => NS_PROJECT ),
	'+wikinews' => array( 'Wikinews' => NS_PROJECT ),
	'+wikiquote' => array( 'Wikiquote' => NS_PROJECT ),
	'+wikisource' => array( 'Wikisource' => NS_PROJECT ),
	'+wikiversity' => array( 'Wikiversity' => NS_PROJECT ),
	'+wikipedia' => array( 'Wikipedia' => NS_PROJECT ),
	'+wiktionary' => array( 'Wiktionary' => NS_PROJECT ),
	'+wikivoyage' => array(
		'Wikivoyage' => NS_PROJECT,
		'WV' => NS_PROJECT,
	),

Maybe that list needs a bit of cleaning. Please comment.

Details

	Subject	Repo	Branch	Lines +/-
	Remove redundant NS_PROJECT entries from wgNamespacesAliases	operations/mediawiki-config	master	+0 -41

Customize query in gerrit

Event Timeline

MarcoAurelio created this task.Mar 27 2016, 3:37 PM

Restricted Application added subscribers: JEumerus, Matanya, Aklapper. · View Herald TranscriptMar 27 2016, 3:37 PM

MarcoAurelio added subscribers: Dereckson, Krenair.Mar 27 2016, 3:38 PM

Your analysis is correct. You can clean that without issue, that will be help to facilitate readability.

Test procedure will be straightforward and trivial: once the change is merged, you can check one per one if the alias are still working.

Dereckson triaged this task as Low priority.Mar 27 2016, 4:45 PM

Dereckson moved this task from Backlog to Config - to process on the Wikimedia-Site-requests board.Mar 27 2016, 4:47 PM

Thanks for your review, @Dereckson. I can work in a patch for this, as it'll be easy, but it's not on my priority list right now so it might take a while before I submit a patch for this. I think Low priority (as it's right now) or even lowest is the most appropriate status of this task.

Okay. About priority, lowest for a cleaning task escaping only narrowly the definition of technical debt would be a little too low. AFAIK, it escapes the definition, as we don't depend of the cleaning to be able to amend properly the configuration.

Restricted Application added a subscriber: Urbanecm. · View Herald TranscriptApr 3 2016, 8:50 PM

Dereckson renamed this task from Redundant wgNamespacesAliases? to Remove redundant entries from wgNamespacesAliases.Apr 14 2016, 1:20 PM

Dereckson claimed this task.May 4 2016, 10:59 AM

Change 286812 had a related patch set uploaded (by Dereckson):
Remove redundant NS_PROJECT entries from wgNamespacesAliases

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

gerritbot added a project: Patch-For-Review.May 4 2016, 11:07 AM

Dereckson moved this task from Config - to process to To deploy on the Wikimedia-Site-requests board.May 4 2016, 11:08 AM

Change 286812 merged by jenkins-bot:
Remove redundant NS_PROJECT entries from wgNamespacesAliases

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

Patch was merged.

Remove redundant entries from wgNamespacesAliasesClosed, ResolvedPublicActions

Description

Details

Event Timeline

Remove redundant entries from wgNamespacesAliases
Closed, ResolvedPublic
Actions