svn commit: r1768215 - /ofbiz/trunk/applications/manufacturing/src/main/java/org/apache/ofbiz/manufacturing/jobshopmgt/ProductionRunServices.java

[diveshdutta]

Author: diveshdutta
Date: Sat Nov  5 13:38:20 2016
New Revision: 1768215

URL: http://svn.apache.org/viewvc?rev=1768215&view=rev
Log:
New feature: Provided an option to specify the Facility location while producing the inventory once production run gets completed.

(OFBIZ-7108)

Thanks: Swapnil Shah for suggesting this improvement and Anuj Jain for providing the patch

Modified:
    ofbiz/trunk/applications/manufacturing/src/main/java/org/apache/ofbiz/manufacturing/jobshopmgt/ProductionRunServices.java

Modified: ofbiz/trunk/applications/manufacturing/src/main/java/org/apache/ofbiz/manufacturing/jobshopmgt/ProductionRunServices.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/manufacturing/src/main/java/org/apache/ofbiz/manufacturing/jobshopmgt/ProductionRunServices.java?rev=1768215&r1=1768214&r2=1768215&view=diff
==============================================================================
--- ofbiz/trunk/applications/manufacturing/src/main/java/org/apache/ofbiz/manufacturing/jobshopmgt/ProductionRunServices.java (original)
+++ ofbiz/trunk/applications/manufacturing/src/main/java/org/apache/ofbiz/manufacturing/jobshopmgt/ProductionRunServices.java Sat Nov  5 13:38:20 2016
@@ -1673,9 +1673,10 @@ public class ProductionRunServices {
         }
         // the inventory item unit cost is the product's standard cost
         BigDecimal unitCost = ZERO;
+        GenericValue facility = null;
         try {
             // get the currency
-            GenericValue facility = productionRun.getGenericValue().getRelatedOne("Facility", false);
+            facility = productionRun.getGenericValue().getRelatedOne("Facility", false);
             Map<String, Object> outputMap = dispatcher.runSync("getPartyAccountingPreferences", UtilMisc.<String, Object>toMap("userLogin", userLogin, "organizationPartyId", facility.getString("ownerPartyId")));
             GenericValue partyAccountingPreference = (GenericValue)outputMap.get("partyAccountingPreference");
             if (partyAccountingPreference == null) {
@@ -1686,6 +1687,18 @@ public class ProductionRunServices {
             if (unitCost == null) {
                 unitCost = ZERO;
             }
+            
+         // Before creating InvntoryItem and InventoryItemDetails, check weather the record of ProductFacility exist in the system or not
+            GenericValue productFacility = EntityQuery.use(delegator). from("ProductFacility").where("productId", productionRun.getProductProduced().getString("productId")
+             , "facilityId", facility.get("facilityId")).queryOne();
+            
+            if (productFacility == null) {
+             Map<String, Object> createProductFacilityCtx = new HashMap<String, Object>();
+             createProductFacilityCtx.put("productId", productionRun.getProductProduced().getString("productId"));
+             createProductFacilityCtx.put("facilityId", facility.get("facilityId"));
+             createProductFacilityCtx.put("userLogin", userLogin);
+             dispatcher.runSync("createProductFacility", createProductFacilityCtx);
+            }
 
         } catch (GenericServiceException gse) {
             Debug.logWarning(gse.getMessage(), module);
@@ -1694,7 +1707,7 @@ public class ProductionRunServices {
             Debug.logWarning(e.getMessage(), module);
             return ServiceUtil.returnError(e.getMessage());
         }
-
+        
         if ("SERIALIZED_INV_ITEM".equals(inventoryItemTypeId)) {
             try {
                 int numOfItems = quantity.intValue();