jenkins-bot has submitted this change and it was merged.
Change subject: SearchUpdate cleanups for CirrusSearch
......................................................................
SearchUpdate cleanups for CirrusSearch
SearchUpdate hook no longer exists since I2063fe05, so don't bother
registering for it. Also return true for textAlreadyUpdatedForIndex()
since we handle all our text processing in getTextFromContent()
Change-Id: I87a5b8cc4603d02327759d95842bd22f76692f18
---
M CirrusSearch.body.php
M CirrusSearch.php
2 files changed, 4 insertions(+), 1 deletion(-)
Approvals:
Manybubbles: Looks good to me, approved
jenkins-bot: Verified
diff --git a/CirrusSearch.body.php b/CirrusSearch.body.php
index 22c88aa..2673975 100644
--- a/CirrusSearch.body.php
+++ b/CirrusSearch.body.php
@@ -424,6 +424,10 @@
}
return $text;
}
+
+ public function textAlreadyUpdatedForIndex() {
+ return true;
+ }
}
/**
diff --git a/CirrusSearch.php b/CirrusSearch.php
index 57c8e83..312cc4d 100644
--- a/CirrusSearch.php
+++ b/CirrusSearch.php
@@ -122,7 +122,6 @@
* Hooks
* Also check Setup for other hooks.
*/
-$wgHooks['SearchUpdate'][] = function() { return false; };
$wgHooks['LinksUpdateComplete'][] = 'CirrusSearch::linksUpdateCompletedHook';
/**
--
To view, visit https://gerrit.wikimedia.org/r/79077
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I87a5b8cc4603d02327759d95842bd22f76692f18
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/CirrusSearch
Gerrit-Branch: master
Gerrit-Owner: Demon <[email protected]>
Gerrit-Reviewer: Manybubbles <[email protected]>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
