svn commit: r451382 - /incubator/ofbiz/trunk/framework/entity/src/org/ofbiz/entity/transaction/TransactionUtil.java

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

svn commit: r451382 - /incubator/ofbiz/trunk/framework/entity/src/org/ofbiz/entity/transaction/TransactionUtil.java

jonesde
Author: jonesde
Date: Fri Sep 29 11:52:42 2006
New Revision: 451382

URL: http://svn.apache.org/viewvc?view=rev&rev=451382
Log:
A couple small cleanups, showing a current location of warnings as the messages advertise

Modified:
    incubator/ofbiz/trunk/framework/entity/src/org/ofbiz/entity/transaction/TransactionUtil.java

Modified: incubator/ofbiz/trunk/framework/entity/src/org/ofbiz/entity/transaction/TransactionUtil.java
URL: http://svn.apache.org/viewvc/incubator/ofbiz/trunk/framework/entity/src/org/ofbiz/entity/transaction/TransactionUtil.java?view=diff&rev=451382&r1=451381&r2=451382
==============================================================================
--- incubator/ofbiz/trunk/framework/entity/src/org/ofbiz/entity/transaction/TransactionUtil.java (original)
+++ incubator/ofbiz/trunk/framework/entity/src/org/ofbiz/entity/transaction/TransactionUtil.java Fri Sep 29 11:52:42 2006
@@ -543,7 +543,7 @@
         Exception e = (Exception) transactionBeginStack.get();
         if (e == null) {
             Exception e2 = new Exception("Current Stack Trace");
-            Debug.logWarning("WARNING: In clearTransactionBeginStack no stack placeholder was in place, here is the current location: ", module);
+            Debug.logWarning(e2, "WARNING: In clearTransactionBeginStack no stack placeholder was in place, here is the current location: ", module);
             return null;
         } else {
             transactionBeginStack.set(null);
@@ -551,11 +551,12 @@
         }
     }
     public static Exception getTransactionBeginStack() {
-        if (transactionBeginStack.get() == null) {
+        Exception e = (Exception) transactionBeginStack.get();
+        if (e == null) {
             Exception e2 = new Exception("Current Stack Trace");
-            Debug.logWarning("WARNING: In getTransactionBeginStack no stack placeholder was in place, here is the current location: ", module);
+            Debug.logWarning(e2, "WARNING: In getTransactionBeginStack no stack placeholder was in place, here is the current location: ", module);
         }
-        return (Exception) transactionBeginStack.get();
+        return e;
     }
 
     // =======================================