Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0030662Openbravo ERPZ. Otherspublic2015-08-24 11:002015-09-04 12:38
Reportermaite 
Assigned Toaferraz 
PriorityimmediateSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target Version3.0PR15Q2.5Fixed in Version 
Merge Request Status
Review Assigned Todmiguelez
OBNetwork customer
Web browser
ModulesAdvanced Payables and Receivables Mngmt
Support ticket
Regression levelProduction - Confirmed Stable
Regression date2014-09-23
Regression introduced in release3.0PR14Q4
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/179d97ee602d [^]
Triggers an Emergency PackNo
Summary0030662: Payment Method not reloaded when setting Business Partner in Add Payment process from Transaction tab
DescriptionPayment Method not reloaded when setting Business Partner in Add Payment process from Transaction tab
Steps To Reproduce1. Access "Cuenta de Banco" financial account
2. Create new transaction and press + button to add payment
3. Realize that Payment Method field is filled with "Cheque"
4. Select "Alimentos y Supermercados, S.A" business partner and realize that Payment Method combo is not reloaded with business partner default payment method (Transferencia), which was standard behavior before add transaction refactor
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
blocks defect 00306413.0PR15Q4 closed aferraz Payment Method not reloaded when setting Business Partner in Add Payment process from Transaction tab 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2015-08-25 12:00aferrazTypedefect => backport
2015-08-25 12:00aferrazTarget Version3.0PR15Q4 => 3.0PR15Q2.4
2015-08-25 14:54aferrazTarget Version3.0PR15Q2.4 => 3.0PR15Q2.5
2015-08-31 09:19hgbotCheckin
2015-08-31 09:19hgbotNote Added: 0079822
2015-08-31 09:19hgbotStatusscheduled => resolved
2015-08-31 09:19hgbotResolutionopen => fixed
2015-08-31 09:19hgbotFixed in SCM revision => http://code.openbravo.com/erp/backports/3.0PR15Q2.5/rev/1f95905fc6c0286eb4c2ac8eb9b61ac13636da08 [^]
2015-08-31 09:21dmiguelezReview Assigned To => dmiguelez
2015-08-31 09:21dmiguelezNote Added: 0079823
2015-08-31 09:21dmiguelezStatusresolved => closed
2015-09-03 11:55aferrazNote Added: 0080044
2015-09-03 11:55aferrazStatusclosed => new
2015-09-03 11:55aferrazResolutionfixed => open
2015-09-03 12:30aferrazSeveritymajor => minor
2015-09-04 12:32hgbotCheckin
2015-09-04 12:32hgbotNote Added: 0080097
2015-09-04 12:38dmiguelezStatusnew => scheduled
2015-09-04 12:38dmiguelezNote Added: 0080111
2015-09-04 12:38dmiguelezStatusscheduled => resolved
2015-09-04 12:38dmiguelezResolutionopen => fixed
2015-09-04 12:38dmiguelezNote Added: 0080112
2015-09-04 12:38dmiguelezStatusresolved => closed

Notes
(0079822)
hgbot   
2015-08-31 09:19   
Repository: erp/backports/3.0PR15Q2.5
Changeset: 1f95905fc6c0286eb4c2ac8eb9b61ac13636da08
Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com>
Date: Tue Aug 25 14:00:35 2015 +0200
URL: http://code.openbravo.com/erp/backports/3.0PR15Q2.5/rev/1f95905fc6c0286eb4c2ac8eb9b61ac13636da08 [^]

Fixes issue 30662: Payment Method not reloaded in Add Payment process

Payment Method was not being reloaded when setting Business Partner in Add Payment process opened from Transaction tab.
ReceivedFromPaymentMethodActionHandler was not breaking the loop when a valid payment method was found.
Also, payment method name will be retrieved to display it correctly in Add Payment window.

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/actionHandler/ReceivedFromPaymentMethodActionHandler.java
M modules/org.openbravo.advpaymentmngt/web/org.openbravo.advpaymentmngt/js/ob-aprm-addPayment.js
---
(0079823)
dmiguelez   
2015-08-31 09:21   
Code Review + Testing Ok
(0080044)
aferraz   
2015-09-03 11:55   
Reopened as the following scenario is not working properly:

- Go to Financial Account window, search by "Caja".
- Move to Payment Method tab and create "Transferencia" payment method as active = false
- Go to Business Partner window, select "Alimentos y Supermercados" and set "Transferencia" payment method and "Caja" financial account as default in Customer tab.
- Go back to "Caja" Financial Account
- Move to Transactions tab and add a new one (BP Deposit).
- Click on the "+" option of Payment field.
- In the "Add Payment" window, select "Alimentos y Supermercados" in the field "Received From".
- Realize that "Transferencia" payment method is shown, although it is not active for this financial account.
(0080097)
hgbot   
2015-09-04 12:32   
Repository: erp/backports/3.0PR15Q2.5
Changeset: 24b660fd01aeb612b7f1e8648f38469efffb9cf6
Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com>
Date: Fri Sep 04 10:26:47 2015 +0200
URL: http://code.openbravo.com/erp/backports/3.0PR15Q2.5/rev/24b660fd01aeb612b7f1e8648f38469efffb9cf6 [^]

Related to issue 30662: Reload payment method only if it is active

Reload payment method when changing business partner in the selector only if business partner default payment method is active for the current financial account.
Also ReceivedFromPaymentMethodActionHandler performance has been improved.

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/actionHandler/ReceivedFromPaymentMethodActionHandler.java
---
(0080111)
dmiguelez   
2015-09-04 12:38   
Additional commit to complete fix:
http://code.openbravo.com/erp/backports/3.0PR15Q2.5/rev/24b660fd01aeb612b7f1e8648f38469efffb9cf6 [^]
(0080112)
dmiguelez   
2015-09-04 12:38   
Code Review + Testing Ok