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

IDEMPIERE-638 Check log level before calling logging method / s_log.config

This commit is contained in:
Richard Morales 2013-02-21 12:11:33 -05:00
parent 379aa8ca1a
commit fee2df24c9
8 changed files with 26 additions and 26 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
org.adempiere.base.process/src/org/compiere/process/DocumentTypeVerify.java
View File

@ -88,7 +88,7 @@ public class DocumentTypeVerify extends SvrProcess
{ {
String name = rs.getString(2); String name = rs.getString(2);
String value = rs.getString(1); String value = rs.getString(1);
s_log.config(name + "=" + value); if (s_log.isLoggable(Level.CONFIG)) s_log.config(name + "=" + value);
MDocType dt = new MDocType (ctx, value, name, trxName); MDocType dt = new MDocType (ctx, value, name, trxName);
if (dt.save()) if (dt.save())
{ {
@ -166,7 +166,7 @@ public class DocumentTypeVerify extends SvrProcess
int Client_ID = rs.getInt(1); int Client_ID = rs.getInt(1);
int C_Period_ID = rs.getInt(2); int C_Period_ID = rs.getInt(2);
String DocBaseType = rs.getString(3); String DocBaseType = rs.getString(3);
s_log.config("AD_Client_ID=" + Client_ID if (s_log.isLoggable(Level.CONFIG)) s_log.config("AD_Client_ID=" + Client_ID
+ ", C_Period_ID=" + C_Period_ID + ", DocBaseType=" + DocBaseType); + ", C_Period_ID=" + C_Period_ID + ", DocBaseType=" + DocBaseType);
// //
MPeriodControl pc = new MPeriodControl (ctx, Client_ID, MPeriodControl pc = new MPeriodControl (ctx, Client_ID,

4
org.adempiere.base/src/org/compiere/model/MCost.java
View File

@ -725,7 +725,7 @@ public class MCost extends X_M_Cost
if (cost.is_new()) if (cost.is_new())
{ {
if (cost.save()) if (cost.save())
s_log.config("Std.Cost for " + product.getName() if (s_log.isLoggable(Level.CONFIG)) s_log.config("Std.Cost for " + product.getName()
+ " - " + as.getName()); + " - " + as.getName());
else else
s_log.warning("Not created: Std.Cost for " + product.getName() s_log.warning("Not created: Std.Cost for " + product.getName()
@ -743,7 +743,7 @@ public class MCost extends X_M_Cost
if (cost.is_new()) if (cost.is_new())
{ {
if (cost.save()) if (cost.save())
s_log.config("Std.Cost for " + product.getName() if (s_log.isLoggable(Level.CONFIG)) s_log.config("Std.Cost for " + product.getName()
+ " - " + o.getName() + " - " + o.getName()
+ " - " + as.getName()); + " - " + as.getName());
else else

26
org.adempiere.base/src/org/compiere/model/MCostDetail.java
View File

@ -80,7 +80,7 @@ public class MCostDetail extends X_M_CostDetail
.append(" AND M_AttributeSetInstance_ID=").append(M_AttributeSetInstance_ID); .append(" AND M_AttributeSetInstance_ID=").append(M_AttributeSetInstance_ID);
int no = DB.executeUpdate(sql.toString(), trxName); int no = DB.executeUpdate(sql.toString(), trxName);
if (no != 0) if (no != 0)
s_log.config("Deleted #" + no); if (s_log.isLoggable(Level.CONFIG)) s_log.config("Deleted #" + no);
MCostDetail cd = get (as.getCtx(), "C_OrderLine_ID=?", MCostDetail cd = get (as.getCtx(), "C_OrderLine_ID=?",
C_OrderLine_ID, M_AttributeSetInstance_ID, as.getC_AcctSchema_ID(), trxName); C_OrderLine_ID, M_AttributeSetInstance_ID, as.getC_AcctSchema_ID(), trxName);
// //
@ -113,7 +113,7 @@ public class MCostDetail extends X_M_CostDetail
{ {
ok = cd.process(); ok = cd.process();
} }
s_log.config("(" + ok + ") " + cd); if (s_log.isLoggable(Level.CONFIG)) s_log.config("(" + ok + ") " + cd);
return ok; return ok;
} // createOrder } // createOrder
@ -147,7 +147,7 @@ public class MCostDetail extends X_M_CostDetail
.append(" AND M_AttributeSetInstance_ID=").append(M_AttributeSetInstance_ID); .append(" AND M_AttributeSetInstance_ID=").append(M_AttributeSetInstance_ID);
int no = DB.executeUpdate(sql.toString(), trxName); int no = DB.executeUpdate(sql.toString(), trxName);
if (no != 0) if (no != 0)
s_log.config("Deleted #" + no); if (s_log.isLoggable(Level.CONFIG)) s_log.config("Deleted #" + no);
MCostDetail cd = get (as.getCtx(), "C_InvoiceLine_ID=?", MCostDetail cd = get (as.getCtx(), "C_InvoiceLine_ID=?",
C_InvoiceLine_ID, M_AttributeSetInstance_ID, as.getC_AcctSchema_ID(), trxName); C_InvoiceLine_ID, M_AttributeSetInstance_ID, as.getC_AcctSchema_ID(), trxName);
// //
@ -180,7 +180,7 @@ public class MCostDetail extends X_M_CostDetail
{ {
ok = cd.process(); ok = cd.process();
} }
s_log.config("(" + ok + ") " + cd); if (s_log.isLoggable(Level.CONFIG)) s_log.config("(" + ok + ") " + cd);
return ok; return ok;
} // createInvoice } // createInvoice
@ -214,7 +214,7 @@ public class MCostDetail extends X_M_CostDetail
.append(" AND M_AttributeSetInstance_ID=").append(M_AttributeSetInstance_ID); .append(" AND M_AttributeSetInstance_ID=").append(M_AttributeSetInstance_ID);
int no = DB.executeUpdate(sql.toString(), trxName); int no = DB.executeUpdate(sql.toString(), trxName);
if (no != 0) if (no != 0)
s_log.config("Deleted #" + no); if (s_log.isLoggable(Level.CONFIG)) s_log.config("Deleted #" + no);
MCostDetail cd = get (as.getCtx(), "M_InOutLine_ID=?", MCostDetail cd = get (as.getCtx(), "M_InOutLine_ID=?",
M_InOutLine_ID, M_AttributeSetInstance_ID, as.getC_AcctSchema_ID(), trxName); M_InOutLine_ID, M_AttributeSetInstance_ID, as.getC_AcctSchema_ID(), trxName);
// //
@ -248,7 +248,7 @@ public class MCostDetail extends X_M_CostDetail
{ {
ok = cd.process(); ok = cd.process();
} }
s_log.config("(" + ok + ") " + cd); if (s_log.isLoggable(Level.CONFIG)) s_log.config("(" + ok + ") " + cd);
return ok; return ok;
} // createShipment } // createShipment
@ -281,7 +281,7 @@ public class MCostDetail extends X_M_CostDetail
.append(" AND M_AttributeSetInstance_ID=").append(M_AttributeSetInstance_ID); .append(" AND M_AttributeSetInstance_ID=").append(M_AttributeSetInstance_ID);
int no = DB.executeUpdate(sql.toString(), trxName); int no = DB.executeUpdate(sql.toString(), trxName);
if (no != 0) if (no != 0)
s_log.config("Deleted #" + no); if (s_log.isLoggable(Level.CONFIG)) s_log.config("Deleted #" + no);
MCostDetail cd = get (as.getCtx(), "M_InventoryLine_ID=?", MCostDetail cd = get (as.getCtx(), "M_InventoryLine_ID=?",
M_InventoryLine_ID, M_AttributeSetInstance_ID, as.getC_AcctSchema_ID(), trxName); M_InventoryLine_ID, M_AttributeSetInstance_ID, as.getC_AcctSchema_ID(), trxName);
// //
@ -314,7 +314,7 @@ public class MCostDetail extends X_M_CostDetail
{ {
ok = cd.process(); ok = cd.process();
} }
s_log.config("(" + ok + ") " + cd); if (s_log.isLoggable(Level.CONFIG)) s_log.config("(" + ok + ") " + cd);
return ok; return ok;
} // createInventory } // createInventory
@ -349,7 +349,7 @@ public class MCostDetail extends X_M_CostDetail
.append(" AND M_AttributeSetInstance_ID=").append(M_AttributeSetInstance_ID); .append(" AND M_AttributeSetInstance_ID=").append(M_AttributeSetInstance_ID);
int no = DB.executeUpdate(sql.toString(), trxName); int no = DB.executeUpdate(sql.toString(), trxName);
if (no != 0) if (no != 0)
s_log.config("Deleted #" + no); if (s_log.isLoggable(Level.CONFIG)) s_log.config("Deleted #" + no);
StringBuilder msget = new StringBuilder( "M_MovementLine_ID=? AND IsSOTrx=") StringBuilder msget = new StringBuilder( "M_MovementLine_ID=? AND IsSOTrx=")
.append((from ? "'Y'" : "'N'")); .append((from ? "'Y'" : "'N'"));
MCostDetail cd = get (as.getCtx(),msget.toString(), MCostDetail cd = get (as.getCtx(),msget.toString(),
@ -385,7 +385,7 @@ public class MCostDetail extends X_M_CostDetail
{ {
ok = cd.process(); ok = cd.process();
} }
s_log.config("(" + ok + ") " + cd); if (s_log.isLoggable(Level.CONFIG)) s_log.config("(" + ok + ") " + cd);
return ok; return ok;
} // createMovement } // createMovement
@ -418,7 +418,7 @@ public class MCostDetail extends X_M_CostDetail
.append(" AND M_AttributeSetInstance_ID=").append(M_AttributeSetInstance_ID); .append(" AND M_AttributeSetInstance_ID=").append(M_AttributeSetInstance_ID);
int no = DB.executeUpdate(sql.toString(), trxName); int no = DB.executeUpdate(sql.toString(), trxName);
if (no != 0) if (no != 0)
s_log.config("Deleted #" + no); if (s_log.isLoggable(Level.CONFIG)) s_log.config("Deleted #" + no);
MCostDetail cd = get (as.getCtx(), "M_ProductionLine_ID=?", MCostDetail cd = get (as.getCtx(), "M_ProductionLine_ID=?",
M_ProductionLine_ID, M_AttributeSetInstance_ID, as.getC_AcctSchema_ID(), trxName); M_ProductionLine_ID, M_AttributeSetInstance_ID, as.getC_AcctSchema_ID(), trxName);
// //
@ -451,7 +451,7 @@ public class MCostDetail extends X_M_CostDetail
{ {
ok = cd.process(); ok = cd.process();
} }
s_log.config("(" + ok + ") " + cd); if (s_log.isLoggable(Level.CONFIG)) s_log.config("(" + ok + ") " + cd);
return ok; return ok;
} // createProduction } // createProduction
@ -547,7 +547,7 @@ public class MCostDetail extends X_M_CostDetail
else else
counterError++; counterError++;
} }
s_log.config("OK=" + counterOK + ", Errors=" + counterError); if (s_log.isLoggable(Level.CONFIG)) s_log.config("OK=" + counterOK + ", Errors=" + counterError);
return counterError == 0; return counterError == 0;
} // processProduct } // processProduct

6
org.adempiere.base/src/org/compiere/model/MCostQueue.java
View File

@ -259,7 +259,7 @@ public class MCostQueue extends X_M_CostQueue
lastPrice = queue.getCurrentCostPrice(); lastPrice = queue.getCurrentCostPrice();
BigDecimal costBatch = lastPrice.multiply(remainingQty); BigDecimal costBatch = lastPrice.multiply(remainingQty);
cost = cost.add(costBatch); cost = cost.add(costBatch);
s_log.config("ASI=" + queue.getM_AttributeSetInstance_ID() if (s_log.isLoggable(Level.CONFIG)) s_log.config("ASI=" + queue.getM_AttributeSetInstance_ID()
+ " - Cost=" + lastPrice + " * Qty=" + remainingQty + "(!) = " + costBatch); + " - Cost=" + lastPrice + " * Qty=" + remainingQty + "(!) = " + costBatch);
return cost; return cost;
} }
@ -281,7 +281,7 @@ public class MCostQueue extends X_M_CostQueue
// Done // Done
if (remainingQty.signum() == 0) if (remainingQty.signum() == 0)
{ {
s_log.config("Cost=" + cost); if (s_log.isLoggable(Level.CONFIG)) s_log.config("Cost=" + cost);
return cost; return cost;
} }
if (firstPrice == null) if (firstPrice == null)
@ -303,7 +303,7 @@ public class MCostQueue extends X_M_CostQueue
BigDecimal costBatch = lastPrice.multiply(remainingQty); BigDecimal costBatch = lastPrice.multiply(remainingQty);
s_log.fine("RemainingQty=" + remainingQty + " * LastPrice=" + lastPrice + " = " + costBatch); s_log.fine("RemainingQty=" + remainingQty + " * LastPrice=" + lastPrice + " = " + costBatch);
cost = cost.add(costBatch); cost = cost.add(costBatch);
s_log.config("Cost=" + cost); if (s_log.isLoggable(Level.CONFIG)) s_log.config("Cost=" + cost);
return cost; return cost;
} // getCosts } // getCosts

2
org.adempiere.base/src/org/compiere/model/MInvoice.java
View File

@ -1191,7 +1191,7 @@ public class MInvoice extends X_C_Invoice implements DocAction
finally { finally {
DB.close(rs); DB.close(rs);
} }
s_log.config("#" + counter); if (s_log.isLoggable(Level.CONFIG)) s_log.config("#" + counter);
/**/ /**/
} // setIsPaid } // setIsPaid

2
org.adempiere.base/src/org/compiere/model/MPayment.java
View File

@ -872,7 +872,7 @@ public final class MPayment extends X_C_Payment
rs = null; rs = null;
pstmt = null; pstmt = null;
} }
s_log.config("#" + counter); if (s_log.isLoggable(Level.CONFIG)) s_log.config("#" + counter);
} // setIsAllocated } // setIsAllocated
/************************************************************************** /**************************************************************************

6
org.adempiere.base/src/org/compiere/print/MPrintPaper.java
View File

@ -18,6 +18,7 @@ package org.compiere.print;
import java.sql.ResultSet; import java.sql.ResultSet;
import java.util.Properties; import java.util.Properties;
import java.util.logging.Level;
import javax.print.attribute.EnumSyntax; import javax.print.attribute.EnumSyntax;
import javax.print.attribute.Size2DSyntax; import javax.print.attribute.Size2DSyntax;
@ -52,8 +53,7 @@ public class MPrintPaper extends X_AD_PrintPaper
/** /**
* *
*/ */
private static final long serialVersionUID = 8284757169766494111L; private static final long serialVersionUID = -3609557177958141344L;
/** /**
* Get Paper * Get Paper
@ -70,7 +70,7 @@ public class MPrintPaper extends X_AD_PrintPaper
s_papers.put(key, pp); s_papers.put(key, pp);
} }
else else
s_log.config("AD_PrintPaper_ID=" + AD_PrintPaper_ID); if (s_log.isLoggable(Level.CONFIG)) s_log.config("AD_PrintPaper_ID=" + AD_PrintPaper_ID);
return pp; return pp;
} // get } // get

2
org.adempiere.base/src/org/compiere/wf/MWorkflow.java
View File

@ -121,7 +121,7 @@ public class MWorkflow extends X_AD_Workflow
list.toArray(wfs); list.toArray(wfs);
s_cacheDocValue.put (oldKey, wfs); s_cacheDocValue.put (oldKey, wfs);
} }
s_log.config("#" + s_cacheDocValue.size()); if (s_log.isLoggable(Level.CONFIG)) s_log.config("#" + s_cacheDocValue.size());
} }
// Look for Entry // Look for Entry
MWorkflow[] retValue = (MWorkflow[])s_cacheDocValue.get(key); MWorkflow[] retValue = (MWorkflow[])s_cacheDocValue.get(key);