HV-740 Disabling around construct callbacks

hibernate · Mar 11, 2013 · 26fcd6d · 26fcd6d
1 parent 106ab01
commit 26fcd6d
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/...src/main/java/org/hibernate/validator/internal/cdi/interceptor/ValidationInterceptor.java b/...src/main/java/org/hibernate/validator/internal/cdi/interceptor/ValidationInterceptor.java
@@ -23,7 +23,6 @@
 import java.util.Set;
 import javax.annotation.Priority;
 import javax.inject.Inject;
-import javax.interceptor.AroundConstruct;
 import javax.interceptor.AroundInvoke;
 import javax.interceptor.Interceptor;
 import javax.interceptor.InvocationContext;
@@ -107,12 +106,10 @@ public Object validateMethodInvocation(InvocationContext ctx) throws Exception {
 	 *
 	 * @param ctx The context of the intercepted constructor invocation.
 	 *
-	 * @return The result of the constructor invocation.
-	 *
 	 * @throws Exception Any exception caused by the intercepted constructor invocation. A {@link ConstraintViolationException}
 	 * in case at least one constraint violation occurred either during parameter or return value validation.
 	 */
-	@AroundConstruct
+//	@AroundConstruct
 	@SuppressWarnings("unchecked")
 	public void validateConstructorInvocation(InvocationContext ctx) throws Exception {
 		ExecutableValidator executableValidator = validator.forExecutables();
@@ -128,11 +125,12 @@ public void validateConstructorInvocation(InvocationContext ctx) throws Exceptio
 			);
 		}
 
-		Object result = ctx.proceed();
+		ctx.proceed();
+		Object createdObject = ctx.getTarget();
 
 		violations = validator.forExecutables().validateConstructorReturnValue(
 				ctx.getConstructor(),
-				result
+				createdObject
 		);
 
 		if ( !violations.isEmpty() ) {

diff --git a/integration/weld/src/test/java/org/hibernate/validator/integration/cdi/InterceptorTest.java b/integration/weld/src/test/java/org/hibernate/validator/integration/cdi/InterceptorTest.java
@@ -25,6 +25,7 @@
 import org.jboss.shrinkwrap.api.ShrinkWrap;
 import org.jboss.shrinkwrap.api.asset.EmptyAsset;
 import org.jboss.shrinkwrap.api.spec.JavaArchive;
+import org.junit.Ignore;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 
@@ -67,6 +68,7 @@ public void testInjection() throws Exception {
 	}
 
 	@Test
+	@Ignore
 	public void testInstanceInjection() throws Exception {
 		assertNotNull( repeaterInstance );
 		try {