svn commit: r506575 - in /ofbiz/trunk/applications: ecommerce/webapp/ecommerce/WEB-INF/actions/customer/ order/webapp/ordermgr/WEB-INF/actions/entry/catalog/ order/webapp/ordermgr/WEB-INF/actions/order/ order/webapp/ordermgr/WEB-INF/actions/requirement/

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

svn commit: r506575 - in /ofbiz/trunk/applications: ecommerce/webapp/ecommerce/WEB-INF/actions/customer/ order/webapp/ordermgr/WEB-INF/actions/entry/catalog/ order/webapp/ordermgr/WEB-INF/actions/order/ order/webapp/ordermgr/WEB-INF/actions/requirement/

jonesde
Author: jonesde
Date: Mon Feb 12 10:50:26 2007
New Revision: 506575

URL: http://svn.apache.org/viewvc?view=rev&rev=506575
Log:
Changed requestParameters to parameters in a bunch of places, as that is the one that should nearly always be usedfor consistency, and to support the request to session etc defaulting pattern; also removed a number of statements to manually create the Map instead of using the one always created by the Screen Widget

Modified:
    ofbiz/trunk/applications/ecommerce/webapp/ecommerce/WEB-INF/actions/customer/editcontactmech.bsh
    ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/category.bsh
    ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/categorydetail.bsh
    ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/quickadd.bsh
    ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/sidedeepcategory.bsh
    ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/order/findOrders.bsh
    ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/requirement/ApprovedProductRequirements.bsh

Modified: ofbiz/trunk/applications/ecommerce/webapp/ecommerce/WEB-INF/actions/customer/editcontactmech.bsh
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/ecommerce/webapp/ecommerce/WEB-INF/actions/customer/editcontactmech.bsh?view=diff&rev=506575&r1=506574&r2=506575
==============================================================================
--- ofbiz/trunk/applications/ecommerce/webapp/ecommerce/WEB-INF/actions/customer/editcontactmech.bsh (original)
+++ ofbiz/trunk/applications/ecommerce/webapp/ecommerce/WEB-INF/actions/customer/editcontactmech.bsh Mon Feb 12 10:50:26 2007
@@ -67,25 +67,24 @@
 }
 
 tryEntity = context.get("tryEntity");
-requestParameters = UtilHttp.getParameterMap(request);
 
 contactMechData = context.get("contactMech");
-if (!tryEntity.booleanValue()) contactMechData = requestParameters;
+if (!tryEntity.booleanValue()) contactMechData = parameters;
 if (contactMechData == null) contactMechData = new HashMap();
 if (contactMechData != null) context.put("contactMechData", contactMechData);
 
 partyContactMechData = context.get("partyContactMech");
-if (!tryEntity.booleanValue()) partyContactMechData = requestParameters;
+if (!tryEntity.booleanValue()) partyContactMechData = parameters;
 if (partyContactMechData == null) partyContactMechData = new HashMap();
 if (partyContactMechData != null) context.put("partyContactMechData", partyContactMechData);
 
 postalAddressData = context.get("postalAddress");
-if (!tryEntity.booleanValue()) postalAddressData = requestParameters;
+if (!tryEntity.booleanValue()) postalAddressData = parameters;
 if (postalAddressData == null) postalAddressData = new HashMap();
 if (postalAddressData != null) context.put("postalAddressData", postalAddressData);
 
 telecomNumberData = context.get("telecomNumber");
-if (!tryEntity.booleanValue()) telecomNumberData = requestParameters;
+if (!tryEntity.booleanValue()) telecomNumberData = parameters;
 if (telecomNumberData == null) telecomNumberData = new HashMap();
 if (telecomNumberData != null) context.put("telecomNumberData", telecomNumberData);
 

Modified: ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/category.bsh
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/category.bsh?view=diff&rev=506575&r1=506574&r2=506575
==============================================================================
--- ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/category.bsh (original)
+++ ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/category.bsh Mon Feb 12 10:50:26 2007
@@ -29,10 +29,9 @@
 detailScreen = "categorydetail";
 catalogName = CatalogWorker.getCatalogName(request);
 delegator = request.getAttribute("delegator");
-requestParameters = UtilHttp.getParameterMap(request);
 productCategoryId = request.getAttribute("productCategoryId");
 if (productCategoryId == null) {
-    productCategoryId = requestParameters.get("category_id");
+    productCategoryId = parameters.get("category_id");
 }
 context.put("productCategoryId", productCategoryId);
 

Modified: ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/categorydetail.bsh
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/categorydetail.bsh?view=diff&rev=506575&r1=506574&r2=506575
==============================================================================
--- ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/categorydetail.bsh (original)
+++ ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/categorydetail.bsh Mon Feb 12 10:50:26 2007
@@ -28,8 +28,8 @@
 import org.ofbiz.product.catalog.*;
 import org.ofbiz.product.category.CategoryContentWrapper;
 
-viewSize = requestParameters.get("VIEW_SIZE");
-viewIndex = requestParameters.get("VIEW_INDEX");
+viewSize = parameters.get("VIEW_SIZE");
+viewIndex = parameters.get("VIEW_INDEX");
 currentCatalogId = CatalogWorker.getCurrentCatalogId(request);
 
 // set the default view size

Modified: ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/quickadd.bsh
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/quickadd.bsh?view=diff&rev=506575&r1=506574&r2=506575
==============================================================================
--- ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/quickadd.bsh (original)
+++ ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/quickadd.bsh Mon Feb 12 10:50:26 2007
@@ -30,9 +30,8 @@
 import org.ofbiz.product.catalog.*;
 
 dispatcher = request.getAttribute("dispatcher");
-requestParameters = UtilHttp.getParameterMap(request);
 currentCatalogId = CatalogWorker.getCurrentCatalogId(request);
-categoryId = requestParameters.get("category_id");
+categoryId = parameters.get("category_id");
 if (categoryId == null || categoryId.length() <= 0)
     categoryId = CatalogWorker.getCatalogQuickaddCategoryPrimary(request);
     

Modified: ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/sidedeepcategory.bsh
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/sidedeepcategory.bsh?view=diff&rev=506575&r1=506574&r2=506575
==============================================================================
--- ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/sidedeepcategory.bsh (original)
+++ ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/sidedeepcategory.bsh Mon Feb 12 10:50:26 2007
@@ -27,10 +27,8 @@
 import org.ofbiz.product.category.*;
 import javolution.util.FastMap;
 
-requestParameters = UtilHttp.getParameterMap(request);
-
 CategoryWorker.getRelatedCategories(request, "topLevelList", CatalogWorker.getCatalogTopCategoryId(request, CatalogWorker.getCurrentCatalogId(request)), true);
-curCategoryId = UtilFormatOut.checkNull(requestParameters.get("category_id"), requestParameters.get("CATEGORY_ID"));
+curCategoryId = UtilFormatOut.checkNull(parameters.get("category_id"), parameters.get("CATEGORY_ID"));
 request.setAttribute("curCategoryId", curCategoryId);
 CategoryWorker.setTrail(request, curCategoryId);
 

Modified: ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/order/findOrders.bsh
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/order/findOrders.bsh?view=diff&rev=506575&r1=506574&r2=506575
==============================================================================
--- ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/order/findOrders.bsh (original)
+++ ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/order/findOrders.bsh Mon Feb 12 10:50:26 2007
@@ -387,7 +387,7 @@
     productStore = ProductStoreWorker.getProductStore(request);
     List filterInventoryProblems = new ArrayList();
     
-    if ("Y".equals(requestParameters.get("filterInventoryProblems")) && UtilValidate.isNotEmpty(orderHeaderList)) {
+    if ("Y".equals(parameters.get("filterInventoryProblems")) && UtilValidate.isNotEmpty(orderHeaderList)) {
         paramList = paramList + "&amp;filterInventoryProblems=" + "Y";
         for (ohi = orderHeaderList.iterator(); ohi.hasNext();) {
             orderHeader = (GenericValue) ohi.next();
@@ -403,19 +403,19 @@
     List filterPOsWithRejectedItems = new ArrayList();
     List filterPartiallyReceivedPOs = new ArrayList();
     
-    if ("Y".equals(requestParameters.get("filterPOsWithRejectedItems"))) {
+    if ("Y".equals(parameters.get("filterPOsWithRejectedItems"))) {
                         paramList = paramList + "&amp;filterPOsWithRejectedItems=" + "Y";
     }    
-    if ("Y".equals(requestParameters.get("filterPOsOpenPastTheirETA"))) {
+    if ("Y".equals(parameters.get("filterPOsOpenPastTheirETA"))) {
                         paramList = paramList + "&amp;filterPOsOpenPastTheirETA=" + "Y";
     }    
-    if ("Y".equals(requestParameters.get("filterPartiallyReceivedPOs"))) {
+    if ("Y".equals(parameters.get("filterPartiallyReceivedPOs"))) {
                         paramList = paramList + "&amp;filterPartiallyReceivedPOs=" + "Y";
     }
         
-    if (("Y".equals(requestParameters.get("filterPartiallyReceivedPOs")) ||
-            "Y".equals(requestParameters.get("filterPOsOpenPastTheirETA")) ||
-            "Y".equals(requestParameters.get("filterPOsWithRejectedItems"))) &&
+    if (("Y".equals(parameters.get("filterPartiallyReceivedPOs")) ||
+            "Y".equals(parameters.get("filterPOsOpenPastTheirETA")) ||
+            "Y".equals(parameters.get("filterPOsWithRejectedItems"))) &&
             UtilValidate.isNotEmpty(orderHeaderList)) {
         for (ohi = orderHeaderList.iterator(); ohi.hasNext();) {
             orderHeader = (GenericValue) ohi.next();
@@ -426,18 +426,18 @@
             orderReadHelper = OrderReadHelper.getHelper(orderHeader);
             if ("PURCHASE_ORDER".equals(orderHeader.getString("orderTypeId"))) {                
                 if (orderReadHelper.getRejectedOrderItems() &&
-                        "Y".equals(requestParameters.get("filterPOsWithRejectedItems"))) {                    
+                        "Y".equals(parameters.get("filterPOsWithRejectedItems"))) {                    
                     filterPOsWithRejectedItems.add(orderHeader.get("orderId"));
                     paramList = paramList + "&amp;filterPOsWithRejectedItems=" + "Y";
                     continue;
                 }
                 if (orderReadHelper.getPastEtaOrderItems(orderHeader.get("orderId")) &&
-                        "Y".equals(requestParameters.get("filterPOsOpenPastTheirETA"))) {
+                        "Y".equals(parameters.get("filterPOsOpenPastTheirETA"))) {
                     filterPOsOpenPastTheirETA.add(orderHeader.get("orderId"));
                     continue;
                 }
                 if (orderReadHelper.getPartiallyReceivedItems() &&
-                        "Y".equals(requestParameters.get("filterPartiallyReceivedPOs"))) {
+                        "Y".equals(parameters.get("filterPartiallyReceivedPOs"))) {
                     filterPartiallyReceivedPOs.add(orderHeader.get("orderId"));
                     paramList = paramList + "&amp;filterPartiallyReceivedPOs=" + "Y";
                     continue;

Modified: ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/requirement/ApprovedProductRequirements.bsh
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/requirement/ApprovedProductRequirements.bsh?view=diff&rev=506575&r1=506574&r2=506575
==============================================================================
--- ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/requirement/ApprovedProductRequirements.bsh (original)
+++ ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/requirement/ApprovedProductRequirements.bsh Mon Feb 12 10:50:26 2007
@@ -17,8 +17,7 @@
 
 import org.ofbiz.base.util.UtilMisc;
 
-dispatcher = request.getAttribute("dispatcher");
-prepare = dispatcher.runSync("prepareFind", UtilMisc.toMap("inputFields", requestParameters, "entityName", "Requirement"));
-results = dispatcher.runSync("getRequirementsForSupplier", UtilMisc.toMap("requirementConditions", prepare.get("entityConditionList"), "partyId", requestParameters.get("partyId")));
+prepare = dispatcher.runSync("prepareFind", UtilMisc.toMap("inputFields", parameters, "entityName", "Requirement"));
+results = dispatcher.runSync("getRequirementsForSupplier", UtilMisc.toMap("requirementConditions", prepare.get("entityConditionList"), "partyId", parameters.get("partyId")));
 context.put("requirementsForSupplier", results.get("requirementsForSupplier"));
 context.put("quantityReport", UtilMisc.toMap("distinctProductCount", results.get("distinctProductCount"), "quantityTotal", results.get("quantityTotal")));