From 177111b5598e4301c2d4e2de7c86b7e0a8acef88 Mon Sep 17 00:00:00 2001
From: Albert Lee <allee8285@apache.org>
Date: Wed, 27 Feb 2008 17:46:00 +0000
Subject: [PATCH] Potential NPE where custom can be null, Boolean.TRUE or
 Boolean.FALSE. Follow the pattern in update() and deleet()

git-svn-id: https://svn.apache.org/repos/asf/openjpa/trunk@631663 13f79535-47bb-0310-9956-ffa450edef68
---
 .../org/apache/openjpa/jdbc/kernel/AbstractUpdateManager.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/AbstractUpdateManager.java b/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/AbstractUpdateManager.java
index 46886b036..bf527d0b2 100644
--- a/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/AbstractUpdateManager.java
+++ b/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/AbstractUpdateManager.java
@@ -187,7 +187,7 @@ public abstract class AbstractUpdateManager
         Boolean custom = mapping.isCustomInsert(sm, store);
         if (!Boolean.FALSE.equals(custom))
             mapping.customInsert(sm, store);
-        if (custom.equals(Boolean.TRUE))
+        if (Boolean.TRUE.equals(custom))
             return;
 
         ClassMapping sup = mapping.getJoinablePCSuperclassMapping();