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.

Sign up for GitHub

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

Jump to bottom

[13.0] MRP : Traceback when creating Workorder By Going in Workcenter. #38992

Closed
dipsshah42 opened this issue Oct 18, 2019 · 1 comment
Closed

[13.0] MRP : Traceback when creating Workorder By Going in Workcenter. #38992

dipsshah42 opened this issue Oct 18, 2019 · 1 comment
Labels
13.0 Logistics about stock, mrp, delivery, purchase

Comments

@dipsshah42
Copy link
Contributor

dipsshah42 commented Oct 18, 2019 •
edited
Loading

Impacted versions: 13.0

Steps to reproduce:

  • MRP > Create Any work center and Save it.
  • Open that work center > Go into 2nd smart button named 'Lost hours' something.
  • which will open the Productivity Losses menu.
  • Create any record manually here and create a work order from here.

Current behavior:

  • Traceback is throwing.

Expected behavior:

  • Should handle in well manner.

Video/Screenshot link (optional):
Screenshot
Video

Thanks!
@pedrobaeza pedrobaeza added 13.0 Logistics about stock, mrp, delivery, purchase labels Oct 18, 2019
@sswapnesh sswapnesh mentioned this issue Oct 19, 2019
Closed
@sswapnesh
Copy link
Contributor

Create and Edit should be Removed from there as It doesn't make any sense to create a record from there.

Added fix at #39071.
However, Still, you will get You have to produce at least one False. due to

odoo/addons/mrp/models/mrp_abstract_workorder.py

Line 57 in e398db8

raise UserError(_('You have to produce at least one %s.') % self.product_uom_id.name)

mart-e pushed a commit to sswapnesh/odoo that referenced this issue Oct 23, 2019
@sswapnesh @mart-e
[FIX] mrp: always set a value to computed fiec
c93a46c 
Records were filtered so not every record was set in the for loop

Fixes odoo#38992
Closes odoo#39071
fw-bot pushed a commit to odoo-dev/odoo that referenced this issue Oct 23, 2019
@sswapnesh @fw-bot
[FIX] mrp: always set a value to computed fiec
c89eed4 
Records were filtered so not every record was set in the for loop

Fixes odoo#38992
Closes odoo#39071

X-original-commit: f7d5ba6
@fw-bot fw-bot mentioned this issue Oct 23, 2019
Closed
robodoo pushed a commit that referenced this issue Oct 23, 2019
@sswapnesh @fw-bot
[FIX] mrp: always set a value to computed fiec
2fc9280 
Records were filtered so not every record was set in the for loop

Fixes #38992
Closes #39071

closes #39234

X-original-commit: f7d5ba6
Signed-off-by: Martin Trigaux (mat) <mat@odoo.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
13.0 Logistics about stock, mrp, delivery, purchase
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[FIX] mrp: Compute is_produced if needed sswapnesh/odoo
3 participants
@pedrobaeza @sswapnesh @dipsshah42