details: https://code.openbravo.com/erp/devel/pi/rev/e8c1e6c5054d
changeset: 16073:e8c1e6c5054d
user: Naiara Martinez <naiara.martinez <at> openbravo.com>
date: Wed Apr 11 12:29:37 2012 +0200
summary: fixed issue 20093 APRM Set a correct error message and catch the
exception
diffstat:
modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/AD_MESSAGE.xml
| 11 ++++++++++
modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/dao/AdvPaymentMngtDao.java
| 6 ++++-
2 files changed, 16 insertions(+), 1 deletions(-)
diffs (37 lines):
diff -r 8c1ee5492e66 -r e8c1e6c5054d
modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/AD_MESSAGE.xml
---
a/modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/AD_MESSAGE.xml
Wed Apr 11 11:54:38 2012 +0200
+++
b/modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/AD_MESSAGE.xml
Wed Apr 11 12:29:37 2012 +0200
@@ -749,6 +749,17 @@
<!--C5133F7FD55D4AF6B8B7FB527BFB9920-->
<AD_MODULE_ID><![CDATA[A918E3331C404B889D69AA9BFAFB23AC]]></AD_MODULE_ID>
<!--C5133F7FD55D4AF6B8B7FB527BFB9920--></AD_MESSAGE>
+<!--D304C035EC724B6A874E574BE59BC7BF--><AD_MESSAGE>
+<!--D304C035EC724B6A874E574BE59BC7BF-->
<AD_MESSAGE_ID><![CDATA[D304C035EC724B6A874E574BE59BC7BF]]></AD_MESSAGE_ID>
+<!--D304C035EC724B6A874E574BE59BC7BF-->
<AD_CLIENT_ID><![CDATA[0]]></AD_CLIENT_ID>
+<!--D304C035EC724B6A874E574BE59BC7BF--> <AD_ORG_ID><![CDATA[0]]></AD_ORG_ID>
+<!--D304C035EC724B6A874E574BE59BC7BF--> <ISACTIVE><![CDATA[Y]]></ISACTIVE>
+<!--D304C035EC724B6A874E574BE59BC7BF-->
<VALUE><![CDATA[APRM_PaymentMethod]]></VALUE>
+<!--D304C035EC724B6A874E574BE59BC7BF--> <MSGTEXT><![CDATA[Selected payment
method doesn't exist.]]></MSGTEXT>
+<!--D304C035EC724B6A874E574BE59BC7BF--> <MSGTYPE><![CDATA[E]]></MSGTYPE>
+<!--D304C035EC724B6A874E574BE59BC7BF-->
<AD_MODULE_ID><![CDATA[A918E3331C404B889D69AA9BFAFB23AC]]></AD_MODULE_ID>
+<!--D304C035EC724B6A874E574BE59BC7BF--></AD_MESSAGE>
+
<!--D374CA8D2FF541E5BF1149FCFDAEED86--><AD_MESSAGE>
<!--D374CA8D2FF541E5BF1149FCFDAEED86-->
<AD_MESSAGE_ID><![CDATA[D374CA8D2FF541E5BF1149FCFDAEED86]]></AD_MESSAGE_ID>
<!--D374CA8D2FF541E5BF1149FCFDAEED86-->
<AD_CLIENT_ID><![CDATA[0]]></AD_CLIENT_ID>
diff -r 8c1ee5492e66 -r e8c1e6c5054d
modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/dao/AdvPaymentMngtDao.java
---
a/modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/dao/AdvPaymentMngtDao.java
Wed Apr 11 11:54:38 2012 +0200
+++
b/modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/dao/AdvPaymentMngtDao.java
Wed Apr 11 12:29:37 2012 +0200
@@ -1319,7 +1319,11 @@
obc.add(Restrictions.eq(FinAccPaymentMethod.PROPERTY_ACTIVE, true));
obc.setFilterOnReadableClients(false);
obc.setFilterOnReadableOrganization(false);
- return obc.list().get(0);
+ try {
+ return obc.list().get(0);
+ } catch (IndexOutOfBoundsException e) {
+ throw new OBException(FIN_Utility.messageBD("APRM_PaymentMethod"));
+ }
}
public boolean isAutomatedExecutionPayment(FIN_FinancialAccount account,
------------------------------------------------------------------------------
For Developers, A Lot Can Happen In A Second.
Boundary is the first to Know...and Tell You.
Monitor Your Applications in Ultra-Fine Resolution. Try it FREE!
http://p.sf.net/sfu/Boundary-d2dvs2
_______________________________________________
Openbravo-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openbravo-commits
