Menu

#132 Fix for #506549 bug scrollable rs prob

v2.4 (stable)
closed-fixed
nobody
JBossCX (28)
5
2002-04-05
2002-01-21
Mike Park
No

Here is the patch that fixes this:

The old call to con.prepareStatement(sql,int,int)

was implemented like this:

public PreparedStatement prepareStatement(String sql,
int resultSetType, int resultSetConcurrency) throws
SQLException {
return this.prepareStatement(sql);

}

it should be like so:

public PreparedStatement prepareStatement(String sql,
int resultSetType, int resultSetConcurrency) throws
SQLException {
if(con == null) throw new SQLException(CLOSED);
try {
return con.prepareStatement(sql,
resultSetType, resultSetConcurrency);
} catch(SQLException e) {
setError(e);
throw e;
}

}

I have tested this out (although not with the
testsuite) and it works well - seems like a simple
oversight considering the implementations of
prepareStatment() and prepareCall() around it were
done correctly.

I have included the minimally changed
XAClientConnection.java file

Discussion

  • Mike Park

    Mike Park - 2002-01-21

    XAClientConnection.java fix for bug #506549

     
  • Scott M Stark

    Scott M Stark - 2002-04-05

    Logged In: YES
    user_id=175228

    Applied for 2.4.5

     
  • Scott M Stark

    Scott M Stark - 2002-04-05
    • status: open --> closed-fixed
     

Log in to post a comment.