Bharat Devnani (Open ERP) has proposed merging 
lp:~openerp-dev/openobject-addons/trunk-bug-794412-bde into 
lp:openobject-addons.

Requested reviews:
  OpenERP Core Team (openerp)
Related bugs:
  Bug #794412 in OpenERP Addons: "delivery order with back order (after manual 
split) is wrongly indicated as shipped (and 100% picked)"
  https://bugs.launchpad.net/openobject-addons/+bug/794412

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-794412-bde/+merge/86771

Hello Sir,

I have applied some changes in sale/sale.py in order to fix the bug.

Thanks & Regards,
Devnani Bharat R.
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-794412-bde/+merge/86771
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/trunk-bug-794412-bde.
=== modified file 'sale/sale.py'
--- sale/sale.py	2011-12-22 12:09:59 +0000
+++ sale/sale.py	2011-12-23 05:19:31 +0000
@@ -122,7 +122,14 @@
 
         for order in self.browse(cr, uid, ids, context=context):
             if order.shipped:
-                res[order.id] = 100.0
+                cnt = 0
+                for picking in order.picking_ids:
+                    if picking.state == 'done':
+                        cnt += 1
+                if len(order.picking_ids) == cnt:
+                    res[order.id] = 100.0
+                else:
+                    res[order.id] = tmp[order.id]['total'] and (100.0 * tmp[order.id]['picked'] / tmp[order.id]['total']) or 0.0
             else:
                 res[order.id] = tmp[order.id]['total'] and (100.0 * tmp[order.id]['picked'] / tmp[order.id]['total']) or 0.0
         return res

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to