Review: Needs Fixing
some improvements to do:
98 + state: draft and 165 + state: draft => Not needed
120 + from tools.translate import _ and 147 + from tools.translate import
=> not needed
product_product_pendrive2 => this product is not used anywhere, remove if not
needed
varify the cost price of product created and cost price taken in PO line is not
same
403 + partner_address_id: base.res_partner_address_7 => ref of partner
address_id is not of Axelor
change DistriPC => Distrib PC
468 + partner_address_id: base.res_partner_address_7 => ref of partner
address_id is not of ASUStek
485 +
partner_id=self.pool.get('res.partner').search(cr,uid,[('name','=','ASUStek')])[0]
=> instead of hardcoded search on name of ASUStek you can find from the
reference of base.res_partner_asus
499 + assert (state=='approved') => Put this with previous code
thanks
--
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-purchase-coverage-aag/+merge/76209
Your team OpenERP R&D Team is subscribed to branch
lp:~openerp-dev/openobject-addons/trunk-purchase-coverage-aag.
_______________________________________________
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
