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] mrp: write with multiple workorders in self #101580

Closed

Conversation

MerlinGuillaume
Copy link
Contributor

This PR #97805 wrongly assumed that self contained a single workorder. This would raise an error if there were multiple records in self.

Solution:
Modify calls on self to workorder (as we already iterate through them)

This PR odoo#97805 wrongly assumed that
self contained a single workorder. This would raise an error if there
were multiple records in self.

Solution:
Modify calls on `self` to `workorder` (as we already iterate through
them)
@robodoo
Copy link
Contributor

robodoo commented Sep 29, 2022

Pull request status dashboard

@C3POdoo C3POdoo requested a review from a team September 29, 2022 11:13
@C3POdoo C3POdoo added the RD research & development, internal work label Sep 29, 2022
@Whenrow
Copy link
Contributor

Whenrow commented Sep 30, 2022

robodoo r+

robodoo pushed a commit that referenced this pull request Sep 30, 2022
This PR #97805 wrongly assumed that
self contained a single workorder. This would raise an error if there
were multiple records in self.

Solution:
Modify calls on `self` to `workorder` (as we already iterate through
them)

closes #101580

Signed-off-by: William Henrotin (whe) <whe@odoo.com>
@robodoo robodoo temporarily deployed to merge September 30, 2022 08:28 Inactive
@robodoo robodoo closed this Sep 30, 2022
@fw-bot fw-bot deleted the 13.0-fix_workorder_ensure_one-megu branch October 14, 2022 08:47
gamarino pushed a commit to numaes/numa-public-odoo that referenced this pull request Jan 11, 2023
This PR odoo/odoo#97805 wrongly assumed that
self contained a single workorder. This would raise an error if there
were multiple records in self.

Solution:
Modify calls on `self` to `workorder` (as we already iterate through
them)

closes odoo/odoo#101580

Signed-off-by: William Henrotin (whe) <whe@odoo.com>
zamberjo pushed a commit to aurestic/OpenUpgrade that referenced this pull request Mar 1, 2023
This PR odoo/odoo#97805 wrongly assumed that
self contained a single workorder. This would raise an error if there
were multiple records in self.

Solution:
Modify calls on `self` to `workorder` (as we already iterate through
them)

closes odoo/odoo#101580

Signed-off-by: William Henrotin (whe) <whe@odoo.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants