ResultSetIterator.hasNext() now throws IllegalStateException instead of

RuntimeException to wrap cases of SQLException

ResultSetIterator.next() now throws IllegalStateException instead of
RuntimeException to wrap cases of SQLException

Author: garydgregory

src/changes/changes.xml

@@ -33,7 +33,8 @@
       <action type="fix" dev="ggregory" due-to="Gary Gregory">BeanComparator.compare(T, T) now throws IllegalArgumentException instead of RuntimeException to wrap all cases of ReflectiveOperationException.</action>
       <action type="fix" dev="ggregory" due-to="Gary Gregory">MappedMethodReference.get() now throws IllegalStateException instead of RuntimeException to wrap cases of NoSuchMethodException.</action>
       <action type="fix" dev="ggregory" due-to="Gary Gregory">ResultSetIterator.get(String) now throws IllegalArgumentException instead of RuntimeException to wrap cases of SQLException.</action>
-      <action type="fix" dev="ggregory" due-to="Gary Gregory">ResultSetIterator.hasNext() now throws IllegalArgumentException instead of RuntimeException to wrap cases of SQLException.</action>
+      <action type="fix" dev="ggregory" due-to="Gary Gregory">ResultSetIterator.hasNext() now throws IllegalStateException instead of RuntimeException to wrap cases of SQLException.</action>
+      <action type="fix" dev="ggregory" due-to="Gary Gregory">ResultSetIterator.next() now throws IllegalStateException instead of RuntimeException to wrap cases of SQLException.</action>
       <action type="fix" dev="ggregory" due-to="Gary Gregory">ResultSetIterator.set(String, Object) now throws IllegalArgumentException instead of RuntimeException to wrap cases of SQLException.</action>
       <action type="fix" dev="ggregory" due-to="Gary Gregory">ResultSetIterator.set(String, String, Object) now throws IllegalArgumentException instead of RuntimeException to wrap cases of SQLException.</action>
       <!-- ADD -->

src/main/java/org/apache/commons/beanutils/ResultSetIterator.java

@@ -176,7 +176,7 @@ public boolean hasNext() {
             advance();
             return !eof;
         } catch (final SQLException e) {
-            throw new IllegalArgumentException("hasNext():  SQLException:  " + e, e);
+            throw new IllegalStateException("hasNext():  SQLException:  " + e, e);
         }
     }
 
@@ -195,7 +195,7 @@ public DynaBean next() {
             current = false;
             return this;
         } catch (final SQLException e) {
-            throw new IllegalArgumentException("next():  SQLException:  " + e, e);
+            throw new IllegalStateException("next():  SQLException:  " + e, e);
         }
 
     }