Merged in from DEV trunk...

KTS-3059 "Unsupported documents are not indexed rightly so but the Documents Indexer Statistics's Documents Indexed shows a number contrary this" Fixed. Committed By: Conrad Vermeulen Reviewed By: Megan Watson KTS-1594 "Support for Office 2007 Documents" Fixed. Committed By: Conrad Vermeulen Reviewed By: Megan Watson KTS-2400 "Using double quotes in the new search system may be problematic" Fixed. Committed By: Conrad Vermeulen Reviewed By: Megan Watson git-svn-id: https://kt-dms.svn.sourceforge.net/svnroot/kt-dms/STABLE/trunk@8147 c91229c3-7414-0410-bfa2-8a42b809f60b

Merged in from DEV trunk...
KTS-3059 "Unsupported documents are not indexed rightly so but the Documents Indexer Statistics's Documents Indexed shows a number contrary this" Fixed. Committed By: Conrad Vermeulen Reviewed By: Megan Watson KTS-1594 "Support for Office 2007 Documents" Fixed. Committed By: Conrad Vermeulen Reviewed By: Megan Watson KTS-2400 "Using double quotes in the new search system may be problematic" Fixed. Committed By: Conrad Vermeulen Reviewed By: Megan Watson git-svn-id: https://kt-dms.svn.sourceforge.net/svnroot/kt-dms/STABLE/trunk@8147 c91229c3-7414-0410-bfa2-8a42b809f60b
kevin_fourie
1 parent 327b80ed
Showing 5 changed files with 27 additions and 11 deletions
plugins/ktstandard/KTSubscriptions.php
plugins/search2/reporting/IndexErrors.php
plugins/search2/reporting/PendingDocuments.php
search2/indexing/indexers/JavaXMLRPCLuceneIndexer.inc.php
templates/ktcore/search2/adv_query_builder.smarty
@@ -5,32 +5,32 @@
  * KnowledgeTree Open Source Edition
  * Document Management Made Simple
  * Copyright (C) 2004 - 2008 The Jam Warehouse Software (Pty) Limited
- * 
+ *
  * This program is free software; you can redistribute it and/or modify it under
  * the terms of the GNU General Public License version 3 as published by the
  * Free Software Foundation.
- * 
+ *
  * This program is distributed in the hope that it will be useful, but WITHOUT
  * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
  * FOR A PARTICULAR PURPOSE.  See the GNU General Public License for more
  * details.
- * 
+ *
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
- * 
+ *
  * You can contact The Jam Warehouse Software (Pty) Limited, Unit 1, Tramber Place,
  * Blake Street, Observatory, 7925 South Africa. or email info@knowledgetree.com.
- * 
+ *
  * The interactive user interfaces in modified source and object code versions
  * of this program must display Appropriate Legal Notices, as required under
  * Section 5 of the GNU General Public License version 3.
- * 
+ *
  * In accordance with Section 7(b) of the GNU General Public License version 3,
  * these Appropriate Legal Notices must retain the display of the "Powered by
- * KnowledgeTree" logo and retain the original copyright notice. If the display of the 
+ * KnowledgeTree" logo and retain the original copyright notice. If the display of the
  * logo is not reasonably feasible for technical reasons, the Appropriate Legal Notices
- * must display the words "Powered by KnowledgeTree" and retain the original 
- * copyright notice. 
+ * must display the words "Powered by KnowledgeTree" and retain the original
+ * copyright notice.
  * Contributor( s): ______________________________________
  *
  */
@@ -326,6 +326,8 @@ class KTDiscussionSubscriptionTrigger {
         $oFolder = Folder::get($oDocument->getFolderID());
         $oSubscriptionEvent->DiscussDocument($oDocument, $oFolder);
+        $indexer = Indexer::get();
+        $indexer->index($oDocument, 'D');
     }
 }
 // }}}
@@ -84,6 +84,11 @@ class IndexErrorsDispatcher extends KTAdminDispatcher {
         foreach($aIndexerValues as $key=>$doc)
         {
         	$extractor=$indexer->getExtractor($doc['extractor']);
+        	if (is_null($extractor))
+ 			{
+ 				$doc['extractor'] = 'n/a';
+ 				continue;
+ 			}
         	$doc['extractor'] = $extractor->getDisplayName();
         	$aIndexerValues[$key] = $doc;
         }
@@ -58,6 +58,11 @@ class PendingDocumentsDispatcher extends KTAdminDispatcher
  		foreach($aPendingDocs as $key=>$doc)
  		{
  			$extractor = $indexer->getExtractor($doc['extractor']);
+ 			if (is_null($extractor))
+ 			{
+ 				$doc['extractor'] = 'n/a';
+ 				continue;
+ 			}
  			$doc['extractor'] = $extractor->getDisplayName();
  			$aPendingDocs[$key] = $doc;
  		}
@@ -118,6 +118,10 @@ class JavaXMLRPCLuceneIndexer extends Indexer
     	try
     	{
     		$discussion = Indexer::getDiscussionText($docid);
+    		if (empty($discussion))
+    		{
+    			return true;
+    		}
     		return $this->lucene->updateDiscussion($docid, $discussion);
     	}
     	catch(Exception $e)
@@ -714,12 +714,12 @@ function buildExpression()
 			var start = getElByName(gid, 'field' + gid + '_' + fid + 'start');
 			var end = getElByName(gid, 'field' + gid + '_' + fid + 'end');
-			var expr = expr = fieldname + ' ' + fieldop + ' "' + start.value + '"';
+			var expr = expr = fieldname + ' ' + fieldop + ' "' + start.value.replace(/\"/g, "'") + '"';
 			switch (fieldop)
 			{
 				case 'between':
 				case 'BETWEEN':
-					expr += ' AND "' + end.value + '"';
+					expr += ' AND "' + end.value.replace(/\"/g, "'") + '"';
 					break;
 				default:
 					break;