antville/helma
antville/helma
1
1
Fork 0
Code Issues 5 Pull requests 5 Projects Releases 5 Activity Actions

* Set connection readonly flag to true in executeRetrieval() and false in executeUpdate().

Browse source 
* Remove a few unneeded local variables.
This commit is contained in:
hns 2005-08-12 13:18:11 +00:00
parent a878d5919a
commit ac0fc677a5
1 changed files with 8 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
src/helma/scripting/rhino/extensions/DatabaseObject.java
View file

@ -198,18 +198,17 @@ public class DatabaseObject {
ResultSet resultSet = null;
try {
connection.setReadOnly(true);
statement = connection.createStatement();
resultSet = statement.executeQuery(sql); // will return true if first result is a result set
RowSet rowSet = new RowSet(sql, this, statement, resultSet);
return rowSet;
} catch(SQLException e) {
return new RowSet(sql, this, statement, resultSet);
} catch (SQLException e) {
// System.err.println("##Cannot retrieve: " + e);
// e.printStackTrace();
lastError = e;
try {
if (statement!=null) statement.close();
if (statement != null) statement.close();
} catch (Exception ignored) {
}
statement = null;
@ -226,11 +225,12 @@ public class DatabaseObject {
}
Statement statement = null;
try {
try {
connection.setReadOnly(false);
statement = connection.createStatement();
count = statement.executeUpdate(sql); // will return true if first result is a result set
} catch(SQLException e) {
} catch (SQLException e) {
// System.err.println("##Cannot retrieve: " + e);
// e.printStackTrace();
lastError = e;
@ -594,8 +594,7 @@ class RowSet {
public String[] getSpecialPropertyNames() {
String [] ns = {"length"};
return ns;
return new String[] {"length"};
}