[18.0][IMP] mrp_byproduct_auto_create_lot: add lot propagation to byproducts#1707
[18.0][IMP] mrp_byproduct_auto_create_lot: add lot propagation to byproducts#1707yostashiro wants to merge 1 commit intoOCA:18.0from
Conversation
yostashiro
changed the title
yostashiro
force-pushed
the
18.0-imp-mrp_byproduct_auto_create_lot
branch
from
16b02f7 to
6b4ed8a
Compare
Add option to propagate main product's lot to byproducts instead of generating unique lots. Configurable at company level with product-level override. Unused method _get_lot_sequence() is removed from mrp.production.
yostashiro
force-pushed
the
18.0-imp-mrp_byproduct_auto_create_lot
branch
from
6b4ed8a to
4c6cbf6
Compare
AungKoKoLin1997
left a comment
AungKoKoLin1997
left a comment
There was a problem hiding this comment.
Functional and Code review: LGTM
This comment was marked as duplicate.
This comment was marked as duplicate.
Sorry, something went wrong.
|
|
||
| 1. Go to *Settings \> Manufacturing* and set "Propagate Lot to Byproducts". | ||
| 2. Optionally override at product level via "Propagate Lot to Byproduct" field | ||
| (after the "Auto Create Lot" field in the General Information tab). |
There was a problem hiding this comment.
| (after the "Auto Create Lot" field in the General Information tab). | |
| (after the byproducts' "Auto Create Lot" field in the General Information tab). |
There was a problem hiding this comment.
@nobuQuartile That's not the design. The current description should be okay.
| return self.env["ir.sequence"].next_by_code("stock.lot.serial") | ||
| for line in lines: | ||
| if propagate_lot_to_byproduct and production.lot_producing_id: | ||
| line.lot_name = production.lot_producing_id.name |
There was a problem hiding this comment.
When multiple units of a serial-tracked byproduct are produced, an error occurs indicating that the same serial number has been created multiple times.
This might be something we should add to the roadmap
There was a problem hiding this comment.
This might be something we should add to the roadmap
@nobuQuartile Please help do so.
| _inherit = "res.company" | ||
|
|
||
| propagate_lot_to_byproduct = fields.Selection( | ||
| selection=[("no", "No"), ("yes", "Yes")], |
There was a problem hiding this comment.
| selection=[("no", "No"), ("yes", "Yes")], | |
| selection=[("yes", "Yes"),("no", "No")], |
I think it’s more common to put “Yes” before “No.” Was there any reason for the order?
3 participants
Add option to propagate main product's lot to byproducts instead of generating unique lots. Configurable at company level with product-level override.
Unused method _get_lot_sequence() is removed from mrp.production.
@qrtl QT5959