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.

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

[FIX] Only allow adding receivable entries to sale order payment #30

Merged
merged 1 commit into from
Jan 19, 2015

Conversation

dudunato
Copy link

It should only be possible to link a payment/credit from a receivable account
to a sale order, so filter out all other account types

The accounting behavior doesn't make sense when it is showing all kind of entries. It should only show receivable entries.

To reproduce the bug:

1 - Go to "Quotation and Sales"
2 - Choose an order
3 - Go to "Automation Information" tab
4 - Click the Edit button
5 - Click the Add payments button

After these steps there will be receivable entries in that table, but also entries from the other side of the payment transactions, for example, the bank entry

It should only be possible to link a payment/credit from a receivable account
to a sale order, so filter out all other account types
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 3338f4d on dudunato:7.0_payment_receivable_filter into ff2c3d7 on OCA:7.0.

@pedrobaeza
Copy link
Member

Seems reasonable 👍

@guewen
Copy link
Member

guewen commented Jan 19, 2015

👍

@challs
Copy link

challs commented Jan 19, 2015

👍 from me too.

I have been working with Dudu on this problem together and have verified I think it is correct from an accounting perspective.

pedrobaeza added a commit that referenced this pull request Jan 19, 2015
[FIX] Only allow adding receivable entries to sale order payment
@pedrobaeza pedrobaeza merged commit 0250e81 into OCA:7.0 Jan 19, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants