details:   /erp/devel/pi/rev/1528be065b3b
changeset: 8797:1528be065b3b
user:      Adrián Romero <adrianromero <at> openbravo.com>
date:      Thu Nov 04 19:23:56 2010 +0100
summary:   Fixes issue 0015039: Wrong message in Service Project when you 
select new Price List
SE_ProjectLine callout was mistakenly applying parameters it was intrepreting a 
pricelistversion_id as a pricelist_id
Now the callout is interpreting properly arguments

diffstat:

 src/org/openbravo/erpCommon/ad_callouts/SE_ProjectLine_Value.java |  31 
++++-----
 1 files changed, 13 insertions(+), 18 deletions(-)

diffs (49 lines):

diff -r c7d985e43280 -r 1528be065b3b 
src/org/openbravo/erpCommon/ad_callouts/SE_ProjectLine_Value.java
--- a/src/org/openbravo/erpCommon/ad_callouts/SE_ProjectLine_Value.java Thu Nov 
04 17:21:24 2010 +0100
+++ b/src/org/openbravo/erpCommon/ad_callouts/SE_ProjectLine_Value.java Thu Nov 
04 19:23:56 2010 +0100
@@ -64,7 +64,7 @@
   }
 
   private void printPage(HttpServletResponse response, VariablesSecureApp vars,
-      String strmProductId, String strPriceListId, String strTabId, String 
strProjectId,
+      String strmProductId, String strPriceListVersion, String strTabId, 
String strProjectId,
       String strPhaseId, String strADOrgID) throws IOException, 
ServletException {
     if (log4j.isDebugEnabled())
       log4j.debug("Output: dataSheet");
@@ -92,24 +92,19 @@
       String strMWarehouseID = data1[0].warehouse;
       String strProjCat = data1[0].projcat;
 
-      if (!strPriceListId.equals("")) {
-        String strPriceListVersion = 
SEProjectLineValueData.selectPriceListVersion(this,
-            strPriceListId, strDate);
-        if (!strPriceListVersion.equals("")) {
-          String plannedprice = SEProjectLineValueData.selectPlannedPrice(this,
-              strPriceListVersion, strmProductId, strProjectId);
-          if (plannedprice != null && !plannedprice.equals("")) {
-            if (!strProjCat.equals("S")) {
-              resultado.append("new Array(\"inpplannedprice\", "
-                  + (plannedprice.equals("") ? "\"\"" : plannedprice) + 
"),\n");
-            } else {
-              resultado.append("new Array(\"inppriceactual\", "
-                  + (plannedprice.equals("") ? "\"\"" : plannedprice) + ")\n");
-            }
-          } else
-            strMessage = "PriceNotFound";
+      if (!strPriceListVersion.equals("")) {
+        String plannedprice = SEProjectLineValueData.selectPlannedPrice(this, 
strPriceListVersion,
+            strmProductId, strProjectId);
+        if (plannedprice != null && !plannedprice.equals("")) {
+          if (!strProjCat.equals("S")) {
+            resultado.append("new Array(\"inpplannedprice\", "
+                + (plannedprice.equals("") ? "\"\"" : plannedprice) + "),\n");
+          } else {
+            resultado.append("new Array(\"inppriceactual\", "
+                + (plannedprice.equals("") ? "\"\"" : plannedprice) + ")\n");
+          }
         } else
-          strMessage = "PriceListVersionNotFound";
+          strMessage = "PriceNotFound";
       }
       if (!strProjCat.equals("S")) {
         if (strCBPartnerLocationID != null && 
!strCBPartnerLocationID.equals("")

------------------------------------------------------------------------------
The Next 800 Companies to Lead America's Growth: New Video Whitepaper
David G. Thomson, author of the best-selling book "Blueprint to a 
Billion" shares his insights and actions to help propel your 
business during the next growth cycle. Listen Now!
http://p.sf.net/sfu/SAP-dev2dev
_______________________________________________
Openbravo-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openbravo-commits

Reply via email to