From 432cfe0bc9a6dfc7ad7ba8e24ae72cd5d3ad1e8a Mon Sep 17 00:00:00 2001 From: teo_sarca Date: Mon, 4 Feb 2008 23:16:05 +0000 Subject: [PATCH] BF [ 1874419 ] JDBC Statement not close in a finally block - fixed for MChangeLog * reorganize imports --- base/src/org/compiere/model/POInfo.java | 37 +++++++++++++------------ 1 file changed, 19 insertions(+), 18 deletions(-) diff --git a/base/src/org/compiere/model/POInfo.java b/base/src/org/compiere/model/POInfo.java index 54b0939c6d..4b89be1cea 100644 --- a/base/src/org/compiere/model/POInfo.java +++ b/base/src/org/compiere/model/POInfo.java @@ -16,12 +16,20 @@ *****************************************************************************/ package org.compiere.model; -import java.io.*; -import java.math.*; -import java.sql.*; -import java.util.*; -import java.util.logging.*; -import org.compiere.util.*; +import java.io.Serializable; +import java.math.BigDecimal; +import java.sql.PreparedStatement; +import java.sql.ResultSet; +import java.sql.SQLException; +import java.util.ArrayList; +import java.util.Properties; +import java.util.logging.Level; + +import org.compiere.util.CCache; +import org.compiere.util.CLogger; +import org.compiere.util.DB; +import org.compiere.util.DisplayType; +import org.compiere.util.Env; /** * Persistet Object Info. @@ -142,11 +150,12 @@ public class POInfo implements Serializable sql.append(" AND e.AD_Language='").append(Env.getAD_Language(m_ctx)).append("'"); // PreparedStatement pstmt = null; + ResultSet rs = null; try { pstmt = DB.prepareStatement(sql.toString(), trxName); pstmt.setInt(1, m_AD_Table_ID); - ResultSet rs = pstmt.executeQuery(); + rs = pstmt.executeQuery(); while (rs.next()) { if (m_TableName == null) @@ -184,22 +193,14 @@ public class POInfo implements Serializable IsTranslated, IsEncrypted); list.add(col); } - rs.close(); } catch (SQLException e) { CLogger.get().log(Level.SEVERE, sql.toString(), e); } - finally - { - if( pstmt != null) - { - try - { - pstmt.close(); - } - catch (SQLException e) {} - } + finally { + DB.close(rs, pstmt); + rs = null; pstmt = null; } // convert to array m_columns = new POInfoColumn[list.size()];