You are viewing a plain text version of this content. The canonical link for it is here.
Posted to torque-dev@db.apache.org by he...@apache.org on 2003/01/08 12:06:30 UTC
cvs commit: jakarta-turbine-torque/src/java/org/apache/torque/util Transaction.java
henning 2003/01/08 03:06:30
Modified: src/java/org/apache/torque/util Transaction.java
Log:
Just log the error message, because this could be from
<peerclass>.save() -> Error -> safeRollback
and then, right after the safeRollback, the real exception is
thrown. This is hidden here.
Throwing a NullPointerException was a bad idea anyway... :-)
Revision Changes Path
1.4 +2 -2 jakarta-turbine-torque/src/java/org/apache/torque/util/Transaction.java
Index: Transaction.java
===================================================================
RCS file: /home/cvs/jakarta-turbine-torque/src/java/org/apache/torque/util/Transaction.java,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -r1.3 -r1.4
--- Transaction.java 12 Dec 2002 14:53:28 -0000 1.3
+++ Transaction.java 8 Jan 2003 11:06:30 -0000 1.4
@@ -176,7 +176,7 @@
{
if (con == null)
{
- throw new NullPointerException("Connection object was null. "
+ category.error("Connection object was null. "
+ "This could be due to a misconfiguration of the "
+ "DataSourceFactory. Check the logs and Torque.properties "
+ "to better determine the cause.");