Fixing issue with Reporter data fetching with JDBC 23 driver.

Jacek Gębal · Jacek Gębal · commit 571d95ccc3c8 · 2026-04-22T19:09:24.000+03:00
diff --git a/src/main/java/org/utplsql/api/outputBuffer/AbstractOutputBuffer.java b/src/main/java/org/utplsql/api/outputBuffer/AbstractOutputBuffer.java
@@ -92,7 +92,7 @@ public void fetchAvailable(Connection conn, Consumer<String> onLineFetched) thro
             cstmt.execute();
             cstmt.setFetchSize(fetchSize);
 
-            try (ResultSet resultSet = (ResultSet) cstmt.getObject(1)) {
+            try (ResultSet resultSet = (ResultSet) cstmt.getObject("lines_cursor")) {
                 while (resultSet.next()) {
                     onLineFetched.accept(resultSet.getString("text"));
                 }
@@ -114,7 +114,7 @@ public List<String> fetchAll(Connection conn) throws SQLException {
             cstmt.execute();
             cstmt.setFetchSize(fetchSize);
 
-            try (ResultSet resultSet = (ResultSet) cstmt.getObject(1)) {
+            try (ResultSet resultSet = (ResultSet) cstmt.getObject("lines_cursor")) {
 
                 List<String> outputLines = new ArrayList<>();
                 while (resultSet.next()) {
diff --git a/src/main/java/org/utplsql/api/outputBuffer/CompatibilityOutputBufferPre310.java b/src/main/java/org/utplsql/api/outputBuffer/CompatibilityOutputBufferPre310.java
@@ -20,9 +20,9 @@ class CompatibilityOutputBufferPre310 extends AbstractOutputBuffer {
 
     @Override
     protected CallableStatement getLinesCursorStatement(Connection conn) throws SQLException {
-        CallableStatement cstmt = conn.prepareCall("BEGIN ? := ut_output_buffer.get_lines_cursor(?); END;");
-        cstmt.registerOutParameter(1, OracleTypes.CURSOR);
-        cstmt.setString(2, getReporter().getId());
+        CallableStatement cstmt = conn.prepareCall("begin :lines_cursor := ut_output_buffer.get_lines_cursor(:reporter_id); end;");
+        cstmt.registerOutParameter("lines_cursor", OracleTypes.CURSOR);
+        cstmt.setString("reporter_id", getReporter().getId());
         return cstmt;
     }
 }
diff --git a/src/main/java/org/utplsql/api/outputBuffer/DefaultOutputBuffer.java b/src/main/java/org/utplsql/api/outputBuffer/DefaultOutputBuffer.java
@@ -28,15 +28,15 @@ class DefaultOutputBuffer extends AbstractOutputBuffer {
     @Override
     protected CallableStatement getLinesCursorStatement(Connection conn) throws SQLException {
         Reporter reporter = getReporter();
-        String plsql = "DECLARE" +
-                "  l_rep " + reporter.getTypeName() + " := " + reporter.getTypeName() +"(); " +
-                "BEGIN" +
-                "  l_rep.set_reporter_id(:1); " +
-                "  :2 := l_rep.get_lines_cursor(); " +
-                "END;";
+        String plsql = "declare" +
+                "  l_rep " + reporter.getTypeName() + " := " + reporter.getTypeName() + "(); " +
+                "begin" +
+                "  l_rep.set_reporter_id(:reporter_id); " +
+                "  :lines_cursor := l_rep.get_lines_cursor(); " +
+                "end;";
         OracleCallableStatement cstmt = (OracleCallableStatement) conn.prepareCall(plsql);
-        cstmt.setString(1, reporter.getId());
-        cstmt.registerOutParameter(2, OracleTypes.CURSOR);
+        cstmt.setString("reporter_id", reporter.getId());
+        cstmt.registerOutParameter("lines_cursor", OracleTypes.CURSOR);
         return cstmt;
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -20,9 +20,9 @@ class CompatibilityOutputBufferPre310 extends AbstractOutputBuffer {`
`20`	`20`
`21`	`21`	`@Override`
`22`	`22`	`protected CallableStatement getLinesCursorStatement(Connection conn) throws SQLException {`
`23`		`- CallableStatement cstmt = conn.prepareCall("BEGIN ? := ut_output_buffer.get_lines_cursor(?); END;");`
`24`		`- cstmt.registerOutParameter(1, OracleTypes.CURSOR);`
`25`		`- cstmt.setString(2, getReporter().getId());`
	`23`	`+ CallableStatement cstmt = conn.prepareCall("begin :lines_cursor := ut_output_buffer.get_lines_cursor(:reporter_id); end;");`
	`24`	`+ cstmt.registerOutParameter("lines_cursor", OracleTypes.CURSOR);`
	`25`	`+ cstmt.setString("reporter_id", getReporter().getId());`
`26`	`26`	`return cstmt;`
`27`	`27`	`}`
`28`	`28`	`}`