From e6396deb4e93808cfd8e63b7f0d5f460986767a3 Mon Sep 17 00:00:00 2001 From: Christophe Simonis Date: Fri, 27 Feb 2015 15:57:13 +0100 Subject: [PATCH] [IMP] models: log source when "Comparing apples and oranges" --- openerp/models.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/openerp/models.py b/openerp/models.py index e264fdaeddd..7ba80a12f86 100644 --- a/openerp/models.py +++ b/openerp/models.py @@ -49,7 +49,7 @@ import pytz import re import time from collections import defaultdict, MutableMapping -from inspect import getmembers +from inspect import getmembers, currentframe import babel.dates import dateutil.relativedelta @@ -66,6 +66,7 @@ from .osv import fields from .osv.query import Query from .tools import frozendict, lazy_property, ormcache from .tools.config import config +from .tools.func import frame_codeinfo from .tools.misc import CountingStream, DEFAULT_SERVER_DATETIME_FORMAT, DEFAULT_SERVER_DATE_FORMAT from .tools.safe_eval import safe_eval as eval from .tools.translate import _ @@ -5474,7 +5475,9 @@ class BaseModel(object): """ Test whether two recordsets are equivalent (up to reordering). """ if not isinstance(other, BaseModel): if other: - _logger.warning("Comparing apples and oranges: %s == %s", self, other) + filename, lineno = frame_codeinfo(currentframe(), 1) + _logger.warning("Comparing apples and oranges: %r == %r (%s:%s)", + self, other, filename, lineno) return False return self._name == other._name and set(self._ids) == set(other._ids)