Review: Needs Fixing

Hello, 

Thanks for the cleanup.

You still should use the function _company_default_get of model company instead 
of browsing user.

This function will do a lookup on setup company defaults and then have a 
fallback on user company_id.


Just a detail in your __init__.py you should use explicit import: 

 from . import document_page_multi_company

Regards

Nicolas

-- 
https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/7.0-document-page-multi-company/+merge/182661
Your team Savoir-faire Linux' OpenERP is subscribed to branch 
lp:~savoirfairelinux-openerp/knowledge-addons/7.0-document-page-multi-company.

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

Reply via email to