details:   https://code.openbravo.com/erp/devel/pi/rev/ffb026aa78c8
changeset: 19621:ffb026aa78c8
user:      Eduardo Argal Guibert <eduardo.argal <at> openbravo.com>
date:      Fri Feb 08 16:12:33 2013 +0100
summary:   Fixes bug 23013 Part III

diffstat:

 
modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/ad_actionbutton/MatchTransaction.java
 |  6 +++++-
 1 files changed, 5 insertions(+), 1 deletions(-)

diffs (31 lines):

diff -r 96d1681cd223 -r ffb026aa78c8 
modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/ad_actionbutton/MatchTransaction.java
--- 
a/modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/ad_actionbutton/MatchTransaction.java
       Fri Feb 08 14:24:33 2013 +0100
+++ 
b/modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/ad_actionbutton/MatchTransaction.java
       Fri Feb 08 16:12:33 2013 +0100
@@ -430,7 +430,8 @@
       data = getMatchedBankStatementLinesData(vars, strFinancialAccountId, 
strReconciliationId,
           strPaymentTypeFilter, strShowCleared, strHideDate, executeMatching);
     } catch (Exception e) {
-      log4j.debug("Output: Exception ocurred while retrieving Bank Statement 
Lines.");
+      log4j.debug("Output: Exception ocurred while retrieving Bank Statement 
Lines.", e);
+      e.printStackTrace();
     } finally {
       OBContext.restorePreviousMode();
     }
@@ -504,6 +505,7 @@
         final String COLOR_WEAK = "#99CC66";
         final String COLOR_WHITE = "white";
         boolean alreadyMatched = false;
+
         FIN_BankStatementLine line = 
OBDal.getInstance().get(FIN_BankStatementLine.class,
             FIN_BankStatementLines[i].getId());
 
@@ -514,7 +516,9 @@
           // try to match if exception is thrown continue
           try {
             matched = matchingTransaction.match(line, excluded);
+            OBDal.getInstance().getConnection().commit();
           } catch (Exception e) {
+            OBDal.getInstance().rollbackAndClose();
             matched = new FIN_MatchedTransaction(null, 
FIN_MatchedTransaction.NOMATCH);
           }
           // When hide flag checked then exclude matchings for transactions 
out of date range

------------------------------------------------------------------------------
Free Next-Gen Firewall Hardware Offer
Buy your Sophos next-gen firewall before the end March 2013 
and get the hardware for free! Learn more.
http://p.sf.net/sfu/sophos-d2d-feb
_______________________________________________
Openbravo-commits mailing list
Openbravo-commits@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openbravo-commits

Reply via email to