Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0033987Openbravo ERPZ. Otherspublic2016-09-14 10:252016-09-26 22:05
Reportermaite 
Assigned Tomarkmm82 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version3.0PR16Q4 
Merge Request Status
Review Assigned Toaferraz
OBNetwork customerOBPS
Web browser
ModulesAdvanced Payables and Receivables Mngmt
Support ticket43651
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0033987: It is not possible to VOID a payment that generates credit
DescriptionError 20501 is obtained when trying to void a payment that generates credit
Steps To Reproduce0. Access Financial Account "Cuenta de Banco" and configure "Transferencia" payment method as "automatic", "Simple Execution Process" and "deferred=yes"
1. Access Payment In window and create record for "Cuenta de Banco", "transferencia", "Alimentos y Supermercados" and Amount=500
2. Run Add details process, NOT select any invoice and set option "Leave the credit to be used later". Process payment
3. Press "Reactivate" button and select VOID option
4. Error 20501 is raised

Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-09-14 10:25maiteNew Issue
2016-09-14 10:25maiteAssigned To => Triage Finance
2016-09-14 10:25maiteOBNetwork customer => Yes
2016-09-14 10:25maiteModules => Advanced Payables and Receivables Mngmt
2016-09-14 10:25maiteSupport ticket => 43651
2016-09-14 10:25maiteResolution time => 1475532000
2016-09-14 10:25maiteTriggers an Emergency Pack => No
2016-09-14 10:25maiteIssue Monitored: networkb
2016-09-15 11:13aferrazAssigned ToTriage Finance => markmm82
2016-09-20 17:30markmm82Statusnew => scheduled
2016-09-20 20:13markmm82Note Added: 0090122
2016-09-23 14:46hgbotCheckin
2016-09-23 14:46hgbotNote Added: 0090208
2016-09-23 14:46hgbotStatusscheduled => resolved
2016-09-23 14:46hgbotResolutionopen => fixed
2016-09-23 14:46hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/a570fdbe9083a550e3c628e7157f73af6efc56aa [^]
2016-09-23 14:47aferrazReview Assigned To => aferraz
2016-09-23 14:47aferrazNote Added: 0090209
2016-09-23 14:47aferrazStatusresolved => closed
2016-09-23 14:47aferrazFixed in Version => 3.0PR16Q4
2016-09-26 22:05hudsonbotCheckin
2016-09-26 22:05hudsonbotNote Added: 0090276

Notes
(0090122)
markmm82   
2016-09-20 20:13   
Test Plan:
0. Access Financial Account "Cuenta de Banco" and configure "Transferencia" payment method as "automatic", "Simple Execution Process" and "deferred=yes"
1. Access Payment In window and create record for "Cuenta de Banco", "transferencia", "Alimentos y Supermercados" and Amount=500
2. Run Add details process, NOT select any invoice and set option "Leave the credit to be used later". Process payment
3. Press "Reactivate" button and select VOID option
4. Notice the process is completed successfully and Payment has Status = Void.
(0090208)
hgbot   
2016-09-23 14:46   
Repository: erp/devel/pi
Changeset: a570fdbe9083a550e3c628e7157f73af6efc56aa
Author: Mark <markmm82 <at> gmail.com>
Date: Thu Sep 22 07:12:39 2016 -0400
URL: http://code.openbravo.com/erp/devel/pi/rev/a570fdbe9083a550e3c628e7157f73af6efc56aa [^]

Fixes issue 33987: It is possible to VOID a payment that generates credit

While payment is processed, as it will be voided it updates its generated credit. When all changes are flushed and aprm_fin_pmt_check_trg function is fired, it also takes into account status of the NEW.STATUS object, to only compare differences between amounts and generated credits if the new status is different than RPVOID.

---
M modules/org.openbravo.advpaymentmngt/src-db/database/model/triggers/APRM_FIN_PMT_CHECK_TRG.xml
---
(0090209)
aferraz   
2016-09-23 14:47   
Code review + Testing OK
(0090276)
hudsonbot   
2016-09-26 22:05   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/d32e95b900dd [^]
Maturity status: Test