openerp-india team mailing list archive

[Alexis de Lattre <alexis@xxxxxxxxxx>]

In fact, my previous comment was wrong, the merge proposal doesn't work : here is the bug scenario with the code of the merge proposal :
You have a picking with 3 products :
1) you do a partial delivery of 1 product -> the generated invoice is OK and has 1 product.
2) you deliver the last 2 products -> you get an error message "This picking list does not require invoicing."

When you look at the code and the exact behavior of the 2 pickings, you
understand that, at step 2, the "active_id" variable in the do_partial()
function of stock_invoice_directly/wizard/stock_invoice.py contains the
ID the "done" picking with 1 products, and not the ID of the picking
with 2 products.

-- 
You received this bug notification because you are a member of OpenERP
Indian Team, which is subscribed to OpenERP Addons.
https://bugs.launchpad.net/bugs/1060259

Title:
  partial picking wizard generate wrong invoice

Status in OpenERP Addons (modules):
  Fix Released

Bug description:
  When you use Receive Products wizard (stock.partial.picking) from a
  "To Be Invoiced" picking (defined from the purchase order property
  Based on incoming shipments), the result invoice reflects the products
  and quantities of the backorder (the picking don't received yet)
  instead to reflects the real products and quantities received and
  registered in the Receive Products wizard.

  I added this code to fix this :

  === modificado archivo stock_invoice_directly/wizard/stock_invoice.py
  --- stock_invoice_directly/wizard/stock_invoice.py	2011-10-16 01:28:00 +0000
  +++ stock_invoice_directly/wizard/stock_invoice.py	2012-10-02 14:22:35 +0000
  @@ -32,7 +32,7 @@
           result = super(invoice_directly, self).do_partial(cr, uid, ids, context)
           partial = self.browse(cr, uid, ids[0], context)
           context.update(active_model='stock.picking',
  -                       active_ids=[partial.picking_id.id])
  +                       active_ids=[partial.picking_id.backorder_id.id])
           if partial.picking_id.invoice_state == '2binvoiced':
               return {
                   'name': 'Create Invoice',

  Openerp Addons trunk7.0 r.7555

To manage notifications about this bug go to:
https://bugs.launchpad.net/openobject-addons/+bug/1060259/+subscriptions