You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openjpa.apache.org by pp...@apache.org on 2010/03/17 23:11:02 UTC

svn commit: r924518 - in /openjpa/trunk/openjpa-persistence/src/main: java/org/apache/openjpa/persistence/EntityManagerImpl.java resources/org/apache/openjpa/persistence/localizer.properties

Author: ppoddar
Date: Wed Mar 17 22:11:02 2010
New Revision: 924518

URL: http://svn.apache.org/viewvc?rev=924518&view=rev
Log:
THrow IllegalArg exception on null argument

Modified:
    openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/EntityManagerImpl.java
    openjpa/trunk/openjpa-persistence/src/main/resources/org/apache/openjpa/persistence/localizer.properties

Modified: openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/EntityManagerImpl.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/EntityManagerImpl.java?rev=924518&r1=924517&r2=924518&view=diff
==============================================================================
--- openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/EntityManagerImpl.java (original)
+++ openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/EntityManagerImpl.java Wed Mar 17 22:11:02 2010
@@ -1557,6 +1557,8 @@ public class EntityManagerImpl
     }
 
     public void detach(Object entity) {
+        if (entity == null)
+            throw new IllegalArgumentException(_loc.get("null-detach").getMessage());
         assertNotCloseInvoked();
         _broker.detach(entity, this);
     }

Modified: openjpa/trunk/openjpa-persistence/src/main/resources/org/apache/openjpa/persistence/localizer.properties
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-persistence/src/main/resources/org/apache/openjpa/persistence/localizer.properties?rev=924518&r1=924517&r2=924518&view=diff
==============================================================================
--- openjpa/trunk/openjpa-persistence/src/main/resources/org/apache/openjpa/persistence/localizer.properties (original)
+++ openjpa/trunk/openjpa-persistence/src/main/resources/org/apache/openjpa/persistence/localizer.properties Wed Mar 17 22:11:02 2010
@@ -229,3 +229,4 @@ create-emf-depend-error: Failed to creat
 invalid-version-attribute: Persistence version attribute value "{0}" is not valid.  Using version "{1}" by default.
 not-jpql-or-criteria-query: Query is neither a JPQL SELECT nor a Criteria API query.
 cache-retrieve-override: The setting of CacheRetrieveMode.USE is ignored and set to BYPASS for refresh operation.  
+null-detach: Can not detach null entity
\ No newline at end of file