Changed the namespace on the discussions trigger, added text. Added check to ind…

…exer to only reindex the document and not reprocess after an update. Committed by: Megan Watson

Changed the namespace on the discussions trigger, added text. Added check to ind…
…exer to only reindex the document and not reprocess after an update. Committed by: Megan Watson
Megan Watson
1 parent 3faaec34
Showing 3 changed files with 19 additions and 3 deletions
lib/subscriptions/subscriptions.inc.php
plugins/ktstandard/KTSubscriptions.php
search2/indexing/indexerCore.inc.php
@@ -5,7 +5,7 @@
  * KnowledgeTree Community Edition
  * Document Management Made Simple
  * Copyright (C) 2008, 2009, 2010 KnowledgeTree Inc.
- * 
+ *
  *
  * 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
@@ -580,6 +580,7 @@ class SubscriptionContent {
         $restoreArchivedDocumentText = _kt('The document "').$info['object_name']._kt('" has been restored by an administrator');
         $downloadDocumentText = _kt('The document "').$info['object_name']._kt('"');
         $documentAlertText = _kt('An alert on the document "').$info['object_name']._kt('" has been added or modified');
+        $discussDocumentText = _kt('A discussion on the document "').$info['object_name']._kt('" has been updated');
         if($info['location_name'] !== NULL && !$bulk_action){
             $addFolderText .= _kt(' to "').$info['location_name']._kt('"');
@@ -701,6 +702,14 @@ class SubscriptionContent {
                 $links .= '&#160;|&#160;<a href="'.$url.'">'._kt('Clear Alert').'</a>';
                 $url = $rootUrl.'/notify.php?id='.$info['notify_id'].'&notify_action=viewall';
                 $links .= '&#160;|&#160;<a href="'.$url.'">'._kt('View all alerts on this document').'</a>';
+                break;
+            case 'DiscussDocument':
+                $text = $discussDocumentText;
+                $url = $rootUrl.'/notify.php?id='.$info['notify_id'];
+                $links = '<a href="'.$url.'">'._kt('View Document').'</a>';
+                $url = $rootUrl.'/notify.php?id='.$info['notify_id'].'&notify_action=clear';
+                $links .= '&#160;|&#160;<a href="'.$url.'">'._kt('Clear Alert').'</a>';
+                break;
         }
         if($info['actor_name'] !== NULL && $info['event_type'] != 'RestoredArchivedDocument'){
@@ -5,7 +5,7 @@
  * KnowledgeTree Community Edition
  * Document Management Made Simple
  * Copyright (C) 2008, 2009, 2010 KnowledgeTree Inc.
- * 
+ *
  *
  * 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
@@ -65,6 +65,7 @@ class KTSubscriptionPlugin extends KTPlugin {
             'ktstandard.subscription.documentsubscription');
         $this->registerAction('documentsubscriptionaction', 'KTDocumentUnsubscriptionAction',
             'ktstandard.subscription.documentunsubscription');
+
         $this->registerTrigger('checkout', 'postValidate', 'KTCheckoutSubscriptionTrigger',
             'ktstandard.triggers.subscription.checkout');
         $this->registerTrigger('edit', 'postValidate', 'KTEditSubscriptionTrigger',
@@ -76,7 +77,8 @@ class KTSubscriptionPlugin extends KTPlugin {
         $this->registerTrigger('archive', 'postValidate', 'KTArchiveSubscriptionTrigger',
             'ktstandard.triggers.subscription.archive');
         $this->registerTrigger('discussion', 'postValidate', 'KTDiscussionSubscriptionTrigger',
-            'ktstandard.triggers.subscription.archive');
+            'ktstandard.triggers.subscription.discussion');
+
         $this->registerAction('foldersubscriptionaction', 'KTFolderSubscriptionAction',
             'ktstandard.subscription.foldersubscription');
         $this->registerAction('foldersubscriptionaction', 'KTFolderUnsubscriptionAction',
@@ -643,6 +643,11 @@ abstract class Indexer
         $default->log->debug("index: Queuing indexing of $document_id");
+        // If we're indexing a discussion, re-processing is not needed.
+        if($what === 'D'){
+            return true;
+        }
+
         // Appending the process queue to the index for convenience
         // Don't want to complicate matters by creating too many new classes and files
         Indexer::unqueueDocFromProcessing($document_id);