#283: websearch: spires short codes in invenio search
------------------------+---------------------------------------------------
Reporter: jblayloc | Owner: jblayloc
Type: defect | Status: in_work
Priority: major | Milestone:
Component: WebSearch | Version:
Resolution: | Keywords: INSPIRE Syntax Oct
------------------------+---------------------------------------------------
Changes (by jblayloc):
* component: *general* => WebSearch
Comment:
To investigate:
One theory relies on the fact that in create_basic_search_units the
keyword substitution is done only in case parameter 'm' is None, and
parameter 'p' does not start with /, ', ", and parameter 'f' is not
'author' or 'exactauthor'. So if we're dealing with a search of the form
'inst:CERN', can we be certain that 'm' is not getting set somewhere along
the way between the User and create_basic_search_units?
Another theory is that wash_field is not being used in enough places.
It seems likely both of these are true.
It's also possible that the problem could be in search_unit_in_bibxxx, but
that seems least likely right now because it's easy to read.
Strategy:
Probably two things should happen. First, the reason it's not working
should be diagnosed and repaired (in case something other than this
precise piece of functionality is broken). Second, this bunch of
functionality should go to live somewhere else. These substitutions
should be happening before search_engine ever sees the user's request.
See also #292.
Open question:
How to adequately unit test functionality that is explicitly recommended
be turned off in the default site instance? Probably it should be turned
on in Atlantis, at least with one or two example syntax pieces.
--
Ticket URL: <http://invenio-software.org/ticket/283#comment:5>
Invenio <http://invenio-software.org>
