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 manual create sourcing #34

Merged
merged 4 commits into from
Nov 13, 2014
Merged

Conversation

lepistone
Copy link
Member

Fixes #33.

Fixes OCA#33.

This is due to the fact that with the v8 API the methods that compute
function fields are called not only on save, but also as automatic
onchanges. In that situation, `self` is not a true record, but a virtual
NewId that contains only the values present in the view.

That is cool in theory, but in cases like that it can crash because the
requisition_id is empty as it is not present in the view.

This fix does the computation that relies on requisition_id only when it
is available. I do not check explicitly whether self if a NewId, so that
if we manage to fix the view so that we know the requisition_id, the
amount_total_company will be updated on the fly.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.03%) when pulling f2a1a0f on lepistone:fix-manual-create-sourcing into 9fd6a2a on OCA:8.0.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.03%) when pulling f2a1a0f on lepistone:fix-manual-create-sourcing into 9fd6a2a on OCA:8.0.

# instead of crashing. The result of that is that the
# amount_total_company field will be updated only when we save.
# It might be that that can be improved by improving the view. This
# check will pose no problem in that case.
Copy link
Member

Choose a reason for hiding this comment

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

should we log something in that case?

Copy link
Member Author

Choose a reason for hiding this comment

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

agreed: that should be a warn.

@lepistone
Copy link
Member Author

@rdeheele @gurneyalex done.

@coveralls
Copy link

coveralls commented Nov 12, 2014

Coverage Status

Coverage decreased (-0.04%) to 79.138% when pulling 3d27db6 on lepistone:fix-manual-create-sourcing into 9fd6a2a on OCA:8.0.

@gurneyalex
Copy link
Member

👍

gurneyalex added a commit that referenced this pull request Nov 13, 2014
@gurneyalex gurneyalex merged commit 9963177 into OCA:8.0 Nov 13, 2014
@lepistone lepistone deleted the fix-manual-create-sourcing branch November 13, 2014 13:01
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

4 participants