Merge lp:~vauxoo/openobject-addons/addons-6.1-mrp_bug_bom-dev-julio into lp:openobject-addons/6.1
Proposed by
Julio Serna-http://www.vauxoo.com
Status: | Needs review |
---|---|
Proposed branch: | lp:~vauxoo/openobject-addons/addons-6.1-mrp_bug_bom-dev-julio |
Merge into: | lp:openobject-addons/6.1 |
Diff against target: |
18 lines (+7/-1) 1 file modified
mrp/mrp.py (+7/-1) |
To merge this branch: | bzr merge lp:~vauxoo/openobject-addons/addons-6.1-mrp_bug_bom-dev-julio |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Chris Biersbach (OpenERP) | Pending | ||
Olivier Dony (Odoo) | Pending | ||
Moisés López - http://www.vauxoo.com | Pending | ||
Review via email: mp+123865@code.launchpad.net |
Description of the change
modifies query that gets the BoM, validating if the BoM is active or if the date is in valid range
To post a comment you must log in.
Unmerged revisions
- 6984. By Julio Serna-http://www.vauxoo.com
-
[FIX][mrp] modified function _bom_find to get the BoM considering if active or dates
Hi Julio, '%Y-%m- %d %H:%M:%S')), '%Y-%m- %d %H:%M:%S'))]
Your fix works great.
You may also use search() instead,
domain = [('product_id', '=', product_id), ('bom_id', '=', False), '&',
'|', ('date_start', '=', False), ('date_start', '<=', time.strftime(
'|', ('date_stop', '=', False), ('date_stop', '>=', time.strftime(
How about this one?
I will fix using search() for 7.0.
Thanks!