details: https://code.openbravo.com/erp/devel/pi/rev/b39a4d259c57 changeset: 33213:b39a4d259c57 user: Nono Carballo <nonofce <at> gmail.com> date: Wed Jan 24 13:09:52 2018 +0100 summary: Fixes issue 37676: "Project Phase" field does not show any value (although Project is selected) in Expense Sheet window
In callouts, for fields representing Ids, an empty string is returned instead of null. diffstat: src/org/openbravo/erpCommon/ad_callouts/SE_Expense_BP_Project.java | 8 ++++---- src/org/openbravo/erpCommon/ad_callouts/SE_InOut_Organization.java | 2 +- src/org/openbravo/erpCommon/ad_callouts/SE_Invoice_BPartner.java | 2 +- src/org/openbravo/erpCommon/ad_callouts/SE_Order_BPartner.java | 10 +++++----- src/org/openbravo/erpCommon/ad_callouts/SE_Order_Organization.java | 2 +- src/org/openbravo/erpCommon/ad_callouts/SE_Project_BPartner.java | 8 ++++---- src/org/openbravo/erpCommon/ad_callouts/SE_Trxtype_Transaction.java | 8 ++++---- src/org/openbravo/erpCommon/ad_callouts/SL_GlobalUse_Product.java | 2 +- src/org/openbravo/erpCommon/ad_callouts/SL_InOutLine_Product.java | 2 +- src/org/openbravo/erpCommon/ad_callouts/SL_InOut_BPartner.java | 8 ++++---- src/org/openbravo/erpCommon/ad_callouts/SL_Internal_Consumption_Conversion.java | 2 +- src/org/openbravo/erpCommon/ad_callouts/SL_Internal_Consumption_Product.java | 2 +- src/org/openbravo/erpCommon/ad_callouts/SL_InvAmtUpd_ProductRefDate.java | 2 +- src/org/openbravo/erpCommon/ad_callouts/SL_Inventory_Product.java | 2 +- src/org/openbravo/erpCommon/ad_callouts/SL_Invoice_DocType.java | 2 +- src/org/openbravo/erpCommon/ad_callouts/SL_Invoice_Product.java | 2 +- src/org/openbravo/erpCommon/ad_callouts/SL_Journal_MultiAcctSchema.java | 2 +- src/org/openbravo/erpCommon/ad_callouts/SL_Movement_Product.java | 2 +- src/org/openbravo/erpCommon/ad_callouts/SL_Order_DocType.java | 2 +- src/org/openbravo/erpCommon/ad_callouts/SL_Order_Product.java | 10 +++++----- src/org/openbravo/erpCommon/ad_callouts/SL_Production_Product.java | 2 +- src/org/openbravo/erpCommon/ad_callouts/SL_RefreshSequenceAndLength.java | 4 ++-- src/org/openbravo/erpCommon/ad_callouts/SL_RequisitionLine_Product.java | 2 +- src/org/openbravo/erpCommon/ad_callouts/SL_SequenceProduct_Product.java | 2 +- 24 files changed, 45 insertions(+), 45 deletions(-) diffs (truncated from 426 to 300 lines): diff -r 21e9102f2f7d -r b39a4d259c57 src/org/openbravo/erpCommon/ad_callouts/SE_Expense_BP_Project.java --- a/src/org/openbravo/erpCommon/ad_callouts/SE_Expense_BP_Project.java Mon Jan 22 17:22:09 2018 +0100 +++ b/src/org/openbravo/erpCommon/ad_callouts/SE_Expense_BP_Project.java Wed Jan 24 13:09:52 2018 +0100 @@ -40,8 +40,8 @@ // If project changed if (StringUtils.equals(strChanged, "inpcProjectId")) { // Reset Project Phase and Project Task fields - info.addResult("inpcProjectphaseId", null); - info.addResult("inpcProjecttaskId", null); + info.addResult("inpcProjectphaseId", ""); + info.addResult("inpcProjecttaskId", ""); // If project changed, select project's business partner (if any). if (StringUtils.isNotEmpty(strProjectId)) { @@ -73,8 +73,8 @@ else { strProjectId = SEExpenseBPProjectData.selectProjectId(this, strBPartnerId); info.addResult("inpcProjectId", strProjectId); - info.addResult("inpcProjectphaseId", null); - info.addResult("inpcProjecttaskId", null); + info.addResult("inpcProjectphaseId", ""); + info.addResult("inpcProjecttaskId", ""); } } } diff -r 21e9102f2f7d -r b39a4d259c57 src/org/openbravo/erpCommon/ad_callouts/SE_InOut_Organization.java --- a/src/org/openbravo/erpCommon/ad_callouts/SE_InOut_Organization.java Mon Jan 22 17:22:09 2018 +0100 +++ b/src/org/openbravo/erpCommon/ad_callouts/SE_InOut_Organization.java Wed Jan 24 13:09:52 2018 +0100 @@ -55,7 +55,7 @@ info.addResult("inpmWarehouseId", td[0].getField("id")); } } else { - info.addResult("inpmWarehouseId", null); + info.addResult("inpmWarehouseId", ""); } } } diff -r 21e9102f2f7d -r b39a4d259c57 src/org/openbravo/erpCommon/ad_callouts/SE_Invoice_BPartner.java --- a/src/org/openbravo/erpCommon/ad_callouts/SE_Invoice_BPartner.java Mon Jan 22 17:22:09 2018 +0100 +++ b/src/org/openbravo/erpCommon/ad_callouts/SE_Invoice_BPartner.java Wed Jan 24 13:09:52 2018 +0100 @@ -76,7 +76,7 @@ BpartnerMiscData[] data = BpartnerMiscData.select(this, strBPartner); if (data == null || data.length == 0) { - info.addResult("inpcBpartnerLocationId", null); + info.addResult("inpcBpartnerLocationId", ""); } else { diff -r 21e9102f2f7d -r b39a4d259c57 src/org/openbravo/erpCommon/ad_callouts/SE_Order_BPartner.java --- a/src/org/openbravo/erpCommon/ad_callouts/SE_Order_BPartner.java Mon Jan 22 17:22:09 2018 +0100 +++ b/src/org/openbravo/erpCommon/ad_callouts/SE_Order_BPartner.java Wed Jan 24 13:09:52 2018 +0100 @@ -151,7 +151,7 @@ } info.endSelect(); } else { - info.addResult("inpmWarehouseId", null); + info.addResult("inpmWarehouseId", ""); } // Sales Representative @@ -178,7 +178,7 @@ info.endSelect(); } else { - info.addResult("inpsalesrepId", null); + info.addResult("inpsalesrepId", ""); } // Invoice Rule @@ -213,7 +213,7 @@ info.endSelect(); } else { - info.addResult("inpinvoicerule", null); + info.addResult("inpinvoicerule", ""); } // Project @@ -282,7 +282,7 @@ info.endSelect(); } else { - info.addResult("inpdeliveryrule", null); + info.addResult("inpdeliveryrule", ""); } } @@ -319,7 +319,7 @@ info.endSelect(); } else { - info.addResult("inpadUserId", null); + info.addResult("inpadUserId", ""); } // Message diff -r 21e9102f2f7d -r b39a4d259c57 src/org/openbravo/erpCommon/ad_callouts/SE_Order_Organization.java --- a/src/org/openbravo/erpCommon/ad_callouts/SE_Order_Organization.java Mon Jan 22 17:22:09 2018 +0100 +++ b/src/org/openbravo/erpCommon/ad_callouts/SE_Order_Organization.java Wed Jan 24 13:09:52 2018 +0100 @@ -71,7 +71,7 @@ info.addResult("inpmWarehouseId", td[0].getField("id")); } } else { - info.addResult("inpmWarehouseId", null); + info.addResult("inpmWarehouseId", ""); } } diff -r 21e9102f2f7d -r b39a4d259c57 src/org/openbravo/erpCommon/ad_callouts/SE_Project_BPartner.java --- a/src/org/openbravo/erpCommon/ad_callouts/SE_Project_BPartner.java Mon Jan 22 17:22:09 2018 +0100 +++ b/src/org/openbravo/erpCommon/ad_callouts/SE_Project_BPartner.java Wed Jan 24 13:09:52 2018 +0100 @@ -85,7 +85,7 @@ } info.endSelect(); } else { - info.addResult("inpcBpartnerLocationId", null); + info.addResult("inpcBpartnerLocationId", ""); } // Sales Representative @@ -108,7 +108,7 @@ } info.endSelect(); } else { - info.addResult("inpsalesrepId", null); + info.addResult("inpsalesrepId", ""); } // Business Partner - User/Contacts @@ -135,7 +135,7 @@ } info.endSelect(); } else { - info.addResult("inpadUserId", null); + info.addResult("inpadUserId", ""); } // Business Partner - Bill To Address @@ -164,7 +164,7 @@ } info.endSelect(); } else { - info.addResult("inpbilltoId", null); + info.addResult("inpbilltoId", ""); } // Payment Rule, Payment Term, Price List, Payment Method diff -r 21e9102f2f7d -r b39a4d259c57 src/org/openbravo/erpCommon/ad_callouts/SE_Trxtype_Transaction.java --- a/src/org/openbravo/erpCommon/ad_callouts/SE_Trxtype_Transaction.java Mon Jan 22 17:22:09 2018 +0100 +++ b/src/org/openbravo/erpCommon/ad_callouts/SE_Trxtype_Transaction.java Wed Jan 24 13:09:52 2018 +0100 @@ -30,17 +30,17 @@ final String strTrxType = info.getStringParameter("inptrxtype", null); String strWindowId = info.getStringParameter("inpwindowId", null); if ("BF".equals(strTrxType)) { - info.addResult("inpfinPaymentId", null); - info.addResult("inpcGlitemId", null); + info.addResult("inpfinPaymentId", ""); + info.addResult("inpcGlitemId", ""); info.addResult("inpissotrx", "Y"); info.vars.setSessionValue(strWindowId + "|IsSOTrx", "Y"); } else if ("BPW".equals(strTrxType)) { - info.addResult("inpfinPaymentId", null); + info.addResult("inpfinPaymentId", ""); info.addResult("inpdepositamt", BigDecimal.ZERO); info.addResult("inpissotrx", "N"); info.vars.setSessionValue(strWindowId + "|IsSOTrx", "N"); } else if ("BPD".equals(strTrxType)) { - info.addResult("inpfinPaymentId", null); + info.addResult("inpfinPaymentId", ""); info.addResult("inppaymentamt", BigDecimal.ZERO); info.addResult("inpissotrx", "Y"); info.vars.setSessionValue(strWindowId + "|IsSOTrx", "Y"); diff -r 21e9102f2f7d -r b39a4d259c57 src/org/openbravo/erpCommon/ad_callouts/SL_GlobalUse_Product.java --- a/src/org/openbravo/erpCommon/ad_callouts/SL_GlobalUse_Product.java Mon Jan 22 17:22:09 2018 +0100 +++ b/src/org/openbravo/erpCommon/ad_callouts/SL_GlobalUse_Product.java Wed Jan 24 13:09:52 2018 +0100 @@ -90,7 +90,7 @@ } info.endSelect(); } else { - info.addResult("inpmProductUomId", null); + info.addResult("inpmProductUomId", ""); } } } diff -r 21e9102f2f7d -r b39a4d259c57 src/org/openbravo/erpCommon/ad_callouts/SL_InOutLine_Product.java --- a/src/org/openbravo/erpCommon/ad_callouts/SL_InOutLine_Product.java Mon Jan 22 17:22:09 2018 +0100 +++ b/src/org/openbravo/erpCommon/ad_callouts/SL_InOutLine_Product.java Wed Jan 24 13:09:52 2018 +0100 @@ -161,7 +161,7 @@ } info.endSelect(); } else { - info.addResult("inpmProductUomId", null); + info.addResult("inpmProductUomId", ""); } } diff -r 21e9102f2f7d -r b39a4d259c57 src/org/openbravo/erpCommon/ad_callouts/SL_InOut_BPartner.java --- a/src/org/openbravo/erpCommon/ad_callouts/SL_InOut_BPartner.java Mon Jan 22 17:22:09 2018 +0100 +++ b/src/org/openbravo/erpCommon/ad_callouts/SL_InOut_BPartner.java Wed Jan 24 13:09:52 2018 +0100 @@ -89,7 +89,7 @@ } info.endSelect(); } else { - info.addResult("inpcBpartnerLocationId", null); + info.addResult("inpcBpartnerLocationId", ""); } // Sales Representative @@ -113,11 +113,11 @@ } info.endSelect(); } else { - info.addResult("inpsalesrepId", null); + info.addResult("inpsalesrepId", ""); } // Project - info.addResult("inpcProjectId", null); + info.addResult("inpcProjectId", ""); // Business Partner Contact try { @@ -143,7 +143,7 @@ } info.endSelect(); } else { - info.addResult("inpadUserId", null); + info.addResult("inpadUserId", ""); } // Delivery Rule diff -r 21e9102f2f7d -r b39a4d259c57 src/org/openbravo/erpCommon/ad_callouts/SL_Internal_Consumption_Conversion.java --- a/src/org/openbravo/erpCommon/ad_callouts/SL_Internal_Consumption_Conversion.java Mon Jan 22 17:22:09 2018 +0100 +++ b/src/org/openbravo/erpCommon/ad_callouts/SL_Internal_Consumption_Conversion.java Wed Jan 24 13:09:52 2018 +0100 @@ -81,7 +81,7 @@ // Set Product UOM blank when Quantity Order is blank if (StringUtils.equals(strChanged, "inpquantityorder") && StringUtils.isEmpty(strQuantityOrder) && StringUtils.isNotEmpty(strMProductUOMID)) { - info.addResult("inpmProductUomId", null); + info.addResult("inpmProductUomId", ""); } } diff -r 21e9102f2f7d -r b39a4d259c57 src/org/openbravo/erpCommon/ad_callouts/SL_Internal_Consumption_Product.java --- a/src/org/openbravo/erpCommon/ad_callouts/SL_Internal_Consumption_Product.java Mon Jan 22 17:22:09 2018 +0100 +++ b/src/org/openbravo/erpCommon/ad_callouts/SL_Internal_Consumption_Product.java Wed Jan 24 13:09:52 2018 +0100 @@ -99,7 +99,7 @@ } info.endSelect(); } else { - info.addResult("inpmProductUomId", null); + info.addResult("inpmProductUomId", ""); } } diff -r 21e9102f2f7d -r b39a4d259c57 src/org/openbravo/erpCommon/ad_callouts/SL_InvAmtUpd_ProductRefDate.java --- a/src/org/openbravo/erpCommon/ad_callouts/SL_InvAmtUpd_ProductRefDate.java Mon Jan 22 17:22:09 2018 +0100 +++ b/src/org/openbravo/erpCommon/ad_callouts/SL_InvAmtUpd_ProductRefDate.java Wed Jan 24 13:09:52 2018 +0100 @@ -65,7 +65,7 @@ if (StringUtils.equals(isWarehouseDimension, "N")) { warehouseId = null; - info.addResult("inpmWarehouseId", null); + info.addResult("inpmWarehouseId", ""); } if (product == null) { diff -r 21e9102f2f7d -r b39a4d259c57 src/org/openbravo/erpCommon/ad_callouts/SL_Inventory_Product.java --- a/src/org/openbravo/erpCommon/ad_callouts/SL_Inventory_Product.java Mon Jan 22 17:22:09 2018 +0100 +++ b/src/org/openbravo/erpCommon/ad_callouts/SL_Inventory_Product.java Wed Jan 24 13:09:52 2018 +0100 @@ -138,7 +138,7 @@ } info.endSelect(); } else { - info.addResult("inpmProductUomId", null); + info.addResult("inpmProductUomId", ""); } } diff -r 21e9102f2f7d -r b39a4d259c57 src/org/openbravo/erpCommon/ad_callouts/SL_Invoice_DocType.java --- a/src/org/openbravo/erpCommon/ad_callouts/SL_Invoice_DocType.java Mon Jan 22 17:22:09 2018 +0100 +++ b/src/org/openbravo/erpCommon/ad_callouts/SL_Invoice_DocType.java Wed Jan 24 13:09:52 2018 +0100 @@ -66,7 +66,7 @@ info.addResult("inppaymentrule", "P"); } else { // Payment Rule - No Form of Payment for non Credit Memos Document type. - info.addResult("inppaymentrule", null); + info.addResult("inppaymentrule", ""); } } } diff -r 21e9102f2f7d -r b39a4d259c57 src/org/openbravo/erpCommon/ad_callouts/SL_Invoice_Product.java --- a/src/org/openbravo/erpCommon/ad_callouts/SL_Invoice_Product.java Mon Jan 22 17:22:09 2018 +0100 +++ b/src/org/openbravo/erpCommon/ad_callouts/SL_Invoice_Product.java Wed Jan 24 13:09:52 2018 +0100 @@ -239,7 +239,7 @@ } info.endSelect(); } else { - info.addResult("inpmProductUomId", null); ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot _______________________________________________ Openbravo-commits mailing list Openbravo-commits@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openbravo-commits