diff --git a/org.adempiere.base/src/org/adempiere/base/DefaultModelFactory.java b/org.adempiere.base/src/org/adempiere/base/DefaultModelFactory.java index 83abb2d5ff..e703749398 100644 --- a/org.adempiere.base/src/org/adempiere/base/DefaultModelFactory.java +++ b/org.adempiere.base/src/org/adempiere/base/DefaultModelFactory.java @@ -222,7 +222,7 @@ public class DefaultModelFactory implements IModelFactory { String classTableName = clazz.getField("Table_Name").get(null).toString(); if (!tableName.equals(classTableName)) { - s_log.finest("Invalid class for table: " + className+" (tableName="+tableName+", classTableName="+classTableName+")"); + if (s_log.isLoggable(Level.FINEST)) s_log.finest("Invalid class for table: " + className+" (tableName="+tableName+", classTableName="+classTableName+")"); return null; } } @@ -241,7 +241,7 @@ public class DefaultModelFactory implements IModelFactory { catch (Exception e) { } - s_log.finest("Not found: " + className); + if (s_log.isLoggable(Level.FINEST)) s_log.finest("Not found: " + className); return null; } // getPOclass diff --git a/org.adempiere.base/src/org/compiere/model/MCost.java b/org.adempiere.base/src/org/compiere/model/MCost.java index 7e26d01d67..2f369d4e91 100644 --- a/org.adempiere.base/src/org/compiere/model/MCost.java +++ b/org.adempiere.base/src/org/compiere/model/MCost.java @@ -172,7 +172,7 @@ public class MCost extends X_M_Cost String cm = rs.getString(3); percent = rs.getBigDecimal(4); //M_CostElement_ID = rs.getInt(5); - s_log.finest("CurrentCostPrice=" + currentCostPrice + if (s_log.isLoggable(Level.FINEST)) s_log.finest("CurrentCostPrice=" + currentCostPrice + ", CurrentCostPriceLL=" + currentCostPriceLL + ", CostElementType=" + costElementType + ", CostingMethod=" + cm @@ -205,7 +205,7 @@ public class MCost extends X_M_Cost } if (count > 1) // Print summary - s_log.finest("MaterialCost=" + materialCostEach + if (s_log.isLoggable(Level.FINEST)) s_log.finest("MaterialCost=" + materialCostEach + ", OtherCosts=" + otherCostEach + ", Percentage=" + percentage); diff --git a/org.adempiere.base/src/org/compiere/model/MSequence.java b/org.adempiere.base/src/org/compiere/model/MSequence.java index 84c5582a12..8c13c87dd0 100644 --- a/org.adempiere.base/src/org/compiere/model/MSequence.java +++ b/org.adempiere.base/src/org/compiere/model/MSequence.java @@ -154,12 +154,11 @@ public class MSequence extends X_AD_Sequence pstmt.setQueryTimeout(QUERY_TIME_OUT); } rs = pstmt.executeQuery(); - if (CLogMgt.isLevelFinest()) - s_log.finest("AC=" + conn.getAutoCommit() + ", RO=" + conn.isReadOnly() - + " - Isolation=" + conn.getTransactionIsolation() + "(" + Connection.TRANSACTION_READ_COMMITTED - + ") - RSType=" + pstmt.getResultSetType() + "(" + ResultSet.TYPE_SCROLL_SENSITIVE - + "), RSConcur=" + pstmt.getResultSetConcurrency() + "(" + ResultSet.CONCUR_UPDATABLE - + ")"); + if (s_log.isLoggable(Level.FINEST)) s_log.finest("AC=" + conn.getAutoCommit() + ", RO=" + conn.isReadOnly() + + " - Isolation=" + conn.getTransactionIsolation() + "(" + Connection.TRANSACTION_READ_COMMITTED + + ") - RSType=" + pstmt.getResultSetType() + "(" + ResultSet.TYPE_SCROLL_SENSITIVE + + "), RSConcur=" + pstmt.getResultSetConcurrency() + "(" + ResultSet.CONCUR_UPDATABLE + + ")"); if (rs.next()) { @@ -283,7 +282,7 @@ public class MSequence extends X_AD_Sequence } - //s_log.finest (retValue + " - Table=" + TableName + " [" + trx + "]"); + //if (s_log.isLoggable(Level.FINEST)) s_log.finest (retValue + " - Table=" + TableName + " [" + trx + "]"); return retValue; } // getNextID diff --git a/org.adempiere.base/src/org/compiere/print/MPrintFormat.java b/org.adempiere.base/src/org/compiere/print/MPrintFormat.java index 4fe50953c6..b13f432d3a 100644 --- a/org.adempiere.base/src/org/compiere/print/MPrintFormat.java +++ b/org.adempiere.base/src/org/compiere/print/MPrintFormat.java @@ -623,7 +623,7 @@ public class MPrintFormat extends X_AD_PrintFormat if (pfi != null) { printFormatItemList.add (pfi); - s_log.finest("Tab: " + pfi); + if (s_log.isLoggable(Level.FINEST)) s_log.finest("Tab: " + pfi); } } @@ -844,7 +844,7 @@ public class MPrintFormat extends X_AD_PrintFormat if (pfi != null) { list.add (pfi); - s_log.finest("Tab: " + pfi); + if (s_log.isLoggable(Level.FINEST)) s_log.finest("Tab: " + pfi); } } } @@ -878,7 +878,7 @@ public class MPrintFormat extends X_AD_PrintFormat if (pfi != null) { list.add (pfi); - s_log.finest("Table: " + pfi); + if (s_log.isLoggable(Level.FINEST)) s_log.finest("Table: " + pfi); } } } @@ -962,7 +962,7 @@ public class MPrintFormat extends X_AD_PrintFormat break; counter += no; } // for - s_log.finest("#" + counter); + if (s_log.isLoggable(Level.FINEST)) s_log.finest("#" + counter); } // copyTranslationItems diff --git a/org.adempiere.base/src/org/compiere/util/Evaluator.java b/org.adempiere.base/src/org/compiere/util/Evaluator.java index 629a399508..db4eb744bd 100644 --- a/org.adempiere.base/src/org/compiere/util/Evaluator.java +++ b/org.adempiere.base/src/org/compiere/util/Evaluator.java @@ -58,7 +58,7 @@ public class Evaluator } String variable = logic.substring(first+1, second-1); String eval = source.get_ValueAsString (variable); - s_log.finest(variable + "=" + eval); + if (s_log.isLoggable(Level.FINEST)) s_log.finest(variable + "=" + eval); if (eval == null || eval.length() == 0) return false; // @@ -195,9 +195,8 @@ public class Evaluator // Logical Comparison boolean result = evaluateLogicTuple (firstEval, operand, secondEval); // - if (CLogMgt.isLevelFinest()) - s_log.finest(logic - + " => \"" + firstEval + "\" " + operand + " \"" + secondEval + "\" => " + result); + if (s_log.isLoggable(Level.FINEST)) s_log.finest(logic + + " => \"" + firstEval + "\" " + operand + " \"" + secondEval + "\" => " + result); // return result; } // evaluateLogicTuple