Patches item #506673, was opened at 2002-01-21 14:22
You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=376687&aid=506673&group_id=22866

Category: JBossCX
Group: v2.4 (stable)
Status: Open
Resolution: None
Priority: 5
Submitted By: Mike Park (johnnycannuk)
Assigned to: Nobody/Anonymous (nobody)
Summary: Fix for #506549 bug scrollable rs prob

Initial Comment:
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

----------------------------------------------------------------------

You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=376687&aid=506673&group_id=22866

_______________________________________________
Jboss-development mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/jboss-development

Reply via email to