Merge lp://qastaging/~openerp-dev/openobject-addons/7.0-opw-596283-rha into lp://qastaging/openobject-addons/7.0
Status: | Approved |
---|---|
Approved by: | Naresh(OpenERP) |
Approved revision: | 9386 |
Proposed branch: | lp://qastaging/~openerp-dev/openobject-addons/7.0-opw-596283-rha |
Merge into: | lp://qastaging/openobject-addons/7.0 |
Diff against target: |
29 lines (+8/-1) 2 files modified
sale_stock/stock.py (+3/-0) stock_location/stock_location.py (+5/-1) |
To merge this branch: | bzr merge lp://qastaging/~openerp-dev/openobject-addons/7.0-opw-596283-rha |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Naresh(OpenERP) (community) | Approve | ||
Rifakat Husen (OpenERP) (community) | Needs Resubmitting | ||
Review via email: mp+178896@code.qastaging.launchpad.net |
Description of the change
When it is multi step delivery where invoicing is based on delivery, wrong picking is getting invoiced. An internal picking is getting invoiced(Invoice Control=`To Be Invoiced`) instead of delivery order(Invoice Control='Not Applicable')
Steps:
- install sale_stock module
- open location `Output`, change chaining type to `Manual Opearation` from 'Automatic No Step Added'
- create sales qoutation, take any product, any qty, set invoicing option, Create Invoice = On Delivery Order
- validate quotation and check pickings
- Internal Picking is having Invoice Control as To Be Invoiced and Delivery Order is having Not Applicable, perhaps result shoud be opposite.
Fix: Only Delivery Order should be invoiced, so enable invoicing for the picking with state = out.
Unmerged revisions
- 9386. By Rifakat Husen (OpenERP)
-
[IMP] better way to update a dict
- 9385. By Rifakat Husen (OpenERP)
-
[fix] stock_location: solve problem also from stock_location module where _prepare_
chained_ picking( ) is overridden
Hello Rifakat,
The fix doesn't seems to work with stock_location installed ! The method *_prepare_ chained_ picking* overridden in this module will change the value of invoicing control back to 'none'.
can you check please ?
Thanks,
Naresh Soni,
OpenERP Enterprise Services !