savoirfairelinux-openerp team mailing list archive

[Nicolas Bessi - Camptocamp <nicolas.bessi@xxxxxxxxxxxxxx>]

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.