Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[10.0] account_check_printing_report_base check reports report incorrect amount in "Payment" column #258

Closed
brendanmasson opened this issue May 24, 2019 · 1 comment
Closed

[10.0] account_check_printing_report_base check reports report incorrect amount in "Payment" column #258

brendanmasson opened this issue May 24, 2019 · 1 comment
Milestone
10.0

Comments

@brendanmasson
Copy link

For invoices with multiple payments, amounts listed in "Payment" column are the same as amounts in the "Original Amount".

The issue is that the report is aggregating amounts from all of the invoice's payments rather than selecting the ones in the report.
brendanmasson added a commit to brendanmasson/account-payment that referenced this issue May 25, 2019
@brendanmasson
[FIX] account_check_printing_report_base: invoice with multiple payments
c39bb92 
Fix check printing to work with invoices that have multiple payments.
The amount in the "Payment" column should only include amounts related
to the selected payment.

Fixes OCA#258.
@brendanmasson brendanmasson mentioned this issue May 25, 2019
Closed
@pedrobaeza
Copy link
Member

Fixed in #259, so closing

@pedrobaeza pedrobaeza added this to the 10.0 milestone May 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
10.0
Development

No branches or pull requests
2 participants
@pedrobaeza @brendanmasson