opendev/elastic-recheck
Code Issues Proposed changes

Correct conditional branching in field search

Browse Source 
* elastic_recheck/results.py(Hit): Simple but subtle typographical
error in branching conditionals caused at_attr to be searched for in
_source even if attr was already present there. Brown bag fix.

Change-Id: I730f7b7c74a9d772edd0bf483f0089523cb5f6e8
This commit is contained in:
Jeremy Stanley 2013-10-24 18:58:45 +00:00
parent 29368fc2aa
commit 552edc0f8a
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
elastic_recheck/results.py
View File

@ -127,7 +127,7 @@ class Hit(object):
at_attr = "@%s" % attr
if attr in self._hit['_source']:
result = first(self._hit['_source'][attr])
if at_attr in self._hit['_source']:
elif at_attr in self._hit['_source']:
result = first(self._hit['_source'][at_attr])
elif attr in self._hit['_source']['@fields']:
result = first(self._hit['_source']['@fields'][attr])