IDEMPIERE-3798:Move iDempiere default branch to openjdk 10 (warning deprecated of constructer)

constructer with expression

org.adempiere.base.callout/src/org/compiere/model/CalloutCashJournal.java

@@ -84,7 +84,7 @@ public class CalloutCashJournal extends CalloutEngine
 			rs = pstmt.executeQuery();
 			if (rs.next())
 			{
-				mTab.setValue("C_Currency_ID", new Integer(rs.getInt(2)));
+				mTab.setValue("C_Currency_ID", Integer.valueOf(rs.getInt(2)));
 				BigDecimal PayAmt = rs.getBigDecimal(3);
 				BigDecimal DiscountAmt = rs.getBigDecimal(5);
 				boolean isSOTrx = "Y".equals(rs.getString(4));

org.adempiere.base.callout/src/org/compiere/model/CalloutGLJournal.java

@@ -103,7 +103,7 @@ public class CalloutGLJournal extends CalloutEngine
 				pstmt = null;
 			}
 			if (C_Period_ID != 0)
-				mTab.setValue("C_Period_ID", new Integer(C_Period_ID));
+				mTab.setValue("C_Period_ID", Integer.valueOf(C_Period_ID));
 		}
 
 		//  When C_Period_ID is changed, check if in DateAcct range and set to end date if not

org.adempiere.base.callout/src/org/compiere/model/CalloutInOut.java

@@ -61,28 +61,28 @@ public class CalloutInOut extends CalloutEngine
 		{
 			mTab.setValue("DateOrdered", order.getDateOrdered());
 			mTab.setValue("POReference", order.getPOReference());
-			mTab.setValue("AD_Org_ID", new Integer(order.getAD_Org_ID()));
+			mTab.setValue("AD_Org_ID", Integer.valueOf(order.getAD_Org_ID()));
-			mTab.setValue("AD_OrgTrx_ID", new Integer(order.getAD_OrgTrx_ID()));
+			mTab.setValue("AD_OrgTrx_ID", Integer.valueOf(order.getAD_OrgTrx_ID()));
-			mTab.setValue("C_Activity_ID", new Integer(order.getC_Activity_ID()));
+			mTab.setValue("C_Activity_ID", Integer.valueOf(order.getC_Activity_ID()));
-			mTab.setValue("C_Campaign_ID", new Integer(order.getC_Campaign_ID()));
+			mTab.setValue("C_Campaign_ID", Integer.valueOf(order.getC_Campaign_ID()));
-			mTab.setValue("C_Project_ID", new Integer(order.getC_Project_ID()));
+			mTab.setValue("C_Project_ID", Integer.valueOf(order.getC_Project_ID()));
-			mTab.setValue("User1_ID", new Integer(order.getUser1_ID()));
+			mTab.setValue("User1_ID", Integer.valueOf(order.getUser1_ID()));
-			mTab.setValue("User2_ID", new Integer(order.getUser2_ID()));
+			mTab.setValue("User2_ID", Integer.valueOf(order.getUser2_ID()));
-			mTab.setValue("M_Warehouse_ID", new Integer(order.getM_Warehouse_ID()));
+			mTab.setValue("M_Warehouse_ID", Integer.valueOf(order.getM_Warehouse_ID()));
 			//
 			mTab.setValue("DeliveryRule", order.getDeliveryRule());
 			mTab.setValue("DeliveryViaRule", order.getDeliveryViaRule());
-			mTab.setValue("M_Shipper_ID", new Integer(order.getM_Shipper_ID()));
+			mTab.setValue("M_Shipper_ID", Integer.valueOf(order.getM_Shipper_ID()));
 			mTab.setValue("FreightCostRule", order.getFreightCostRule());
 			mTab.setValue("FreightAmt", order.getFreightAmt());
 
-			mTab.setValue("C_BPartner_ID", new Integer(order.getC_BPartner_ID()));
+			mTab.setValue("C_BPartner_ID", Integer.valueOf(order.getC_BPartner_ID()));
 
 			//[ 1867464 ]
-			mTab.setValue("C_BPartner_Location_ID", new Integer(order.getC_BPartner_Location_ID()));
+			mTab.setValue("C_BPartner_Location_ID", Integer.valueOf(order.getC_BPartner_Location_ID()));
 			
 			if (order.getAD_User_ID() > 0)
-				mTab.setValue("AD_User_ID", new Integer(order.getAD_User_ID()));
+				mTab.setValue("AD_User_ID", Integer.valueOf(order.getAD_User_ID()));
 			else
 				mTab.setValue("AD_User_ID", null);
 		}
@@ -128,28 +128,28 @@ public class CalloutInOut extends CalloutEngine
 		{
 			mTab.setValue("DateOrdered", originalReceipt.getDateOrdered());
 			mTab.setValue("POReference", originalReceipt.getPOReference());
-			mTab.setValue("AD_Org_ID", new Integer(originalReceipt.getAD_Org_ID()));
+			mTab.setValue("AD_Org_ID", Integer.valueOf(originalReceipt.getAD_Org_ID()));
-			mTab.setValue("AD_OrgTrx_ID", new Integer(originalReceipt.getAD_OrgTrx_ID()));
+			mTab.setValue("AD_OrgTrx_ID", Integer.valueOf(originalReceipt.getAD_OrgTrx_ID()));
-			mTab.setValue("C_Activity_ID", new Integer(originalReceipt.getC_Activity_ID()));
+			mTab.setValue("C_Activity_ID", Integer.valueOf(originalReceipt.getC_Activity_ID()));
-			mTab.setValue("C_Campaign_ID", new Integer(originalReceipt.getC_Campaign_ID()));
+			mTab.setValue("C_Campaign_ID", Integer.valueOf(originalReceipt.getC_Campaign_ID()));
-			mTab.setValue("C_Project_ID", new Integer(originalReceipt.getC_Project_ID()));
+			mTab.setValue("C_Project_ID", Integer.valueOf(originalReceipt.getC_Project_ID()));
-			mTab.setValue("User1_ID", new Integer(originalReceipt.getUser1_ID()));
+			mTab.setValue("User1_ID", Integer.valueOf(originalReceipt.getUser1_ID()));
-			mTab.setValue("User2_ID", new Integer(originalReceipt.getUser2_ID()));
+			mTab.setValue("User2_ID", Integer.valueOf(originalReceipt.getUser2_ID()));
-			mTab.setValue("M_Warehouse_ID", new Integer(originalReceipt.getM_Warehouse_ID()));
+			mTab.setValue("M_Warehouse_ID", Integer.valueOf(originalReceipt.getM_Warehouse_ID()));
 			//
 			mTab.setValue("DeliveryRule", originalReceipt.getDeliveryRule());
 			mTab.setValue("DeliveryViaRule", originalReceipt.getDeliveryViaRule());
-			mTab.setValue("M_Shipper_ID", new Integer(originalReceipt.getM_Shipper_ID()));
+			mTab.setValue("M_Shipper_ID", Integer.valueOf(originalReceipt.getM_Shipper_ID()));
 			mTab.setValue("FreightCostRule", originalReceipt.getFreightCostRule());
 			mTab.setValue("FreightAmt", originalReceipt.getFreightAmt());
 
-			mTab.setValue("C_BPartner_ID", new Integer(originalReceipt.getC_BPartner_ID()));
+			mTab.setValue("C_BPartner_ID", Integer.valueOf(originalReceipt.getC_BPartner_ID()));
 
 			//[ 1867464 ]
-			mTab.setValue("C_BPartner_Location_ID", new Integer(originalReceipt.getC_BPartner_Location_ID()));
+			mTab.setValue("C_BPartner_Location_ID", Integer.valueOf(originalReceipt.getC_BPartner_Location_ID()));
 
 			if (originalReceipt.getAD_User_ID() > 0)
-				mTab.setValue("AD_User_ID", new Integer(originalReceipt.getAD_User_ID()));
+				mTab.setValue("AD_User_ID", Integer.valueOf(originalReceipt.getAD_User_ID()));
 			else
 				mTab.setValue("AD_User_ID", null);
 		}
@@ -290,13 +290,13 @@ public class CalloutInOut extends CalloutEngine
 				if (!IsSOTrx)
 				{
 					//	Location
-					Integer ii = new Integer(rs.getInt("C_BPartner_Location_ID"));
+					Integer ii = Integer.valueOf(rs.getInt("C_BPartner_Location_ID"));
 					if (rs.wasNull())
 						mTab.setValue("C_BPartner_Location_ID", null);
 					else
 						mTab.setValue("C_BPartner_Location_ID", ii);
 					//	Contact
-					ii = new Integer(rs.getInt("AD_User_ID"));
+					ii = Integer.valueOf(rs.getInt("AD_User_ID"));
 					if (rs.wasNull())
 						mTab.setValue("AD_User_ID", null);
 					else
@@ -361,12 +361,12 @@ public class CalloutInOut extends CalloutEngine
 			if (rs.next())
 			{
 				//	Org
-				Integer ii = new Integer(rs.getInt(1));
+				Integer ii = Integer.valueOf(rs.getInt(1));
 				int AD_Org_ID = Env.getContextAsInt(ctx, WindowNo, "AD_Org_ID");
 				if (AD_Org_ID != ii.intValue())
 					mTab.setValue("AD_Org_ID", ii);
 				//	Locator
-				ii = new Integer(rs.getInt(2));
+				ii = Integer.valueOf(rs.getInt(2));
 				if (rs.wasNull())
 					Env.setContext(ctx, WindowNo, 0, "M_Locator_ID", null);
 				else
@@ -416,17 +416,17 @@ public class CalloutInOut extends CalloutEngine
 		if (ol.get_ID() != 0)
 		{
 			if (ol.getC_Charge_ID() > 0 && ol.getM_Product_ID() <= 0) {
-				mTab.setValue("C_Charge_ID", new Integer(ol.getC_Charge_ID()));
+				mTab.setValue("C_Charge_ID", Integer.valueOf(ol.getC_Charge_ID()));
 				mTab.setValue("M_Product_ID", null);
 				mTab.setValue("M_AttributeSetInstance_ID", null);
 			}
 			else {
-				mTab.setValue("M_Product_ID", new Integer(ol.getM_Product_ID()));
+				mTab.setValue("M_Product_ID", Integer.valueOf(ol.getM_Product_ID()));
-				mTab.setValue("M_AttributeSetInstance_ID", new Integer(ol.getM_AttributeSetInstance_ID()));
+				mTab.setValue("M_AttributeSetInstance_ID", Integer.valueOf(ol.getM_AttributeSetInstance_ID()));
 				mTab.setValue("C_Charge_ID", null);
 			}
 			//
-			mTab.setValue("C_UOM_ID", new Integer(ol.getC_UOM_ID()));
+			mTab.setValue("C_UOM_ID", Integer.valueOf(ol.getC_UOM_ID()));
 			BigDecimal MovementQty = ol.getQtyOrdered().subtract(ol.getQtyDelivered());
 			BigDecimal runningqty = DB.getSQLValueBDEx(null, "SELECT SUM(MovementQty) FROM M_InOutLine WHERE M_InOut_ID=? AND M_InOutLine_ID!=? AND C_OrderLine_ID=?",
 					Env.getContextAsInt(ctx, WindowNo, "M_InOut_ID"),
@@ -442,14 +442,14 @@ public class CalloutInOut extends CalloutEngine
 					.divide(ol.getQtyOrdered(), 12, BigDecimal.ROUND_HALF_UP);
 			mTab.setValue("QtyEntered", QtyEntered);
 			//
-			mTab.setValue("C_Activity_ID", new Integer(ol.getC_Activity_ID()));
+			mTab.setValue("C_Activity_ID", Integer.valueOf(ol.getC_Activity_ID()));
-			mTab.setValue("C_Campaign_ID", new Integer(ol.getC_Campaign_ID()));
+			mTab.setValue("C_Campaign_ID", Integer.valueOf(ol.getC_Campaign_ID()));
-			mTab.setValue("C_Project_ID", new Integer(ol.getC_Project_ID()));
+			mTab.setValue("C_Project_ID", Integer.valueOf(ol.getC_Project_ID()));
-			mTab.setValue("C_ProjectPhase_ID", new Integer(ol.getC_ProjectPhase_ID()));
+			mTab.setValue("C_ProjectPhase_ID", Integer.valueOf(ol.getC_ProjectPhase_ID()));
-			mTab.setValue("C_ProjectTask_ID", new Integer(ol.getC_ProjectTask_ID()));
+			mTab.setValue("C_ProjectTask_ID", Integer.valueOf(ol.getC_ProjectTask_ID()));
-			mTab.setValue("AD_OrgTrx_ID", new Integer(ol.getAD_OrgTrx_ID()));
+			mTab.setValue("AD_OrgTrx_ID", Integer.valueOf(ol.getAD_OrgTrx_ID()));
-			mTab.setValue("User1_ID", new Integer(ol.getUser1_ID()));
+			mTab.setValue("User1_ID", Integer.valueOf(ol.getUser1_ID()));
-			mTab.setValue("User2_ID", new Integer(ol.getUser2_ID()));
+			mTab.setValue("User2_ID", Integer.valueOf(ol.getUser2_ID()));
 		}
 		return "";
 	}	//	orderLine
@@ -474,30 +474,30 @@ public class CalloutInOut extends CalloutEngine
 		if (rl.get_ID() != 0)
 		{
 			if (rl.getC_Charge_ID() > 0 && rl.getM_Product_ID() <= 0) {
-				mTab.setValue("C_Charge_ID", new Integer(rl.getC_Charge_ID()));
+				mTab.setValue("C_Charge_ID", Integer.valueOf(rl.getC_Charge_ID()));
 				mTab.setValue("M_Product_ID", null);
 				mTab.setValue("M_AttributeSetInstance_ID", null);
 			}
 			else {
-				mTab.setValue("M_Product_ID", new Integer(rl.getM_Product_ID()));
+				mTab.setValue("M_Product_ID", Integer.valueOf(rl.getM_Product_ID()));
-				mTab.setValue("M_AttributeSetInstance_ID", new Integer(rl.getM_AttributeSetInstance_ID()));
+				mTab.setValue("M_AttributeSetInstance_ID", Integer.valueOf(rl.getM_AttributeSetInstance_ID()));
 				mTab.setValue("C_Charge_ID", null);
 			}
 			//
-			mTab.setValue("C_UOM_ID", new Integer(rl.getC_UOM_ID()));
+			mTab.setValue("C_UOM_ID", Integer.valueOf(rl.getC_UOM_ID()));
 			BigDecimal MovementQty = rl.getQty().subtract(rl.getQtyDelivered());
 			mTab.setValue("MovementQty", MovementQty);
 			BigDecimal QtyEntered = MovementQty;
 			mTab.setValue("QtyEntered", QtyEntered);
 			//
-			mTab.setValue("C_Activity_ID", new Integer(rl.getC_Activity_ID()));
+			mTab.setValue("C_Activity_ID", Integer.valueOf(rl.getC_Activity_ID()));
-			mTab.setValue("C_Campaign_ID", new Integer(rl.getC_Campaign_ID()));
+			mTab.setValue("C_Campaign_ID", Integer.valueOf(rl.getC_Campaign_ID()));
-			mTab.setValue("C_Project_ID", new Integer(rl.getC_Project_ID()));
+			mTab.setValue("C_Project_ID", Integer.valueOf(rl.getC_Project_ID()));
-			mTab.setValue("C_ProjectPhase_ID", new Integer(rl.getC_ProjectPhase_ID()));
+			mTab.setValue("C_ProjectPhase_ID", Integer.valueOf(rl.getC_ProjectPhase_ID()));
-			mTab.setValue("C_ProjectTask_ID", new Integer(rl.getC_ProjectTask_ID()));
+			mTab.setValue("C_ProjectTask_ID", Integer.valueOf(rl.getC_ProjectTask_ID()));
-			mTab.setValue("AD_OrgTrx_ID", new Integer(rl.getAD_OrgTrx_ID()));
+			mTab.setValue("AD_OrgTrx_ID", Integer.valueOf(rl.getAD_OrgTrx_ID()));
-			mTab.setValue("User1_ID", new Integer(rl.getUser1_ID()));
+			mTab.setValue("User1_ID", Integer.valueOf(rl.getUser1_ID()));
-			mTab.setValue("User2_ID", new Integer(rl.getUser2_ID()));
+			mTab.setValue("User2_ID", Integer.valueOf(rl.getUser2_ID()));
 		}
 		return "";
 	}	//	rmaLine
@@ -525,10 +525,10 @@ public class CalloutInOut extends CalloutEngine
 			&& Env.getContextAsInt(ctx, WindowNo, Env.TAB_INFO, "M_AttributeSetInstance_ID") != 0)
 		{
 			mTab.setValue("M_AttributeSetInstance_ID",
-				new Integer(Env.getContextAsInt(ctx, WindowNo, Env.TAB_INFO, "M_AttributeSetInstance_ID")));
+				Integer.valueOf(Env.getContextAsInt(ctx, WindowNo, Env.TAB_INFO, "M_AttributeSetInstance_ID")));
 			M_Locator_ID = Env.getContextAsInt(ctx, WindowNo, Env.TAB_INFO, "M_Locator_ID");
 			if (M_Locator_ID != 0)
-				mTab.setValue("M_Locator_ID", new Integer(M_Locator_ID));
+				mTab.setValue("M_Locator_ID", Integer.valueOf(M_Locator_ID));
 		}
 		else
 			mTab.setValue("M_AttributeSetInstance_ID", null);

org.adempiere.base.callout/src/org/compiere/model/CalloutInvoice.java

@@ -161,7 +161,7 @@ public class CalloutInvoice extends CalloutEngine
 				}
 				
 				//	PriceList & IsTaxIncluded & Currency
-				Integer ii = new Integer(rs.getInt(IsSOTrx ? "M_PriceList_ID" : "PO_PriceList_ID"));
+				Integer ii = Integer.valueOf(rs.getInt(IsSOTrx ? "M_PriceList_ID" : "PO_PriceList_ID"));
 				if (!rs.wasNull())
 					mTab.setValue("M_PriceList_ID", ii);
 				else
@@ -171,13 +171,13 @@ public class CalloutInvoice extends CalloutEngine
 					{
 						MPriceList pl = new MPriceList(ctx, i, null);
 						if (IsSOTrx == pl.isSOPriceList())
-							mTab.setValue("M_PriceList_ID", new Integer(i));
+							mTab.setValue("M_PriceList_ID", Integer.valueOf(i));
 						else
 						{
 							String sql2 = "SELECT M_PriceList_ID FROM M_PriceList WHERE AD_Client_ID=? AND IsSOPriceList=? AND IsActive='Y' ORDER BY IsDefault DESC";
 							ii = DB.getSQLValue (null, sql2, Env.getAD_Client_ID(ctx), IsSOTrx);
 							if (ii != 0)
-								mTab.setValue("M_PriceList_ID", new Integer(ii));
+								mTab.setValue("M_PriceList_ID", Integer.valueOf(ii));
 						}
 					}
 				}
@@ -189,7 +189,7 @@ public class CalloutInvoice extends CalloutEngine
 				if (Env.getContext(ctx, WindowNo, "DocBaseType").endsWith("C"))	//	Credits are Payment Term
 					s = X_C_Invoice.PAYMENTRULE_OnCredit;
 				//  Payment Term
-				ii = new Integer(rs.getInt(IsSOTrx ? "C_PaymentTerm_ID" : "PO_PaymentTerm_ID"));
+				ii = Integer.valueOf(rs.getInt(IsSOTrx ? "C_PaymentTerm_ID" : "PO_PaymentTerm_ID"));
 				if (!rs.wasNull())
 					mTab.setValue("C_PaymentTerm_ID", ii);
 
@@ -206,7 +206,7 @@ public class CalloutInvoice extends CalloutEngine
 				if (locID == 0)
 					mTab.setValue("C_BPartner_Location_ID", null);
 				else
-					mTab.setValue("C_BPartner_Location_ID", new Integer(locID));
+					mTab.setValue("C_BPartner_Location_ID", Integer.valueOf(locID));
 
 				//	Contact - overwritten by InfoBP selection
 				int contID = rs.getInt("AD_User_ID");
@@ -219,7 +219,7 @@ public class CalloutInvoice extends CalloutEngine
 				if (contID == 0)
 					mTab.setValue("AD_User_ID", null);
 				else
-					mTab.setValue("AD_User_ID", new Integer(contID));
+					mTab.setValue("AD_User_ID", Integer.valueOf(contID));
 
 				//	CreditAvailable
 				if (IsSOTrx)
@@ -358,9 +358,9 @@ public class CalloutInvoice extends CalloutEngine
 		mTab.setValue("PriceLimit", pp.getPriceLimit());
 		mTab.setValue("PriceActual", pp.getPriceStd());
 		mTab.setValue("PriceEntered", pp.getPriceStd());
-		mTab.setValue("C_Currency_ID", new Integer(pp.getC_Currency_ID()));
+		mTab.setValue("C_Currency_ID", Integer.valueOf(pp.getC_Currency_ID()));
 	//	mTab.setValue("Discount", pp.getDiscount());
-		mTab.setValue("C_UOM_ID", new Integer(pp.getC_UOM_ID()));
+		mTab.setValue("C_UOM_ID", Integer.valueOf(pp.getC_UOM_ID()));
 		Env.setContext(ctx, WindowNo, "EnforcePriceLimit", pp.isEnforcePriceLimit() ? "Y" : "N");
 		Env.setContext(ctx, WindowNo, "DiscountSchema", pp.isDiscountSchema() ? "Y" : "N");
 		//
@@ -393,7 +393,7 @@ public class CalloutInvoice extends CalloutEngine
 		}
 		mTab.setValue("M_AttributeSetInstance_ID", null);
 		mTab.setValue("S_ResourceAssignment_ID", null);
-		mTab.setValue("C_UOM_ID", new Integer(100));	//	EA
+		mTab.setValue("C_UOM_ID", Integer.valueOf(100));	//	EA
 
 		Env.setContext(ctx, WindowNo, "DiscountSchema", "N");
 		String sql = "SELECT ChargeAmt FROM C_Charge WHERE C_Charge_ID=?";
@@ -490,7 +490,7 @@ public class CalloutInvoice extends CalloutEngine
 		if (C_Tax_ID == 0)
 			mTab.fireDataStatusEEvent(CLogger.retrieveError());
 		else
-			mTab.setValue("C_Tax_ID", new Integer(C_Tax_ID));
+			mTab.setValue("C_Tax_ID", Integer.valueOf(C_Tax_ID));
 		//
 		return amt (ctx, WindowNo, mTab, mField, value);
 	}	//	tax

org.adempiere.base.callout/src/org/compiere/model/CalloutInvoiceBatch.java

@@ -108,7 +108,7 @@ public class CalloutInvoiceBatch extends CalloutEngine
 				if (Env.getContext(ctx, WindowNo, "DocBaseType").endsWith("C"))	//	Credits are Payment Term
 					s = X_C_Invoice.PAYMENTRULE_OnCredit;
 				//  Payment Term
-				Integer ii = new Integer(rs.getInt(IsSOTrx ? "C_PaymentTerm_ID" : "PO_PaymentTerm_ID"));
+				Integer ii = Integer.valueOf(rs.getInt(IsSOTrx ? "C_PaymentTerm_ID" : "PO_PaymentTerm_ID"));
 				if (!rs.wasNull())
 					mTab.setValue("C_PaymentTerm_ID", ii);
 
@@ -125,7 +125,7 @@ public class CalloutInvoiceBatch extends CalloutEngine
 				if (locID == 0)
 					mTab.setValue("C_BPartner_Location_ID", null);
 				else
-					mTab.setValue("C_BPartner_Location_ID", new Integer(locID));
+					mTab.setValue("C_BPartner_Location_ID", Integer.valueOf(locID));
 
 				//	Contact - overwritten by InfoBP selection
 				int contID = rs.getInt("AD_User_ID");
@@ -138,7 +138,7 @@ public class CalloutInvoiceBatch extends CalloutEngine
 				if (contID == 0)
 					mTab.setValue("AD_User_ID", null);
 				else
-					mTab.setValue("AD_User_ID", new Integer(contID));
+					mTab.setValue("AD_User_ID", Integer.valueOf(contID));
 
 				//	CreditAvailable
 				if (IsSOTrx)
@@ -324,7 +324,7 @@ public class CalloutInvoiceBatch extends CalloutEngine
 		if (C_Tax_ID == 0)
 			mTab.fireDataStatusEEvent(CLogger.retrieveError());
 		else
-			mTab.setValue("C_Tax_ID", new Integer(C_Tax_ID));
+			mTab.setValue("C_Tax_ID", Integer.valueOf(C_Tax_ID));
 		//
 		return amt (ctx, WindowNo, mTab, mField, value);
 	}	//	tax

org.adempiere.base.callout/src/org/compiere/model/CalloutOrder.java

@@ -186,7 +186,7 @@ public class CalloutOrder extends CalloutEngine
 					if (s != null && s.length() != 0)
 						mTab.setValue("PaymentRule", s);
 					//	Payment Term
-					Integer ii =new Integer(rs.getInt(IsSOTrx ? "C_PaymentTerm_ID" : "PO_PaymentTerm_ID"));
+					Integer ii =Integer.valueOf(rs.getInt(IsSOTrx ? "C_PaymentTerm_ID" : "PO_PaymentTerm_ID"));
 					if (!rs.wasNull())
 						mTab.setValue("C_PaymentTerm_ID", ii);
 					//	InvoiceRule
@@ -278,7 +278,7 @@ public class CalloutOrder extends CalloutEngine
 				}
 
 				//	PriceList (indirect: IsTaxIncluded & Currency)
-				Integer ii = new Integer(rs.getInt(IsSOTrx ? "M_PriceList_ID" : "PO_PriceList_ID"));
+				Integer ii = Integer.valueOf(rs.getInt(IsSOTrx ? "M_PriceList_ID" : "PO_PriceList_ID"));
 				if (!rs.wasNull())
 					mTab.setValue("M_PriceList_ID", ii);
 				else
@@ -288,13 +288,13 @@ public class CalloutOrder extends CalloutEngine
 					{
 						MPriceList pl = new MPriceList(ctx, i, null);
 						if (IsSOTrx == pl.isSOPriceList())
-							mTab.setValue("M_PriceList_ID", new Integer(i));
+							mTab.setValue("M_PriceList_ID", Integer.valueOf(i));
 						else
 						{
 							String sql2 = "SELECT M_PriceList_ID FROM M_PriceList WHERE AD_Client_ID=? AND IsSOPriceList=? AND IsActive='Y' ORDER BY IsDefault DESC";
 							ii = DB.getSQLValue (null, sql2, Env.getAD_Client_ID(ctx), IsSOTrx);
 							if (ii != 0)
-								mTab.setValue("M_PriceList_ID", new Integer(ii));
+								mTab.setValue("M_PriceList_ID", Integer.valueOf(ii));
 						}
 					}
 				}
@@ -325,7 +325,7 @@ public class CalloutOrder extends CalloutEngine
 				if (bill_Location_ID == 0)
 					mTab.setValue("Bill_Location_ID", null);
 				else
-					mTab.setValue("Bill_Location_ID", new Integer(bill_Location_ID));
+					mTab.setValue("Bill_Location_ID", Integer.valueOf(bill_Location_ID));
 				// Ship-To Location
 				if (shipTo_ID == 0)
 					shipTo_ID = rs.getInt("C_BPartner_Location_ID");
@@ -333,7 +333,7 @@ public class CalloutOrder extends CalloutEngine
 				if (shipTo_ID == 0)
 					mTab.setValue("C_BPartner_Location_ID", null);
 				else
-					mTab.setValue("C_BPartner_Location_ID", new Integer(shipTo_ID));
+					mTab.setValue("C_BPartner_Location_ID", Integer.valueOf(shipTo_ID));
 
 				//	Contact - overwritten by InfoBP selection
 				int contID = rs.getInt("AD_User_ID");
@@ -347,8 +347,8 @@ public class CalloutOrder extends CalloutEngine
 					mTab.setValue("AD_User_ID", null);
 				else
 				{
-					mTab.setValue("AD_User_ID", new Integer(contID));
+					mTab.setValue("AD_User_ID", Integer.valueOf(contID));
-					mTab.setValue("Bill_User_ID", new Integer(contID));
+					mTab.setValue("Bill_User_ID", Integer.valueOf(contID));
 				}
 
 				//	CreditAvailable
@@ -399,7 +399,7 @@ public class CalloutOrder extends CalloutEngine
 					if (s != null && s.length() != 0)
 						mTab.setValue("PaymentRule", s);
 					//	Payment Term
-					ii = new Integer(rs.getInt(IsSOTrx ? "C_PaymentTerm_ID" : "PO_PaymentTerm_ID"));
+					ii = Integer.valueOf(rs.getInt(IsSOTrx ? "C_PaymentTerm_ID" : "PO_PaymentTerm_ID"));
 					if (!rs.wasNull())
 						mTab.setValue("C_PaymentTerm_ID", ii);
 					//	InvoiceRule
@@ -479,7 +479,7 @@ public class CalloutOrder extends CalloutEngine
 			if (rs.next())
 			{
 				//	PriceList (indirect: IsTaxIncluded & Currency)
-				Integer ii = new Integer(rs.getInt(IsSOTrx ? "M_PriceList_ID" : "PO_PriceList_ID"));
+				Integer ii = Integer.valueOf(rs.getInt(IsSOTrx ? "M_PriceList_ID" : "PO_PriceList_ID"));
 				if (!rs.wasNull())
 					mTab.setValue("M_PriceList_ID", ii);
 				else
@@ -489,13 +489,13 @@ public class CalloutOrder extends CalloutEngine
 					{
 						MPriceList pl = new MPriceList(ctx, i, null);
 						if (IsSOTrx == pl.isSOPriceList())
-							mTab.setValue("M_PriceList_ID", new Integer(i));
+							mTab.setValue("M_PriceList_ID", Integer.valueOf(i));
 						else
 						{
 							String sql2 = "SELECT M_PriceList_ID FROM M_PriceList WHERE AD_Client_ID=? AND IsSOPriceList=? AND IsActive='Y' ORDER BY IsDefault DESC";
 							ii = DB.getSQLValue (null, sql2, Env.getAD_Client_ID(ctx), IsSOTrx);
 							if (ii != 0)
-								mTab.setValue("M_PriceList_ID", new Integer(ii));
+								mTab.setValue("M_PriceList_ID", Integer.valueOf(ii));
 						}
 					}
 				}
@@ -518,7 +518,7 @@ public class CalloutOrder extends CalloutEngine
 				if (bill_Location_ID == 0)
 					mTab.setValue("Bill_Location_ID", null);
 				else
-					mTab.setValue("Bill_Location_ID", new Integer(bill_Location_ID));
+					mTab.setValue("Bill_Location_ID", Integer.valueOf(bill_Location_ID));
 
 				//	Contact - overwritten by InfoBP selection
 				int contID = rs.getInt("AD_User_ID");
@@ -531,7 +531,7 @@ public class CalloutOrder extends CalloutEngine
 				if (contID == 0)
 					mTab.setValue("Bill_User_ID", null);
 				else
-					mTab.setValue("Bill_User_ID", new Integer(contID));
+					mTab.setValue("Bill_User_ID", Integer.valueOf(contID));
 
 				//	CreditAvailable
 				if (IsSOTrx)
@@ -577,7 +577,7 @@ public class CalloutOrder extends CalloutEngine
 					if (s != null && s.length() != 0)
 						mTab.setValue("PaymentRule", s);
 					//	Payment Term
-					ii = new Integer(rs.getInt(IsSOTrx ? "C_PaymentTerm_ID" : "PO_PaymentTerm_ID"));
+					ii = Integer.valueOf(rs.getInt(IsSOTrx ? "C_PaymentTerm_ID" : "PO_PaymentTerm_ID"));
 					if (!rs.wasNull())
 						mTab.setValue("C_PaymentTerm_ID", ii);
 					//	InvoiceRule
@@ -783,9 +783,9 @@ public class CalloutOrder extends CalloutEngine
 		mTab.setValue("PriceLimit", pp.getPriceLimit());
 		mTab.setValue("PriceActual", pp.getPriceStd());
 		mTab.setValue("PriceEntered", pp.getPriceStd());
-		mTab.setValue("C_Currency_ID", new Integer(pp.getC_Currency_ID()));
+		mTab.setValue("C_Currency_ID", Integer.valueOf(pp.getC_Currency_ID()));
 		mTab.setValue("Discount", pp.getDiscount());
-		mTab.setValue("C_UOM_ID", new Integer(pp.getC_UOM_ID()));
+		mTab.setValue("C_UOM_ID", Integer.valueOf(pp.getC_UOM_ID()));
 		mTab.setValue("QtyOrdered", mTab.getValue("QtyEntered"));
 		Env.setContext(ctx, WindowNo, "EnforcePriceLimit", pp.isEnforcePriceLimit() ? "Y" : "N");
 		Env.setContext(ctx, WindowNo, "DiscountSchema", pp.isDiscountSchema() ? "Y" : "N");
@@ -795,7 +795,7 @@ public class CalloutOrder extends CalloutEngine
 		//	Integer wh = (Integer)mTab.getValue("M_Warehouse_ID");
 		//	if (wh.intValue() != M_Warehouse_ID)
 		//	{
-		//		mTab.setValue("M_Warehouse_ID", new Integer(M_Warehouse_ID));
+		//		mTab.setValue("M_Warehouse_ID", Integer.valueOf(M_Warehouse_ID));
 		//		ADialog.warn(,WindowNo, "WarehouseChanged");
 		//	}
 
@@ -822,7 +822,7 @@ public class CalloutOrder extends CalloutEngine
 				{
 					Integer C_OrderLine_ID = (Integer)mTab.getValue("C_OrderLine_ID");
 					if (C_OrderLine_ID == null)
-						C_OrderLine_ID = new Integer(0);
+						C_OrderLine_ID = Integer.valueOf(0);
 					BigDecimal notReserved = MOrderLine.getNotReserved(ctx,
 						M_Warehouse_ID, M_Product_ID, M_AttributeSetInstance_ID,
 						C_OrderLine_ID.intValue());
@@ -869,7 +869,7 @@ public class CalloutOrder extends CalloutEngine
 		}
 		mTab.setValue("M_AttributeSetInstance_ID", null);
 		mTab.setValue("S_ResourceAssignment_ID", null);
-		mTab.setValue("C_UOM_ID", new Integer(100));	//	EA
+		mTab.setValue("C_UOM_ID", Integer.valueOf(100));	//	EA
 
 		Env.setContext(ctx, WindowNo, "DiscountSchema", "N");
 		String sql = "SELECT ChargeAmt FROM C_Charge WHERE C_Charge_ID=?";
@@ -976,7 +976,7 @@ public class CalloutOrder extends CalloutEngine
 		if (C_Tax_ID == 0)
 			mTab.fireDataStatusEEvent(CLogger.retrieveError());
 		else
-			mTab.setValue("C_Tax_ID", new Integer(C_Tax_ID));
+			mTab.setValue("C_Tax_ID", Integer.valueOf(C_Tax_ID));
 		//
 		if (steps) log.warning("fini");
 		return amt(ctx, WindowNo, mTab, mField, value);
@@ -1315,7 +1315,7 @@ public class CalloutOrder extends CalloutEngine
 				{
 					Integer C_OrderLine_ID = (Integer)mTab.getValue("C_OrderLine_ID");
 					if (C_OrderLine_ID == null)
-						C_OrderLine_ID = new Integer(0);
+						C_OrderLine_ID = Integer.valueOf(0);
 					BigDecimal notReserved = MOrderLine.getNotReserved(ctx,
 						M_Warehouse_ID, M_Product_ID, M_AttributeSetInstance_ID,
 						C_OrderLine_ID.intValue());

org.adempiere.base.callout/src/org/compiere/model/CalloutPaymentAllocate.java

@@ -95,9 +95,9 @@ public class CalloutPaymentAllocate extends CalloutEngine
 			rs = pstmt.executeQuery();
 			if (rs.next())
 			{
-			//	mTab.setValue("C_BPartner_ID", new Integer(rs.getInt(1)));
+			//	mTab.setValue("C_BPartner_ID", Integer.valueOf(rs.getInt(1)));
 			//	int C_Currency_ID = rs.getInt(2);					//	Set Invoice Currency
-			//	mTab.setValue("C_Currency_ID", new Integer(C_Currency_ID));
+			//	mTab.setValue("C_Currency_ID", Integer.valueOf(C_Currency_ID));
 				//
 				BigDecimal InvoiceOpen = rs.getBigDecimal(3);		//	Set Invoice OPen Amount
 				if (InvoiceOpen == null)

org.adempiere.base.callout/src/org/compiere/model/CalloutProductCategory.java

@@ -43,7 +43,7 @@ public class CalloutProductCategory extends CalloutEngine
 
 		Integer productCategoryId = (Integer) mTab.getValue(MProductCategory.COLUMNNAME_M_Product_Category_ID);
 		if (productCategoryId == null)
-			productCategoryId = new Integer(0);
+			productCategoryId = Integer.valueOf(0);
 		
 		if (productCategoryId.intValue() > 0) {
 			MProductCategory pc = new MProductCategory(ctx, productCategoryId.intValue(), null);

org.adempiere.base.callout/src/org/compiere/model/CalloutProduction.java

@@ -51,7 +51,7 @@ public class CalloutProduction extends CalloutEngine
 		if (Env.getContextAsInt(ctx, WindowNo, Env.TAB_INFO, "M_Product_ID") == M_Product_ID.intValue()
 			&& Env.getContextAsInt(ctx, WindowNo, Env.TAB_INFO, "M_AttributeSetInstance_ID") != 0)
 		{
-			mTab.setValue("M_AttributeSetInstance_ID", new Integer(Env.getContextAsInt(ctx, WindowNo, Env.TAB_INFO, "M_AttributeSetInstance_ID")));
+			mTab.setValue("M_AttributeSetInstance_ID", Integer.valueOf(Env.getContextAsInt(ctx, WindowNo, Env.TAB_INFO, "M_AttributeSetInstance_ID")));
 		}
 		else
 		{

org.adempiere.base.callout/src/org/compiere/model/CalloutRequest.java

@@ -138,7 +138,7 @@ public class CalloutRequest extends CalloutEngine
 		MRequestType rt = MRequestType.get(ctx, R_RequestType_ID);
 		int R_Status_ID = rt.getDefaultR_Status_ID();
 		if (R_Status_ID != 0)
-			mTab.setValue("R_Status_ID", new Integer(R_Status_ID));
+			mTab.setValue("R_Status_ID", Integer.valueOf(R_Status_ID));
 		
 		return "";
 	}	//	type

org.adempiere.base.callout/src/org/compiere/model/CalloutTimeExpense.java

@@ -98,7 +98,7 @@ public class CalloutTimeExpense extends CalloutEngine
 					if (priceActual == null)
 						priceActual = rs.getBigDecimal("PriceLimit");
 					//	Currency
-					Integer ii = new Integer(rs.getInt("C_Currency_ID"));
+					Integer ii = Integer.valueOf(rs.getInt("C_Currency_ID"));
 					if (!rs.wasNull())
 						mTab.setValue("C_Currency_ID", ii);
 				}
@@ -143,7 +143,7 @@ public class CalloutTimeExpense extends CalloutEngine
 						if (priceActual == null)
 							priceActual = rs.getBigDecimal("PriceLimit");
 						//	Currency
-						Integer ii = new Integer(rs.getInt("C_Currency_ID"));
+						Integer ii = Integer.valueOf(rs.getInt("C_Currency_ID"));
 						if (!rs.wasNull())
 							mTab.setValue("C_Currency_ID", ii);
 					}

org.adempiere.base.process/src/org/adempiere/process/Export.java

@@ -173,7 +173,7 @@ public class Export extends SvrProcess
 			if (rs.next())
 			{
 				HashMap<String, Integer> variableMap = new HashMap<String, Integer>();
-				variableMap.put(TOTAL_SEGMENTS, new Integer(1));
+				variableMap.put(TOTAL_SEGMENTS, Integer.valueOf(1));
 				
 				Element rootElement = outDocument.createElement(exportFormat.getValue());
 				rootElement.appendChild(outDocument.createComment(exportFormat.getDescription()));
@@ -395,11 +395,11 @@ public class Export extends SvrProcess
 		if (variableName != null && !"".equals(variableName) ) {
 			Integer var = variableMap.get(variableName);
 			if (var == null) {
-				var = new Integer(0);
+				var = Integer.valueOf(0);
 			}
 			int intValue = var.intValue();
 			intValue++;
-			variableMap.put(variableName, new Integer(intValue));
+			variableMap.put(variableName, Integer.valueOf(intValue));
 		}
 	}
 	

org.adempiere.base.process/src/org/compiere/process/AcctSchemaCopyAcct.java

@@ -122,7 +122,7 @@ public class AcctSchemaCopyAcct extends SvrProcess
 			String columnName = pp.getName();
 			MAccount sourceAccount = MAccount.get(getCtx(), sourceC_ValidCombination_ID);
 			MAccount targetAccount = createAccount(targetAS, sourceAccount);
-			target.setValue(columnName, new Integer(targetAccount.getC_ValidCombination_ID()));
+			target.setValue(columnName, Integer.valueOf(targetAccount.getC_ValidCombination_ID()));
 		}
 		if (!target.save())
 			throw new AdempiereSystemError("Could not Save GL");
@@ -147,7 +147,7 @@ public class AcctSchemaCopyAcct extends SvrProcess
 			String columnName = pp.getName();
 			MAccount sourceAccount = MAccount.get(getCtx(), sourceC_ValidCombination_ID);
 			MAccount targetAccount = createAccount(targetAS, sourceAccount);
-			target.setValue(columnName, new Integer(targetAccount.getC_ValidCombination_ID()));
+			target.setValue(columnName, Integer.valueOf(targetAccount.getC_ValidCombination_ID()));
 		}
 		if (!target.save())
 			throw new AdempiereSystemError("Could not Save Default");

org.adempiere.base.process/src/org/compiere/process/DunningPrint.java

@@ -159,7 +159,7 @@ public class DunningPrint extends SvrProcess
 			//	query
 			MQuery query = new MQuery("C_Dunning_Header_v");
 			query.addRestriction("C_DunningRunEntry_ID", MQuery.EQUAL, 
-				new Integer(entry.getC_DunningRunEntry_ID()));
+				Integer.valueOf(entry.getC_DunningRunEntry_ID()));
 
 			//	Engine
 			PrintInfo info = new PrintInfo(

org.adempiere.base.process/src/org/compiere/process/InvoicePrint.java

@@ -275,7 +275,7 @@ public class InvoicePrint extends SvrProcess
 				format.setTranslationLanguage(language);
 				//	query
 				MQuery query = new MQuery("C_Invoice_Header_v");
-				query.addRestriction("C_Invoice_ID", MQuery.EQUAL, new Integer(C_Invoice_ID));
+				query.addRestriction("C_Invoice_ID", MQuery.EQUAL, Integer.valueOf(C_Invoice_ID));
 
 				//	Engine
 				PrintInfo info = new PrintInfo(

org.adempiere.base.process/src/org/compiere/process/ReplicationLocal.java

@@ -170,7 +170,7 @@ public class ReplicationLocal extends SvrProcess
 			+ "WHERE rt.IsActive='Y' AND t.IsActive='Y'"
 			+ " AND AD_ReplicationStrategy_ID=? "	//	#1
 			+ "ORDER BY t.LoadSeq";
-		RowSet rowset = getRowSet(sql, new Object[]{new Integer(m_replication.getAD_ReplicationStrategy_ID())});
+		RowSet rowset = getRowSet(sql, new Object[]{Integer.valueOf(m_replication.getAD_ReplicationStrategy_ID())});
 		if (rowset == null)
 			throw new Exception("setupRemote - No RowSet Data");
 
@@ -222,7 +222,7 @@ public class ReplicationLocal extends SvrProcess
 			+ " AND AD_ReplicationStrategy_ID=?" 	//	#1
 			+ " AND rt.ReplicationType='M' "		//	Merge
 			+ "ORDER BY t.LoadSeq";
-		RowSet rowset = getRowSet(sql, new Object[]{new Integer(m_replication.getAD_ReplicationStrategy_ID())});
+		RowSet rowset = getRowSet(sql, new Object[]{Integer.valueOf(m_replication.getAD_ReplicationStrategy_ID())});
 		try
 		{
 			while (rowset.next())
@@ -400,7 +400,7 @@ public class ReplicationLocal extends SvrProcess
 			+ " AND AD_ReplicationStrategy_ID=?" 	//	#1
 			+ " AND rt.ReplicationType='R' "		//	Reference
 			+ "ORDER BY t.LoadSeq";
-		RowSet rowset = getRowSet(sql, new Object[]{new Integer(m_replication.getAD_ReplicationStrategy_ID())});
+		RowSet rowset = getRowSet(sql, new Object[]{Integer.valueOf(m_replication.getAD_ReplicationStrategy_ID())});
 		try
 		{
 			while (rowset.next())

org.adempiere.base/src/com/akunagroup/uk/postcode/AddressLookup.java

@@ -252,7 +252,7 @@ public class AddressLookup implements AddressLookupInterface {
 						NodeList firstValueList = firstElement.getElementsByTagName("Value");
 						Element ValueElement = (Element) firstValueList.item(0);
 						if (log.isLoggable(Level.FINE)){log.log(Level.FINE, "Value: "  + ValueElement.getChildNodes().item(0).getNodeValue().replaceAll("\n","").trim() + " Node: " + i);}
-						resultCode = (int) new Integer(ValueElement.getChildNodes().item(0).getNodeValue().replaceAll("\n","").trim());
+						resultCode = (int) Integer.valueOf(ValueElement.getChildNodes().item(0).getNodeValue().replaceAll("\n","").trim());
 					}
 						
 			}

org.adempiere.base/src/org/adempiere/base/DefaultModelFactory.java

@@ -269,7 +269,7 @@ public class DefaultModelFactory implements IModelFactory {
 				s_log.warning("No transaction Constructor for " + clazz + " (" + msg + ")");
 			}
 
-			PO po = constructor!=null ? (PO)constructor.newInstance(new Object[] {Env.getCtx(), new Integer(Record_ID), trxName}) : null;
+			PO po = constructor!=null ? (PO)constructor.newInstance(new Object[] {Env.getCtx(), Integer.valueOf(Record_ID), trxName}) : null;
 			return po;
 		}
 		catch (Exception e)

org.adempiere.base/src/org/adempiere/model/GenericPO.java

@@ -114,7 +114,7 @@ public class GenericPO extends PO {
 		if (AD_OrgTrx_ID == 0)
 			set_Value("AD_OrgTrx_ID", null);
 		else
-			set_Value("AD_OrgTrx_ID", new Integer(AD_OrgTrx_ID));
+			set_Value("AD_OrgTrx_ID", Integer.valueOf(AD_OrgTrx_ID));
 	}
 
 	/**

org.adempiere.base/src/org/adempiere/model/MBroadcastMessage.java

@@ -50,7 +50,7 @@ public class MBroadcastMessage extends X_AD_BroadcastMessage
 
 	public static MBroadcastMessage get (Properties ctx, int AD_BroadcastMessage_ID)
 	{
-		Integer key = new Integer(AD_BroadcastMessage_ID);
+		Integer key = Integer.valueOf(AD_BroadcastMessage_ID);
 		MBroadcastMessage retValue = (MBroadcastMessage)s_cache.get(key);
 		if (retValue == null)
 		{

org.adempiere.base/src/org/adempiere/util/AbstractDocumentSearch.java

@@ -216,7 +216,7 @@ public abstract class AbstractDocumentSearch {
 					rsSO = pstmtSO.executeQuery();
 					Vector<Integer> idSO = new Vector<Integer>();
 					while (rsSO.next()) {
-						idSO.add(new Integer(rsSO.getInt(1)));
+						idSO.add(Integer.valueOf(rsSO.getInt(1)));
 					}
 					if (role.getWindowAccess(msd.getAD_Window_ID()) != null) {
 						msglog = new StringBuilder("Open Window: ").append(msd.getAD_Window_ID()).append(" / Table: ")
@@ -225,7 +225,7 @@ public abstract class AbstractDocumentSearch {
 
 						if (idSO.size() == 0 && (searchString == null || searchString.trim().length() == 0)) {
 							// No search string - open the window with new record
-							idSO.add(new Integer(0));
+							idSO.add(Integer.valueOf(0));
 						}
 
 						openWindow(idSO, table.getTableName(), msd.getAD_Window_ID());
@@ -239,7 +239,7 @@ public abstract class AbstractDocumentSearch {
 					rsPO = pstmtPO.executeQuery();
 					Vector<Integer> idPO = new Vector<Integer>();
 					while (rsPO.next()) {
-						idPO.add(new Integer(rsPO.getInt(1)));
+						idPO.add(Integer.valueOf(rsPO.getInt(1)));
 					}
 					if (role.getWindowAccess(msd.getPO_Window_ID()) != null) {
 						msglog = new StringBuilder("Open Window: ").append(msd.getPO_Window_ID()).append(" / Table: ")

org.adempiere.base/src/org/adempiere/util/GenerateModelJPA.java

@@ -406,7 +406,7 @@ public class GenerateModelJPA
 					sb.append("if (").append (columnName).append (" <= 0) ")
 						.append(setValue).append(" (\"").append(columnName).append("\", null); else \n");
 			}
-			sb.append(setValue).append(" (\"").append(columnName).append("\", new Integer(").append(columnName).append("));");
+			sb.append(setValue).append(" (\"").append(columnName).append("\", Integer.valueOf(").append(columnName).append("));");
 		}
 		else if (clazz.equals(Boolean.class))
 			sb.append(setValue).append(" (\"").append(columnName).append("\", Boolean.valueOf(").append(columnName).append("));");

org.adempiere.base/src/org/compiere/acct/Doc.java

@@ -1036,12 +1036,12 @@ public abstract class Doc
 		}
 		//  Get All Currencies
 		HashSet<Integer> set = new HashSet<Integer>();
-		set.add(new Integer(getC_Currency_ID()));
+		set.add(Integer.valueOf(getC_Currency_ID()));
 		for (int i = 0; p_lines != null && i < p_lines.length; i++)
 		{
 			int C_Currency_ID = p_lines[i].getC_Currency_ID();
 			if (C_Currency_ID != NO_CURRENCY)
-				set.add(new Integer(C_Currency_ID));
+				set.add(Integer.valueOf(C_Currency_ID));
 		}
 
 		//  just one and the same

org.adempiere.base/src/org/compiere/acct/Doc_Order.java

@@ -159,7 +159,7 @@ public class Doc_Order extends Doc
 		for (int i = 0; i < oLines.length; i++)
 		{
 			MOrderLine line = oLines[i];
-			qtys.put(new Integer(line.getC_OrderLine_ID()), line.getQtyOrdered());
+			qtys.put(Integer.valueOf(line.getC_OrderLine_ID()), line.getQtyOrdered());
 		}
 		//
 		ArrayList<DocLine> list = new ArrayList<DocLine>();
@@ -182,7 +182,7 @@ public class Doc_Order extends Doc
 				DocLine docLine = new DocLine (line, this);
 				//	Quantity - not more then OrderLine
 				//	Issue: Split of Requisition to multiple POs & different price
-				Integer key = new Integer(line.getC_OrderLine_ID());
+				Integer key = Integer.valueOf(line.getC_OrderLine_ID());
 				BigDecimal maxQty = qtys.get(key);
 				BigDecimal Qty = line.getQty().max(maxQty);
 				if (Qty.signum() == 0)

org.adempiere.base/src/org/compiere/acct/Fact.java

@@ -378,7 +378,7 @@ public final class Fact
 			for (int i = 0; i < m_lines.size(); i++)
 			{
 				FactLine line = (FactLine)m_lines.get(i);
-				Integer key = new Integer(line.getAD_Org_ID());
+				Integer key = Integer.valueOf(line.getAD_Org_ID());
 				BigDecimal bal = line.getSourceBalance();
 				BigDecimal oldBal = (BigDecimal)map.get(key);
 				if (oldBal != null)
@@ -445,7 +445,7 @@ public final class Fact
 			for (int i = 0; i < m_lines.size(); i++)
 			{
 				FactLine line = (FactLine)m_lines.get(i);
-				Integer key = new Integer(line.getAD_Org_ID());
+				Integer key = Integer.valueOf(line.getAD_Org_ID());
 			//	BigDecimal balance = line.getSourceBalance();
 				Balance oldBalance = (Balance)map.get(key);
 				if (oldBalance == null)

org.adempiere.base/src/org/compiere/cm/WebProjectDeploy.java

@@ -94,7 +94,7 @@ public class WebProjectDeploy extends SvrProcess
 		//	Stage
 		MCStage[] stages = MCStage.getStages(m_project);
 		for (int i = 0; i < stages.length; i++)
-			m_map.put(new Integer(stages[i].getCM_CStage_ID()), stages[i]);
+			m_map.put(Integer.valueOf(stages[i].getCM_CStage_ID()), stages[i]);
 		
 		//	Copy Stage Tree
 		MTree treeS = new MTree (getCtx(), m_project.getAD_TreeCMS_ID(), false, false, get_TrxName());
@@ -106,7 +106,7 @@ public class WebProjectDeploy extends SvrProcess
 		for (int i = 0; i < containers.length; i++)
 		{
 			MContainer container = containers[i];
-			if (!m_idList.contains(new Integer(container.getCM_Container_ID())))
+			if (!m_idList.contains(Integer.valueOf(container.getCM_Container_ID())))
 			{
 				String name = container.getName();
 				if (container.delete(true)) {
@@ -155,14 +155,14 @@ public class WebProjectDeploy extends SvrProcess
 	private void copyStage (MTreeNode node, String path)
 	{
 		org.compiere.cm.CacheHandler thisHandler = new org.compiere.cm.CacheHandler(org.compiere.cm.CacheHandler.convertJNPURLToCacheURL(getCtx().getProperty("java.naming.provider.url")), log, getCtx(), get_TrxName());
-		Integer ID = new Integer(node.getNode_ID());
+		Integer ID = Integer.valueOf(node.getNode_ID());
 		MCStage stage = m_map.get(ID);
 		//	
 		int size = node.getChildCount();
 		for (int i = 0; i < size; i++)
 		{
 			MTreeNode child = (MTreeNode)node.getChildAt(i);
-			ID = new Integer(child.getNode_ID());
+			ID = Integer.valueOf(child.getNode_ID());
 			stage = m_map.get(ID);
 			if (stage == null)
 			{

org.adempiere.base/src/org/compiere/model/CalloutEngine.java

@@ -103,9 +103,9 @@ public class CalloutEngine implements Callout
 		{
 			Object[] args = null;
 			if (argLength == 6)
-				args = new Object[] {ctx, new Integer(WindowNo), mTab, mField, value, oldValue};
+				args = new Object[] {ctx, Integer.valueOf(WindowNo), mTab, mField, value, oldValue};
 			else
-				args = new Object[] {ctx, new Integer(WindowNo), mTab, mField, value}; 
+				args = new Object[] {ctx, Integer.valueOf(WindowNo), mTab, mField, value}; 
 			retValue = (String)method.invoke(this, args);
 		}
 		catch (Exception e)

org.adempiere.base/src/org/compiere/model/GridField.java

@@ -791,14 +791,14 @@ public class GridField
 			&& (m_vo.ColumnName.equals("AD_Client_ID") || m_vo.ColumnName.equals("AD_Org_ID")))
 		{
 			if (log.isLoggable(Level.FINE)) log.fine("[SystemAccess] " + m_vo.ColumnName + "=0");
-			return new Integer(0);
+			return Integer.valueOf(0);
 		}
 		//	Set Org to System, if Client access
 		else if (X_AD_Table.ACCESSLEVEL_SystemPlusClient.equals(Env.getContext(m_vo.ctx, m_vo.WindowNo, m_vo.TabNo, GridTab.CTX_AccessLevel))
 			&& m_vo.ColumnName.equals("AD_Org_ID"))
 		{
 			if (log.isLoggable(Level.FINE)) log.fine("[ClientAccess] " + m_vo.ColumnName + "=0");
-			return new Integer(0);
+			return Integer.valueOf(0);
 		}
 		
 		return null;
@@ -1057,17 +1057,17 @@ public class GridField
 					int ii = Integer.parseInt(value);
 					if (ii < 0)
 						return null;
-					return new Integer(ii);
+					return Integer.valueOf(ii);
 				}
 				catch (Exception e)
 				{
 					log.warning("Cannot parse: " + value + " - " + e.getMessage());
 				}
-				return new Integer(0);
+				return Integer.valueOf(0);
 			}
 			//	Integer
 			if (m_vo.displayType == DisplayType.Integer)
-				return new Integer(value);
+				return Integer.valueOf(value);
 			
 			//	Number
 			if (DisplayType.isNumeric(m_vo.displayType))
@@ -2057,7 +2057,7 @@ public class GridField
 				|| (DisplayType.isID(dt) && getColumnName().endsWith("_ID")))
 			{
 				int i = Integer.parseInt(newValue);
-				setValue (new Integer(i), inserting);
+				setValue (Integer.valueOf(i), inserting);
 			}
 			//	Return BigDecimal
 			else if (DisplayType.isNumeric(dt))

org.adempiere.base/src/org/compiere/model/GridFieldVO.java

@@ -870,7 +870,7 @@ public class GridFieldVO implements Serializable
 	public static class SeqNoComparator implements Comparator<GridFieldVO> {
 		@Override
 		public int compare(GridFieldVO gf1, GridFieldVO gf2) {
-			return (new Integer(gf1.SeqNo)).compareTo(new Integer(gf2.SeqNo));
+			return (Integer.valueOf(gf1.SeqNo)).compareTo(Integer.valueOf(gf2.SeqNo));
 		}
 	}
 

org.adempiere.base/src/org/compiere/model/GridTab.java

@@ -1818,7 +1818,7 @@ public class GridTab implements DataStatusListener, Evaluatee, Serializable
 				if (rs.next())
 				{
 					//	{0} - Number of lines
-					Integer lines = new Integer(rs.getInt(1));
+					Integer lines = Integer.valueOf(rs.getInt(1));
 					arguments[0] = lines;
 					//	{1} - Line net
 					Double net = Double.valueOf(rs.getDouble(2));
@@ -1905,7 +1905,7 @@ public class GridTab implements DataStatusListener, Evaluatee, Serializable
 				if (rs.next())
 				{
 					//	{0} - Number of lines
-					Integer lines = new Integer(rs.getInt(1));
+					Integer lines = Integer.valueOf(rs.getInt(1));
 					arguments[0] = lines;
 					//	{1} - Line toral
 					Double lineTotal = Double.valueOf(rs.getDouble(3));
@@ -1979,7 +1979,7 @@ public class GridTab implements DataStatusListener, Evaluatee, Serializable
 				if (rs.next())
 				{
 					//	{0} - Number of lines
-					Integer lines = new Integer(rs.getInt(1));
+					Integer lines = Integer.valueOf(rs.getInt(1));
 					arguments[0] = lines;
 					//	{1} - Line total
 					Double total = Double.valueOf(rs.getDouble(2));
@@ -2234,7 +2234,7 @@ public class GridTab implements DataStatusListener, Evaluatee, Serializable
 			rs = pstmt.executeQuery();
 			while (rs.next())
 			{
-				Integer key = new Integer(rs.getInt(1));
+				Integer key = Integer.valueOf(rs.getInt(1));
 				m_Lock.add(key);
 			}
 		}
@@ -2264,7 +2264,7 @@ public class GridTab implements DataStatusListener, Evaluatee, Serializable
 		if (m_Lock == null || m_Lock.isEmpty())
 			return false;
 		//
-		Integer key = new Integer(m_mTable.getKeyID (m_currentRow));
+		Integer key = Integer.valueOf(m_mTable.getKeyID (m_currentRow));
 		return m_Lock.contains(key);
 	}	//	isLocked
 
@@ -3190,9 +3190,9 @@ public class GridTab implements DataStatusListener, Evaluatee, Serializable
 			lineNoCurrentRow = (Integer) m_mTable.getValueAt(from, lineCol);
 			lineNoNextRow = (Integer) m_mTable.getValueAt(to, lineCol);
 		} else if (m_mTable.getValueAt(from, lineCol) instanceof BigDecimal) {
-			lineNoCurrentRow = new Integer(((BigDecimal) m_mTable.getValueAt(from, lineCol))
+			lineNoCurrentRow = Integer.valueOf(((BigDecimal) m_mTable.getValueAt(from, lineCol))
 					.intValue());
-			lineNoNextRow = new Integer(((BigDecimal) m_mTable.getValueAt(to, lineCol))
+			lineNoNextRow = Integer.valueOf(((BigDecimal) m_mTable.getValueAt(to, lineCol))
 					.intValue());
 		} else {
 			log.fine("unknown value format - return");

org.adempiere.base/src/org/compiere/model/GridTable.java

@@ -1303,8 +1303,8 @@ public class GridTable extends AbstractTableModel
 
 		//  Save old value
 		m_oldValue = new Object[3];
-		m_oldValue[0] = new Integer(row);
+		m_oldValue[0] = Integer.valueOf(row);
-		m_oldValue[1] = new Integer(col);
+		m_oldValue[1] = Integer.valueOf(col);
 		m_oldValue[2] = oldValue;
 
 		//	Set Data item
@@ -1894,7 +1894,7 @@ public class GridTable extends AbstractTableModel
 								if (dd instanceof Integer)
 									iii = (Integer)dd;
 								else
-									iii = new Integer(dd.toString());
+									iii = Integer.valueOf(dd.toString());
 								if (encrypted)
 									iii = (Integer)encrypt(iii, getAD_Client_ID());
 								if (manualUpdate)
@@ -3242,7 +3242,7 @@ public class GridTable extends AbstractTableModel
 				//	Integer, ID, Lookup
 				if (displayType == DisplayType.Integer || (DisplayType.isID(displayType) && !(columnName.equals("EntityType") || columnName.equals("AD_Language"))))
 				{
-					rowData[j] = new Integer(rs.getInt(j+1));	//	Integer
+					rowData[j] = Integer.valueOf(rs.getInt(j+1));	//	Integer
 					if (rs.wasNull())
 						rowData[j] = null;
 				}

org.adempiere.base/src/org/compiere/model/MAcctSchema.java

@@ -62,7 +62,7 @@ public class MAcctSchema extends X_C_AcctSchema
 	public static MAcctSchema get (Properties ctx, int C_AcctSchema_ID, String trxName)
 	{
 		//  Check Cache
-		Integer key = new Integer(C_AcctSchema_ID);
+		Integer key = Integer.valueOf(C_AcctSchema_ID);
 		MAcctSchema retValue = (MAcctSchema)s_cache.get(key);
 		if (retValue != null)
 			return retValue;
@@ -93,7 +93,7 @@ public class MAcctSchema extends X_C_AcctSchema
 	public static synchronized MAcctSchema[] getClientAcctSchema (Properties ctx, int AD_Client_ID, String trxName)
 	{
 		//  Check Cache
-		Integer key = new Integer(AD_Client_ID);
+		Integer key = Integer.valueOf(AD_Client_ID);
 		if (s_schema.containsKey(key))
 			return (MAcctSchema[])s_schema.get(key);
 

org.adempiere.base/src/org/compiere/model/MAging.java

@@ -116,9 +116,9 @@ public class MAging extends X_T_Aging
 
 		//	Optional
 	//	setC_Invoice_ID (C_Invoice_ID);		// may be zero
-		set_ValueNoCheck ("C_Invoice_ID", new Integer(C_Invoice_ID));
+		set_ValueNoCheck ("C_Invoice_ID", Integer.valueOf(C_Invoice_ID));
 	//	setC_InvoicePaySchedule_ID(C_InvoicePaySchedule_ID);	//	may be zero
-		set_Value ("C_InvoicePaySchedule_ID", new Integer(C_InvoicePaySchedule_ID));
+		set_Value ("C_InvoicePaySchedule_ID", Integer.valueOf(C_InvoicePaySchedule_ID));
 		setIsListInvoices(C_Invoice_ID != 0);
 		//
 		setDueDate(DueDate);		//	only sensible if List invoices
@@ -156,9 +156,9 @@ public class MAging extends X_T_Aging
 
 		//	Optional
 	//	setC_Invoice_ID (C_Invoice_ID);		// may be zero
-		set_ValueNoCheck ("C_Invoice_ID", new Integer(C_Invoice_ID));
+		set_ValueNoCheck ("C_Invoice_ID", Integer.valueOf(C_Invoice_ID));
 	//	setC_InvoicePaySchedule_ID(C_InvoicePaySchedule_ID);	//	may be zero
-		set_Value ("C_InvoicePaySchedule_ID", new Integer(C_InvoicePaySchedule_ID));
+		set_Value ("C_InvoicePaySchedule_ID", Integer.valueOf(C_InvoicePaySchedule_ID));
 		setIsListInvoices(C_Invoice_ID != 0);
 		//
 		setDueDate(DueDate);		//	only sensible if List invoices

org.adempiere.base/src/org/compiere/model/MBPartner.java

@@ -63,7 +63,7 @@ public class MBPartner extends X_C_BPartner
 		//	Reset
 		if (template != null)
 		{
-			template.set_ValueNoCheck ("C_BPartner_ID", new Integer(0));
+			template.set_ValueNoCheck ("C_BPartner_ID", Integer.valueOf(0));
 			template.set_ValueNoCheck ("C_BPartner_UU", (String)null);
 			template.setAD_OrgBP_ID(0);
 			template.setLogo_ID(0);

org.adempiere.base/src/org/compiere/model/MBPartnerLocation.java

@@ -105,7 +105,7 @@ public class MBPartnerLocation extends X_C_BPartner_Location {
 		this(bp.getCtx(), 0, bp.get_TrxName());
 		setClientOrg(bp);
 		// may (still) be 0
-		set_ValueNoCheck("C_BPartner_ID", new Integer(bp.getC_BPartner_ID()));
+		set_ValueNoCheck("C_BPartner_ID", Integer.valueOf(bp.getC_BPartner_ID()));
 	} // MBPartner_Location
 
 	/**

org.adempiere.base/src/org/compiere/model/MChangeLog.java

@@ -72,7 +72,7 @@ public class MChangeLog extends X_AD_ChangeLog
 			pstmt = DB.prepareStatement(sql, null);
 			rs = pstmt.executeQuery();
 			while (rs.next())
-				list.add(new Integer(rs.getInt(1)));
+				list.add(Integer.valueOf(rs.getInt(1)));
 		}
 		catch (Exception e)
 		{

org.adempiere.base/src/org/compiere/model/MConversionType.java

@@ -66,7 +66,7 @@ public class MConversionType extends X_C_ConversionType
 			+ "ORDER BY IsDefault DESC, AD_Client_ID DESC";
 		C_ConversionType_ID = DB.getSQLValue(null, sql, AD_Client_ID);
 		//	Return
-		s_cache.put(key, new Integer(C_ConversionType_ID));
+		s_cache.put(key, Integer.valueOf(C_ConversionType_ID));
 		return C_ConversionType_ID;
 	}	//	getDefault
 	

org.adempiere.base/src/org/compiere/model/MCurrency.java

@@ -125,7 +125,7 @@ public class MCurrency extends X_C_Currency
 	public static MCurrency get (Properties ctx, int C_Currency_ID)
 	{
 		//	Try Cache
-		Integer key = new Integer(C_Currency_ID);
+		Integer key = Integer.valueOf(C_Currency_ID);
 		MCurrency retValue = (MCurrency)s_currencies.get(key);
 		if (retValue != null)
 			return retValue;

org.adempiere.base/src/org/compiere/model/MDesktop.java

@@ -223,7 +223,7 @@ public class MDesktop
 			while (rs.next())
 			{
 				int AD_Workbench_ID = rs.getInt(1);
-				m_workbenches.add (new Integer(AD_Workbench_ID));
+				m_workbenches.add (Integer.valueOf(AD_Workbench_ID));
 			}
 		}
 		catch (SQLException e)

org.adempiere.base/src/org/compiere/model/MInOutLine.java

@@ -283,7 +283,7 @@ public class MInOutLine extends X_M_InOutLine
 		if (M_Locator_ID < 0)
 			throw new IllegalArgumentException ("M_Locator_ID is mandatory.");
 		//	set to 0 explicitly to reset
-		set_Value (COLUMNNAME_M_Locator_ID, new Integer(M_Locator_ID));
+		set_Value (COLUMNNAME_M_Locator_ID, Integer.valueOf(M_Locator_ID));
 	}	//	setM_Locator_ID
 
 	/**

org.adempiere.base/src/org/compiere/model/MInvoiceLine.java

@@ -192,7 +192,7 @@ public class MInvoiceLine extends X_C_InvoiceLine
 		m_C_BPartner_ID = invoice.getC_BPartner_ID();
 		m_C_BPartner_Location_ID = invoice.getC_BPartner_Location_ID();
 		m_IsSOTrx = invoice.isSOTrx();
-		m_precision = new Integer(invoice.getPrecision());
+		m_precision = Integer.valueOf(invoice.getPrecision());
 	}	//	setOrder
 
 	/**
@@ -343,7 +343,7 @@ public class MInvoiceLine extends X_C_InvoiceLine
 	public void setM_AttributeSetInstance_ID (int M_AttributeSetInstance_ID)
 	{
 		if (M_AttributeSetInstance_ID == 0)		//	 0 is valid ID
-			set_Value("M_AttributeSetInstance_ID", new Integer(0));
+			set_Value("M_AttributeSetInstance_ID", Integer.valueOf(0));
 		else
 			super.setM_AttributeSetInstance_ID (M_AttributeSetInstance_ID);
 	}	//	setM_AttributeSetInstance_ID
@@ -768,7 +768,7 @@ public class MInvoiceLine extends X_C_InvoiceLine
 			log.warning("getPrecision = " + i + " - set to 2");
 			i = 2;
 		}
-		m_precision = new Integer(i);
+		m_precision = Integer.valueOf(i);
 		return m_precision.intValue();
 	}	//	getPrecision
 

org.adempiere.base/src/org/compiere/model/MInvoiceTax.java

@@ -159,7 +159,7 @@ public class MInvoiceTax extends X_C_InvoiceTax
 	 */
 	protected void setPrecision (int precision)
 	{
-		m_precision = new Integer(precision);
+		m_precision = Integer.valueOf(precision);
 	}	//	setPrecision
 
 	/**

org.adempiere.base/src/org/compiere/model/MLocatorLookup.java

@@ -229,7 +229,7 @@ public final class MLocatorLookup extends Lookup implements Serializable
 		//
 		int key = loc.getM_Locator_ID();
 		if (saveInCache)
-			m_lookup.put(new Integer(key), loc);
+			m_lookup.put(Integer.valueOf(key), loc);
 		NamePair retValue = new KeyNamePair(key, loc.toString());
 		return retValue;
 	}	//	getDirect
@@ -375,7 +375,7 @@ public final class MLocatorLookup extends Lookup implements Serializable
 				{
 					int M_Locator_ID = rs.getInt(1);
 					MLocator loc = MLocator.get(m_ctx, M_Locator_ID);
-					m_lookup.put(new Integer(M_Locator_ID), loc);
+					m_lookup.put(Integer.valueOf(M_Locator_ID), loc);
 				}
 			}
 			catch (SQLException e)

org.adempiere.base/src/org/compiere/model/MLookup.java

@@ -101,7 +101,7 @@ public final class MLookup extends Lookup implements Serializable
 	/** Number of max rows to load	*/
 	private static final int	MAX_ROWS = 5000;
 	/**	Indicator for Null			*/
-	private static Integer 		MINUS_ONE = new Integer(-1);
+	private static Integer 		MINUS_ONE = Integer.valueOf(-1);
 
 	/** The Lookup Info Value Object        */
 	private MLookupInfo         m_info = null;
@@ -539,7 +539,7 @@ public final class MLookup extends Lookup implements Serializable
 					int keyValue = rs.getInt(1);
 					KeyNamePair p = new KeyNamePair(keyValue, name.toString());
 					if (saveInCache)		//	save if
-						m_lookup.put(new Integer(keyValue), p);
+						m_lookup.put(Integer.valueOf(keyValue), p);
 					directValue = p;
 				}
 				else
@@ -878,7 +878,7 @@ public final class MLookup extends Lookup implements Serializable
 					{
 						int key = rs.getInt(1);
 						KeyNamePair p = new KeyNamePair(key, name.toString());
-						m_lookup.put(new Integer(key), p);
+						m_lookup.put(Integer.valueOf(key), p);
 					}
 					else
 					{

org.adempiere.base/src/org/compiere/model/MOrderLine.java

@@ -233,7 +233,7 @@ public class MOrderLine extends X_C_OrderLine
 	public void setHeaderInfo (MOrder order)
 	{
 		m_parent = order;
-		m_precision = new Integer(order.getPrecision());
+		m_precision = Integer.valueOf(order.getPrecision());
 		m_M_PriceList_ID = order.getM_PriceList_ID();
 		m_IsSOTrx = order.isSOTrx();
 	}	//	setHeaderInfo
@@ -410,7 +410,7 @@ public class MOrderLine extends X_C_OrderLine
 			+ "FROM C_Currency c INNER JOIN C_Order x ON (x.C_Currency_ID=c.C_Currency_ID) "
 			+ "WHERE x.C_Order_ID=?";
 		int i = DB.getSQLValue(get_TrxName(), sql, getC_Order_ID());
-		m_precision = new Integer(i);
+		m_precision = Integer.valueOf(i);
 		return m_precision.intValue();
 	}	//	getPrecision
 	
@@ -481,7 +481,7 @@ public class MOrderLine extends X_C_OrderLine
 	public void setM_AttributeSetInstance_ID (int M_AttributeSetInstance_ID)
 	{
 		if (M_AttributeSetInstance_ID == 0)		//	 0 is valid ID
-			set_Value("M_AttributeSetInstance_ID", new Integer(0));
+			set_Value("M_AttributeSetInstance_ID", Integer.valueOf(0));
 		else
 			super.setM_AttributeSetInstance_ID (M_AttributeSetInstance_ID);
 	}	//	setM_AttributeSetInstance_ID

org.adempiere.base/src/org/compiere/model/MOrderTax.java

@@ -178,7 +178,7 @@ public class MOrderTax extends X_C_OrderTax
 	 */
 	protected void setPrecision (int precision)
 	{
-		m_precision = new Integer(precision);
+		m_precision = Integer.valueOf(precision);
 	}	//	setPrecision
 
 	/**

org.adempiere.base/src/org/compiere/model/MPInstance.java

@@ -276,7 +276,7 @@ public class MPInstance extends X_AD_PInstance
 			if (log.isLoggable(Level.INFO)) log.info("Set to 0 from " + Record_ID);
 			Record_ID = 0;
 		}
-		set_ValueNoCheck ("Record_ID", new Integer(Record_ID));
+		set_ValueNoCheck ("Record_ID", Integer.valueOf(Record_ID));
 	}	//	setRecord_ID
 
 	/**

org.adempiere.base/src/org/compiere/model/MPOSTerminal.java

@@ -65,7 +65,7 @@ public class MPOSTerminal extends X_U_POSTerminal
      */
     public static MPOSTerminal get(Properties ctx, int U_POSTerminal_ID)
     {
-        Integer key = new Integer(U_POSTerminal_ID);
+        Integer key = Integer.valueOf(U_POSTerminal_ID);
         MPOSTerminal retValue = (MPOSTerminal)s_cache.get(key);
         if (retValue == null)
         {
@@ -133,7 +133,7 @@ public class MPOSTerminal extends X_U_POSTerminal
         
         if (success)
         {
-            s_cache.remove(new Integer(get_ID()));
+            s_cache.remove(Integer.valueOf(get_ID()));
         }
         
         return success;

org.adempiere.base/src/org/compiere/model/MPeriod.java

@@ -68,7 +68,7 @@ public class MPeriod extends X_C_Period
 		if (C_Period_ID <= 0)
 			return null;
 		//
-		Integer key = new Integer(C_Period_ID);
+		Integer key = Integer.valueOf(C_Period_ID);
 		MPeriod retValue = (MPeriod) s_cache.get (key);
 		if (retValue != null)
 			return retValue;
@@ -170,7 +170,7 @@ public class MPeriod extends X_C_Period
 			while (rs.next())
 			{
 				MPeriod period = new MPeriod(ctx, rs, trxName);
-				Integer key = new Integer(period.getC_Period_ID());
+				Integer key = Integer.valueOf(period.getC_Period_ID());
 				s_cache.put (key, period);
 				if (period.isStandardPeriod())
 					retValue = period;

org.adempiere.base/src/org/compiere/model/MPrivateAccess.java

@@ -94,7 +94,7 @@ public class MPrivateAccess extends X_AD_Private_Access
 			pstmt.setInt(2, AD_User_ID);
 			ResultSet rs = pstmt.executeQuery();
 			while (rs.next())
-				list.add(new Integer(rs.getInt(1))); 
+				list.add(Integer.valueOf(rs.getInt(1))); 
 			rs.close();
 			pstmt.close();
 			pstmt = null;

org.adempiere.base/src/org/compiere/model/MProductCategory.java

@@ -86,7 +86,7 @@ public class MProductCategory extends X_M_Product_Category
 			pstmt.setInt (1, M_Product_ID);
 			rs = pstmt.executeQuery ();
 			if (rs.next ())
-				category = new Integer(rs.getInt(1));
+				category = Integer.valueOf(rs.getInt(1));
 		}
 		catch (Exception e)
 		{

org.adempiere.base/src/org/compiere/model/MQuery.java

@@ -311,7 +311,7 @@ public class MQuery implements Serializable
 		MQuery query = new MQuery();
 		if (columnName.endsWith("_ID"))
 			query.setTableName(columnName.substring(0, columnName.length()-3));
-		query.addRestriction(columnName, EQUAL, new Integer(value));
+		query.addRestriction(columnName, EQUAL, Integer.valueOf(value));
 		query.setRecordCount(1);	//	guess
 		return query;
 	}	//	getEqualQuery
@@ -527,7 +527,7 @@ public class MQuery implements Serializable
 		int Code)
 	{
 		Restriction r = new Restriction (ColumnName, Operator,
-			new Integer(Code), null, null, true, 0);
+			Integer.valueOf(Code), null, null, true, 0);
 		m_list.add(r);
 	}	//	addRestriction
 
@@ -1024,7 +1024,7 @@ class Restriction  implements Serializable
 		if (code instanceof Boolean)
 			Code = ((Boolean)code).booleanValue() ? "Y" : "N";
 		else if (code instanceof KeyNamePair)
-			Code = new Integer(((KeyNamePair)code).getKey());
+			Code = Integer.valueOf(((KeyNamePair)code).getKey());
 		else if (code instanceof ValueNamePair)
 			Code = ((ValueNamePair)code).getValue();
 		else

org.adempiere.base/src/org/compiere/model/MRMATax.java

@@ -171,7 +171,7 @@ public class MRMATax extends X_M_RMATax
 	 */
 	protected void setPrecision (int precision)
 	{
-		m_precision = new Integer(precision);
+		m_precision = Integer.valueOf(precision);
 	}	//	setPrecision
 
 	/**

org.adempiere.base/src/org/compiere/model/MRegistrationAttribute.java

@@ -64,7 +64,7 @@ public class MRegistrationAttribute extends X_A_RegistrationAttribute
 			while (rs.next())
 			{
 				MRegistrationAttribute value = new MRegistrationAttribute(ctx, rs, null);
-				Integer key = new Integer(value.getA_RegistrationAttribute_ID());
+				Integer key = Integer.valueOf(value.getA_RegistrationAttribute_ID());
 				s_cache.put(key, value);
 				list.add(value);
 			}
@@ -93,7 +93,7 @@ public class MRegistrationAttribute extends X_A_RegistrationAttribute
 	 */
 	public static MRegistrationAttribute get (Properties ctx, int A_RegistrationAttribute_ID, String trxName)
 	{
-		Integer key = new Integer(A_RegistrationAttribute_ID);
+		Integer key = Integer.valueOf(A_RegistrationAttribute_ID);
 		MRegistrationAttribute retValue = (MRegistrationAttribute)s_cache.get(key);
 		if (retValue == null)
 		{

org.adempiere.base/src/org/compiere/model/MRequest.java

@@ -119,8 +119,8 @@ public class MRequest extends X_R_Request
 		int R_RequestType_ID, String Summary, boolean isSelfService, String trxName)
 	{
 		this(ctx, 0, trxName);
-		set_Value ("SalesRep_ID", new Integer(SalesRep_ID));	//	could be 0
+		set_Value ("SalesRep_ID", Integer.valueOf(SalesRep_ID));	//	could be 0
-		set_Value ("R_RequestType_ID", new Integer(R_RequestType_ID));
+		set_Value ("R_RequestType_ID", Integer.valueOf(R_RequestType_ID));
 		setSummary (Summary);
 		setIsSelfService(isSelfService);
 		getRequestType();

org.adempiere.base/src/org/compiere/model/MRole.java

@@ -902,7 +902,7 @@ public final class MRole extends X_AD_Role
 			rs = pstmt.executeQuery();
 			while (rs.next())
 			{
-				Integer ii = new Integer(rs.getInt(1));
+				Integer ii = Integer.valueOf(rs.getInt(1));
 				m_tableAccessLevel.put(ii, rs.getString(2));
 				String tableName = rs.getString(3); 
 				m_tableName.put(tableName, ii);
@@ -1345,7 +1345,7 @@ public final class MRole extends X_AD_Role
 		//	AccessLevel
 		//		1 = Org - 2 = Client - 4 = System
 		//		3 = Org+Client - 6 = Client+System - 7 = All
-		String roleAccessLevel = (String)m_tableAccessLevel.get(new Integer(AD_Table_ID));
+		String roleAccessLevel = (String)m_tableAccessLevel.get(Integer.valueOf(AD_Table_ID));
 		if (roleAccessLevel == null)
 		{
 			if (log.isLoggable(Level.FINE)) log.fine("NO - No AccessLevel - AD_Table_ID=" + AD_Table_ID);
@@ -1542,7 +1542,7 @@ public final class MRole extends X_AD_Role
 				pstmt.setInt(1, getAD_Role_ID());
 				rs = pstmt.executeQuery();
 				while (rs.next()) {
-					Integer winId = new Integer(rs.getInt(1));
+					Integer winId = Integer.valueOf(rs.getInt(1));
 					if ("N".equals(rs.getString(3))) {
 						// inactive window on direct access
 						if (m_windowAccess.containsKey(winId)) {
@@ -1627,7 +1627,7 @@ public final class MRole extends X_AD_Role
 				pstmt.setInt(1, getAD_Role_ID());
 				rs = pstmt.executeQuery();
 				while (rs.next()) {
-					Integer procId = new Integer(rs.getInt(1));
+					Integer procId = Integer.valueOf(rs.getInt(1));
 					if ("N".equals(rs.getString(3))) {
 						// inactive process on direct access
 						if (m_processAccess.containsKey(procId)) {
@@ -1707,7 +1707,7 @@ public final class MRole extends X_AD_Role
 				pstmt.setInt(1, getAD_Role_ID());
 				rs = pstmt.executeQuery();
 				while (rs.next()) {
-					Integer taskId = new Integer(rs.getInt(1));
+					Integer taskId = Integer.valueOf(rs.getInt(1));
 					if ("N".equals(rs.getString(3))) {
 						// inactive task on direct access
 						if (m_taskAccess.containsKey(taskId)) {
@@ -1787,7 +1787,7 @@ public final class MRole extends X_AD_Role
 				pstmt.setInt(1, getAD_Role_ID());
 				rs = pstmt.executeQuery();
 				while (rs.next()) {
-					Integer formId = new Integer(rs.getInt(1));
+					Integer formId = Integer.valueOf(rs.getInt(1));
 					if ("N".equals(rs.getString(3))) {
 						// inactive form on direct access
 						if (m_formAccess.containsKey(formId)) {
@@ -1867,7 +1867,7 @@ public final class MRole extends X_AD_Role
 				pstmt.setInt(1, getAD_Role_ID());
 				rs = pstmt.executeQuery();
 				while (rs.next()) {
-					Integer formId = new Integer(rs.getInt(1));
+					Integer formId = Integer.valueOf(rs.getInt(1));
 					if ("N".equals(rs.getString(3))) {
 						// inactive workflow on direct access
 						if (m_workflowAccess.containsKey(formId)) {
@@ -3137,7 +3137,7 @@ public final class MRole extends X_AD_Role
 				pstmt.setInt(1, getAD_Role_ID());
 				rs = pstmt.executeQuery();
 				while (rs.next()) {
-					Integer infoId = new Integer(rs.getInt(1));
+					Integer infoId = Integer.valueOf(rs.getInt(1));
 					if ("N".equals(rs.getString(2))) {
 						// inactive info on direct access
 						if (m_infoAccess.containsKey(infoId)) {

org.adempiere.base/src/org/compiere/model/MSequence.java

@@ -1229,7 +1229,7 @@ public class MSequence extends X_AD_Sequence
 				try
 				{
 					int no = DB.getNextID(0, "Test", null);
-					s_list.add(new Integer(no));
+					s_list.add(Integer.valueOf(no));
 				//	System.out.println("#" + m_i + ": " + no);
 				}
 				catch (Exception e)

org.adempiere.base/src/org/compiere/model/MSession.java

@@ -59,7 +59,7 @@ public class MSession extends X_AD_Session
 		int AD_Session_ID = Env.getContextAsInt(ctx, "#AD_Session_ID");
 		MSession session = null;
 		if (AD_Session_ID > 0)
-			session = (MSession)s_sessions.get(new Integer(AD_Session_ID));
+			session = (MSession)s_sessions.get(Integer.valueOf(AD_Session_ID));
 		// Try to load
 		if (session == null && AD_Session_ID > 0)
 		{
@@ -76,7 +76,7 @@ public class MSession extends X_AD_Session
 			session.saveEx();
 			AD_Session_ID = session.getAD_Session_ID();
 			Env.setContext (ctx, "#AD_Session_ID", AD_Session_ID);
-			s_sessions.put (new Integer(AD_Session_ID), session);
+			s_sessions.put (Integer.valueOf(AD_Session_ID), session);
 		}	
 		return session;
 	}	//	get
@@ -94,14 +94,14 @@ public class MSession extends X_AD_Session
 		int AD_Session_ID = Env.getContextAsInt(ctx, "#AD_Session_ID");
 		MSession session = null;
 		if (AD_Session_ID > 0)
-			session = (MSession)s_sessions.get(new Integer(AD_Session_ID));
+			session = (MSession)s_sessions.get(Integer.valueOf(AD_Session_ID));
 		if (session == null)
 		{
 			session = new MSession (ctx, Remote_Addr, Remote_Host, WebSession, null);	//	remote session
 			session.saveEx();
 			AD_Session_ID = session.getAD_Session_ID();
 			Env.setContext(ctx, "#AD_Session_ID", AD_Session_ID);
-			s_sessions.put(new Integer(AD_Session_ID), session);
+			s_sessions.put(Integer.valueOf(AD_Session_ID), session);
 		}	
 		return session;
 	}	//	get
@@ -238,7 +238,7 @@ public class MSession extends X_AD_Session
 	{
 		setProcessed(true);
 		saveEx();
-		s_sessions.remove(new Integer(getAD_Session_ID()));
+		s_sessions.remove(Integer.valueOf(getAD_Session_ID()));
 		if (log.isLoggable(Level.INFO)) log.info(TimeUtil.formatElapsed(getCreated(), getUpdated()));
 	}	//	logout
 

org.adempiere.base/src/org/compiere/model/MStatus.java

@@ -69,7 +69,7 @@ public class MStatus extends X_R_Status
 	 */
 	public static MStatus getDefault (Properties ctx, int R_RequestType_ID)
 	{
-		Integer key = new Integer(R_RequestType_ID);
+		Integer key = Integer.valueOf(R_RequestType_ID);
 		MStatus retValue = (MStatus)s_cacheDefault.get(key);
 		if (retValue != null)
 			return retValue;

org.adempiere.base/src/org/compiere/model/MStyle.java

@@ -37,7 +37,7 @@ public class MStyle extends X_AD_Style {
 
 	public static MStyle get (Properties ctx, int AD_Style_ID)
 	{
-		Integer key = new Integer(AD_Style_ID);
+		Integer key = Integer.valueOf(AD_Style_ID);
 		MStyle retValue = (MStyle)s_cache.get(key);
 		if (retValue == null)
 		{

org.adempiere.base/src/org/compiere/model/MTree_NodeCMC.java

@@ -149,6 +149,6 @@ public class MTree_NodeCMC extends X_AD_TreeNodeCMC
 	 */
 	public void setParent_ID (int Parent_ID)
 	{
-		set_Value ("Parent_ID", new Integer(Parent_ID));
+		set_Value ("Parent_ID", Integer.valueOf(Parent_ID));
 	}
 }	//	MTree_NodeCMC

org.adempiere.base/src/org/compiere/model/MUser.java

@@ -136,7 +136,7 @@ public class MUser extends X_AD_User
 	 */
 	public static MUser get (Properties ctx, int AD_User_ID)
 	{
-		Integer key = new Integer(AD_User_ID);
+		Integer key = Integer.valueOf(AD_User_ID);
 		MUser retValue = (MUser)s_cache.get(key);
 		if (retValue == null)
 		{

org.adempiere.base/src/org/compiere/model/MUserRoles.java

@@ -120,7 +120,7 @@ public class MUserRoles extends X_AD_User_Roles
 	 */
 	public void setAD_User_ID (int AD_User_ID)
 	{
-		set_ValueNoCheck ("AD_User_ID", new Integer(AD_User_ID));
+		set_ValueNoCheck ("AD_User_ID", Integer.valueOf(AD_User_ID));
 	}	//	setAD_User_ID
 	
 	/** 
@@ -130,7 +130,7 @@ public class MUserRoles extends X_AD_User_Roles
 	 **/
 	public void setAD_Role_ID (int AD_Role_ID)
 	{
-		set_ValueNoCheck ("AD_Role_ID", new Integer(AD_Role_ID));
+		set_ValueNoCheck ("AD_Role_ID", Integer.valueOf(AD_Role_ID));
 	}	//	setAD_Role_ID
 
 	@Override

org.adempiere.base/src/org/compiere/model/MWarehouse.java

@@ -63,7 +63,7 @@ public class MWarehouse extends X_M_Warehouse
 	 */
 	public static MWarehouse get (Properties ctx, int M_Warehouse_ID, String trxName)
 	{
-		Integer key = new Integer(M_Warehouse_ID);
+		Integer key = Integer.valueOf(M_Warehouse_ID);
 		MWarehouse retValue = (MWarehouse)s_cache.get(key);
 		if (retValue != null)
 			return retValue;

org.adempiere.base/src/org/compiere/model/PO.java

@@ -256,7 +256,7 @@ public abstract class PO
 	private HashMap<String,Object>	m_attributes = null;
 
 	/** Zero Integer				*/
-	protected static final Integer I_ZERO = new Integer(0);
+	protected static final Integer I_ZERO = Integer.valueOf(0);
 	/** Accounting Columns			*/
 	private ArrayList <String>	s_acctColumns = null;
 
@@ -657,7 +657,7 @@ public abstract class PO
 		{
 			int result = ((Integer)nValue).intValue();
 			result -= ((Integer)oValue).intValue();
-			return new Integer(result);
+			return Integer.valueOf(result);
 		}
 		//
 		log.warning("Invalid type - New=" + nValue);
@@ -830,7 +830,7 @@ public abstract class PO
 					&& p_info.getColumnClass(index) == Integer.class)
 				try
 				{
-					m_newValues[index] = new Integer((String)value);
+					m_newValues[index] = Integer.valueOf((String)value);
 				}
 				catch (NumberFormatException e)
 				{
@@ -1283,7 +1283,7 @@ public abstract class PO
 		if (ID > 0)
 		{
 			setKeyInfo();
-			m_IDs = new Object[] {new Integer(ID)};
+			m_IDs = new Object[] {Integer.valueOf(ID)};
 			//m_KeyColumns = new String[] {p_info.getTableName() + "_ID"};
 			load(trxName);
 		}
@@ -1399,7 +1399,7 @@ public abstract class PO
 			try
 			{
 				if (clazz == Integer.class)
-					m_oldValues[index] = decrypt(index, new Integer(rs.getInt(columnName)));
+					m_oldValues[index] = decrypt(index, Integer.valueOf(rs.getInt(columnName)));
 				else if (clazz == BigDecimal.class)
 					m_oldValues[index] = decrypt(index, rs.getBigDecimal(columnName));
 				else if (clazz == Boolean.class)
@@ -1475,7 +1475,7 @@ public abstract class PO
 			try
 			{
 				if (clazz == Integer.class)
-					m_oldValues[index] = new Integer(value);
+					m_oldValues[index] = Integer.valueOf(value);
 				else if (clazz == BigDecimal.class)
 					m_oldValues[index] = new BigDecimal(value);
 				else if (clazz == Boolean.class)
@@ -1634,9 +1634,9 @@ public abstract class PO
 			else if (colName.equals("IsActive"))
 				m_newValues[i] = Boolean.TRUE;
 			else if (colName.equals("AD_Client_ID"))
-				m_newValues[i] = new Integer(Env.getAD_Client_ID(p_ctx));
+				m_newValues[i] = Integer.valueOf(Env.getAD_Client_ID(p_ctx));
 			else if (colName.equals("AD_Org_ID"))
-				m_newValues[i] = new Integer(Env.getAD_Org_ID(p_ctx));
+				m_newValues[i] = Integer.valueOf(Env.getAD_Org_ID(p_ctx));
 			else if (colName.equals("Processed"))
 				m_newValues[i] = Boolean.FALSE;
 			else if (colName.equals("Processing"))
@@ -1748,7 +1748,7 @@ public abstract class PO
 	 */
 	final protected void setAD_Client_ID (int AD_Client_ID)
 	{
-		set_ValueNoCheck ("AD_Client_ID", new Integer(AD_Client_ID));
+		set_ValueNoCheck ("AD_Client_ID", Integer.valueOf(AD_Client_ID));
 	}	//	setAD_Client_ID
 
 	/**
@@ -1769,7 +1769,7 @@ public abstract class PO
 	 */
 	final public void setAD_Org_ID (int AD_Org_ID)
 	{
-		set_ValueNoCheck ("AD_Org_ID", new Integer(AD_Org_ID));
+		set_ValueNoCheck ("AD_Org_ID", Integer.valueOf(AD_Org_ID));
 	}	//	setAD_Org_ID
 
 	/**
@@ -1875,7 +1875,7 @@ public abstract class PO
 	 */
 	final protected void setUpdatedBy (int AD_User_ID)
 	{
-		set_ValueNoCheck ("UpdatedBy", new Integer(AD_User_ID));
+		set_ValueNoCheck ("UpdatedBy", Integer.valueOf(AD_User_ID));
 	}	//	setAD_User_ID
 
 	/**	Cache						*/
@@ -2502,7 +2502,7 @@ public abstract class PO
 				if (!changes && !updatedBy)
 				{
 					int AD_User_ID = Env.getContextAsInt(p_ctx, "#AD_User_ID");
-					set_ValueNoCheck("UpdatedBy", new Integer(AD_User_ID));
+					set_ValueNoCheck("UpdatedBy", Integer.valueOf(AD_User_ID));
 					sql.append("UpdatedBy=").append(AD_User_ID);
 					changes = true;
 					updatedBy = true;
@@ -2681,7 +2681,7 @@ public abstract class PO
 			if (!updatedBy)	//	UpdatedBy not explicitly set
 			{
 				int AD_User_ID = Env.getContextAsInt(p_ctx, "#AD_User_ID");
-				set_ValueNoCheck("UpdatedBy", new Integer(AD_User_ID));
+				set_ValueNoCheck("UpdatedBy", Integer.valueOf(AD_User_ID));
 				if (withValues)
 				{
 					sql.append(",UpdatedBy=").append(AD_User_ID);
@@ -2755,7 +2755,7 @@ public abstract class PO
 				log.severe("No NextID (" + no + ")");
 				return saveFinish (true, false);
 			}
-			m_IDs[0] = new Integer(no);
+			m_IDs[0] = Integer.valueOf(no);
 			set_ValueNoCheck(m_KeyColumns[0], m_IDs[0]);
 		}
 		//uuid secondary key
@@ -4341,7 +4341,7 @@ public abstract class PO
 			pstmt = DB.prepareStatement(sql.toString(), trxName);
 			rs = pstmt.executeQuery();
 			while (rs.next())
-				list.add(new Integer(rs.getInt(1)));
+				list.add(Integer.valueOf(rs.getInt(1)));
 		}
 		catch (SQLException e)
 		{

org.adempiere.base/src/org/compiere/model/PO_Record.java

@@ -108,7 +108,7 @@ public class PO_Record
 			//	DELETE FROM table WHERE AD_Table_ID=#1 AND Record_ID=#2
 			if (s_cascades[i] != AD_Table_ID)
 			{
-				Object[] params = new Object[]{new Integer(AD_Table_ID), new Integer(Record_ID)};
+				Object[] params = new Object[]{Integer.valueOf(AD_Table_ID), Integer.valueOf(Record_ID)};
 				StringBuffer sql = new StringBuffer ("DELETE FROM ")
 					.append(s_cascadeNames[i])
 					.append(" WHERE AD_Table_ID=? AND Record_ID=?");
@@ -129,7 +129,7 @@ public class PO_Record
 			if (s_parents[j] == AD_Table_ID)
 			{
 				int AD_Table_IDchild = s_parentChilds[j];
-				Object[] params = new Object[]{new Integer(AD_Table_IDchild), new Integer(Record_ID)};
+				Object[] params = new Object[]{Integer.valueOf(AD_Table_IDchild), Integer.valueOf(Record_ID)};
 				for (int i = 0; i < s_cascades.length; i++)
 				{
 					StringBuffer sql = new StringBuffer ("DELETE FROM ")

org.adempiere.base/src/org/compiere/model/StandardTaxProvider.java

@@ -45,7 +45,7 @@ public class StandardTaxProvider implements ITaxProvider {
 		{
 			MOrderLine line = lines[i];
 			totalLines = totalLines.add(line.getLineNetAmt());
-			Integer taxID = new Integer(line.getC_Tax_ID());
+			Integer taxID = Integer.valueOf(line.getC_Tax_ID());
 			if (!taxList.contains(taxID))
 			{
 				MTax tax = new MTax(order.getCtx(), taxID, order.get_TrxName());
@@ -302,7 +302,7 @@ public class StandardTaxProvider implements ITaxProvider {
 		{
 			MRMALine line = lines[i];
 			totalLines = totalLines.add(line.getLineNetAmt());
-			Integer taxID = new Integer(line.getC_Tax_ID());
+			Integer taxID = Integer.valueOf(line.getC_Tax_ID());
 			if (!taxList.contains(taxID))
 			{
 				MTax tax = new MTax(rma.getCtx(), taxID, rma.get_TrxName());

org.adempiere.base/src/org/compiere/model/TranslationTable.java

@@ -72,7 +72,7 @@ public class TranslationTable
 			return s_activeLanguages.intValue();
 		int no = DB.getSQLValue(null, 
 			"SELECT COUNT(*) FROM AD_Language WHERE IsActive='Y' AND IsSystemLanguage='Y'");
-		s_activeLanguages = new Integer(no);
+		s_activeLanguages = Integer.valueOf(no);
 		return s_activeLanguages.intValue();
 	}	//	getActiveLanguages
 	

org.adempiere.base/src/org/compiere/print/MPrintColor.java

@@ -92,7 +92,7 @@ public class MPrintColor extends X_AD_PrintColor
 	{
 	//	if (AD_PrintColor_ID == 0)
 	//		return new MPrintColor (ctx, 0);
-		Integer key = new Integer(AD_PrintColor_ID);
+		Integer key = Integer.valueOf(AD_PrintColor_ID);
 		MPrintColor pc = (MPrintColor)s_colors.get(key);
 		if (pc == null)
 		{

org.adempiere.base/src/org/compiere/print/MPrintFont.java

@@ -205,7 +205,7 @@ public class MPrintFont extends X_AD_PrintFont
 	 */
 	static public MPrintFont get (int AD_PrintFont_ID)
 	{
-		Integer key = new Integer(AD_PrintFont_ID);
+		Integer key = Integer.valueOf(AD_PrintFont_ID);
 		MPrintFont pf = (MPrintFont)s_fonts.get(key);
 		if (pf == null)
 		{

org.adempiere.base/src/org/compiere/print/MPrintFormat.java

@@ -147,7 +147,7 @@ public class MPrintFormat extends X_AD_PrintFormat
 		{
 			//	Sort Order and Column must be > 0
 			if (m_items[i].getSortNo() != 0 && m_items[i].getAD_Column_ID() != 0)
-				map.put(new Integer(m_items[i].getSortNo()), new Integer(m_items[i].getAD_Column_ID()));
+				map.put(Integer.valueOf(m_items[i].getSortNo()), Integer.valueOf(m_items[i].getAD_Column_ID()));
 		}
 		//	Get SortNo and Sort them
 		Integer[] keys = new Integer[map.keySet().size()];
@@ -174,7 +174,7 @@ public class MPrintFormat extends X_AD_PrintFormat
 		for (int i = 0; i < m_items.length; i++)
 		{
 			if (m_items[i].getAD_Column_ID() != 0 && m_items[i].isPrinted())
-				list.add(new Integer(m_items[i].getAD_Column_ID()));
+				list.add(Integer.valueOf(m_items[i].getAD_Column_ID()));
 		}
 		//	Convert
 		int[] retValue = new int[list.size()];
@@ -1087,7 +1087,7 @@ public class MPrintFormat extends X_AD_PrintFormat
 	 */
 	static public MPrintFormat get (Properties ctx, int AD_PrintFormat_ID, boolean readFromDisk)
 	{
-		Integer key = new Integer(AD_PrintFormat_ID);
+		Integer key = Integer.valueOf(AD_PrintFormat_ID);
 		MPrintFormat pf = null;
 		if (!readFromDisk)
 			pf = (MPrintFormat)s_formats.get(key);
@@ -1154,7 +1154,7 @@ public class MPrintFormat extends X_AD_PrintFormat
 	 */
 	static public void deleteFromCache (int AD_PrintFormat_ID)
 	{
-		Integer key = new Integer(AD_PrintFormat_ID);
+		Integer key = Integer.valueOf(AD_PrintFormat_ID);
 		s_formats.put(key, null);
 	}	//	deleteFromCache
 
@@ -1197,9 +1197,9 @@ public class MPrintFormat extends X_AD_PrintFormat
 		// add sql parameter
 		List<Object> lsParameter = new ArrayList<Object>();
 
-		lsParameter.add(new Integer(AD_Table_ID));
+		lsParameter.add(Integer.valueOf(AD_Table_ID));
 		if (AD_Window_ID > 0)
-			lsParameter.add(new Integer(AD_Window_ID));		
+			lsParameter.add(Integer.valueOf(AD_Window_ID));		
 		
 		// init query
 		Query query = new Query(Env.getCtx(), MPrintFormat.Table_Name, sqlWhere, trxName);

org.adempiere.base/src/org/compiere/print/MPrintFormatItem.java

@@ -392,7 +392,7 @@ public class MPrintFormatItem extends X_AD_PrintFormatItem
 	public String getColumnName()
 	{
 		if (m_columnName == null)	//	Get Column Name from AD_Column not index
-			m_columnName = getColumnName (new Integer(getAD_Column_ID()));
+			m_columnName = getColumnName (Integer.valueOf(getAD_Column_ID()));
 		return m_columnName;
 	}	//	getColumnName
 

org.adempiere.base/src/org/compiere/print/MPrintPaper.java

@@ -62,7 +62,7 @@ public class MPrintPaper extends X_AD_PrintPaper
 	 */
 	static public MPrintPaper get (int AD_PrintPaper_ID)
 	{
-		Integer key = new Integer(AD_PrintPaper_ID);
+		Integer key = Integer.valueOf(AD_PrintPaper_ID);
 		MPrintPaper pp = (MPrintPaper)s_papers.get(key);
 		if (pp == null)
 		{

org.adempiere.base/src/org/compiere/print/PrintData.java

@@ -252,9 +252,9 @@ public class PrintData implements Serializable
 	{
 		m_matrix.addRow(nodes);
 		if (functionRow)
-			m_functionRows.add(new Integer(m_matrix.getRowIndex()));
+			m_functionRows.add(Integer.valueOf(m_matrix.getRowIndex()));
 		if (m_hasLevelNo && levelNo != 0)
-			addNode(new PrintDataElement(LEVEL_NO, new Integer(levelNo), DisplayType.Integer, null));
+			addNode(new PrintDataElement(LEVEL_NO, Integer.valueOf(levelNo), DisplayType.Integer, null));
 	}	//	addRow
 
 	/**
@@ -301,7 +301,7 @@ public class PrintData implements Serializable
 	 */
 	public boolean isFunctionRow (int row)
 	{
-		return m_functionRows.contains(new Integer(row));
+		return m_functionRows.contains(Integer.valueOf(row));
 	}	//	isFunctionRow
 
 	/**
@@ -310,7 +310,7 @@ public class PrintData implements Serializable
 	 */
 	public boolean isFunctionRow ()
 	{
-		return m_functionRows.contains(new Integer(m_matrix.getRowIndex()));
+		return m_functionRows.contains(Integer.valueOf(m_matrix.getRowIndex()));
 	}	//	isFunctionRow
 
 	/**

org.adempiere.base/src/org/compiere/print/ReportEngine.java

@@ -729,7 +729,7 @@ queued-job-count = 0  (class javax.print.attribute.standard.QueuedJobCount)
 						{
 							td td = new td();
 							tr.addElement(td);
-							Object obj = m_printData.getNode(new Integer(item.getAD_Column_ID()));
+							Object obj = m_printData.getNode(Integer.valueOf(item.getAD_Column_ID()));
 							if (obj == null){
 								td.addElement(" ");
 								if (colSuppressRepeats[printColIndex]){
@@ -945,7 +945,7 @@ queued-job-count = 0  (class javax.print.attribute.standard.QueuedJobCount)
 						else
 						{
 							printColIndex++;
-							Object obj = m_printData.getNode(new Integer(item.getAD_Column_ID()));
+							Object obj = m_printData.getNode(Integer.valueOf(item.getAD_Column_ID()));
 							String data = "";
 							if (obj == null){
 								if (colSuppressRepeats[printColIndex]){

org.adempiere.base/src/org/compiere/print/layout/ImageElement.java

@@ -89,7 +89,7 @@ public class ImageElement extends PrintElement
 	 */
 	public static ImageElement get (int AD_PrintFormatItem_ID)
 	{
-		Object key = new Integer(AD_PrintFormatItem_ID);
+		Object key = Integer.valueOf(AD_PrintFormatItem_ID);
 		ImageElement image = (ImageElement)s_cache.get(key);
 		if (image == null)
 		{

org.adempiere.base/src/org/compiere/print/layout/LayoutEngine.java

@@ -1233,7 +1233,7 @@ public class LayoutEngine implements Pageable, Printable, Doc
 		int AD_Column_ID = item.getAD_Column_ID();
 		if (log.isLoggable(Level.INFO)) log.info(format + " - Item=" + item.getName() + " (" + AD_Column_ID + ")");
 		//
-		Object obj = data.getNode(new Integer(AD_Column_ID));
+		Object obj = data.getNode(Integer.valueOf(AD_Column_ID));
 		//	Object obj = data.getNode(item.getColumnName());	//	slower
 		if (obj == null)
 		{
@@ -1267,7 +1267,7 @@ public class LayoutEngine implements Pageable, Printable, Doc
 			return null;
 		}
 		MQuery query = new MQuery (format.getAD_Table_ID());
-		query.addRestriction(item.getColumnName(), MQuery.EQUAL, new Integer(Record_ID));
+		query.addRestriction(item.getColumnName(), MQuery.EQUAL, Integer.valueOf(Record_ID));
 		format.setTranslationViewQuery(query);
 		if (log.isLoggable(Level.FINE))
 			log.fine(query.toString());
@@ -1363,7 +1363,7 @@ public class LayoutEngine implements Pageable, Printable, Doc
 		String FieldAlignmentType, boolean isForm)
 	{
 		//	Get Data
-		Object obj = m_data.getNode(new Integer(item.getAD_Column_ID()));
+		Object obj = m_data.getNode(Integer.valueOf(item.getAD_Column_ID()));
 		if (obj == null)
 			return null;
 		else if (obj instanceof PrintDataElement)
@@ -1479,7 +1479,7 @@ public class LayoutEngine implements Pageable, Printable, Doc
 	 */
 	private PrintElement createImageElement (MPrintFormatItem item, PrintData printData)
 	{
-		Object obj = printData.getNode(new Integer(item.getAD_Column_ID())); 
+		Object obj = printData.getNode(Integer.valueOf(item.getAD_Column_ID())); 
 		if (obj == null)
 			return null;
 		else if (obj instanceof PrintDataElement)
@@ -1518,7 +1518,7 @@ public class LayoutEngine implements Pageable, Printable, Doc
 	private PrintElement createBarcodeElement (MPrintFormatItem item, PrintData printData)
 	{
 		//	Get Data
-		Object obj = printData.getNode(new Integer(item.getAD_Column_ID()));
+		Object obj = printData.getNode(Integer.valueOf(item.getAD_Column_ID()));
 		if (obj == null)
 			return null;
 		else if (obj instanceof PrintDataElement)
@@ -1620,7 +1620,7 @@ public class LayoutEngine implements Pageable, Printable, Doc
 			{
 				if (item.isNextLine() && item.getBelowColumn() != 0)
 				{
-					additionalLines.put(new Integer(col), new Integer(item.getBelowColumn()-1));
+					additionalLines.put(Integer.valueOf(col), Integer.valueOf(item.getBelowColumn()-1));
 					if (!item.isSuppressNull())
 					{
 						item.setIsSuppressNull(true);	//	display size will be set to 0 in TableElement
@@ -1676,13 +1676,13 @@ public class LayoutEngine implements Pageable, Printable, Doc
 			printData.setRowIndex(row);
 			if (printData.isFunctionRow())
 			{
-				functionRows.add(new Integer(row));
+				functionRows.add(Integer.valueOf(row));
 				rowColFont.put(new Point(row, TableElement.ALL), tf.getFunct_Font());
 				rowColColor.put(new Point(row, TableElement.ALL), tf.getFunctFG_Color());
 				rowColBackground.put(new Point(row, TableElement.ALL), tf.getFunctBG_Color());
 				if (printData.isPageBreak())
 				{
-					pageBreak.add(new Integer(row));
+					pageBreak.add(Integer.valueOf(row));
 					if (log.isLoggable(Level.FINER))
 						log.finer("PageBreak row=" + row);
 				}
@@ -1737,7 +1737,7 @@ public class LayoutEngine implements Pageable, Printable, Doc
 					{
 						Object obj = null;
 						if (item.getAD_Column_ID() > 0) // teo_sarca, [ 1673542 ]
-							obj = printData.getNode(new Integer(item.getAD_Column_ID()));
+							obj = printData.getNode(Integer.valueOf(item.getAD_Column_ID()));
 						if (obj == null)
 							;
 						else if (obj instanceof PrintDataElement)

org.adempiere.base/src/org/compiere/print/layout/StringElement.java

@@ -380,7 +380,7 @@ public class StringElement extends PrintElement
 			String tableName = MQuery.getZoomTableName(columnName);
 			Object code = m_ID.getID();
 			if (m_ID instanceof KeyNamePair)
-				code = new Integer(((KeyNamePair)m_ID).getKey());
+				code = Integer.valueOf(((KeyNamePair)m_ID).getKey());
 			//
 			MQuery query = new MQuery(tableName);
 			query.addRestriction(columnName, MQuery.EQUAL, code);

org.adempiere.base/src/org/compiere/print/layout/TableElement.java

@@ -301,9 +301,9 @@ public class TableElement extends PrintElement
 		{
 			int col = dataCol;
 			//	Print below existing column
-			if (m_additionalLines.containsKey(new Integer(dataCol)))
+			if (m_additionalLines.containsKey(Integer.valueOf(dataCol)))
 			{
-				col = ((Integer)m_additionalLines.get(new Integer(dataCol))).intValue();
+				col = ((Integer)m_additionalLines.get(Integer.valueOf(dataCol))).intValue();
 				if (log.isLoggable(Level.FINEST)) log.finest("DataColumn=" + dataCol + ", BelowColumn=" + col);
 			}
 			float colWidth = 0;
@@ -614,7 +614,7 @@ public class TableElement extends PrintElement
 		if (m_firstPage.height >= p_height && m_pageBreak.size() == 0)
 		{
 			if (log.isLoggable(Level.FINEST)) log.finest("Page Y=1 - PageHeight=" + m_firstPage.height + " - TableHeight=" + p_height);
-			m_firstRowOnPage.add(new Integer(0));	//	Y
+			m_firstRowOnPage.add(Integer.valueOf(0));	//	Y
 			m_pageHeight.add(Float.valueOf(p_height));	//	Y index only
 		}
 		//	multiple pages on Y | Axis
@@ -652,7 +652,7 @@ public class TableElement extends PrintElement
 				if (pageBreak)
 				{
 					availableHeight = firstPage ? m_firstPage.height : m_nextPages.height;
-					m_firstRowOnPage.add(new Integer(dataRow+addlRows));	//	Y
+					m_firstRowOnPage.add(Integer.valueOf(dataRow+addlRows));	//	Y
 					if (!firstPage)
 					{
 						m_pageHeight.add(Float.valueOf(usedHeight));	//	Y index only
@@ -688,7 +688,7 @@ public class TableElement extends PrintElement
 		{
 			if (log.isLoggable(Level.FINEST)) log.finest("Page X=1 - PageWidth=" + m_firstPage.width 
 					+ " - TableWidth=" + p_width);
-			m_firstColumnOnPage.add(new Integer(0));	//	X
+			m_firstColumnOnPage.add(Integer.valueOf(0));	//	X
 			//
 			distributeColumns (m_firstPage.width-(int)p_width, 0, m_columnWidths.size());
 		}
@@ -706,7 +706,7 @@ public class TableElement extends PrintElement
 					if (col != 0)
 						distributeColumns (availableWidth, lastStart, col);
 					//
-					m_firstColumnOnPage.add(new Integer(col));	//	X
+					m_firstColumnOnPage.add(Integer.valueOf(col));	//	X
 					if (log.isLoggable(Level.FINEST)) log.finest("Page X=" + m_firstColumnOnPage.size()
 							+ " - Col=" + col);
 					lastStart = col;
@@ -1073,7 +1073,7 @@ public class TableElement extends PrintElement
 		String tableName = MQuery.getZoomTableName(columnName);
 		Object code = pp.getID();
 		if (pp instanceof KeyNamePair)
-			code = new Integer(((KeyNamePair)pp).getKey());
+			code = Integer.valueOf(((KeyNamePair)pp).getKey());
 		//
 		MQuery query = new MQuery(tableName);
 		query.addRestriction(columnName, MQuery.EQUAL, code, null, pp.toString());
@@ -1612,8 +1612,8 @@ public class TableElement extends PrintElement
             else
             {
                 //  next line is a funcion column -> underline this
-                boolean nextIsFunction = m_functionRows.contains(new Integer(row+1));
+                boolean nextIsFunction = m_functionRows.contains(Integer.valueOf(row+1));
-                if (nextIsFunction && m_functionRows.contains(new Integer(row)))
+                if (nextIsFunction && m_functionRows.contains(Integer.valueOf(row)))
                     nextIsFunction = false;     //  this is a function line too
                 if (nextIsFunction)
                 {

org.adempiere.base/src/org/compiere/report/FinReportJasper.java

@@ -61,7 +61,7 @@ public class FinReportJasper extends FinReport
 		for (int i = 0; i < oldpara.length; i++)
 			list.add (oldpara[i]);
 		// and add the T_Report_AD_PInstance_ID parameter
-		list.add (new ProcessInfoParameter("T_Report_AD_PInstance_ID", new Integer(getAD_PInstance_ID()), null, null, null));
+		list.add (new ProcessInfoParameter("T_Report_AD_PInstance_ID", Integer.valueOf(getAD_PInstance_ID()), null, null, null));
 		ProcessInfoParameter[] pars = new ProcessInfoParameter[list.size()];
 		list.toArray(pars);
 

org.adempiere.base/src/org/compiere/report/MReportTree.java

@@ -303,7 +303,7 @@ public class MReportTree
 				MTreeNode nn = (MTreeNode)en.nextElement();
 				if (!nn.isSummary())
 				{
-					list.add(new Integer(nn.getNode_ID()));
+					list.add(Integer.valueOf(nn.getNode_ID()));
 					if (log.isLoggable(Level.FINEST)) log.finest("- " + nn);
 				}
 				else
@@ -311,7 +311,7 @@ public class MReportTree
 			}
 		}
 		else	//	not found or not summary 
-			list.add(new Integer(ID));
+			list.add(Integer.valueOf(ID));
 		//
 		Integer[] retValue = new Integer[list.size()];
 		list.toArray(retValue);

org.adempiere.base/src/org/compiere/report/core/RModel.java

@@ -226,7 +226,7 @@ public class RModel implements Serializable
 		if (log.isLoggable(Level.CONFIG)) log.config( "RModel.setGroup col=" + col);
 		if (col < 0 || col >= m_data.cols.size())
 			return;
-		Integer ii = new Integer(col);
+		Integer ii = Integer.valueOf(col);
 		if (!m_data.groups.contains(ii))
 			m_data.groups.add(ii);
 	}   //  setGroup
@@ -262,7 +262,7 @@ public class RModel implements Serializable
 		if (log.isLoggable(Level.CONFIG)) log.config( "RModel.setFunction col=" + col + " - " + function);
 		if (col < 0 || col >= m_data.cols.size())
 			return;
-		m_data.functions.put(new Integer(col), function);
+		m_data.functions.put(Integer.valueOf(col), function);
 	}   //  setFunction
 
 	/*************************************************************************/

org.adempiere.base/src/org/compiere/report/core/RModelData.java

@@ -150,7 +150,7 @@ public class RModelData
 					else if (rc.getColClass() == Double.class)
 						row.add(Double.valueOf(rs.getDouble(index++)));
 					else if (rc.getColClass() == Integer.class)
-						row.add(new Integer(rs.getInt(index++)));
+						row.add(Integer.valueOf(rs.getInt(index++)));
 					else if (rc.getColClass() == Timestamp.class)
 						row.add(rs.getTimestamp(index++));
 					else if (rc.getColClass() == Boolean.class)
@@ -290,7 +290,7 @@ public class RModelData
 						}
 					}   //  for all columns
 					//
-					m_groupRows.add(new Integer(rows.size())); //  group row indicator
+					m_groupRows.add(Integer.valueOf(rows.size())); //  group row indicator
 					rows.add(newRow);
 					groupBysValue[level] = row.get(idx);
 				}
@@ -347,7 +347,7 @@ public class RModelData
 			//  remove empty row added earlier to force group change
 			if (gSize > 0)
 				rows.remove(rows.size()-1);
-			m_groupRows.add(new Integer(rows.size())); //  group row indicator
+			m_groupRows.add(Integer.valueOf(rows.size())); //  group row indicator
 			rows.add(newRow);
 		}
 		if (log.isLoggable(Level.FINE)) log.fine("End Rows=" + rows.size());
@@ -367,7 +367,7 @@ public class RModelData
 		{
 			m_groupRowsIndicator = new ArrayList<Boolean>(rows.size());
 			for (int r = 0; r < rows.size(); r++)
-				m_groupRowsIndicator.add(Boolean.valueOf(m_groupRows.contains(new Integer(r))));
+				m_groupRowsIndicator.add(Boolean.valueOf(m_groupRows.contains(Integer.valueOf(r))));
 		}
 		if (row < 0 || row >= m_groupRowsIndicator.size())
 			return false;

org.adempiere.base/src/org/compiere/util/DB.java

@@ -961,7 +961,7 @@ public final class DB
 	 */
 	public static int executeUpdate (String sql, int param, String trxName, int timeOut)
 	{
-		return executeUpdate (sql, new Object[]{new Integer(param)}, false, trxName, timeOut);
+		return executeUpdate (sql, new Object[]{Integer.valueOf(param)}, false, trxName, timeOut);
 	}	//	executeUpdate
 
 	/**
@@ -990,7 +990,7 @@ public final class DB
 	 */
 	public static int executeUpdate (String sql, int param, boolean ignoreError, String trxName, int timeOut)
 	{
-		return executeUpdate (sql, new Object[]{new Integer(param)}, ignoreError, trxName, timeOut);
+		return executeUpdate (sql, new Object[]{Integer.valueOf(param)}, ignoreError, trxName, timeOut);
 	}	//	executeUpdate
 
 	/**

org.adempiere.base/src/org/eevolution/model/MDDOrderLine.java

@@ -285,7 +285,7 @@ public class MDDOrderLine extends X_DD_OrderLine
 	public void setM_AttributeSetInstance_ID (int M_AttributeSetInstance_ID)
 	{
 		if (M_AttributeSetInstance_ID == 0)		//	 0 is valid ID
-			set_Value("M_AttributeSetInstance_ID", new Integer(0));
+			set_Value("M_AttributeSetInstance_ID", Integer.valueOf(0));
 		else
 			super.setM_AttributeSetInstance_ID (M_AttributeSetInstance_ID);
 	}	//	setM_AttributeSetInstance_ID

org.adempiere.extend/src/compiere/model/CalloutUser.java

@@ -112,7 +112,7 @@ public class CalloutUser extends CalloutEngine
 				if (locID == 0)
 					mTab.setValue("C_BPartner_Location_ID", null);
 				else
-					mTab.setValue("C_BPartner_Location_ID", new Integer(locID));
+					mTab.setValue("C_BPartner_Location_ID", Integer.valueOf(locID));
 
 				//	Contact - overwritten by InfoBP selection
 				int contID = rs.getInt("AD_User_ID");
@@ -125,7 +125,7 @@ public class CalloutUser extends CalloutEngine
 				if (contID == 0)
 					mTab.setValue("AD_User_ID", null);
 				else
-					mTab.setValue("AD_User_ID", new Integer(contID));
+					mTab.setValue("AD_User_ID", Integer.valueOf(contID));
 			}
 		}
 		catch (SQLException e)

org.adempiere.extend/src/test/AdempiereTestCase.java

@@ -148,8 +148,8 @@ public class AdempiereTestCase extends TestCase
 
 		//m_Ctx = new Properties();
 		m_Ctx = Env.getCtx();
-		m_Ctx.setProperty("#AD_User_ID", new Integer(AD_User_ID_Value).toString());
+		m_Ctx.setProperty("#AD_User_ID", Integer.valueOf(AD_User_ID_Value).toString());
-		m_Ctx.setProperty("#AD_Client_ID", new Integer(AD_Client_ID_Value).toString());
+		m_Ctx.setProperty("#AD_Client_ID", Integer.valueOf(AD_Client_ID_Value).toString());
 
 		if (fileName_Value.length() < 1) {
 			assertEquals("Please specify path to idempiere.properties file!", true, false);

org.adempiere.extend/src/test/performance/SingleMProductTest.java

@@ -45,7 +45,7 @@ public class SingleMProductTest extends TestCase {
 		AD_User_ID_Value = Integer.parseInt(testProperties.getProperty(AD_User_ID_Key, AD_User_ID_DefaultValue) );
 		
 		m_Ctx = new Properties();
-		m_Ctx.setProperty("#AD_User_ID", new Integer(AD_User_ID_Value).toString());
+		m_Ctx.setProperty("#AD_User_ID", Integer.valueOf(AD_User_ID_Value).toString());
 		System.out.println("m_Ctx: " + m_Ctx);
 		
 		if (fileName_Value.length() < 1) {

org.adempiere.install/src/org/compiere/install/util/AppsAction.java

@@ -100,7 +100,7 @@ public final class AppsAction extends AbstractAction
 			if (ch != ' ')
 			{
 				toolTipText = toolTipText.substring(0, pos) + toolTipText.substring(pos+1);
-				putValue(Action.MNEMONIC_KEY, new Integer(ch.hashCode()));
+				putValue(Action.MNEMONIC_KEY, Integer.valueOf(ch.hashCode()));
 			}
 		}
 		//
@@ -125,7 +125,7 @@ public final class AppsAction extends AbstractAction
 		putValue(Action.SHORT_DESCRIPTION, toolTipText);	//	Tooltip
 		putValue(Action.ACTION_COMMAND_KEY, m_action);      //  ActionCammand
 		putValue(Action.ACCELERATOR_KEY, accelerator);      //  KeyStroke
-	//	putValue(Action.MNEMONIC_KEY, new Integer(0));      //  Mnemonic
+	//	putValue(Action.MNEMONIC_KEY, Integer.valueOf(0));      //  Mnemonic
 	//	putValue(Action.DEFAULT, text);						//	Not Used
 
 		//	Create Button

org.adempiere.pipo/src/org/adempiere/pipo2/PackOut.java

@@ -197,7 +197,7 @@ public class PackOut
 		SAXTransformerFactory packoutFactory = (SAXTransformerFactory) SAXTransformerFactory.newInstance();
 		//indent-number attribute support is not guarantee
 		try {
-			packoutFactory.setAttribute("indent-number", new Integer(4));
+			packoutFactory.setAttribute("indent-number", Integer.valueOf(4));
 		} catch (Exception e) {}
 		TransformerHandler packoutHandler = packoutFactory.newTransformerHandler();
 		Transformer packoutTransformer = packoutHandler.getTransformer();
@@ -247,7 +247,7 @@ public class PackOut
 		SAXTransformerFactory transformerFactory = (SAXTransformerFactory) SAXTransformerFactory.newInstance();
 		//indent-number attribute support is not guarantee
 		try {
-			transformerFactory.setAttribute("indent-number", new Integer(4));
+			transformerFactory.setAttribute("indent-number", Integer.valueOf(4));
 		} catch (Exception e) {}
 		TransformerHandler docHandler = transformerFactory.newTransformerHandler();
 		Transformer transformer = docHandler.getTransformer();

org.adempiere.pipo/src/org/adempiere/pipo2/PoFiller.java

@@ -131,7 +131,7 @@ public class PoFiller{
 
 		if (value!=null && value.trim().length() == 0)
 			value = null;
-		Integer i = value != null ? new Integer(value) : null;
+		Integer i = value != null ? Integer.valueOf(value) : null;
 
 		Object oldValue = po.get_Value(qName);
 		if (oldValue == null && i == null)
@@ -267,7 +267,7 @@ public class PoFiller{
 				//only import official id
 				if (qName.equalsIgnoreCase(po.get_TableName()+"_ID")) {
 					String value = e != null ? e.contents.toString() : null;
-					Integer i = !Util.isEmpty(value) ? new Integer(value) : null;
+					Integer i = !Util.isEmpty(value) ? Integer.valueOf(value) : null;
 					if (i != null && i.intValue() > 0 && i.intValue() <= MTable.MAX_OFFICIAL_ID) {
 						if (po.get_ID() != i.intValue()) {
 							//make sure id have not been used by other client

org.adempiere.replication/src/org/adempiere/process/rpl/exp/ExportHelper.java

@@ -473,11 +473,11 @@ public class ExportHelper {
 		if (variableName != null && !"".equals(variableName) ) {
 			Integer var = variableMap.get(variableName);
 			if (var == null) {
-				var = new Integer(0);
+				var = Integer.valueOf(0);
 			}
 			int intValue = var.intValue();
 			intValue++;
-			variableMap.put(variableName, new Integer(intValue));
+			variableMap.put(variableName, Integer.valueOf(intValue));
 		}
 	}
 

org.adempiere.replication/src/org/adempiere/process/rpl/imp/ImportHelper.java

@@ -128,9 +128,9 @@ public class ImportHelper {
 			throw new Exception(Msg.getMsg(ctx, "XMLVersionAttributeMandatory"));
 		}
 		///Getting Attributes.
-		int ReplicationMode = new Integer(rootElement.getAttribute("ReplicationMode"));
+		int ReplicationMode = Integer.valueOf(rootElement.getAttribute("ReplicationMode"));
 		String ReplicationType = rootElement.getAttribute("ReplicationType");
-		int ReplicationEvent = new Integer(rootElement.getAttribute("ReplicationEvent"));
+		int ReplicationEvent = Integer.valueOf(rootElement.getAttribute("ReplicationEvent"));
 		
 		MClient client = null;
 		client = getAD_ClientByValue(ctx, AD_Client_Value, trxName);
@@ -354,7 +354,7 @@ public class ImportHelper {
 			{
 				refRecord_ID = getID(ctx, referencedExpFormat, referencedNode, line.getValue(), po.get_TrxName());
 				if (log.isLoggable(Level.INFO)) log.info("refRecord_ID = " + refRecord_ID);
-				value = new Integer(refRecord_ID);
+				value = Integer.valueOf(refRecord_ID);
 			}
 			else
 			{
@@ -485,7 +485,7 @@ public class ImportHelper {
 						if (! Util.isEmpty(value.toString()))
 						{
 							int intValue = Integer.parseInt(value.toString());
-							value = new Integer( intValue );
+							value = Integer.valueOf( intValue );
 						}else
 						{
 							value=null;
@@ -654,7 +654,7 @@ public class ImportHelper {
 
 				if (log.isLoggable(Level.INFO)) log.info("record_ID = " + record_ID);
 				
-				cols[col] = new Integer(record_ID);
+				cols[col] = Integer.valueOf(record_ID);
 			} 
 			else 
 			{
@@ -680,7 +680,7 @@ public class ImportHelper {
 				if (!Util.isEmpty(value.toString()))
 				{
 					//double doubleValue = Double.parseDouble(value.toString());
-					value = new Integer(value.toString());
+					value = Integer.valueOf(value.toString());
 					if (DisplayType.ID	==	column.getAD_Reference_ID())
 					{
 						replication_id = (Integer) value;

org.adempiere.report.jasper/src/org/adempiere/report/jasper/ReportStarter.java

@@ -560,16 +560,16 @@ public class ReportStarter implements ProcessCall, ClientProcess
             }
 
             if (Record_ID > 0)
-            	params.put("RECORD_ID", new Integer( Record_ID));
+            	params.put("RECORD_ID", Integer.valueOf( Record_ID));
 
         	// contribution from Ricardo (ralexsander)
             // in iReports you can 'SELECT' AD_Client_ID, AD_Org_ID and AD_User_ID using only AD_PINSTANCE_ID
-            params.put("AD_PINSTANCE_ID", new Integer( AD_PInstance_ID));
+            params.put("AD_PINSTANCE_ID", Integer.valueOf( AD_PInstance_ID));
 
             // FR [3123850] - Add continuosly needed parameters to Jasper Starter - Carlos Ruiz - GlobalQSS
-        	params.put("AD_CLIENT_ID", new Integer( Env.getAD_Client_ID(Env.getCtx())));
+        	params.put("AD_CLIENT_ID", Integer.valueOf( Env.getAD_Client_ID(Env.getCtx())));
-        	params.put("AD_ROLE_ID", new Integer( Env.getAD_Role_ID(Env.getCtx())));
+        	params.put("AD_ROLE_ID", Integer.valueOf( Env.getAD_Role_ID(Env.getCtx())));
-        	params.put("AD_USER_ID", new Integer( Env.getAD_User_ID(Env.getCtx())));
+        	params.put("AD_USER_ID", Integer.valueOf( Env.getAD_User_ID(Env.getCtx())));
 
         	params.put("AD_CLIENT_NAME", Env.getContext(Env.getCtx(), "#AD_Client_Name"));
         	params.put("AD_ROLE_NAME", Env.getContext(Env.getCtx(), "#AD_Role_Name"));

org.adempiere.server/src/main/server/org/compiere/server/Scheduler.java

@@ -586,7 +586,7 @@ public class Scheduler extends AdempiereServer
 	 */
 	protected static MScheduler get(Properties ctx, int AD_Scheduler_ID)
 	{
-		Integer key = new Integer(AD_Scheduler_ID);
+		Integer key = Integer.valueOf(AD_Scheduler_ID);
 		MScheduler retValue = (MScheduler)s_cache.get(key);
 		if (retValue == null)
 		{

org.adempiere.server/src/main/server/org/compiere/server/WorkflowProcessor.java

@@ -381,7 +381,7 @@ public class WorkflowProcessor extends AdempiereServer
 		//	To Activity Owner
 		if (m_client.sendEMail(activity.getAD_User_ID(), subject, message, pdf))
 			counter++;
-		list.add (new Integer(activity.getAD_User_ID()));
+		list.add (Integer.valueOf(activity.getAD_User_ID()));
 
 		//	To Process Owner
 		if (toProcess
@@ -389,7 +389,7 @@ public class WorkflowProcessor extends AdempiereServer
 		{
 			if (m_client.sendEMail(process.getAD_User_ID(), subject, message, pdf))
 				counter++;
-			list.add (new Integer(process.getAD_User_ID()));
+			list.add (Integer.valueOf(process.getAD_User_ID()));
 		}
 		
 		//	To Activity Responsible
@@ -409,11 +409,11 @@ public class WorkflowProcessor extends AdempiereServer
 		//	Processor SuperVisor
 		if (toSupervisor 
 			&& m_model.getSupervisor_ID() != 0
-			&& !list.contains(new Integer(m_model.getSupervisor_ID())))
+			&& !list.contains(Integer.valueOf(m_model.getSupervisor_ID())))
 		{
 			if (m_client.sendEMail(m_model.getSupervisor_ID(), subject, message, pdf))
 				counter++;
-			list.add (new Integer(m_model.getSupervisor_ID()));
+			list.add (Integer.valueOf(m_model.getSupervisor_ID()));
 		}
 
 		return counter;
@@ -439,11 +439,11 @@ public class WorkflowProcessor extends AdempiereServer
 		//	Human
 		else if (MWFResponsible.RESPONSIBLETYPE_Human.equals(responsible.getResponsibleType())
 			&& responsible.getAD_User_ID() != 0
-			&& !list.contains(new Integer(responsible.getAD_User_ID())))
+			&& !list.contains(Integer.valueOf(responsible.getAD_User_ID())))
 		{
 			if (m_client.sendEMail(responsible.getAD_User_ID(), subject, message, pdf))
 				counter++;
-			list.add (new Integer(responsible.getAD_User_ID()));
+			list.add (Integer.valueOf(responsible.getAD_User_ID()));
 		}
 		//	Org of the Document
 		else if (MWFResponsible.RESPONSIBLETYPE_Organization.equals(responsible.getResponsibleType()))
@@ -453,11 +453,11 @@ public class WorkflowProcessor extends AdempiereServer
 			{
 				MOrgInfo org = MOrgInfo.get (getCtx(), document.getAD_Org_ID(), null);
 				if (org.getSupervisor_ID() != 0
-					&& !list.contains(new Integer(org.getSupervisor_ID())))
+					&& !list.contains(Integer.valueOf(org.getSupervisor_ID())))
 				{
 					if (m_client.sendEMail(org.getSupervisor_ID(), subject, message, pdf))
 						counter++;
-					list.add (new Integer(org.getSupervisor_ID()));
+					list.add (Integer.valueOf(org.getSupervisor_ID()));
 				}
 			}
 		}
@@ -472,11 +472,11 @@ public class WorkflowProcessor extends AdempiereServer
 				if (!roles.isActive())
 					continue;
 				int AD_User_ID = roles.getAD_User_ID();
-				if (!list.contains(new Integer(AD_User_ID)))
+				if (!list.contains(Integer.valueOf(AD_User_ID)))
 				{
 					if (m_client.sendEMail(AD_User_ID, subject, message, pdf))
 						counter++;
-					list.add (new Integer(AD_User_ID));
+					list.add (Integer.valueOf(AD_User_ID));
 				}
 			}
 		}

org.adempiere.ui.swing/src/org/adempiere/plaf/AdempiereTheme.java

@@ -405,8 +405,8 @@ public class AdempiereTheme extends com.jgoodies.looks.plastic.PlasticTheme {
                 "ScrollBar.thumbHighlight",
                 getPrimaryControlHighlight(),
                 PlasticScrollBarUI.MAX_BUMPS_WIDTH_KEY,
-                new Integer(22),
+                Integer.valueOf(22),
-                PlasticScrollBarUI.MAX_BUMPS_WIDTH_KEY, new Integer(30),
+                PlasticScrollBarUI.MAX_BUMPS_WIDTH_KEY, Integer.valueOf(30),
                 //"TabbedPane.selected", getWhite(),
                 "TabbedPane.selectHighlight",  new ColorUIResource(231, 218, 188),
                 ExtendedTheme.ERROR_BG_KEY,

org.adempiere.ui.swing/src/org/compiere/acct/AcctViewerData.java

@@ -209,7 +209,7 @@ class AcctViewerData
 				//
 				ValueNamePair pp = new ValueNamePair(tableName, name);
 				cb.addItem(pp);
-				tableInfo.put (tableName, new Integer(id));
+				tableInfo.put (tableName, Integer.valueOf(id));
 				if (id == AD_Table_ID)
 					select = pp;
 			}

org.adempiere.ui.swing/src/org/compiere/apps/APanel.java

@@ -856,7 +856,7 @@ public final class APanel extends CPanel
 						//	Is this tab included?
 						if (includedMap.size() > 0)
 						{
-							GridController parent = (GridController)includedMap.get(new Integer(gTab.getAD_Tab_ID()));
+							GridController parent = (GridController)includedMap.get(Integer.valueOf(gTab.getAD_Tab_ID()));
 							if (parent != null)
 							{
 								// FR [ 1757088 ]
@@ -2333,7 +2333,7 @@ public final class APanel extends CPanel
 		{
 			if (queryColumn.endsWith("_ID"))
 				query.addRestriction(queryColumn, MQuery.EQUAL,
-					new Integer(Env.getContextAsInt(m_ctx, m_curWindowNo, queryColumn)),
+					Integer.valueOf(Env.getContextAsInt(m_ctx, m_curWindowNo, queryColumn)),
 					infoName, infoDisplay);
 			else
 				query.addRestriction(queryColumn, MQuery.EQUAL,
@@ -2366,7 +2366,7 @@ public final class APanel extends CPanel
 		{
 			if (link.endsWith("_ID"))
 				query.addRestriction(link, MQuery.EQUAL,
-					new Integer(Env.getContextAsInt(m_ctx, m_curWindowNo, link)));
+					Integer.valueOf(Env.getContextAsInt(m_ctx, m_curWindowNo, link)));
 			else
 				query.addRestriction(link, MQuery.EQUAL,
 					Env.getContext(m_ctx, m_curWindowNo, link));

org.adempiere.ui.swing/src/org/compiere/apps/ARequest.java

@@ -220,47 +220,47 @@ public class ARequest implements ActionListener
 		{
 			GridTab tab = frame.getAPanel().getCurrentTab();
 			tab.dataNew (false);
-			tab.setValue("AD_Table_ID", new Integer(m_AD_Table_ID));
+			tab.setValue("AD_Table_ID", Integer.valueOf(m_AD_Table_ID));
-			tab.setValue("Record_ID", new Integer(m_Record_ID));
+			tab.setValue("Record_ID", Integer.valueOf(m_Record_ID));
 			//
 			if (m_C_BPartner_ID != 0)
-				tab.setValue("C_BPartner_ID", new Integer(m_C_BPartner_ID));
+				tab.setValue("C_BPartner_ID", Integer.valueOf(m_C_BPartner_ID));
 			//
 			if (m_AD_Table_ID == MBPartner.Table_ID)
-				tab.setValue("C_BPartner_ID", new Integer(m_Record_ID));
+				tab.setValue("C_BPartner_ID", Integer.valueOf(m_Record_ID));
 			else if (m_AD_Table_ID == MUser.Table_ID)
-				tab.setValue("AD_User_ID", new Integer(m_Record_ID));
+				tab.setValue("AD_User_ID", Integer.valueOf(m_Record_ID));
 			//
 			else if (m_AD_Table_ID == MProject.Table_ID)
-				tab.setValue("C_Project_ID", new Integer(m_Record_ID));
+				tab.setValue("C_Project_ID", Integer.valueOf(m_Record_ID));
 			else if (m_AD_Table_ID == MAsset.Table_ID)
-				tab.setValue("A_Asset_ID", new Integer(m_Record_ID));
+				tab.setValue("A_Asset_ID", Integer.valueOf(m_Record_ID));
 			//
 			else if (m_AD_Table_ID == MOrder.Table_ID)
-				tab.setValue("C_Order_ID", new Integer(m_Record_ID));
+				tab.setValue("C_Order_ID", Integer.valueOf(m_Record_ID));
 			else if (m_AD_Table_ID == MInvoice.Table_ID)
-				tab.setValue("C_Invoice_ID", new Integer(m_Record_ID));
+				tab.setValue("C_Invoice_ID", Integer.valueOf(m_Record_ID));
 			//
 			else if (m_AD_Table_ID == MProduct.Table_ID)
-				tab.setValue("M_Product_ID", new Integer(m_Record_ID));
+				tab.setValue("M_Product_ID", Integer.valueOf(m_Record_ID));
 			else if (m_AD_Table_ID == MPayment.Table_ID)
-				tab.setValue("C_Payment_ID", new Integer(m_Record_ID));
+				tab.setValue("C_Payment_ID", Integer.valueOf(m_Record_ID));
 			//
 			else if (m_AD_Table_ID == MInOut.Table_ID)
-				tab.setValue("M_InOut_ID", new Integer(m_Record_ID));
+				tab.setValue("M_InOut_ID", Integer.valueOf(m_Record_ID));
 			else if (m_AD_Table_ID == MRMA.Table_ID)
-				tab.setValue("M_RMA_ID", new Integer(m_Record_ID));
+				tab.setValue("M_RMA_ID", Integer.valueOf(m_Record_ID));
 			//
 			else if (m_AD_Table_ID == MCampaign.Table_ID)
-				tab.setValue("C_Campaign_ID", new Integer(m_Record_ID));
+				tab.setValue("C_Campaign_ID", Integer.valueOf(m_Record_ID));
 			//
 			else if (m_AD_Table_ID == MRequest.Table_ID)
-				tab.setValue(MRequest.COLUMNNAME_R_RequestRelated_ID, new Integer(m_Record_ID));
+				tab.setValue(MRequest.COLUMNNAME_R_RequestRelated_ID, Integer.valueOf(m_Record_ID));
 			// FR [2842165] - Order Ref link from SO line creating new request
 			else if (m_AD_Table_ID == MOrderLine.Table_ID) {
 				MOrderLine oLine = new MOrderLine(Env.getCtx(), m_Record_ID, null);
 				if (oLine != null) {
-					tab.setValue(MOrderLine.COLUMNNAME_C_Order_ID, new Integer(oLine.getC_Order_ID()));
+					tab.setValue(MOrderLine.COLUMNNAME_C_Order_ID, Integer.valueOf(oLine.getC_Order_ID()));
 				}
 			}
 		}