Summary: | Publication date range assumes that years must always be 4 digits | ||
---|---|---|---|
Product: | Koha | Reporter: | Katrin Fischer <katrin.fischer> |
Component: | Searching | Assignee: | Jonathan Druart <jonathan.druart> |
Status: | Failed QA --- | QA Contact: | Testopia <testopia> |
Severity: | normal | ||
Priority: | P5 - low | CC: | jonathan.druart, martin.renvoize, nick |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Bug 11469: Allow search by publication years with number of digits != 4
Bug 11469: Allow search by publication years with number of digits != 4 Bug 11469: Fix search by date range |
Description
Katrin Fischer
2014-01-02 15:42:42 UTC
Created attachment 107628 [details] [review] Bug 11469: Allow search by publication years with number of digits != 4 The adv search did not allow publication years prior to 1000 We are now ready for year 10000 Test plan: Having some book with a publication year=123 (3 digits) Search this book by publication year => Search result must return it Created attachment 107848 [details] [review] Bug 11469: Allow search by publication years with number of digits != 4 The adv search did not allow publication years prior to 1000 We are now ready for year 10000 Test plan: Having some book with a publication year=123 (3 digits) Search this book by publication year => Search result must return it Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Simple patch that works as expected.. must admit I was expecting us to be doing something 'clever' like catching double-digit dates adding a prefix to make it up to four digits.. but as we're not doing anything like that already this patch seems perfect. Signing off. > The adv search did not allow publication years prior to 1000
> We are now ready for year 10000
We might be - but MARC definitely is not :)
Before patch, on opac, search 'Date range: 1990-2000' Succeeds After: fails Created attachment 107939 [details] [review] Bug 11469: Fix search by date range Did some more testing, now -2007 doesn't seem to work with the patches :( |