You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openjpa.apache.org by kw...@apache.org on 2012/06/05 04:23:04 UTC
svn commit: r1346213 -
/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/JDBCStoreManager.java
Author: kwsutter
Date: Tue Jun 5 02:23:03 2012
New Revision: 1346213
URL: http://svn.apache.org/viewvc?rev=1346213&view=rev
Log:
OPENJPA-2204. Detect a potential null value before blindly outputting a trace string.
Modified:
openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/JDBCStoreManager.java
Modified: openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/JDBCStoreManager.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/JDBCStoreManager.java?rev=1346213&r1=1346212&r2=1346213&view=diff
==============================================================================
--- openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/JDBCStoreManager.java (original)
+++ openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/JDBCStoreManager.java Tue Jun 5 02:23:03 2012
@@ -998,7 +998,9 @@ public class JDBCStoreManager implements
if (oid == null)
return null;
if (_log.isTraceEnabled()) {
- _log.trace("find: oid="+oid+" "+vm.getDeclaredTypeMapping().getDescribedType());
+ ClassMapping declaredTypeMapping = vm.getDeclaredTypeMapping();
+ Class<?> describedType = (declaredTypeMapping != null) ? declaredTypeMapping.getDescribedType() : null;
+ _log.trace("find: oid="+oid+", describedType="+describedType);
}
Object pc = _ctx.find(oid, fetch, null, null, 0);
if (pc == null && vm != null) {