From 4eb9059480b786864c5162156674eea2bfbca325 Mon Sep 17 00:00:00 2001 From: Olivier Dony Date: Wed, 21 Aug 2013 14:29:55 +0200 Subject: [PATCH] [IMP] BaseModel.fields_view_get: warn about non-qualified *_view_ref references lp bug: https://launchpad.net/bugs/1214799 fixed bzr revid: odo@openerp.com-20130821122955-8c9z0mi8cu48rne3 --- openerp/osv/orm.py | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/openerp/osv/orm.py b/openerp/osv/orm.py index f96d00815c9..51b0c2b56b9 100644 --- a/openerp/osv/orm.py +++ b/openerp/osv/orm.py @@ -2201,7 +2201,8 @@ class BaseModel(object): sql_res = False parent_view_model = None - view_ref = context.get(view_type + '_view_ref') + view_ref_key = view_type + '_view_ref' + view_ref = context.get(view_ref_key) # Search for a root (i.e. without any parent) view. while True: if view_ref and not view_id: @@ -2211,6 +2212,10 @@ class BaseModel(object): view_ref_res = cr.fetchone() if view_ref_res: view_id = view_ref_res[0] + else: + _logger.warning('%r requires a fully-qualified external id (got: %r for model %s). ' + 'Please use the complete `module.view_id` form instead.', view_ref_key, view_ref, + self._name) if view_id: cr.execute("""SELECT arch,name,field_parent,id,type,inherit_id,model