Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

websearch: fin cn cms and date 2010 and t dog #323

Closed
jrbl opened this Issue · 2 comments

2 participants

@jrbl
Collaborator

Originally on 2010-10-23

On ß release we had a bug in date parsing for the above string, because the regexps were matching the date term as
year:2010 and title:dog
rather than
year:2010

For the sake of rollout, we fixed it by catching exceptions thrown by dateutil and simply using the user's input date string instead.

This is a ticket to go back and correct the regex so it never gets too much stuff.

@jrbl jrbl self-assigned this
@jrbl
Collaborator

Originally on 2010-10-23

This is complete in my branch 323-websearch_date_regexps_problematic; all of the unit tests pass (including the new ones) and the regression tests are no worse than they were.

@invenio-developers
Collaborator

Originally by Joe Blaylock jrbl@slac.stanford.edu on 2010-11-12

In [dc864c5]:

#CommitTicketReference repository="" revision="dc864c5b8c2205b3e43e14fb87beadc5743e5411"
WebSearch: SPIRES syntax date robustness fix

- Less buggy regular expression for matching dates in SPIRES syntax converter,
  and additional unit tests.
  (fixes #323)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.