errorproperty and warningproperty for <sql>. Submitted by Andrew Stevens. PR 38807

git-svn-id: https://svn.apache.org/repos/asf/ant/core/trunk@807228 13f79535-47bb-0310-9956-ffa450edef68

CONTRIBUTORS

@@ -10,6 +10,7 @@ Alison Winters
 Andreas Ames
 Andreas Mross
 Andrew Everitt
+Andrew Stevens
 Andrey Urazov
 Andy Wood
 Anil K. Vijendran

WHATSNEW

@@ -897,6 +897,10 @@ Other changes:
 
  * A new <resourceexists> condition can check whether resources exists.
 
+ * <sql> has two new attributes errorproperty and warningproperty that
+   can be set if an error/warning occurs.
+   Bugzilla Report 38807.
+
 Changes from Ant 1.7.0 TO Ant 1.7.1
 =============================================
 

contributors.xml

@@ -66,6 +66,10 @@
     <first>Andrew</first>
     <last>Everitt</last>
   </name>
+  <name>
+    <first>Andrew</first>
+    <last>Stevens</last>
+  </name>
   <name>
     <first>Andrey</first>
     <last>Urazov</last>

docs/manual/CoreTasks/sql.html

@@ -256,6 +256,18 @@ and <b>abort</b> execution and transaction and fail task.</p>
     ever occurs)</td>
 </tr>
 
+<tr>
+  <td valign="top">errorproperty</td>
+  <td valign="top">The name of a property to set in the event of an
+    error.  <em>Since Ant 1.8.0</em></td>
+  <td align="center" valign="top">No</td>
+</tr>
+<tr>
+  <td valign="top">warningproperty</td>
+  <td valign="top">The name of a property to set in the event of an
+    warning.  <em>Since Ant 1.8.0</em></td>
+  <td align="center" valign="top">No</td>
+</tr>
 </table>
 
 <h3>Parameters specified as nested elements</h3>

src/main/org/apache/tools/ant/taskdefs/SQLExec.java

@@ -250,6 +250,18 @@ public class SQLExec extends JDBCTask {
      */
     private boolean treatWarningsAsErrors = false;
 
+    /**
+     * The name of the property to set in the event of an error
+     * @since Ant 1.8.0
+     */
+    private String errorProperty = null;
+
+    /**
+     * The name of the property to set in the event of a warning
+     * @since Ant 1.8.0
+     */
+    private String warningProperty = null;
+
     /**
      * Set the name of the SQL file to be run.
      * Required unless statements are enclosed in the build file
@@ -521,6 +533,28 @@ public class SQLExec extends JDBCTask {
         csvQuoteChar = s;
     }
 
+    /**
+     * Property to set to "true" if a statement throws an error.
+     *
+     * @param errorProperty the name of the property to set in the
+     * event of an error.
+     * @since Ant 1.8.0
+     */
+    public void setErrorProperty(String errorProperty) {
+        this.errorProperty = errorProperty;
+    }
+
+    /**
+     * Property to set to "true" if a statement produces a warning.
+     *
+     * @param warningProperty the name of the property to set in the
+     * event of a warning.
+     * @since Ant 1.8.0
+     */
+    public void setWarningProperty(String warningProperty) {
+        this.warningProperty = warningProperty;
+    }
+
     /**
      * Load the sql file and then execute it
      * @throws BuildException on error.
@@ -614,11 +648,13 @@ public class SQLExec extends JDBCTask {
                 if (onError.equals("abort")) {
                     throw new BuildException(e, getLocation());
                 }
+                setErrorProperty();
             } catch (SQLException e) {
                 closeQuietly();
                 if (onError.equals("abort")) {
                     throw new BuildException(e, getLocation());
                 }
+                setErrorProperty();
             } finally {
                 try {
                     if (getStatement() != null) {
@@ -758,6 +794,7 @@ public class SQLExec extends JDBCTask {
             if (!onError.equals("continue")) {
                 throw e;
             }
+            setErrorProperty();
         } finally {
             if (resultSet != null) {
                 try {
@@ -1059,5 +1096,22 @@ public class SQLExec extends JDBCTask {
         if (treatWarningsAsErrors && initialWarning != null) {
             throw initialWarning;
         }
+        if (initialWarning != null) {
+            setWarningProperty();
+        }
+    }
+
+    protected final void setErrorProperty() {
+        setProperty(errorProperty);
+    }
+
+    protected final void setWarningProperty() {
+        setProperty(warningProperty);
+    }
+
+    private void setProperty(String name) {
+        if (name != null) {
+            getProject().setNewProperty(name, "true");
+        }
     }
 }