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] account_invoice_merge - record set used instead of list of ids in do_merge #104
Conversation
Updating to 8.0
…voice_merge_bugfixes [FIX] corrected part 'make link between original sale order'
account_invoice_merge/invoice.py
Outdated
@@ -207,8 +206,9 @@ def make_key(br, fields): | |||
[('product_id', '=', so_line.product_id.id), | |||
('invoice_id', '=', new_invoice_id)]) | |||
if invoice_line_ids: | |||
invoice_line_ids_list = [x.id for x in invoice_line_ids] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would be better if you renamed that variable invoice_lines
as search returns a record set. Thus you can do: invoice_lines.ids
to get a list of ids.
Conflicts: account_invoice_merge/models/account_invoice.py
Syncing from upstream OCA/account-invoicing (8.0)
Syncing from upstream OCA/account-invoicing (8.0)
Syncing from upstream OCA/account-invoicing (8.0)
Syncing from upstream OCA/account-invoicing (8.0)
Syncing from upstream OCA/account-invoicing (8.0)
Syncing from upstream OCA/account-invoicing (8.0)
Syncing from upstream OCA/account-invoicing (8.0)
Syncing from upstream OCA/account-invoicing (8.0)
Syncing from upstream OCA/account-invoicing (8.0)
Syncing from upstream OCA/account-invoicing (8.0)
Syncing from upstream OCA/account-invoicing (8.0)
[bt#871] Put new rounding line at the end of the invoice
Seems that was changed a lot from what I reviewed in 2015. The title of this PR is not reflecting the fix. I guess your change was dropped. You should also create a dedicated branch per PR to avoid changing state of a PR. Thus I will close the current PR. |
corrected part 'make link between original sale order'