remove multiple insert in INV_HOLD_REL_TRACE changes done as per chandrakanth instruction.

git-svn-id: http://15.206.35.175/svn/proteus/business-java/trunk@96911 ce508802-f39f-4f6c-b175-0d175dae99d5

webitm-dis/ibase/webitm/ejb/dis/adv/InvHoldConf.java

@@ -239,7 +239,7 @@ public class InvHoldConf extends ActionHandlerEJB implements InvHoldConfLocal ,I
 					rsHoldDet = pHoldDet.executeQuery();
 				//	pHoldDet.clearParameters();
 					
-					while ( rsHoldDet.next() )
+					if ( rsHoldDet.next() )
 					{
 						itemCode = checkNull(rsHoldDet.getString("ITEM_CODE"));
 						siteCode = checkNull(rsHoldDet.getString("SITE_CODE"));
@@ -250,7 +250,7 @@ public class InvHoldConf extends ActionHandlerEJB implements InvHoldConfLocal ,I
 						System.out.println("itemCode : "+itemCode);
 						System.out.println("lotNo : "+lotNo);
 						
-						if(lockLevel.equalsIgnoreCase("I"))
+						if(lockLevel != null && "I".equalsIgnoreCase(lockLevel))
 						{
 							sql = "update item set HOLD_FLAG = 'Y' where ITEM_CODE = ? ";
 							pstmt  = conn.prepareStatement(sql);
@@ -263,7 +263,7 @@ public class InvHoldConf extends ActionHandlerEJB implements InvHoldConfLocal ,I
 							System.out.println("For Item level(item master),Updated hold flag 'Y' for item_code:"+itemCode);
 						}
 						
-						if(lockLevel.equalsIgnoreCase("L"))
+						if(lockLevel != null && "L".equalsIgnoreCase(lockLevel))
 						{
 							sql = "update item_lot_info set HOLD_FLAG = 'Y' where ITEM_CODE = ? and LOT_NO =? ";
 							pstmt  = conn.prepareStatement(sql);
@@ -278,7 +278,7 @@ public class InvHoldConf extends ActionHandlerEJB implements InvHoldConfLocal ,I
 							
 						}
 						
-						System.out.println("1111111");
+						System.out.println("1111111lineNoSl["+lineNoSl+"]");
 						if( lineNoSl != null && lineNoSl.trim().length() > 0 ) //i.e. Line_sl is present
 						{
 							//tranIdTrace = generateTranId("w_inv_hold_rel_trace", conn);
@@ -355,48 +355,59 @@ public class InvHoldConf extends ActionHandlerEJB implements InvHoldConfLocal ,I
 						else //i.e. line_sl is not found
 						{
 							System.out.println("333333");
+						
 							pHoldDet1 = conn.prepareStatement(sqlHoldDet);
 							pHoldDet1.setString( 1, tranId );
 							rsNoSl = pHoldDet1.executeQuery();
 							while ( rsNoSl.next() )
 							{
-								HashMap pkKeyValMap = new HashMap();
-								StringBuffer sb = new StringBuffer("SELECT ITEM_CODE,SITE_CODE,LOC_CODE,LOT_NO,LOT_SL,QUANTITY, HOLD_QTY,QTY_PER_ART FROM STOCK WHERE  QUANTITY > 0 and  "); //change done by kunal on 29/may/14 add condition for stock qty
 								itemCode = checkNull(rsNoSl.getString("ITEM_CODE"));
 								siteCode = checkNull(rsNoSl.getString("SITE_CODE"));
 								locCode = checkNull(rsNoSl.getString("LOC_CODE"));
 								lotNo = checkNull(rsNoSl.getString("LOT_NO"));
 								lotSl = checkNull(rsNoSl.getString("LOT_SL"));
+								
+								
+								sql = "SELECT ITEM_CODE,SITE_CODE,LOC_CODE,LOT_NO,LOT_SL,QUANTITY, HOLD_QTY,QTY_PER_ART FROM STOCK WHERE  QUANTITY > 0 " ;
 
-								ArrayList pkList = new ArrayList();
+								//ArrayList pkList = new ArrayList();
 
 								if ( itemCode != null && itemCode.length() > 0 )
 								{
-									pkKeyValMap.put("ITEM_CODE", itemCode);
-									pkList.add("ITEM_CODE");
+									/*pkKeyValMap.put("ITEM_CODE", itemCode);
+									pkList.add("ITEM_CODE");*/
+									sql = sql + " AND ITEM_CODE ='"+itemCode+"'";
 								}
 								if ( siteCode != null && siteCode.length() > 0 )
 								{
-									pkKeyValMap.put("SITE_CODE", siteCode);
-									pkList.add("SITE_CODE");
+									/*pkKeyValMap.put("SITE_CODE", siteCode);
+									pkList.add("SITE_CODE");*/
+									sql = sql + "AND SITE_CODE = '"+siteCode+"' ";
+									
 								}
 								if ( locCode != null && locCode.length() > 0 )
 								{
-									pkKeyValMap.put("LOC_CODE", locCode);
-									pkList.add("LOC_CODE");
+									/*pkKeyValMap.put("LOC_CODE", locCode);
+									pkList.add("LOC_CODE");*/
+									
+									sql = sql + "AND LOC_CODE = '"+locCode+"' ";
 								}
 								if ( lotNo != null && lotNo.length() > 0 )
 								{
-									pkKeyValMap.put("LOT_NO", lotNo);
-									pkList.add("LOT_NO");
+									/*pkKeyValMap.put("LOT_NO", lotNo);
+									pkList.add("LOT_NO");*/
+									
+									sql = sql + "AND LOT_NO = '"+lotNo+"' ";
 								}
 								if ( lotSl != null && lotSl.length() > 0 )
 								{
-									pkKeyValMap.put("LOT_SL", lotSl);
-									pkList.add("LOT_SL");
+									/*pkKeyValMap.put("LOT_SL", lotSl);
+									pkList.add("LOT_SL");*/
+									
+									sql = sql + "AND LOT_SL = '"+lotSl+"' ";
 								}
-								System.out.println("pkKeyValMap :"+pkKeyValMap);
-								System.out.println("4444444");
+								//System.out.println("pkKeyValMap :"+pkKeyValMap);
+								/*System.out.println("4444444");
 								String key = "";
 								for ( int i = 0; i < pkList.size(); i++ )
 								{
@@ -419,6 +430,11 @@ public class InvHoldConf extends ActionHandlerEJB implements InvHoldConfLocal ,I
 										pstmtStock.setString(i+1, pkKeyValMap.get(key).toString());
 									}
 								}
+								
+								*/
+								PreparedStatement pstmtStock = null;
+								ResultSet rsStock = null;
+								pstmtStock = conn.prepareStatement(sql);
 								rsStock = pstmtStock.executeQuery();
 								while ( rsStock.next() )
 								{
@@ -481,7 +497,7 @@ public class InvHoldConf extends ActionHandlerEJB implements InvHoldConfLocal ,I
 										rs = null;
 									}
 								}
-								pkList = null;
+								//pkList = null;
 								if(pstmtStock != null)
 								{
 									pstmtStock.close();pstmtStock = null;//Addded by chadra shekar on 17-nov-14