details:   https://code.openbravo.com/erp/stable/2.50/rev/48ff2c5071b4
changeset: 9662:48ff2c5071b4
user:      Mikel Irurita <mikel.irurita <at> openbravo.com>
date:      Wed Sep 21 12:45:26 2011 +0200
summary:   Fixes issue 18576: Transaction of GL Item cannot be posted due to a 
NPE error

diffstat:

 src/org/openbravo/erpCommon/ad_forms/DocFINFinAccTransaction.java |  6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diffs (22 lines):

diff -r dcad918db1b3 -r 48ff2c5071b4 
src/org/openbravo/erpCommon/ad_forms/DocFINFinAccTransaction.java
--- a/src/org/openbravo/erpCommon/ad_forms/DocFINFinAccTransaction.java Wed Sep 
21 09:18:48 2011 +0200
+++ b/src/org/openbravo/erpCommon/ad_forms/DocFINFinAccTransaction.java Wed Sep 
21 12:45:26 2011 +0200
@@ -265,15 +265,15 @@
       String Line_ID = data[i].getField("FIN_Finacc_Transaction_ID");
       DocLine_FINFinAccTransaction docLine = new 
DocLine_FINFinAccTransaction(DocumentType,
           Record_ID, Line_ID);
-      FIN_PaymentDetail detail = 
OBDal.getInstance().get(FIN_PaymentDetail.class,
-          data[i].getField("FIN_Payment_Detail_ID"));
       String strPaymentId = data[i].getField("FIN_Payment_ID");
       if (strPaymentId != null && !strPaymentId.equals(""))
         docLine.setFinPaymentId(strPaymentId);
       docLine.m_Record_Id2 = strPaymentId;
       docLine.setIsPrepayment(data[i].getField("isprepayment"));
       docLine.setCGlItemId(data[i].getField("cGlItemId"));
-      if (detail != null) {
+      if (data[i].getField("FIN_Payment_Detail_ID") != null) {
+        FIN_PaymentDetail detail = 
OBDal.getInstance().get(FIN_PaymentDetail.class,
+            data[i].getField("FIN_Payment_Detail_ID"));
         OBContext.setAdminMode();
         try {
           docLine

------------------------------------------------------------------------------
All the data continuously generated in your IT infrastructure contains a
definitive record of customers, application performance, security
threats, fraudulent activity and more. Splunk takes this data and makes
sense of it. Business sense. IT sense. Common sense.
http://p.sf.net/sfu/splunk-d2dcopy1
_______________________________________________
Openbravo-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openbravo-commits

Reply via email to