Page MenuHomePhabricator

Get rid of hacky $wgPageTriageNoIndexTemplates system
Closed, ResolvedPublic0 Estimated Story Points

Description

For some reason, we're looking for a template named {{NOINDEX}} instead of looking for the presence of the magic word.

Whoever wrote the code knew this too:

		// See if article includes any templates that should trigger noindexing
		// TODO: This system is a bit hacky and unintuitive. At some point we
		// may want to switch to a system based on the __NOINDEX__ magic word.

Event Timeline

Restricted Application added subscribers: Liuxinyu970226, Aklapper. · View Herald Transcript

Change 518185 had a related patch set uploaded (by Legoktm; owner: Legoktm):
[mediawiki/extensions/PageTriage@master] Simplify Hooks::shouldShowNoIndex()

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

Change 518185 merged by jenkins-bot:
[mediawiki/extensions/PageTriage@master] Simplify Hooks::shouldShowNoIndex()

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

Legoktm claimed this task.