openerp-community-reviewer team mailing list archive

Thread
Date

Re: lp:~yann-papouin/ocb-server/6.1-bug-944197-register-hook-server-start into lp:ocb-server/6.1

To: Yann Papouin <yann.papouin@xxxxxxxxx>
From: "Stefan Rijnhart \(Therp\)" <stefan@xxxxxxxx>
Date: Fri, 04 Oct 2013 11:18:39 -0000
In-reply-to: <20131004065624.17342.12759.launchpad@ackee.canonical.com>
Reply-to: mp+189219@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing

After reviewing the addons part, I think it is safer if you import SUPERUSER_ID here, and pass it on in the call to _register_hook. Otherwise, if someone updates their 6.1 server and not their addons, their installation will break.

-- 
https://code.launchpad.net/~yann-papouin/ocb-server/6.1-bug-944197-register-hook-server-start/+merge/189219
Your team OpenERP Community Backports Team is subscribed to branch lp:ocb-server/6.1.

References

lp:~yann-papouin/ocb-server/6.1-bug-944197-register-hook-server-start into lp:ocb-server/6.1
From: Yann Papouin, 2013-10-04