Mailing-List: contact commits-help@openjpa.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@openjpa.apache.org
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: svn commit: r707741 - in
 /openjpa/branches/1.2.x/openjpa-kernel/src/main:
 java/org/apache/openjpa/meta/ProxySetupStateManager.java
 resources/org/apache/openjpa/meta/localizer.properties
Date: Fri, 24 Oct 2008 20:35:29 -0000
To: commits@openjpa.apache.org
From: allee8285@apache.org
Message-Id: <20081024203530.1635E2388AB0@eris.apache.org>

Author: allee8285
Date: Fri Oct 24 13:35:29 2008
New Revision: 707741

URL: http://svn.apache.org/viewvc?rev=707741&view=rev
Log:
OPENJPA-752: Enhance exception message text to improve serviceability.

Modified:
    openjpa/branches/1.2.x/openjpa-kernel/src/main/java/org/apache/openjpa/meta/ProxySetupStateManager.java
    openjpa/branches/1.2.x/openjpa-kernel/src/main/resources/org/apache/openjpa/meta/localizer.properties

Modified: openjpa/branches/1.2.x/openjpa-kernel/src/main/java/org/apache/openjpa/meta/ProxySetupStateManager.java
URL: http://svn.apache.org/viewvc/openjpa/branches/1.2.x/openjpa-kernel/src/main/java/org/apache/openjpa/meta/ProxySetupStateManager.java?rev=707741&r1=707740&r2=707741&view=diff
==============================================================================
--- openjpa/branches/1.2.x/openjpa-kernel/src/main/java/org/apache/openjpa/meta/ProxySetupStateManager.java (original)
+++ openjpa/branches/1.2.x/openjpa-kernel/src/main/java/org/apache/openjpa/meta/ProxySetupStateManager.java Fri Oct 24 13:35:29 2008
@@ -25,6 +25,7 @@
 
 import org.apache.openjpa.enhance.PersistenceCapable;
 import org.apache.openjpa.enhance.StateManager;
+import org.apache.openjpa.lib.util.Localizer;
 import org.apache.openjpa.util.InternalException;
 
 /**
@@ -37,6 +38,9 @@
 class ProxySetupStateManager
     implements StateManager {
 
+	private static final Localizer _loc = Localizer
+		.forPackage(ProxySetupStateManager.class);
+
     private Object _object = null;
 
     public void setProxyData(PersistenceCapable pc, ClassMetaData meta) {
@@ -199,39 +203,57 @@
     }
 
     public void providedBooleanField(PersistenceCapable pc, int i, boolean b) {
-        throw new InternalException();
+        throw new InternalException(_loc.get(
+				"unexpected_proxy_sm_attribute_type", pc.getClass().getName(),
+				"boolean"));
     }
 
     public void providedCharField(PersistenceCapable pc, int i, char c) {
-        throw new InternalException();
+        throw new InternalException(_loc.get(
+				"unexpected_proxy_sm_attribute_type", pc.getClass().getName(),
+				"char"));
     }
 
     public void providedByteField(PersistenceCapable pc, int i, byte b) {
-        throw new InternalException();
+        throw new InternalException(_loc.get(
+				"unexpected_proxy_sm_attribute_type", pc.getClass().getName(),
+				"byte"));
     }
 
     public void providedShortField(PersistenceCapable pc, int i, short s) {
-        throw new InternalException();
+        throw new InternalException(_loc.get(
+				"unexpected_proxy_sm_attribute_type", pc.getClass().getName(),
+				"short"));
     }
 
     public void providedIntField(PersistenceCapable pc, int i, int i2) {
-        throw new InternalException();
+        throw new InternalException(_loc.get(
+				"unexpected_proxy_sm_attribute_type", pc.getClass().getName(),
+				"int"));
     }
 
     public void providedLongField(PersistenceCapable pc, int i, long l) {
-        throw new InternalException();
+        throw new InternalException(_loc.get(
+				"unexpected_proxy_sm_attribute_type", pc.getClass().getName(),
+				"long"));
     }
 
     public void providedFloatField(PersistenceCapable pc, int i, float f) {
-        throw new InternalException();
+        throw new InternalException(_loc.get(
+				"unexpected_proxy_sm_attribute_type", pc.getClass().getName(),
+				"float"));
     }
 
     public void providedDoubleField(PersistenceCapable pc, int i, double d) {
-        throw new InternalException();
+        throw new InternalException(_loc.get(
+				"unexpected_proxy_sm_attribute_type", pc.getClass().getName(),
+				"double"));
     }
 
     public void providedStringField(PersistenceCapable pc, int i, String s) {
-        throw new InternalException();
+        throw new InternalException(_loc.get(
+				"unexpected_proxy_sm_attribute_type", pc.getClass().getName(),
+				"String"));
     }
 
     public void providedObjectField(PersistenceCapable pc, int i, Object o) {

Modified: openjpa/branches/1.2.x/openjpa-kernel/src/main/resources/org/apache/openjpa/meta/localizer.properties
URL: http://svn.apache.org/viewvc/openjpa/branches/1.2.x/openjpa-kernel/src/main/resources/org/apache/openjpa/meta/localizer.properties?rev=707741&r1=707740&r2=707741&view=diff
==============================================================================
--- openjpa/branches/1.2.x/openjpa-kernel/src/main/resources/org/apache/openjpa/meta/localizer.properties (original)
+++ openjpa/branches/1.2.x/openjpa-kernel/src/main/resources/org/apache/openjpa/meta/localizer.properties Fri Oct 24 13:35:29 2008
@@ -316,3 +316,8 @@
 interface-badenhance: Dynamic implementation of "{0}" can not be enhanced.
 bad-externalized-value: Value "{0}" was not found in the list of \
 	ExternalValues for field "{2}". Valid values are {1}
+unexpected_proxy_sm_attribute_type: Unexpected attribute type "{1}" for \
+	persistence-capable class "{0}" is detected. If the entity is packaged in \
+    a jar file, this may be caused by one or more inherited class of the \
+    entity not being packaged in the same jar file. Please check all \
+    inherited class(es) are packaged in the same jar file.