From: David Causse <dcausse@wikimedia.org>
Date: Thu, 1 Aug 2019 15:06:50 +0000 (+0200)
Subject: Break the cyclic dependency between SearchEngine and SearchResult
X-Git-Tag: 1.34.0-rc.0~567^2~1
X-Git-Url: https://git.cyclocoop.org/%7B%24www_url%7Dadmin/compta/banques/%22%20.%20generer_url_entite%28%24id_rubrique%2C?a=commitdiff_plain;h=7d1d48ed5f8cae8acb34ccf9046788ebd05cc052;p=lhc%2Fweb%2Fwiklou.git

Break the cyclic dependency between SearchEngine and SearchResult

Change-Id: Icda469eb675ccac6048d7a9d978d51e3e673edab
---

diff --git a/RELEASE-NOTES-1.34 b/RELEASE-NOTES-1.34
index 9a4aeb38f9..6e789487be 100644
--- a/RELEASE-NOTES-1.34
+++ b/RELEASE-NOTES-1.34
@@ -345,6 +345,9 @@ because of Phabricator reports.
   dropped.
 * MessageCache::destroyInstance() has been removed. Instead, call
   MediaWikiTestCase::resetServices().
+* SearchResult protected field $searchEngine is removed and no longer
+  initialized after calling SearchResult::initFromTitle().
+* …
 
 === Deprecations in 1.34 ===
 * The MWNamespace class is deprecated. Use NamespaceInfo.
diff --git a/includes/search/SearchResult.php b/includes/search/SearchResult.php
index b924b29cff..1954e85c7e 100644
--- a/includes/search/SearchResult.php
+++ b/includes/search/SearchResult.php
@@ -51,11 +51,6 @@ class SearchResult {
 	 */
 	protected $mText;
 
-	/**
-	 * @var SearchEngine
-	 */
-	protected $searchEngine;
-
 	/**
 	 * A function returning a set of extension data.
 	 * @var Closure|null
@@ -96,7 +91,6 @@ class SearchResult {
 				$this->mImage = $services->getRepoGroup()->findFile( $this->mTitle );
 			}
 		}
-		$this->searchEngine = $services->newSearchEngine();
 	}
 
 	/**