From c9368fe158635dcb0b1a07150ba0e113ec884ea2 Mon Sep 17 00:00:00 2001
From: "bbaetz%acm.org" <bbaetz%acm.org@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Thu, 20 Feb 2003 07:07:42 +0000
Subject: [PATCH] Bug 193985 - errors from SendSQL aren't being reported
 r,a=justdave

git-svn-id: svn://10.0.0.236/trunk@138028 18797224-902f-48f8-a5cc-f745e15eee43
---
 mozilla/webtools/bugzilla/Bugzilla/DB.pm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/mozilla/webtools/bugzilla/Bugzilla/DB.pm b/mozilla/webtools/bugzilla/Bugzilla/DB.pm
index 0de9b612c52..29935928dee 100644
--- a/mozilla/webtools/bugzilla/Bugzilla/DB.pm
+++ b/mozilla/webtools/bugzilla/Bugzilla/DB.pm
@@ -65,11 +65,11 @@ sub SendSQL {
 
     $_current_sth = Bugzilla->dbh->prepare($str);
 
+    $_current_sth->execute;
+
     # This is really really ugly, but its what we get for not doing
     # error checking for 5 years. See bug 189446 and bug 192531
     $_current_sth->{RaiseError} = 0;
-
-    return $_current_sth->execute;
 }
 
 # Its much much better to use bound params instead of this