r18145 - gnucash/trunk/src/backend/dbi - Move debug lines printing SQL statements being executed before the actual execution so that

[Phil Longstaff]

Author: plongstaff
Date: 2009-06-21 10:38:44 -0400 (Sun, 21 Jun 2009)
New Revision: 18145
Trac: http://svn.gnucash.org/trac/changeset/18145

Modified:
   gnucash/trunk/src/backend/dbi/gnc-backend-dbi.c
Log:
Move debug lines printing SQL statements being executed before the actual execution so that
if there are problems, the SQL statement will be logged.


Modified: gnucash/trunk/src/backend/dbi/gnc-backend-dbi.c
===================================================================
--- gnucash/trunk/src/backend/dbi/gnc-backend-dbi.c	2009-06-21 13:29:35 UTC (rev 18144)
+++ gnucash/trunk/src/backend/dbi/gnc-backend-dbi.c	2009-06-21 14:38:44 UTC (rev 18145)
@@ -1204,12 +1204,12 @@
 	GncDbiSqlStatement* dbi_stmt = (GncDbiSqlStatement*)stmt;
 	dbi_result result;
 														
+	DEBUG( "SQL: %s\n", dbi_stmt->sql->str );
 	result = dbi_conn_query( dbi_conn->conn, dbi_stmt->sql->str );
 	if( result == NULL ) {
 		PERR( "Error executing SQL %s\n", dbi_stmt->sql->str );
 		return NULL;
 	}
-	DEBUG( "SQL: %s\n", dbi_stmt->sql->str );
 	return create_dbi_result( dbi_conn, result );
 }
 
@@ -1222,12 +1222,12 @@
 	gint num_rows;
 	gint status;
 
+	DEBUG( "SQL: %s\n", dbi_stmt->sql->str );
 	result = dbi_conn_query( dbi_conn->conn, dbi_stmt->sql->str );
 	if( result == NULL ) {
 		PERR( "Error executing SQL %s\n", dbi_stmt->sql->str );
 		return -1;
 	}
-	DEBUG( "SQL: %s\n", dbi_stmt->sql->str );
 	num_rows = (gint)dbi_result_get_numrows_affected( result );
 	status = dbi_result_free( result );
 	if( status < 0 ) {
@@ -1294,8 +1294,8 @@
 	dbi_result result;
 	gint status;
 
+	DEBUG( "BEGIN\n" );
 	result = dbi_conn_queryf( dbi_conn->conn, "BEGIN" );
-	DEBUG( "BEGIN\n" );
 	status = dbi_result_free( result );
 	if( status < 0 ) {
 		PERR( "Error in dbi_result_free() result\n" );
@@ -1312,8 +1312,8 @@
 	dbi_result result;
 	gint status;
 
+	DEBUG( "ROLLBACK\n" );
 	result = dbi_conn_queryf( dbi_conn->conn, "ROLLBACK" );
-	DEBUG( "ROLLBACK\n" );
 	status = dbi_result_free( result );
 	if( status < 0 ) {
 		PERR( "Error in dbi_result_free() result\n" );
@@ -1330,8 +1330,8 @@
 	dbi_result result;
 	gint status;
 
+	DEBUG( "COMMIT\n" );
 	result = dbi_conn_queryf( dbi_conn->conn, "COMMIT" );
-	DEBUG( "COMMIT\n" );
 	status = dbi_result_free( result );
 	if( status < 0 ) {
 		PERR( "Error in dbi_result_free() result\n" );