--- old/src/share/classes/com/sun/rowset/JdbcRowSetImpl.java 2012-09-05 16:15:02.000000000 -0400
+++ new/src/share/classes/com/sun/rowset/JdbcRowSetImpl.java 2012-09-05 16:15:02.000000000 -0400
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2003, 2011, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2003, 2012, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -31,7 +31,6 @@
import java.io.*;
import java.math.*;
import java.util.*;
-import java.beans.*;
import javax.sql.rowset.*;
@@ -83,12 +82,6 @@
*/
private ResultSetMetaData resMD;
- /**
- * The property that helps to fire the property changed event when certain
- * properties are changed in the JdbcRowSet
object. This property
- * is being added to satisfy Rave requirements.
- */
- private PropertyChangeSupport propertyChangeSupport;
/**
* The Vector holding the Match Columns
@@ -145,7 +138,6 @@
throw new RuntimeException(ioe);
}
- propertyChangeSupport = new PropertyChangeSupport(this);
initParams();
@@ -268,7 +260,6 @@
throw new RuntimeException(ioe);
}
- propertyChangeSupport = new PropertyChangeSupport(this);
initParams();
// set the defaults
@@ -343,7 +334,6 @@
throw new RuntimeException(ioe);
}
- propertyChangeSupport = new PropertyChangeSupport(this);
initParams();
@@ -363,7 +353,6 @@
// to ensure connection to a db call connect now
// and associate a conn with "this" object
// in this case.
- conn = connect();
setParams();
setReadOnly(true);
@@ -435,7 +424,6 @@
throw new RuntimeException(ioe);
}
- propertyChangeSupport = new PropertyChangeSupport(this);
initParams();
@@ -620,12 +608,7 @@
}
- // An alternate solution is required instead of having the
- // connect method as protected.
- // This is a work around to assist Rave Team
- // :ah
-
- protected Connection connect() throws SQLException {
+ private Connection connect() throws SQLException {
// Get a JDBC connection.
@@ -657,7 +640,6 @@
} else if (getUrl() != null) {
// Check only for getUrl() != null because
// user, passwd can be null
- // Connect using the driver manager.
return DriverManager.getConnection
(getUrl(), getUsername(), getPassword());
@@ -4058,7 +4040,6 @@
if( conn.getHoldability() != HOLD_CURSORS_OVER_COMMIT) {
ResultSet oldVal = rs;
rs = null;
- // propertyChangeSupport.firePropertyChange("ResultSet",oldVal,rs);
}
}
@@ -4121,7 +4102,6 @@
ResultSet oldVal = rs;
rs = null;
- // propertyChangeSupport.firePropertyChange("ResultSet", oldVal,rs);
}
@@ -4248,10 +4228,6 @@
}
- // Over riding the setCommand from BaseRowSet for
- // firing the propertyChangeSupport Event for
- // Rave requirements when this property's value
- // changes.
/**
* Sets this JdbcRowSet
object's command
property to
@@ -4285,20 +4261,13 @@
super.setCommand(command);
ps = null;
rs = null;
- propertyChangeSupport.firePropertyChange("command", oldVal,command);
}
}
else {
super.setCommand(command);
- propertyChangeSupport.firePropertyChange("command", null,command);
}
}
- // Over riding the setDataSourceName from BaseRowSet for
- // firing the propertyChangeSupport Event for
- // Rave requirements when this property's values
- // changes.
-
/**
* Sets the dataSourceName
property for this JdbcRowSet
* object to the given logical name and sets this JdbcRowSet
object's
@@ -4338,19 +4307,13 @@
conn = null;
ps = null;
rs = null;
- propertyChangeSupport.firePropertyChange("dataSourceName",oldVal,dsName);
}
}
else {
super.setDataSourceName(dsName);
- propertyChangeSupport.firePropertyChange("dataSourceName",null,dsName);
}
}
- // Over riding the setUrl from BaseRowSet for
- // firing the propertyChangeSupport Event for
- // Rave requirements when this property's values
- // changes.
/**
* Sets the Url property for this JdbcRowSet
object
@@ -4403,20 +4366,13 @@
conn = null;
ps = null;
rs = null;
- propertyChangeSupport.firePropertyChange("url", oldVal, url);
}
}
else {
super.setUrl(url);
- propertyChangeSupport.firePropertyChange("url", null, url);
}
}
- // Over riding the setUsername from BaseRowSet for
- // firing the propertyChangeSupport Event for
- // Rave requirements when this property's values
- // changes.
-
/**
* Sets the username property for this JdbcRowSet
object
* to the given user name. Because it
@@ -4447,20 +4403,13 @@
conn = null;
ps = null;
rs = null;
- propertyChangeSupport.firePropertyChange("username",oldVal,uname);
}
}
else{
super.setUsername(uname);
- propertyChangeSupport.firePropertyChange("username",null,uname);
}
}
- // Over riding the setPassword from BaseRowSet for
- // firing the propertyChangeSupport Event for
- // Rave requirements when this property's values
- // changes.
-
/**
* Sets the password property for this JdbcRowSet
object
* to the given String
object. Because it
@@ -4490,12 +4439,10 @@
conn = null;
ps = null;
rs = null;
- propertyChangeSupport.firePropertyChange("password",oldVal,password);
}
}
else{
super.setPassword(password);
- propertyChangeSupport.firePropertyChange("password",null,password);
}
}
@@ -4528,7 +4475,6 @@
if(oldVal != type) {
super.setType(type);
- propertyChangeSupport.firePropertyChange("type",oldVal,type);
}
}
@@ -4561,7 +4507,6 @@
if(oldVal != concur) {
super.setConcurrency(concur);
- propertyChangeSupport.firePropertyChange("concurrency",oldVal,concur);
}
}
@@ -4602,7 +4547,6 @@
if(oldVal != transIso) {
super.setTransactionIsolation(transIso);
- propertyChangeSupport.firePropertyChange("transactionIsolation",oldVal,transIso);
}
}
@@ -4632,7 +4576,6 @@
if(oldVal != mRows) {
super.setMaxRows(mRows);
- propertyChangeSupport.firePropertyChange("maxRows",oldVal,mRows);
}
}