Review: Needs Fixing I don't think it is a good idea to merge it as-is if the corresponding addons branch is not merged together: the only installed logging handler is for the 'openerp' namespace, not the root logger. This should normaly result in the addons logs being discarded or displayed differently.
Also, there are a few ugly comments (grep for the added TODO comment to see them). -- https://code.launchpad.net/~openerp-dev/openobject-server/trunk-logging-vmt/+merge/90785 Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-server/trunk-logging-vmt. _______________________________________________ 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
