[ADD]stock:stock_partial_picking.py ADD comments for increase readability of code

bzr revid: ron@tinyerp.com-20110427111822-ow49y4b5l99yeh07
This commit is contained in:
ron@tinyerp.com
2011-04-27 16:48:22 +05:30
parent 61ee65d9f0
commit 2b83cdfff6
2 changed files with 14 additions and 1 deletions
@@ -20,7 +20,6 @@
</record>
<record id="stock_move_memory_tree_in" model="ir.ui.view">
<field name="name">stock.move.memory.tree</field>
<field name="model">stock.move.memory.in</field>
@@ -30,6 +29,9 @@
<field name="product_id" />
<field name="quantity" />
<field name="product_uom" />
<!--When you click the "Validate" button, moves disappear
(product track_incoming or track_outgoing field must be True)
if "attrs="{'required'('tracking','=',True)]}" is specified in prodlot_id field of the tree view -->
<field name="prodlot_id" domain="[('product_id', '=', product_id)]" groups="base.group_extended" />
<field name="tracking" invisible="1"/>
<field name="cost" />
@@ -64,6 +66,9 @@
<field name="product_id" />
<field name="quantity" />
<field name="product_uom" />
<!--When you click the "Validate" button, all moves disappear
(product track_incoming or track_outgoing field must be True)
if "attrs="{'required'('tracking','=',True)]}" is specified in prodlot_id field of the tree view -->
<field name="prodlot_id" domain="[('product_id', '=', product_id)]" groups="base.group_extended" />
<field name="tracking" invisible="1"/>
</tree>
@@ -156,14 +156,22 @@ class stock_partial_picking(osv.osv_memory):
for move in moves_list:
move_uom = move.move_id.product_uom
process_uom = move.product_uom
#Quantiny must be Positive
if move.quantity <= 0:
raise osv.except_osv(_('Warning!'), _('Please provide Proper Quantity !'))
#category id must be same for respective move
if move_uom.category_id.id != process_uom.category_id.id:
raise osv.except_osv(_('Warning'), _('You can not process %s %s as it\'s category is different than category of %s of this move!') % (move.quantity, process_uom.name, move_uom.name))
#Pikcing move product UOM factor must be bigger with respective wizard move product uom factor
if move_uom.factor < process_uom.factor:
raise osv.except_osv(_('Warning'), _('You can not process in UOM "%s" which is smaller than UOM "%s" of the current move.') % (process_uom.name, move_uom.name))
#Compute the wizard Quantity for respective move.
toprocess = uom_obj._compute_qty(cr, uid, move.product_uom.id, move.quantity, move.move_id.product_uom.id)
#Compare wizard Quantity with respective picking move quantity if wizard move quantity bigger then it's giving warning.
if toprocess > move.move_id.product_qty:
raise osv.except_osv(_('Warning'), _('You can not process "%s %s" as the qty is more than "%s %s" of respective move.') % (move.quantity, process_uom.name, move.move_id.product_qty, move_uom.name))
partial_datas['move%s' % (move.move_id.id)] = {