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); } /**