From d86c2834346fc42897dc484b80543245dd13bc34 Mon Sep 17 00:00:00 2001 From: Olivier Dony Date: Fri, 7 Oct 2011 16:49:44 +0200 Subject: [PATCH] [FIX] orm.browse_record: avoid printing unrelated traceback upon KeyError bzr revid: odo@openerp.com-20111007144944-j4obwcbvx9awkj30 --- openerp/osv/orm.py | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/openerp/osv/orm.py b/openerp/osv/orm.py index 303842c91d7..3a3155ed7f1 100644 --- a/openerp/osv/orm.py +++ b/openerp/osv/orm.py @@ -374,11 +374,9 @@ class browse_record(object): else: return attr else: - self.logger.notifyChannel("browse_record", netsvc.LOG_WARNING, - "Field '%s' does not exist in object '%s': \n%s" % ( - name, self, ''.join(traceback.format_exc()))) - raise KeyError("Field '%s' does not exist in object '%s'" % ( - name, self)) + error_msg = "Field '%s' does not exist in object '%s'" % (name, self) + self.logger.notifyChannel("browse_record", netsvc.LOG_WARNING, error_msg) + raise KeyError(error_msg) # if the field is a classic one or a many2one, we'll fetch all classic and many2one fields if col._prefetch: