WELD-1871 Revised BeanLogger.decoratedNoargsConstructorIsPrivate()

weld · Jul 22, 2015 · 268dc83 · 268dc83
1 parent b4ae870
commit 268dc83
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/impl/src/main/java/org/jboss/weld/injection/producer/BeanInjectionTarget.java b/impl/src/main/java/org/jboss/weld/injection/producer/BeanInjectionTarget.java
@@ -35,6 +35,7 @@
 import org.jboss.weld.logging.BeanLogger;
 import org.jboss.weld.manager.BeanManagerImpl;
 import org.jboss.weld.resources.ClassTransformer;
+import org.jboss.weld.util.reflection.Formats;
 
 /**
  * @author Pete Muir
@@ -145,7 +146,7 @@ protected void checkNoArgsConstructor(EnhancedAnnotatedType<T> type) {
         if (constructor == null) {
             throw BeanLogger.LOG.decoratedHasNoNoargsConstructor(this);
         } else if (constructor.isPrivate()) {
-            throw BeanLogger.LOG.decoratedNoargsConstructorIsPrivate(this);
+            throw BeanLogger.LOG.decoratedNoargsConstructorIsPrivate(this, Formats.formatAsStackTraceElement(constructor.getJavaMember()));
         }
     }
 

diff --git a/impl/src/main/java/org/jboss/weld/logging/BeanLogger.java b/impl/src/main/java/org/jboss/weld/logging/BeanLogger.java
@@ -387,8 +387,8 @@ public interface BeanLogger extends WeldLogger {
     @Message(id = 1534, value = "Bean class which has decorators must have a public constructor without parameters: {0}", format = Format.MESSAGE_FORMAT)
     DeploymentException decoratedHasNoNoargsConstructor(Object param1);
 
-    @Message(id = 1535, value = "Constructor without parameters cannot be private in bean class which has decorators: {0}", format = Format.MESSAGE_FORMAT)
-    DeploymentException decoratedNoargsConstructorIsPrivate(Object param1);
+    @Message(id = 1535, value = "Constructor without parameters cannot be private in bean class which has decorators: {0}\n\tat {1}\n  StackTrace:", format = Format.MESSAGE_FORMAT)
+    DeploymentException decoratedNoargsConstructorIsPrivate(Object param1, Object stackElement);
 
     /**
      * ID of this message was originally 0 but in jboss-logging zero is a special value meaning no ID