svn commit: r568395 - /ofbiz/trunk/specialpurpose/oagis/src/org/ofbiz/oagis/OagisInventoryServices.java

[jonesde]

Author: jonesde
Date: Tue Aug 21 20:28:46 2007
New Revision: 568395

URL: http://svn.apache.org/viewvc?rev=568395&view=rev
Log:
A few fixes and cleanups for the AckDel RMA and Status services

Modified:
    ofbiz/trunk/specialpurpose/oagis/src/org/ofbiz/oagis/OagisInventoryServices.java

Modified: ofbiz/trunk/specialpurpose/oagis/src/org/ofbiz/oagis/OagisInventoryServices.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/oagis/src/org/ofbiz/oagis/OagisInventoryServices.java?rev=568395&r1=568394&r2=568395&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/oagis/src/org/ofbiz/oagis/OagisInventoryServices.java (original)
+++ ofbiz/trunk/specialpurpose/oagis/src/org/ofbiz/oagis/OagisInventoryServices.java Tue Aug 21 20:28:46 2007
@@ -841,13 +841,13 @@
                                     if (OagisServices.requireSerialNumberExist != null) {
                                         // according to requireSerialNumberExist make sure serialNumber does or does not exist in database, add an error message as needed
                                         if (OagisServices.requireSerialNumberExist.booleanValue()) {
-                                            if (inventoryItemsBySerialNumber.size() > 0) {
+                                            if (inventoryItemsBySerialNumber.size() == 0) {
                                                 String errMsg = "Referenced serial numbers must already exist, but serial number [" + serialNum + "] was not found.";
                                                 errorMapList.add(UtilMisc.toMap("description", errMsg, "reasonCode", "SerialNumberRequiredButNotFound"));
                                                 continue;
                                             }
                                         } else {
-                                            if (inventoryItemsBySerialNumber.size() == 0) {
+                                            if (inventoryItemsBySerialNumber.size() > 0) {
                                                 String errMsg = "Referenced serial numbers must NOT already exist, but serial number [" + serialNum + "] already exists.";
                                                 errorMapList.add(UtilMisc.toMap("description", errMsg, "reasonCode", "SerialNumberRequiredNotExistButFound"));
                                                 continue;
@@ -1267,28 +1267,24 @@
                             List inventoryItemsBySerialNumber = delegator.findByCondition("InventoryItem", bySerialNumberCondition, null, null);
 
                             // this is a status update, so referenced serial number MUST already exist
-                            if (inventoryItemsBySerialNumber.size() > 0) {
+                            if (inventoryItemsBySerialNumber.size() == 0) {
                                 String errMsg = "Referenced serial numbers must already exist, but serial number [" + serialNum + "] was not found.";
                                 errorMapList.add(UtilMisc.toMap("description", errMsg, "reasonCode", "SerialNumberRequiredButNotFound"));
                                 continue;
                             }
                             
                             GenericValue inventoryItem = EntityUtil.getFirst(inventoryItemsBySerialNumber);
-                            if (inventoryItem != null) {
-                                Map updateInvItmMap = FastMap.newInstance();
-                                updateInvItmMap.put("inventoryItemId", inventoryItem.getString("inventoryItemId"));
-                                updateInvItmMap.put("userLogin", userLogin);
-                                updateInvItmMap.put("statusId", invItemStatusId);
-                                String inventoryItemProductId = inventoryItem.getString("productId");
-                                if (!inventoryItemProductId.equals(productId)) {
-                                    // got a new productId for the serial number; this may happen for refurbishment, etc
-                                    updateInvItmMap.put("productId",productId);
-                                }
-                                dispatcher.runSync("updateInventoryItem", updateInvItmMap);
-                                invItemIds.add(UtilMisc.toMap("inventoryItemId", inventoryItem.getString("inventoryItemId")));
-                            } else {
-                                // TODO: ERROR, referenced InventoryItem by serialNumber not found
+                            Map updateInvItmMap = FastMap.newInstance();
+                            updateInvItmMap.put("inventoryItemId", inventoryItem.getString("inventoryItemId"));
+                            updateInvItmMap.put("userLogin", userLogin);
+                            updateInvItmMap.put("statusId", invItemStatusId);
+                            String inventoryItemProductId = inventoryItem.getString("productId");
+                            if (!inventoryItemProductId.equals(productId)) {
+                                // got a new productId for the serial number; this may happen for refurbishment, etc
+                                updateInvItmMap.put("productId",productId);
                             }
+                            dispatcher.runSync("updateInventoryItem", updateInvItmMap);
+                            invItemIds.add(UtilMisc.toMap("inventoryItemId", inventoryItem.getString("inventoryItemId")));
                         }
                     } else {
                         String inventoryItemTypeId = "NON_SERIAL_INV_ITEM";
@@ -1298,7 +1294,7 @@
                     }
                 }
             } catch (Throwable t) {
-                String errMsg = "System Error processing Acknowledge Delivery RMA message: " + t.toString();
+                String errMsg = "System Error processing Acknowledge Delivery Status message: " + t.toString();
                 Debug.logInfo(t, errMsg, module);
                 // in this case we don't want to return a Confirm BOD, so return an error now
                 return ServiceUtil.returnError(errMsg);