Page MenuHomePhabricator

SearchHighlighter::highlightText() should not be checking if Cite is installed
Open, Needs TriagePublic

Description

		// @todo FIXME: This should prolly be a hook or something
		// instead of hardcoding the name of the Cite extension
		if ( \ExtensionRegistry::getInstance()->isLoaded( 'Cite' ) ) {
			$spat .= '|(<ref>)'; // references via cite extension
			$endPatterns[4] = '/(<ref>)|(<\/ref>)/';
		}

Event Timeline

Legoktm created this task.Feb 10 2015, 5:39 PM
Legoktm raised the priority of this task from to Needs Triage.
Legoktm updated the task description. (Show Details)
Legoktm added a subscriber: Legoktm.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptFeb 10 2015, 5:39 PM

Change 530184 had a related patch set uploaded (by WMDE-Fisch; owner: WMDE-Fisch):
[mediawiki/core@master] Use ExtensionRegistry instead of class name

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

Restricted Application added a project: Discovery-Search. · View Herald TranscriptAug 14 2019, 5:16 PM

Change 530184 merged by jenkins-bot:
[mediawiki/core@master] Use ExtensionRegistry instead of class name

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

WMDE-Fisch updated the task description. (Show Details)Aug 15 2019, 8:26 AM