Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0034385
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] Z. Othersmajoralways2016-11-04 09:152016-12-16 18:38
ReportermaiteView Statuspublic 
Assigned Torqueralta 
PriorityurgentResolutionfixedFixed in Version3.0PR17Q1
StatusclosedFix in branchFixed in SCM revisionc1d19a6b6689
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Toaferraz
Web browser
ModulesAdvanced Payables and Receivables Mngmt
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0034385: Payment of 0 amount created when voiding an unpaid invoice should be created with Exchange Rate record from invoice (if any)

DescriptionAt the moment, in case an original invoice has specific rate set in Exchange Rate tab and it is Voided then reversed invoice inherits this exchange rate. Payment of 0 amount automatically created should also inherit this rate in order to avoid differences when posting it
Steps To Reproduce1. Register Sales Invoice for España organization and any BP. Add any line. Add Exchange Rate between EUR and USD as 5. Complete and Post invoice
2. Run VOID process. Realize that reversed invoice also has this exchange rate of 5. Post invoice
3. Navigate to related payment of 0 and realize that exchange rate tab is empty. In case different rate is defined at Conversion Rates window you will be posting differences when should not be the case
Proposed Solutionpayment of 0 should also copy specific Exchange Rate of invoices it is cancelling
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
related to defect 00309713.0PR16Q1 closedaferraz When voiding an unpaid invoice, payment schedule details should be canceled instead of deleted 
related to defect 0034322 closedrqueralta Payment of 0 amount created when voiding an unpaid invoice is created in Awaiting Execution status 

-  Notes
(0091427)
rqueralta (developer)
2016-11-14 04:15

Test plan:

As F&B International Group Admin

  Go to Sales Order window
  1.Register Sales Invoice for España organization and any BP.
  - Add any line.
  - Add Exchange Rate between EUR and USD as 5.
  - Complete and Post invoice
  2.Run VOID process.
  - Realize that reversed invoice also has this exchange rate of 5.
  - Post invoice
  Go to Payment In window
  - Navigate to related payment of 0.
  - Realize that exchange rate tab is filled with the exhange rate of invoice it is cancelling.
(0091789)
hgbot (developer)
2016-11-24 17:18

Repository: erp/devel/pi
Changeset: c1d19a6b6689346b0e62435877298cf92eaa8068
Author: Rafael Queralta Pozo <rqueralta <at> nauta.cu>
Date: Sun Nov 13 14:52:51 2016 -0500
URL: http://code.openbravo.com/erp/devel/pi/rev/c1d19a6b6689346b0e62435877298cf92eaa8068 [^]

Fixes issue 34385: Payment of 0 amount created when voiding an unpaid invoice

Payment of 0 amount created when voiding an unpaid invoice it is created with Exchange Rate of cancelling invoice.

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/ad_actionbutton/ProcessInvoice.java
---
(0091790)
hgbot (developer)
2016-11-24 17:18

Repository: erp/devel/pi
Changeset: 98b5f1a7b29dfe29c45614d189b1f24c3eb5a4b0
Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com>
Date: Thu Nov 24 17:10:27 2016 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/98b5f1a7b29dfe29c45614d189b1f24c3eb5a4b0 [^]

Related to issue 34385: Code review improvements

Set payment exchange rate foreign amount as zero, as payment amount will be always zero.
Copy every record in Exchange rates tab.
Remove unnecessary query.
Remove unnecessary admin mode.

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/ad_actionbutton/ProcessInvoice.java
---
(0091791)
aferraz (manager)
2016-11-24 17:19

Code review + Testing OK
(0092655)
hudsonbot (developer)
2016-12-16 18:38

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/dc8bf00badd0 [^]
Maturity status: Test
(0092656)
hudsonbot (developer)
2016-12-16 18:38

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/dc8bf00badd0 [^]
Maturity status: Test

- Issue History
Date Modified Username Field Change
2016-11-04 09:15 maite New Issue
2016-11-04 09:15 maite Assigned To => Triage Finance
2016-11-04 09:15 maite Modules => Advanced Payables and Receivables Mngmt
2016-11-04 09:15 maite Resolution time => 1480028400
2016-11-04 09:15 maite Triggers an Emergency Pack => No
2016-11-04 09:15 maite Relationship added related to 0030971
2016-11-04 09:15 maite Issue Monitored: networkb
2016-11-04 10:02 aferraz Assigned To Triage Finance => nonofrancisco
2016-11-04 10:03 aferraz Relationship added related to 0034322
2016-11-11 15:27 rqueralta Assigned To nonofrancisco => rqueralta
2016-11-11 15:27 rqueralta Status new => scheduled
2016-11-14 04:15 rqueralta Note Added: 0091427
2016-11-24 17:18 hgbot Checkin
2016-11-24 17:18 hgbot Note Added: 0091789
2016-11-24 17:18 hgbot Status scheduled => resolved
2016-11-24 17:18 hgbot Resolution open => fixed
2016-11-24 17:18 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/c1d19a6b6689346b0e62435877298cf92eaa8068 [^]
2016-11-24 17:18 hgbot Checkin
2016-11-24 17:18 hgbot Note Added: 0091790
2016-11-24 17:19 aferraz Review Assigned To => aferraz
2016-11-24 17:19 aferraz Note Added: 0091791
2016-11-24 17:19 aferraz Status resolved => closed
2016-11-24 17:19 aferraz Fixed in Version => 3.0PR17Q1
2016-12-16 18:38 hudsonbot Checkin
2016-12-16 18:38 hudsonbot Note Added: 0092655
2016-12-16 18:38 hudsonbot Checkin
2016-12-16 18:38 hudsonbot Note Added: 0092656


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker