[MediaWiki-commits] [Gerrit] SearchUpdate cleanups for CirrusSearch - change (mediawiki...CirrusSearch)

2013-08-14 Thread Demon (Code Review) 
Demon has uploaded a new change for review.

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


Change subject: SearchUpdate cleanups for CirrusSearch
..

SearchUpdate cleanups for CirrusSearch

SearchUpdate hook no longer exists, 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(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/CirrusSearch 
refs/changes/77/79077/1

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: newchange
Gerrit-Change-Id: I87a5b8cc4603d02327759d95842bd22f76692f18
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/CirrusSearch
Gerrit-Branch: master
Gerrit-Owner: Demon ch...@wikimedia.org

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

[MediaWiki-commits] [Gerrit] SearchUpdate cleanups for CirrusSearch - change (mediawiki...CirrusSearch)

2013-08-14 Thread jenkins-bot (Code Review) 
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 ch...@wikimedia.org
Gerrit-Reviewer: Manybubbles never...@wikimedia.org
Gerrit-Reviewer: jenkins-bot

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits