From 545bee300d8038a0ccebb45017f9dfd207050a15 Mon Sep 17 00:00:00 2001 From: Stefan Rijnhart Date: Tue, 26 Jan 2016 12:37:45 +0100 Subject: [PATCH 1/2] [FIX] keep _register_hook's signature and call super --- base_optional_quick_create/model.py | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/base_optional_quick_create/model.py b/base_optional_quick_create/model.py index 00862f0c70d..3963df8d268 100644 --- a/base_optional_quick_create/model.py +++ b/base_optional_quick_create/model.py @@ -37,9 +37,7 @@ def wrapper(cr, uid, name, context=None): _("Can't create quickly. Opening create form")) return wrapper - def _register_hook(self, cr, ids=None): - if ids is None: - ids = self.search(cr, SUPERUSER_ID, []) + def _patch_quick_create(self, cr, ids): for model in self.browse(cr, SUPERUSER_ID, ids): if model.avoid_quick_create: model_name = model.model @@ -50,14 +48,18 @@ def _register_hook(self, cr, ids=None): model_obj.check_quick_create = True return True + def _register_hook(self, cr): + self._patch_quick_create(cr, self.search(cr, SUPERUSER_ID, [])) + return super(ir_model, self)._register_hook(cr) + def create(self, cr, uid, vals, context=None): res_id = super(ir_model, self).create(cr, uid, vals, context=context) - self._register_hook(cr, [res_id]) + self._patch_quick_create(cr, [res_id]) return res_id def write(self, cr, uid, ids, vals, context=None): if isinstance(ids, (int, long)): ids = [ids] res = super(ir_model, self).write(cr, uid, ids, vals, context=context) - self._register_hook(cr, ids) + self._patch_quick_create(cr, ids) return res From 1055313243f96cb8f3557a44e716e51bef79e78e Mon Sep 17 00:00:00 2001 From: Stefan Rijnhart Date: Tue, 26 Jan 2016 13:30:26 +0100 Subject: [PATCH 2/2] [FIX] Odoolint --- base_optional_quick_create/model.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/base_optional_quick_create/model.py b/base_optional_quick_create/model.py index 3963df8d268..01fbee52790 100644 --- a/base_optional_quick_create/model.py +++ b/base_optional_quick_create/model.py @@ -23,7 +23,7 @@ from openerp.tools.translate import _ -class ir_model(orm.Model): +class IrModel(orm.Model): _inherit = 'ir.model' _columns = { @@ -50,16 +50,16 @@ def _patch_quick_create(self, cr, ids): def _register_hook(self, cr): self._patch_quick_create(cr, self.search(cr, SUPERUSER_ID, [])) - return super(ir_model, self)._register_hook(cr) + return super(IrModel, self)._register_hook(cr) def create(self, cr, uid, vals, context=None): - res_id = super(ir_model, self).create(cr, uid, vals, context=context) + res_id = super(IrModel, self).create(cr, uid, vals, context=context) self._patch_quick_create(cr, [res_id]) return res_id def write(self, cr, uid, ids, vals, context=None): if isinstance(ids, (int, long)): ids = [ids] - res = super(ir_model, self).write(cr, uid, ids, vals, context=context) + res = super(IrModel, self).write(cr, uid, ids, vals, context=context) self._patch_quick_create(cr, ids) return res