Code review comment for lp:~vauxoo/addons-vauxoo/mrp_production_make_wzd_dev_luis

Luis,
En el __openerp__.py en la key name se te fue lo mismo que en tu modulo pasado hay que cambiarlo.
En el py del wizard cuando vas utilizar el browse y estas dentro de la misma funcion no es necesario que hagas esto: new_production_obj=self.pool.get('wizard.production.make')
       products=new_production_obj.browse(cr, uid, ids, context=context)[0]
si ya estas ahi mismo con el puro self funciona : new_production_obj=products=self.browse(cr,uid,ids,context=context)[0]
pero a si corre el riesgo de que te marque error si esa función la mandan a llamar fuera del wizard y en ids no te mandan una lista, hay que cambiarlo mejor para que itere dentro del for.

En esta if:
if product.categ_id.location_src_id.id:
  location_src=product.categ_id.location_src_id.id
else:
   location_src=products.location_src_id.id
if product.categ_id.location_dest_id.id:
    location_dest=product.categ_id.location_dest_id.id
else:
    location_dest=products.location_dest_id.id

los puedes reducir si utilizas un or ejemplo:

location_src=product.categ_id.location_src_id.id or products.location_src_id.id