[hibernate-commits] Hibernate SVN: r20835 - search/trunk/hibernate-search/src/main/java/org/hibernate/search/spi.

[hibernate-commits at lists.jboss.org]

Author: epbernard
Date: 2010-10-14 13:31:59 -0400 (Thu, 14 Oct 2010)
New Revision: 20835

Modified:
   search/trunk/hibernate-search/src/main/java/org/hibernate/search/spi/SearchFactoryBuilder.java
Log:
HSEARCH-555 Properly expose as public API the SearchFactoryBuilder

Fix issue where the factory state was copied over after the proeprties were requested

Modified: search/trunk/hibernate-search/src/main/java/org/hibernate/search/spi/SearchFactoryBuilder.java
===================================================================
--- search/trunk/hibernate-search/src/main/java/org/hibernate/search/spi/SearchFactoryBuilder.java	2010-10-14 17:31:27 UTC (rev 20834)
+++ search/trunk/hibernate-search/src/main/java/org/hibernate/search/spi/SearchFactoryBuilder.java	2010-10-14 17:31:59 UTC (rev 20835)
@@ -177,9 +177,11 @@
 		if ( classes.size() == 0 ) {
 			return rootFactory;
 		}
+		factoryState.copyStateFromOldFactory( rootFactory );
+
 		final Properties configurationProperties = factoryState.getConfigurationProperties();
 		BuildContext buildContext = new BuildContext();
-		factoryState.copyStateFromOldFactory( rootFactory );
+		
 		//TODO we don't keep the reflectionManager. Is that an issue?
 		IncrementalSearchConfiguration cfg = new IncrementalSearchConfiguration( classes, configurationProperties );
 		final ReflectionManager reflectionManager = getReflectionManager( cfg );