You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by ja...@apache.org on 2008/06/02 11:56:50 UTC

svn commit: r662380 - /ofbiz/trunk/applications/manufacturing/src/org/ofbiz/manufacturing/mrp/MrpServices.java

Author: jacopoc
Date: Mon Jun  2 02:56:49 2008
New Revision: 662380

URL: http://svn.apache.org/viewvc?rev=662380&view=rev
Log:
Misc cleanups.

Modified:
    ofbiz/trunk/applications/manufacturing/src/org/ofbiz/manufacturing/mrp/MrpServices.java

Modified: ofbiz/trunk/applications/manufacturing/src/org/ofbiz/manufacturing/mrp/MrpServices.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/manufacturing/src/org/ofbiz/manufacturing/mrp/MrpServices.java?rev=662380&r1=662379&r2=662380&view=diff
==============================================================================
--- ofbiz/trunk/applications/manufacturing/src/org/ofbiz/manufacturing/mrp/MrpServices.java (original)
+++ ofbiz/trunk/applications/manufacturing/src/org/ofbiz/manufacturing/mrp/MrpServices.java Mon Jun  2 02:56:49 2008
@@ -441,12 +441,11 @@
         iteratorResult = null;
         GenericValue facility = null;
         try {
-            facility = delegator.findOne("Facility", parameters, false);
+            facility = delegator.findOne("Facility", UtilMisc.toMap("facilityId", facilityId), false);
         } catch (GenericEntityException e) {
             return ServiceUtil.returnError("Problem, we can not find Facility, for more detail look at the log");
         }
         String partyId =  (String)facility.get("ownerPartyId");
-        parameters.put("organizationPartyId", partyId);
         try {
             resultList = delegator.findByAnd("SalesForecast", UtilMisc.toMap("organizationPartyId", partyId));
         } catch (GenericEntityException e) {
@@ -456,7 +455,6 @@
         while (iteratorResult.hasNext()) {
             genericResult = (GenericValue) iteratorResult.next();
             String customTimePeriodId =  genericResult.getString("customTimePeriodId");
-            parameters.put("customTimePeriodId", customTimePeriodId);
             GenericValue customTimePeriod = null;
             try {
                 customTimePeriod = delegator.findOne("CustomTimePeriod", UtilMisc.toMap("customTimePeriodId", customTimePeriodId), false);
@@ -466,7 +464,6 @@
             if (customTimePeriod.getDate("thruDate").before(UtilDateTime.nowDate())) {
                 continue;
             } else {
-                parameters.put("salesForecastId", genericResult.getString("salesForecastId"));
                 List salesForecastDetails = null;
                 Iterator sfdIter = null;
                 try {