svn commit: r1623252 - in /manifoldcf/branches/dev_1x: ./ CHANGES.txt framework/pull-agent/src/main/java/org/apache/manifoldcf/crawler/system/WorkerThread.java

[kwright]

Author: kwright
Date: Mon Sep  8 00:47:03 2014
New Revision: 1623252

URL: http://svn.apache.org/r1623252
Log:
Pull up fix for CONNECTORS-1024 from trunk.

Modified:
    manifoldcf/branches/dev_1x/   (props changed)
    manifoldcf/branches/dev_1x/CHANGES.txt
    
manifoldcf/branches/dev_1x/framework/pull-agent/src/main/java/org/apache/manifoldcf/crawler/system/WorkerThread.java

Propchange: manifoldcf/branches/dev_1x/
------------------------------------------------------------------------------
  Merged /manifoldcf/trunk:r1623251

Modified: manifoldcf/branches/dev_1x/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/manifoldcf/branches/dev_1x/CHANGES.txt?rev=1623252&r1=1623251&r2=1623252&view=diff
==============================================================================
--- manifoldcf/branches/dev_1x/CHANGES.txt (original)
+++ manifoldcf/branches/dev_1x/CHANGES.txt Mon Sep  8 00:47:03 2014
@@ -3,6 +3,10 @@ $Id$
 
 ======================= 1.8-dev =====================
 
+CONNECTORS-1024: Non-null components always failed due to a
+null pointer exception.
+(Karl Wright)
+
 CONNECTORS-1023: Fix Alfresco tests.  They don't yet succeed, but
 at least they run, and the connector can communicate with Alfresco.
 (Karl Wright)

Modified: 
manifoldcf/branches/dev_1x/framework/pull-agent/src/main/java/org/apache/manifoldcf/crawler/system/WorkerThread.java
URL: 
http://svn.apache.org/viewvc/manifoldcf/branches/dev_1x/framework/pull-agent/src/main/java/org/apache/manifoldcf/crawler/system/WorkerThread.java?rev=1623252&r1=1623251&r2=1623252&view=diff
==============================================================================
--- 
manifoldcf/branches/dev_1x/framework/pull-agent/src/main/java/org/apache/manifoldcf/crawler/system/WorkerThread.java
 (original)
+++ 
manifoldcf/branches/dev_1x/framework/pull-agent/src/main/java/org/apache/manifoldcf/crawler/system/WorkerThread.java
 Mon Sep  8 00:47:03 2014
@@ -2114,7 +2114,7 @@ public class WorkerThread extends Thread
     protected void checkMultipleDispositions(String documentIdentifier, String 
componentIdentifier, String componentIdentifierHash)
     {
       if (abortSet.contains(documentIdentifier))
-        throw new IllegalStateException("Multiple document dispositions not 
allowed: Abort cannot be combiend with component disposition; document 
'"+documentIdentifier+"'");
+        throw new IllegalStateException("Multiple document dispositions not 
allowed: Abort cannot be combined with component disposition; document 
'"+documentIdentifier+"'");
       if (documentDeletedSet.contains(documentIdentifier))
         throw new IllegalStateException("Multiple document dispositions not 
allowed: Document delete cannot be combined with component disposition; 
document '"+documentIdentifier+"'");
       if (componentIdentifierHash == null)
@@ -2126,7 +2126,7 @@ public class WorkerThread extends Thread
       else
       {
         Set<String> components = touchedComponentSet.get(documentIdentifier);
-        if (components.contains(componentIdentifierHash))
+        if (components != null && components.contains(componentIdentifierHash))
           throw new IllegalStateException("Multiple document component 
dispositions not allowed: document '"+documentIdentifier+"', component 
'"+componentIdentifier+"'");
       }
     }