[dna-commits] DNA SVN: r1590 - trunk/extensions/dna-connector-store-jpa/src/main/java/org/jboss/dna/connector/store/jpa/model/basic.

[dna-commits at lists.jboss.org]

Author: bcarothers
Date: 2010-01-10 20:21:32 -0500 (Sun, 10 Jan 2010)
New Revision: 1590

Modified:
   trunk/extensions/dna-connector-store-jpa/src/main/java/org/jboss/dna/connector/store/jpa/model/basic/BasicRequestProcessor.java
Log:
DNA-644 BasicJpaRepositoryTckTest has failures related to XPath queries

Applied patch that makes BasicRequestProcessor.process(CreateNodeRequest) call recordChange(request) after successful node additions. If recordChange(...) isn't called, the SearchEngineIndexer is never made aware of the change and the node will not be added to the indexes or returned as a search result. Adding this line makes all 5 failing tests pass. I reviewed the rest of the class, but could not find any other places where this call was missing.

Modified: trunk/extensions/dna-connector-store-jpa/src/main/java/org/jboss/dna/connector/store/jpa/model/basic/BasicRequestProcessor.java
===================================================================
--- trunk/extensions/dna-connector-store-jpa/src/main/java/org/jboss/dna/connector/store/jpa/model/basic/BasicRequestProcessor.java	2010-01-11 00:26:34 UTC (rev 1589)
+++ trunk/extensions/dna-connector-store-jpa/src/main/java/org/jboss/dna/connector/store/jpa/model/basic/BasicRequestProcessor.java	2010-01-11 01:21:32 UTC (rev 1590)
@@ -260,6 +260,7 @@
                                 createProperties(workspace, uuidString, request.properties());
                             }
                             request.setActualLocationOfNode(childLocation);
+                            recordChange(request);
                             return;
                         }
                     }
@@ -306,6 +307,7 @@
             return;
         }
         request.setActualLocationOfNode(actualLocation);
+        recordChange(request);
     }
 
     /**