From c81b0c1cc1a3e16345197429069f597be3ef9196 Mon Sep 17 00:00:00 2001
From: Conrad Vermeulen <conrad@knowledgetree.com>
Date: Fri, 1 Feb 2008 08:56:38 +0000
Subject: [PATCH] KTS-2904 "Indexing process must not remove item from indexing queue if there is a problem with text extraction process" Fixed.

---
 search2/indexing/indexerCore.inc.php | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/search2/indexing/indexerCore.inc.php b/search2/indexing/indexerCore.inc.php
index 4f4b672..d70fee2 100755
--- a/search2/indexing/indexerCore.inc.php
+++ b/search2/indexing/indexerCore.inc.php
@@ -1034,6 +1034,7 @@ abstract class Indexer
 
         		$this->executeHook($extractor, 'pre_extract');
 				$this->executeHook($extractor, 'pre_extract', $mimeType);
+				$removeFromQueue = false;
 
         		if ($extractor->extractTextContent())
         		{
@@ -1048,11 +1049,10 @@ abstract class Indexer
         			if ($indexDiscussion)
         			{
         				$indexStatus = $this->indexDocumentAndDiscussion($docId, $targetFile, $title, $version);
-
+        				$removeFromQueue = $indexStatus;
         				if (!$indexStatus)
         				{
         					$this->logPendingDocumentInfoStatus($docId, sprintf(_kt("Problem indexing document %d - indexDocumentAndDiscussion"),$docId), 'error');
-
         				}
 
         				$extractor->setIndexingStatus($indexStatus);
@@ -1066,6 +1066,7 @@ abstract class Indexer
 						else
 						{
 							$indexStatus = $this->indexDocument($docId, $targetFile, $title, $version);
+							$removeFromQueue = $indexStatus;
 
 							if (!$indexStatus)
 							{
@@ -1095,7 +1096,7 @@ abstract class Indexer
         		}
 
         		@unlink($targetFile);
-        		$removeFromQueue = $indexStatus;
+
         	}
         	else
         	{
--
libgit2 0.21.4