Product UoM conversion error needs definition
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Odoo Addons (MOVED TO GITHUB) |
Fix Released
|
Low
|
OpenERP R&D Addons Team 2 | ||
6.0 |
Confirmed
|
Undecided
|
Unassigned |
Bug Description
It is known that it is not possible to convert UoM PCE to ft. When you incorrectly make a BOM it will show this error
"Conversion from Product UoM m to Default UoM PCE is not possible as they both belong to different Category!."
Makes sense. However this error is not indicative of what actually happened. When this error is returned it is general and not what units of measure are actually being converted. If I have a manufacturing order for a product defined properly as ft, but I configure it incorrectly as PCE it will show this exact error but it should look like this
"Conversion from Product UoM ft to Default UoM PCE is not possible as they both belong to different Category!."
Correctly stating the error and showing the user their mistake. How could I possible know what the problem is when it does not properly state the error.
This case is especially true while running the manufacturing scheduler. This is not a normal scenario but consider I have 9,000 procurement exceptions and while the scheduler is processing them lets say it encounters just this exact problem. The scheduler gives no indication it is running at all but if it encounters this problem it will show this error in the server log.
"Conversion from Product UoM m to Default UoM PCE is not possible as they both belong to different Category!."
So consider not only does this not provide the correct UoM conversion error, it does not state which product is causing this issue which would force the person to manually sort through all 9,000 exceptions on an individual basis to find the problem. SO if this error happens it should say the error correctly,
"Conversion from Product UoM ft to Default UoM PCE is not possible as they both belong to different Category!."
PRODUCT ID :name of product MOVE ID: Relation to database.
Putting the error this way provides a much more solid experience and diagnosis, telling the user exactly what they did wrong and where to fix the problem. Consider again, the error and its context is too broad to be useful in almost any way.
Related branches
- OpenERP Core Team: Pending requested
-
Diff: 54 lines (+9/-3)2 files modifiedmrp_operations/mrp_operations.py (+8/-2)
product/product.py (+1/-1)
Changed in openobject-addons: | |
assignee: | nobody → OpenERP R&D Addons Team 2 (openerp-dev-addons2) |
importance: | Undecided → Low |
status: | New → Confirmed |
Changed in openobject-addons: | |
status: | Fix Committed → Fix Released |
Hello,
Thanks for Reporting.
It has been fixed in lp:~openerp-dev/openobject-addons/ron-dev-addons2
Revision ID: <email address hidden>
Revision num:5142.
It will be available in trunk soon,