Merge lp:~openerp-dev/openobject-addons/7.0-opw-595641-rha into lp:openobject-addons/7.0
Status: | Rejected | ||||
---|---|---|---|---|---|
Rejected by: | Martin Trigaux (OpenERP) | ||||
Proposed branch: | lp:~openerp-dev/openobject-addons/7.0-opw-595641-rha | ||||
Merge into: | lp:openobject-addons/7.0 | ||||
Diff against target: |
14 lines (+2/-2) 1 file modified
stock_invoice_directly/wizard/stock_invoice.py (+2/-2) |
||||
To merge this branch: | bzr merge lp:~openerp-dev/openobject-addons/7.0-opw-595641-rha | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Naresh(OpenERP) | Pending | ||
Review via email: mp+177319@code.launchpad.net |
Description of the change
Enabled warehouse configuration option, "Create and open the invoice when the user finish a delivery order"(module stock_invoice_
Parially processesing the delivery order does automatically invoice the wrong delivery order. It should invoice processed delivery(Backorder) but instead it invoice the delivery which is yet to be processed.
Steps:
- Enable above option
- Create sales order for any product with 3 qty with invoice policy based "On Delivery Order"
- Confirm sales and open delivery order, and process delivery for 2 qty
- It will open an invoice wizard, and create invoice
- It creates invoice for remaining delivery of qty 1 instead of the Backorder of qty 2.
Problem:
Partial delivery keeps open window/form of main/remin delivery that's the reason it's active_id is used to create invoice, but it should actually consider backorder_id.
Fix:
Passed backorder_id as active_id so it will be invoiced.
Hello,
As mentioned on the bug report there were some cases not handled properly.
Instead I have made similar patch but also testing the state of the picking as well. This was merged in 7.0 at revision 9540.