Code review comment for lp:~openerp-community/openobject-addons/lpistone_6.0_lp784499

Revision history for this message
Anup(SerpentCS) (anup-serpent) wrote :

Hello Everyone,

  As per the discussion I have fixed the hard coded query and used the orm search method for the parent account of the company.

  I agree with Leonardo that the account chosen should be of the current company. IMHO LIMIT 1 in query may give a random result of any of the accounts available.

Please share your views.

Thanks.

« Back to merge proposal