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] accessing self.record fails with multiple records #77

Conversation

thomaspaulb
Copy link

Fixes #76

@legalsylvain legalsylvain added this to the 8.0 milestone Jul 26, 2018
@legalsylvain
Copy link
Contributor

@thomaspaulb could you fix trivial travis error. (flake8) ?

kind regards.

@thomaspaulb thomaspaulb force-pushed the 8.0-fix-bug-with-account-fiscal-classification branch from 13f799c to 2d9703a Compare July 27, 2018 10:27
@thomaspaulb
Copy link
Author

@legalsylvain fixed flake issue

Copy link
Contributor

@legalsylvain legalsylvain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need fixing.

regards.

# update or replace 'taxes_id' and 'supplier_taxes_id'
classification = self.fiscal_classification_id
fc_obj = self.env['account.product.fiscal.classification']
fc = fc_obj.browse(classification_id)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

classification_id -> fc_id

@legalsylvain
Copy link
Contributor

thanks, fast tracking this patch.

@legalsylvain legalsylvain merged commit 6567948 into OCA:8.0 Jul 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants