midsuit
/
core-jgi
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Removing some unnecesary code after [ 1886013 ] MSequence.getNextId should be executed outside of trx

This commit is contained in:
Carlos Ruiz 2008-02-06 05:30:22 +00:00
parent 14deebdab7
commit 9c85560ff3
1 changed files with 12 additions and 25 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
base/src/org/compiere/model/MSequence.java
View File

@ -142,22 +142,15 @@ public class MSequence extends X_AD_Sequence
Connection conn = null; Connection conn = null;
PreparedStatement pstmt = null; PreparedStatement pstmt = null;
ResultSet rs = null; ResultSet rs = null;
boolean autocommit = false;
for (int i = 0; i < 3; i++) for (int i = 0; i < 3; i++)
{ {
try try
{ {
//if (trx != null)
//conn = trx.getConnection();
//else
conn = DB.getConnectionID(); conn = DB.getConnectionID();
// Error // Error
if (conn == null) if (conn == null)
return -1; return -1;
//
//jz auto commit off here
autocommit = conn.getAutoCommit();
conn.setAutoCommit(false);
pstmt = conn.prepareStatement(selectSQL, pstmt = conn.prepareStatement(selectSQL,
ResultSet.TYPE_FORWARD_ONLY, ResultSet.CONCUR_UPDATABLE); ResultSet.TYPE_FORWARD_ONLY, ResultSet.CONCUR_UPDATABLE);
pstmt.setString(1, TableName); pstmt.setString(1, TableName);
@ -270,14 +263,9 @@ public class MSequence extends X_AD_Sequence
s_log.log(Level.SEVERE, TableName + " - " + e.getMessage(), e); s_log.log(Level.SEVERE, TableName + " - " + e.getMessage(), e);
try try
{ {
if (conn != null)
conn.rollback(); conn.rollback();
conn.setAutoCommit(autocommit); //jz set back } catch (SQLException e1) { }
if (pstmt != null)
pstmt.close();
}
catch (SQLException e1)
{
}
} }
finally finally
{ {
@ -286,9 +274,6 @@ public class MSequence extends X_AD_Sequence
rs = null; rs = null;
if (conn != null) if (conn != null)
{ {
try {
conn.setAutoCommit(autocommit); //jz set back
} catch (SQLException e) {}
try { try {
conn.close(); conn.close();
} catch (SQLException e) {} } catch (SQLException e) {}
@ -616,10 +601,11 @@ public class MSequence extends X_AD_Sequence
DB.close(rs, pstmt); DB.close(rs, pstmt);
try try
{ {
if (trx == null && conn != null) if (trx == null && conn != null) {
conn.close(); conn.close();
conn = null; conn = null;
} }
}
catch (Exception e) catch (Exception e)
{ {
s_log.log(Level.SEVERE, "(Table) - finish", e); s_log.log(Level.SEVERE, "(Table) - finish", e);
@ -984,10 +970,11 @@ public class MSequence extends X_AD_Sequence
try try
{ {
DB.close(rs, pstmt); DB.close(rs, pstmt);
if (trx == null && conn != null) if (trx == null && conn != null) {
conn.close(); conn.close();
conn = null; conn = null;
} }
}
catch (Exception e) catch (Exception e)
{ {
s_log.log(Level.SEVERE, "(DocType) - finish", e); s_log.log(Level.SEVERE, "(DocType) - finish", e);