From 7a569d3030420f97999bd347cbf66cf431443098 Mon Sep 17 00:00:00 2001
From: Renaud Paquay <rpaquay@microsoft.com>
Date: Fri, 30 Jul 2010 18:44:50 -0700
Subject: [PATCH] Bug fix: Indexing didn't process "unpublish" events

--HG--
branch : dev
---
 .../Handlers/CreateIndexingTaskHandler.cs        | 16 +++++++---------
 1 file changed, 7 insertions(+), 9 deletions(-)

diff --git a/src/Orchard.Web/Modules/Orchard.Indexing/Handlers/CreateIndexingTaskHandler.cs b/src/Orchard.Web/Modules/Orchard.Indexing/Handlers/CreateIndexingTaskHandler.cs
index 604eb4140..074842cf5 100644
--- a/src/Orchard.Web/Modules/Orchard.Indexing/Handlers/CreateIndexingTaskHandler.cs
+++ b/src/Orchard.Web/Modules/Orchard.Indexing/Handlers/CreateIndexingTaskHandler.cs
@@ -22,24 +22,22 @@ namespace Orchard.Indexing.Handlers {
             _indexingTaskManager = indexingTaskManager;
             _indexNotifierHandlers = indexNotifierHandlers;
 
-            OnPublishing<ContentPart>(CreateIndexingTask);
+            OnPublished<ContentPart>(CreateIndexingTask);
             OnRemoved<ContentPart>(RemoveIndexingTask);
         }
 
         void CreateIndexingTask(PublishContentContext context, ContentPart part) {
+            // "Unpublish" case: Same as "remove"
+            if (context.PublishingItemVersionRecord == null) {
+                _indexingTaskManager.CreateDeleteIndexTask(context.ContentItem);
+                return;
+            }
+            // "Publish" case: update index
             _indexingTaskManager.CreateUpdateIndexTask(context.ContentItem);
-            // UpdateIndex();
         }
 
         void RemoveIndexingTask(RemoveContentContext context, ContentPart part) {
             _indexingTaskManager.CreateDeleteIndexTask(context.ContentItem);
-            // UpdateIndex();
-        }
-
-        private void UpdateIndex() {
-            foreach (var handler in _indexNotifierHandlers) {
-                handler.UpdateIndex(SearchIndexName);
-            }
         }
     }
 }