From 826f3d86df8123397da884dca13fedaa07a7180c Mon Sep 17 00:00:00 2001 From: "Redhuan D. Oon" Date: Sun, 2 Nov 2008 10:23:27 +0000 Subject: [PATCH] [ 2214883 ] Remove SQL code and Replace for Query tested by creating Measurement (Achievement) recs - no error --- base/src/org/compiere/model/MAchievement.java | 40 ++++--------------- 1 file changed, 7 insertions(+), 33 deletions(-) diff --git a/base/src/org/compiere/model/MAchievement.java b/base/src/org/compiere/model/MAchievement.java index ccb1533b92..fc1d150ef5 100644 --- a/base/src/org/compiere/model/MAchievement.java +++ b/base/src/org/compiere/model/MAchievement.java @@ -27,6 +27,7 @@ import org.compiere.util.*; * * @author Jorg Janke * @version $Id: MAchievement.java,v 1.2 2006/07/30 00:51:02 jjanke Exp $ + * red1 - [ 2214883 ] Remove SQL code and Replace for Query */ public class MAchievement extends X_PA_Achievement { @@ -48,39 +49,12 @@ public class MAchievement extends X_PA_Achievement */ public static MAchievement[] getOfMeasure (Properties ctx, int PA_Measure_ID) { - ArrayList list = new ArrayList(); - String sql = "SELECT * FROM PA_Achievement " - + "WHERE PA_Measure_ID=? AND IsAchieved='Y' ORDER BY SeqNo, DateDoc"; - PreparedStatement pstmt = null; - try - { - pstmt = DB.prepareStatement (sql, null); - pstmt.setInt (1, PA_Measure_ID); - ResultSet rs = pstmt.executeQuery (); - while (rs.next ()) - list.add (new MAchievement (ctx, rs, null)); - rs.close (); - pstmt.close (); - pstmt = null; - } - catch (Exception e) - { - s_log.log (Level.SEVERE, sql, e); - } - try - { - if (pstmt != null) - pstmt.close (); - pstmt = null; - } - catch (Exception e) - { - pstmt = null; - } - // - MAchievement[] retValue = new MAchievement[list.size ()]; - list.toArray (retValue); - return retValue; + String whereClause ="PA_Measure_ID=? AND IsAchieved='Y'"; + List list = new Query(ctx,MAchievement.Table_Name, whereClause, null) + .setParameters(new Object[]{PA_Measure_ID}).setOrderBy("SeqNo, DateDoc").list(); + + return list.toArray(new MAchievement[list.size()]); + } // getOfMeasure