[jboss-cvs] JBossAS SVN: r80578 - projects/jpa/trunk/deployers/src/main/java/org/jboss/jpa/deployment.

[jboss-cvs-commits at lists.jboss.org]

Author: wolfc
Date: 2008-11-06 05:21:55 -0500 (Thu, 06 Nov 2008)
New Revision: 80578

Modified:
   projects/jpa/trunk/deployers/src/main/java/org/jboss/jpa/deployment/PersistenceUnitDeployment.java
Log:
JBAS-6165: reworked the assertions

Modified: projects/jpa/trunk/deployers/src/main/java/org/jboss/jpa/deployment/PersistenceUnitDeployment.java
===================================================================
--- projects/jpa/trunk/deployers/src/main/java/org/jboss/jpa/deployment/PersistenceUnitDeployment.java	2008-11-06 10:01:55 UTC (rev 80577)
+++ projects/jpa/trunk/deployers/src/main/java/org/jboss/jpa/deployment/PersistenceUnitDeployment.java	2008-11-06 10:21:55 UTC (rev 80578)
@@ -89,6 +89,8 @@
 
    public void create()
    {
+      assert xpcResolver != null : "xpcResolver wasn't set on " + this;
+      
       // To allow for serializable objects to obtain a reference back
       PersistenceUnitRegistry.register(this);
    }
@@ -205,14 +207,14 @@
    
    public XPCResolver getXPCResolver()
    {
-      assert xpcResolver != null : "xpcResolver is null";  
+      assert xpcResolver != null : "xpcResolver is null in " + this;  
       return xpcResolver;
    }
    
    @Inject
    public void setXPCResolver(XPCResolver xpcResolver)
    {
-      assert xpcResolver != null : "xpcResolver is null";
+      // Do not check for null, because MC does uninstall with null
       this.xpcResolver = xpcResolver;
    }