banking-addons-team team mailing list archive

Thread
Date

Re: lp:~therp-nl/banking-addons/ba70-lp1231174-default_method_should_not_raise_at_module_installation_time into lp:banking-addons

To: mp+187677@xxxxxxxxxxxxxxxxxx
From: Pedro Manuel Baeza <pedro.baeza@xxxxxxxxx>
Date: Thu, 26 Sep 2013 07:21:32 -0000
In-reply-to: <20130926070534.21729.34315.launchpad@ackee.canonical.com>
Reply-to: mp+187677@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing code review, no test

Hi, Stefan, can you use the v7 form "from openerp.osv.orm import except_orm"? I also prefer the more compact "context = context or {}" instead of the two lines "if context is None...".

Regards.
-- 
https://code.launchpad.net/~therp-nl/banking-addons/ba70-lp1231174-default_method_should_not_raise_at_module_installation_time/+merge/187677
Your team Banking Addons Core Editors is subscribed to branch lp:banking-addons.

References

lp:~therp-nl/banking-addons/ba70-lp1231174-default_method_should_not_raise_at_module_installation_time into lp:banking-addons
From: Stefan Rijnhart (Therp), 2013-09-26